umożliwiając dostęp do obiektów kontenerowych w C++

Question

Mam klasy jak poniżej:

class Foo { 
private: 
    std::map<std::string, Bar*> bars_by_name; 
    std::map<std::string, Baz*> bazs_by_name; 
}; 

Teraz chciałbym, aby umożliwić użytkownikowi dostęp do obu zbiorów, ale ukryć szczegóły implementacji, że jestem przechowywaniu obiekty na std :: maps. Zamiast tego chciałbym mieć funkcje członkowskie, które zwracają np. iteratory const dla kolekcji i ewentualnie iterator niestandardowy, który zwraca obiekty z obu kolekcji, ponieważ Bar i Baz należą do tej samej hierarchii klas. Biorąc pod uwagę styl, jaki byłby właściwy sposób robienia tego w C++? W Javie prawdopodobnie ustawię typ powrotu metody na Iterable lub zapakuję kolekcję w UnmodifiableCollection.

Answer 1

Możesz odrzucić iteratory kolekcji jako swoje własne.

class Foo { 
    private: 
    typedef std::map<std::string, Bar*> BarContainer; 
    typedef std::map<std::string, Baz*> BazContainer; 
    public: 
    typedef BarContainer::const_iterator ConstBarIterator; 
    ConstBarIterator beginBars() { return bars_by_name.cbegin(); } 
    ConstBarIterator endBars() { return bars_by_name.cend(); } 
    typedef BazContainer::const_iterator ConstBazIterator; 
    ConstBazIterator beginBazs() { return bazs_by_name.cbegin(); } 
    ConstBazIterator endBazs() { return bazs_by_name.cend(); } 

    private: 
    BarContainer bars_by_name; 
    BazContainer bazs_by_name; 
}; 

To oszczędza pracę wdrażaniu własnego iterator, ale pozostawia nam elastyczność zmieniając rodzaje kontenerów później i tylko wymagających rozmówców rekompilacji.

To nie rozwiązuje problemu iterowania ich razem.

Answer 2

Załóż interfejs, który ukrywa szczegóły:

class Foo { 
    public: 
     void AddBar(std::string name, Bar*); 
     void RemoveBar(std::string name); 

     Bar* GetBar(std::string name); 
     Bar* GetBar(std::string name) const; 

     void AddBaz(std::string name, Baz*); 
     void RemoveBaz(std::string name); 

     Baz* GetBaz(std::string name); 
     Baz* GetBaz(std::string name) const; 

    private: 
     std::map<std::string, Bar*> bars_by_name; 
     std::map<std::string, Baz*> bazs_by_name; 
}; 

Dla wszystkich użytkownik wie, można być przechowywanie ich w std :: pair wewnątrz listy, wewnątrz wektora, lub nawet za pomocą równoległych tablic. ... cokolwiek naprawdę.

EDIT:

Fakt, że jesteś przechowując je w mapie, ale chcesz użytkownik iteracyjne nad podstawowych zapachów kontenerowych złym/niewłaściwej konstrukcji. Zezwól im na dostęp indywidualny jak wyżej lub stwórz metody, które zrobią to za nich.