class AllocNode{
.
.
.
.
  Delete_Me(){
     delete this;
  }

}

class Komb{
 std::list<AllocNode*> LIST;

 deleteNodes(){
 //ERSION A

 for(std::list<AllocNode*>::iterator S= LIST.begin(); S != LIST.end();S++)
    delete *S;

//VERSION B

  std::for_each(m_lList.begin(), m_lList.end(),std::mem_fun(&AllocNode::Delete_Me));

//Liste löschen
m_lList.clear();

 }
};

Bei version A iteriere ich die Liste selber und gebe über den iterator jedes Objekt frei, VEriosn be benutzt ich die for_each funktion, und kann sogar eine memberfunktion benutzen ohne sie Delete_Me global zu machen? Nun frage ich ob diese methodik auch gut ist, da sich das objekt selber löscht, und ich nich wie bei Version A von ausen gelöscht wird... kritik?

Wenn ich nun Travesierungen mit forschleifen durch for_Each etc. ersetze, ereich somit eine höhere Performance?

was ich aber nicht mag ist eine memberfunktion, die delete this verwendet. objekte, die sich selbst ins bein schiessen können sind mir suspekt ^^

Aber

AllocNode *A = new AllocNode();
A->delete_Me();

und

AllocNode *A = new AllocNode();
delete A;

sind ja aquvalent oder nich??

AllocNode a;
a.delete_Me();

Aber da Delete_Me() privat ist, und die Liste inter mit poitner verwaltet ist kann zwar ein Objekt statisch angelet werden, aber Delete_Me() nicht aufgerufen werden;)

(ich hab's dir schonmal gesagt - man kann's auch übertreiben mit der Objektorientierung ;))

wenn sie so böse sind, wieso werden sie bereitgestellt... dachte "VERSION B" ist eleganter...

Vielleicht ist das bei dir nicht angekommen, aber finix' Beitrag war ironisch.

Ideal wäre eine "shared_ptr" Liste oder ähnliches, dann tuts ein "list.clear();".

@hustbear: und Version B ist evtl. noch laaanngsammer;)

Nicht unbedingt for_each() wird intern auch nur eine for-Schleife beinhalten, so daß der Compiler mit entsprechend guten Optimierungen (fast) identischen Code aus beiden Versionen erzeugen kann.

@MFK: hmm...sicher? Hört sich ehr zweideutig an

Ich hatte versucht es zu verschleiern aber, tatsächlich, MFK ist mir auf die Schliche gekommen. Mein Beitrag war eher als Kommentar auf deine Furcht vor Verben zu verstehen.

Version B finde ich deswegen Dreck weils 3x mehr Code ist + weniger übersichtlich = vollkommen unnötig.
Nur meine Meinung.

Sehe ich irgendwie genau andersherum. Die Delete_Me-Methode ist natürlich nicht so schön, aber ersetzt man dies durch eine freie Funktion delete_foo finde ich die Zeile wesentlich übersichtlicher und klarer als die Schleife.