<![CDATA[Expression: list iterator not incrementable]]>Einen wunderschönen guten Abend!

Ich habe hier einen Codeausschnitt, welcher in einer Liste mit Schüssen checkt ob diese noch "leben" und falls nicht diese löscht.
Wenn diese Liste nun leer ist, bekommt ich die Fehlermeldung
Expression: list iterator not incrementable.
Hat jemand eine Idee woran das liegen kann? Bin noch mehr als weniger Anfänger in C++...

for (auto it = mList.begin(); it != mList.end(); )
{
                if ((*it)->getIsAlive()==false)
                   {
                        delete(*it);
                       (*it) = nullptr;

                        it = mList.erase(it);
                   }

                 else
                {
                         it++;
                        (*it)->update(frametime);
                }
}

Vielen Dank im vorraus, bei weiteren Fragen kann ich natürlich auch mehr vom Code posten!

]]>https://www.c-plusplus.net/forum/topic/338858/expression-list-iterator-not-incrementableRSS for NodeMon, 13 Apr 2026 01:15:16 GMTFri, 15 Jul 2016 15:14:14 GMT60<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 15:19:23 GMT]]>Einen wunderschönen guten Abend!

Ich habe hier einen Codeausschnitt, welcher in einer Liste mit Schüssen checkt ob diese noch "leben" und falls nicht diese löscht.
Wenn diese Liste nun leer ist, bekommt ich die Fehlermeldung
Expression: list iterator not incrementable.
Hat jemand eine Idee woran das liegen kann? Bin noch mehr als weniger Anfänger in C++...

for (auto it = mList.begin(); it != mList.end(); )
{
                if ((*it)->getIsAlive()==false)
                   {
                        delete(*it);
                       (*it) = nullptr;

                        it = mList.erase(it);
                   }

                 else
                {
                         it++;
                        (*it)->update(frametime);
                }
}

Vielen Dank im vorraus, bei weiteren Fragen kann ich natürlich auch mehr vom Code posten!

]]>https://www.c-plusplus.net/forum/post/2502470https://www.c-plusplus.net/forum/post/2502470Fri, 15 Jul 2016 15:19:23 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 16:02:35 GMT]]>Ich sehe nicht, wie der Codeabschnitt den Fehler erzeugen soll. Du hast ihn aus dem Original kopiert? std::list?

Zeile 13 und 14 sind vertauscht.

Zeile 6 ist sinnlos.

]]>https://www.c-plusplus.net/forum/post/2502475https://www.c-plusplus.net/forum/post/2502475Fri, 15 Jul 2016 16:02:35 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 16:04:09 GMT]]>std::list<>::iterator ist ein bidirektionaler Iterator, ich versteh nicht wo das Problem sein soll.
Darf ich fragen warum du keinen Vector benutzt?

]]>https://www.c-plusplus.net/forum/post/2502476https://www.c-plusplus.net/forum/post/2502476Fri, 15 Jul 2016 16:04:09 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 17:40:18 GMT]]>HALT, STOP,
ich hab hier was verwechselt.
Wenn ich das "it++" oben in der for Schleife anstatt im else mit unterbringe, dann kommt der Fehler wenn die Liste leer ist.
Wenn ich wie jetzt das it++ im else habe, kommt der Fehler wenn ich das erste mal einen Shot spawnen möchte.

Enschuldigt bitte, ich gerate noch durcheinander :p

]]>https://www.c-plusplus.net/forum/post/2502483https://www.c-plusplus.net/forum/post/2502483Fri, 15 Jul 2016 17:40:18 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 17:48:07 GMT]]>Mit einer leeren Liste wird die Schleife garnicht ausgeführt.

]]>https://www.c-plusplus.net/forum/post/2502484https://www.c-plusplus.net/forum/post/2502484Fri, 15 Jul 2016 17:48:07 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Fri, 15 Jul 2016 20:48:44 GMT]]>SatchBoogie schrieb:

Wenn ich das "it++" oben in der for Schleife anstatt im else mit unterbringe, dann kommt der Fehler wenn die Liste leer ist.

Angenommen erase() setzt it auf mList.end() (weil es das letzte Element gelöscht hat) und das ++it im Schleifenkopf will it dann noch weiterbewegen...

SatchBoogie schrieb:

Wenn ich wie jetzt das it++ im else habe, kommt der Fehler wenn ich das erste mal einen Shot spawnen möchte.

Was passiert denn im else Zweig mit dem geschundenen Iterator? erst erhöhen, dann dereferenzieren?! Ist das cool? Insbesondere beim letzten Element?

]]>https://www.c-plusplus.net/forum/post/2502497https://www.c-plusplus.net/forum/post/2502497Fri, 15 Jul 2016 20:48:44 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Sat, 16 Jul 2016 07:49:57 GMT]]>AHH selbsverständlich Furble!
Ich muss natürlich erst mit it arbeiten (also in diesem Falle

(*it)->update(frametime);

und dann it erst erhöhen!
Vielen Dank für den Tipp, ist logisch, aber auch
Übungssache :p

]]>https://www.c-plusplus.net/forum/post/2502527https://www.c-plusplus.net/forum/post/2502527Sat, 16 Jul 2016 07:49:57 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Tue, 19 Jul 2016 11:11:14 GMT]]>SatchBoogie schrieb:

Einen wunderschönen guten Abend!

delete(*it);
                       (*it) = nullptr;

Kleiner Tip am Rande,

du solltest in deiner Liste keine rohen Zeiger speichern. Nimm lieber unique_ptr. Da kannst du die den Kram oben sparen, da die automatisch mit Löschen des Eintrags in der Liste freigegeben werden.

]]>https://www.c-plusplus.net/forum/post/2502763https://www.c-plusplus.net/forum/post/2502763Tue, 19 Jul 2016 11:11:14 GMT<![CDATA[Reply to Expression: list iterator not incrementable on Thu, 21 Jul 2016 10:07:23 GMT]]>genau das Problem hatte ich damals auch. Es handelt sich hierbei um SFML Programmierung mit snem KometenWeltraum Shooter. War damals über den gleichen Fehler gestolpert und habe dann ebenfalls Smart Pointer verwendet. Da die Objekte auf die die Pointer in der Liste zeigen Asteroiden sind, die nach unterschiedlichen Sekunden gelöscht werden und nicht sequentiell, macht ne Liste durchaus Sinn.

]]>https://www.c-plusplus.net/forum/post/2502933https://www.c-plusplus.net/forum/post/2502933Thu, 21 Jul 2016 10:07:23 GMT