Und möchte mittels Iterator darüber iterieren und eine funktion ignition() aufrufen:

for(std::list<MyObject*>::iterator it=list.begin();it!=list.end();++it){
it->ignition();

}
Beim bauen erhalte ich diese Meldung:
error: request for member ‘ignition’ in ‘* tg.std::_List_iterator<_Tp>::operator-> [with _Tp = MyObject*, std::_List_iterator<_Tp>::pointer = MyObject**]()’, which is of non-class type ‘MyObject*’

Wie kann ich die Funktion jedes MyObject aufrufen? Bitte um Hilfe!

[Edit] : tg nach it umbenannt.

Ansonsten hast du sozusagen einen Doppelzeiger hier. Machs so:

(*it)->ignition();

edit: zu spät.

Hat es einen Grund dass du auf tg operierst obwohl dein Iterator it heißt? Du inkrementierst auch den falschen.

Ansonsten hast du sozusagen einen Doppelzeiger hier. Machs so:

(*it)->ignition();

Vielen Dank !

delete (&it);

...

Richtig?

delete &(*it);

Uerst das Objekt holen und dann per Addressoperator die Addresse des Objektes nehmen.

Wäre halbrichtig. Wieso halbrichtig? Weil du kein delete verwenden solltest.
Gibt es einen Grund wieso du die Objekte per new anlegst? Wenn es denn gibt, mann mach lieber eine std::list<std::unique_ptr<MyObject>>.

Uerst das Objekt holen und dann per Addressoperator die Addresse des Objektes nehmen.

Nur sind die Objekte wiederum Zeiger. Also ist das & zuviel.

Ethon schrieb:

Uerst das Objekt holen und dann per Addressoperator die Addresse des Objektes nehmen.

Nur sind die Objekte wiederum Zeiger. Also ist das & zuviel.

Öhhhh ... stimmt.

Ne, in diesem Fall holst du dir die Addresse des Iterators!

delete &(*it);

Uerst das Objekt holen und dann per Addressoperator die Addresse des Objektes nehmen.

Wäre halbrichtig. Wieso halbrichtig? Weil du kein delete verwenden solltest.
Gibt es einen Grund wieso du die Objekte per new anlegst? Wenn es denn gibt, mann mach lieber eine std::list<std::unique_ptr<MyObject>>.

Danke für die Hilfe, hat funtioniert.

Ich muss die Objecte mittels new anlegen weil ich sie auch außerhalb des definierten Bereichs brauche.
Diese Objecte sollen gewisse Zustände speichern und an anderer Stelle verglichen werden.
Die statischen Objecte ohne new erzeugt werden nach verlassen des scopes zerstört obwohl ich sie noch brauche...

Die statischen Objecte ohne new erzeugt werden nach verlassen des scopes zerstört obwohl ich sie noch brauche...

Nimm wie gezeigt std::unique_ptr , der räumt automatisch auf. Oder Boost.PtrContainer...

Ethon schrieb:

Ne, in diesem Fall holst du dir die Addresse des Iterators!

delete &(*it);

Uerst das Objekt holen und dann per Addressoperator die Addresse des Objektes nehmen.

Wäre halbrichtig. Wieso halbrichtig? Weil du kein delete verwenden solltest.
Gibt es einen Grund wieso du die Objekte per new anlegst? Wenn es denn gibt, mann mach lieber eine std::list<std::unique_ptr<MyObject>>.

Danke für die Hilfe, hat funtioniert.

Ich muss die Objecte mittels new anlegen weil ich sie auch außerhalb des definierten Bereichs brauche.
Diese Objecte sollen gewisse Zustände speichern und an anderer Stelle verglichen werden.
Die statischen Objecte ohne new erzeugt werden nach verlassen des scopes zerstört obwohl ich sie noch brauche...

Die Objekte in der Liste doch nicht?
Oder sollen die Objekte länger leben als die Liste? Dann wäre das ein Fall für std::shared_ptr.

Ich muss die Objecte mittels new anlegen weil ich sie auch außerhalb des definierten Bereichs brauche.
Diese Objecte sollen gewisse Zustände speichern und an anderer Stelle verglichen werden.
Die statischen Objecte ohne new erzeugt werden nach verlassen des scopes zerstört obwohl ich sie noch brauche...

Ist dir bewusst, dass in der Liste Kopien der Objekte gespeichert werden, die so lange wie die Liste leben? Wenn ja: Kannst du die Lebenszeit der Liste verlängern?

Und noch die für std::list typische Frage: Hast du die Performance mal mit std::vector verglichen? Der vector zieht die list häufig ab, obwohl einem tolle Containerentscheidungsbildchen was anderes erzählen wollen.