X-Git-Url: http://git.tdb.fi/?p=libs%2Fdatafile.git;a=blobdiff_plain;f=source%2Fcollection.cpp;h=129c0a84f76808c14f0a8164b853e0c56d6ccb97;hp=6d7306357712c2b737f9ff5d8232a6888b5daffb;hb=HEAD;hpb=b26fbba2bf5d53554325762b8022c501b3b8c8fb diff --git a/source/collection.cpp b/source/collection.cpp index 6d73063..e1c25dd 100644 --- a/source/collection.cpp +++ b/source/collection.cpp @@ -1,4 +1,5 @@ #include +#include #include "collection.h" using namespace std; @@ -6,14 +7,25 @@ using namespace std; namespace Msp { namespace DataFile { -Collection::Collection(): - fallback(0) -{ } - Collection::~Collection() { - for(TypeList::iterator i = types.begin(); i!=types.end(); ++i) - delete *i; + for(CollectionItemTypeBase *t: types) + delete t; +} + +void Collection::add_var(const string &name, const CollectionItemTypeBase *type, const Variant &var) +{ + insert_unique(items, name, var); + try + { + if(type) + type->notify_item(name, var); + } + catch(...) + { + remove_existing(items, name); + throw; + } } const Variant &Collection::get_var(const string &name, const CollectionItemTypeBase *type) @@ -39,7 +51,7 @@ const Variant *Collection::find_var(const string &name, const CollectionItemType type->create_item(*this, name); loaded = items.count(name); } - for(SourceList::iterator j=sources.begin(); (!loaded && j!=sources.end()); ++j) + for(auto j=sources.begin(); (!loaded && j!=sources.end()); ++j) { (*j)->load(*this, *type, name); loaded = items.count(name); @@ -51,18 +63,18 @@ const Variant *Collection::find_var(const string &name, const CollectionItemType } i = items.find(name); - return (i!=items.end() ? &i->second : 0); + return (i!=items.end() ? &i->second : nullptr); } void Collection::gather_items(vector *vars, list *names, const CollectionItemTypeBase &type, bool include_sources) const { - for(ItemMap::const_iterator i=items.begin(); i!=items.end(); ++i) - if(type.check_item_type(i->second)) + for(const auto &kvp: items) + if(type.check_item_type(kvp.second)) { if(vars) - vars->push_back(&i->second); + vars->push_back(&kvp.second); if(names) - names->push_back(i->first); + names->push_back(kvp.first); } if(include_sources && names) @@ -74,9 +86,9 @@ unsigned Collection::get_status(const string &name, const CollectionItemTypeBase ItemMap::const_iterator i = items.find(name); if(i==items.end()) { - for(SourceList::const_iterator j=sources.begin(); j!=sources.end(); ++j) - if((*j)->is_loadable(type, name)) - return 2; + auto j = find_if(sources, [&name, &type](const CollectionSource *s){ return s->is_loadable(type, name); }); + if(j!=sources.end()) + return 2; if(fallback) if(CollectionItemTypeBase *fb_type = fallback->get_type(type)) return fallback->get_status(name, *fb_type); @@ -88,18 +100,18 @@ unsigned Collection::get_status(const string &name, const CollectionItemTypeBase CollectionItemTypeBase *Collection::get_type(const CollectionItemTypeBase &type) const { - for(TypeList::const_iterator j=types.begin(); j!=types.end(); ++j) - if((*j)->is_same_type(type)) - return *j; - return 0; + for(CollectionItemTypeBase *t: types) + if(t->is_same_type(type)) + return t; + return nullptr; } CollectionItemTypeBase *Collection::get_type_for_item(const Variant &var) const { - for(TypeList::const_iterator i=types.begin(); i!=types.end(); ++i) - if((*i)->check_item_type(var)) - return *i; - return 0; + for(CollectionItemTypeBase *t: types) + if(t->check_item_type(var)) + return t; + return nullptr; } void Collection::add_source(const CollectionSource &s) @@ -109,44 +121,38 @@ void Collection::add_source(const CollectionSource &s) IO::Seekable *Collection::open_raw(const string &name) const { - for(SourceList::const_iterator i=sources.begin(); i!=sources.end(); ++i) - if(IO::Seekable *io = (*i)->open(name)) + for(const CollectionSource *s: sources) + if(IO::Seekable *io = s->open(name)) return io; - return 0; + return nullptr; } void Collection::gather_names_from_sources(list &names, const CollectionItemTypeBase &type) const { set new_names; - for(SourceList::const_iterator i=sources.begin(); i!=sources.end(); ++i) - { - std::list available_names = (*i)->get_names(type); - for(std::list::iterator j=available_names.begin(); j!=available_names.end(); ++j) - if(!items.count(*j)) - new_names.insert(*j); - } + for(const CollectionSource *s: sources) + for(const string &n: s->get_names(type)) + if(!items.count(n)) + new_names.insert(n); names.insert(names.end(), new_names.begin(), new_names.end()); } void Collection::load_items_from_sources(const CollectionItemTypeBase &type) { - for(SourceList::const_iterator i=sources.begin(); i!=sources.end(); ++i) - { - std::list available_names = (*i)->get_names(type); - for(std::list::iterator j=available_names.begin(); j!=available_names.end(); ++j) - if(!items.count(*j)) + for(const CollectionSource *s: sources) + for(const string &n: s->get_names(type)) + if(!items.count(n)) { bool loaded = false; if(type.can_create()) { - type.create_item(*this, *j); - loaded = items.count(*j); + type.create_item(*this, n); + loaded = items.count(n); } if(!loaded) - (*i)->load(*this, type, *j); + s->load(*this, type, n); } - } } void Collection::set_fallback(Collection *f) @@ -158,15 +164,15 @@ void Collection::set_fallback(Collection *f) Collection::Loader::Loader(Collection &c): coll(c) { - for(TypeList::const_iterator i = coll.types.begin(); i!=coll.types.end(); ++i) - (*i)->add_to_loader(*this); + for(const CollectionItemTypeBase *t: coll.types) + t->add_to_loader(*this); } CollectionItemTypeBase::~CollectionItemTypeBase() { - for(vector::iterator i=extractors.begin(); i!=extractors.end(); ++i) - delete *i; + for(ExtractorBase *e: extractors) + delete e; } void CollectionItemTypeBase::set_keyword(const string &k) @@ -183,8 +189,8 @@ void CollectionItemTypeBase::add_suffix(const string &s) bool CollectionItemTypeBase::match_name(const string &name) const { - for(vector::const_iterator i=suffixes.begin(); i!=suffixes.end(); ++i) - if(name.size()>i->size() && !name.compare(name.size()-i->size(), string::npos, *i)) + for(const string &s: suffixes) + if(name.size()>s.size() && !name.compare(name.size()-s.size(), string::npos, s)) return true; return false; }