„onEachSubelement (...)“ Methode für C++

Question

ich einige Daten in class packen oft Fehler mit öffentlichen/globalen Zugriff zu verhindern und einige gemeinsame Methoden darauf bieten, zum Beispiel:

class GameArea{ 
    std::vector<Enemy*> enemies; 
    std::wstring name; 
public: 
    void makeAllEnemiesScared(); 
    Enemy * getEnemy(unsigned index); 
}; 

GameArea ist nur ein vereinfachtes Beispiel hier. Es wäre eine Art von benutzerdefinierten Container/Menager mit einigen spezialisierten Methoden (aber es ist nicht nur ein Container).

Die ideale Situation ist, wenn ich weiß, welche Operationen an jedem Enemy auf einmal ausgeführt wird, und sie treten an mehreren Stellen, damit ich sie in GameArea direkt erklären können (wie makeAllEnemiesScared()).

Für andere Fälle, die ich mit gehen kann:

for(unsigned i=0; i<gameArea->getEnemiesCount(); i++){ 
    Enemy * enemy = gameArea->getEnemy(i); 
    ... 
} 

Aber es leidet unter einigen Nachteilen:

• Ich kann nicht C++ 11 sauber & schön for(auto &Enemy : enemies) Schleife verwenden,
• Es ist nicht effizient (so viele Anrufe zu getEnemy(index)),
• Es ist nicht ein Zweck für getEnemy(index) zu iterieren werfen alle Elemente - es ist nützlich, wenn wir einzelne oder wenige von ihnen auswählen möchten, hat es auch für index < enemies.size() innen überprüfen - es ist schrecklich, es auf jedem Element in Schleife zu überprüfen.

HINWEIS: denke ich Fälle über, wenn ich etwas ganz Besonderes tun (nicht wert in GameArea getrennt Methode zu schaffen, aber auf jedes Element von GameArea::enemies).

Ich dachte über einige GameArea::onEachEnemy(... function ...) Methode, die eine function (oder vielleicht besser ein Lambda?) Als Parameter verwendet. Ist das eine gute Lösung?

Oder vielleicht anderen Ansatz sollte hier verwendet werden? Wie die std::vector von GameArea zurückgeben - die für mich ein bisschen "hässlich" aussieht. Ich möchte nicht, dass der Benutzer denkt, dass er Artikel direkt zu dieser vector hinzufügen oder entfernen kann.

Answer 1

Eine gute Lösung sein könnte, die nicht Interna Ihrer Klassen aussetzt wie Sie ist, vorgeschlagen, eine Funktion eine Aktion für jedes Objekt aufrufen:

class GameArea { 
... 
    template<typename Func> void ForEachEnemy(Func f) 
    { 
    std::for_each(enemies.begin(), enemies.end(), f); 
    } 
... 

Dann können Sie alles passieren wollen Sie als ForEachEnemy Argument - globale Funktion, boost::bind Ergebnis usw.

Answer 2

Wenn Sie den Vektor selbst oder zumindest Iteratoren aussetzen begin und end, können Sie std::for_each

Dann würden Sie dies als

std::for_each(std::begin(enemies), std::end(enemies), foo); 

verwenden, wo foo die Funktion, die Sie anrufen möchten, ist auf jede Enemy.

Wieder beachten Sie müssen den Vektor selbst aus, da Sie Methoden in GameArea machen könnte Iteratoren zu bekommen, damit der Anruf wie

std::for_each(gameArea->GetEnemyBegin(), gameArea->GetEnemyEnd(), foo); 

Answer 3

Sie könnten das Iterator Muster betrachten. Auf diese Weise können Sie die Kapselung beibehalten, haben aber dennoch etwas, das bequem zu verwenden ist.

Sie können entweder Standard-Container-Iteratoren verfügbar machen oder, wenn Sie eine vollständige Kapselung wünschen, schreiben Sie den Iterator selbst. Wenn Sie den Iterator richtig schreiben können Sie sogar C++ verwenden 11 bereichsbasierte for-Schleifen:

#include <vector> 
#include <string> 
#include <iostream> 

struct Enemy{ std::string name; }; 

class EnemyIterator { 
    friend class GameArea; 
private: 
    std::vector<Enemy>::iterator iterator; 
    EnemyIterator(std::vector<Enemy>::iterator it) : iterator(it){} 
public: 
    EnemyIterator& operator++() { ++iterator; return *this; } 
    Enemy& operator*() { return *iterator; } 
    friend bool operator!=(EnemyIterator lhs, EnemyIterator rhs) { 
    return lhs.iterator != rhs.iterator; 
    } 
}; 

class GameArea{ 
    std::vector<Enemy> enemies; 
public: 
    EnemyIterator begin() { return EnemyIterator(enemies.begin()); } 
    EnemyIterator end() { return EnemyIterator(enemies.end()); } 
    void addEnemy(std::string name) {enemies.push_back(Enemy{name}); } 
}; 

int main() { 
    GameArea area; 
    area.addEnemy("Kobold"); 
    area.addEnemy("Wraith"); 
    area.addEnemy("Ork"); 
    for (auto& enemy : area) 
     std::cout << enemy.name << "\n"; 
} 

Um für C++ 11 gefächerte basierend for-Schleifen mit GameArea Sie arbeiten müssen, eine von dem tun:

• definieren beginnen und das Endglied Funktionen
• define beginnen und enden nicht-Member-Funktionen im gleichen Namensraum
• spezialisiert std :: beginnen und std :: Ende