<![CDATA[Ruft map::erease den Destruktor auf?]]>Wie der Titel schon sagt: ruft map::erease den Destruktor des zu löschenden Objektes auf? Wenn ja, kann man das irgendwie verhindern? Denn bei meinem Destruktor entfernt er das eigene Objekt selbst aus einer Map und das passiert irgenwie in ner Endlosschleife 😕

]]>https://www.c-plusplus.net/forum/topic/177910/ruft-map-erease-den-destruktor-aufRSS for NodeSun, 20 Sep 2026 06:37:39 GMTThu, 05 Apr 2007 10:18:36 GMT60<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 10:18:36 GMT]]>Wie der Titel schon sagt: ruft map::erease den Destruktor des zu löschenden Objektes auf? Wenn ja, kann man das irgendwie verhindern? Denn bei meinem Destruktor entfernt er das eigene Objekt selbst aus einer Map und das passiert irgenwie in ner Endlosschleife 😕

]]>https://www.c-plusplus.net/forum/post/1259898https://www.c-plusplus.net/forum/post/1259898Thu, 05 Apr 2007 10:18:36 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 10:42:51 GMT]]>ja, bei erase werden die gespeicherten objekte ordungsgemäß zerstört sprich deren konstruktor aufgerufen. Gilt auch für Pointer. Allerdings wird hier natürlich nur der Pointer zerstört, nicht jedoch das Objekt auf das er verweist. Diese muessen zuvor "per Hand" mit delete aus dem Speicher entfernt werden.
Wenn du also lediglich die Adressen auf die zu speichernden Objekte in deiner map speicherst, dann wird bei "map.erase()" lediglich die Adressinfo, nicht aber das Objekt gelöscht. Das koennte also eine Möglichkeit für dein Problem darstellen.

]]>https://www.c-plusplus.net/forum/post/1259912https://www.c-plusplus.net/forum/post/1259912Thu, 05 Apr 2007 10:42:51 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 11:29:08 GMT]]>muffmolch schrieb:

...Allerdings wird hier natürlich nur der Pointer zerstört, nicht jedoch das Objekt auf das er verweist....

und das ist auch gut so 😉
schließlich weiß die map ja nicht, ob es da Zeiger auf Heap- oder Stackobjekte in der Hand hält. 😃

Gruß,

Simon2.

]]>https://www.c-plusplus.net/forum/post/1259951https://www.c-plusplus.net/forum/post/1259951Thu, 05 Apr 2007 11:29:08 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 12:52:25 GMT]]>Ein anderes Problem: folgender Code:

this->p_Server->mConnections.erase(GetCurrentThreadId());

Funktioniert bei mir bei der normalen Ausführung nur 1 mal, beim 2. mal nicht mehr, und beim Debugging crasht das Proggy gleich beim ersten mal(Visual C++ zeigt an, dass er in der Funktion equal_range() crasht).
Das zu löschende Objekt existiert 100% prozentig(hab die Thread-IDs vorher 2 mal(1 mal beim anlegen des Objektes und 1 mal vor dem entfernen) ausgeben lassen und sie sind identisch).

]]>https://www.c-plusplus.net/forum/post/1260032https://www.c-plusplus.net/forum/post/1260032Thu, 05 Apr 2007 12:52:25 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 13:46:44 GMT]]>also: erase löscht ALLE elemente zu dem key (bei multimaps kann es ja zu einem schlüssel mehrere elemente geben).
du verwendest vermutlich aber ne stl::map, sprich ein key->ein element.
dabei ist erase unkritisch.
mach doch mal folgendes:

std::cout<<this->p_Server->mConnections.erase(GetCurrentThreadId())<<std::endl;

dann siehst du ob prinzipiell ein element gefunden wurde.
der Fehler wird so wie es aussieht nichts mit der map zu tun haben, sondern mit den darin enthaltenen "threads".

]]>https://www.c-plusplus.net/forum/post/1260106https://www.c-plusplus.net/forum/post/1260106Thu, 05 Apr 2007 13:46:44 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 14:48:26 GMT]]>Der "Fehler" besteht darin, dass das Programm mit dem Aufruf der Funktion crasht...

]]>https://www.c-plusplus.net/forum/post/1260192https://www.c-plusplus.net/forum/post/1260192Thu, 05 Apr 2007 14:48:26 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 19:11:22 GMT]]>Gib mal ein bisschen Kontext.
Davon abgesehen: wenn erase crasht ... dann wirds wohl gröber was haben. erase selbst kann man eigentlich nicht mit Parametern füttern die zu einem Absturz führen könnte, d.h. höchstens die Destruktoren die dann laufen könnten den Absturz verursachen, oder er ist evtl. schon verursacht worden bevor erase läuft, Speicher überschrieben oder sowas.

Andere sache: da du offensichtlich mit Threads arbeitest: synchronisierst du auch brav alle Zugriffe auf diese map (und andere gemeinsam genutzte Daten)?

]]>https://www.c-plusplus.net/forum/post/1260352https://www.c-plusplus.net/forum/post/1260352Thu, 05 Apr 2007 19:11:22 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 19:24:37 GMT]]>Also, ich schreib grad eine Art "Server-Engine", in der alle Verbindungen und der ganze socket-Kram in einer Klasse gemanaged werden. Die einzelnen Verbindungen werden wiederrum in einer eigenen Klasse gemanaged. Die ganzen Verbindungen werden dann in einer Map abgelegt und als Index dient die Thread-ID des Threads, der für die Verbindung "zuständig" ist. Nun wird, wenn Thread, der zuständig ist, "fertig" ist, die Verbindung dazu über delete connection; gelöscht. Im Destruktor von connection wiederrum trägt sich die Verbindung eigenhändig aus der Sammel-Map vom Server aus.
Das komische ist jetzt, dass das ganze bei der 1. Verbindung problemlos funzt und bei der 2. Verbindung bis vor mConnections.erase() im Destruktor kommt(vor map::erase funktioniert noch eine Ausgabe mit printf).

Wegen der Synchronisierung: so wie der Code gebaut ist, kommen nie ein 2 Zugriffe gleichzeitig zustande.

Edit: Hab grad rausgefunden, dass map::erase bei 1. mal zwar nicht crasht aber auch nich die Verbindung löscht.

]]>https://www.c-plusplus.net/forum/post/1260356https://www.c-plusplus.net/forum/post/1260356Thu, 05 Apr 2007 19:24:37 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Thu, 05 Apr 2007 20:23:23 GMT]]>Hi,

kenne mich damit nicht besonders gut in dem Thema aus, aber: Bist Du sicher, dass diese "Connections" das Kopieren überstehen ? Immerhin wird in eine map hinein und "drin herum" einiges kopiert ...

Gruß,

Simon2.

]]>https://www.c-plusplus.net/forum/post/1260373https://www.c-plusplus.net/forum/post/1260373Thu, 05 Apr 2007 20:23:23 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Fri, 06 Apr 2007 11:30:21 GMT]]>Die Map sieht so hier aus:

map<unsigned long, IServerConnection*> mConnections

➡ Da drin gibt es nur Pointer auf die Connections. Das komische ist ja, dass alles einwandfrei funktioniert, nur das löschen nicht.

]]>https://www.c-plusplus.net/forum/post/1260551https://www.c-plusplus.net/forum/post/1260551Fri, 06 Apr 2007 11:30:21 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Fri, 06 Apr 2007 11:55:50 GMT]]>Steht this->p_Server->mConnections.erase(GetCurrentThreadId()); in dem Destruktor, von Connection der aufgerufen wird, wenn du delete connection machst? (ich nehme mal an ja, aber möchte sicher gehen, dass ich deine funktionsweise richtig verstanden hab).

Hat IServerConnection einen virtuellen Destruktor?

Wenn das Löschen nicht funktioniert, dann liegt das aber definitiv nicht an der map, weil hier nur mit einfachen Typen gearbeitet wird und nicht mit deinen Objekten. Ich vermute daher stark, dass der Speicher bis zum Löschen schon längst nicht mehr gültig ist und es deshalb zum Absturz kommt (dass es im Debugger direkt crasht spricht ebenfalls stark dafür).

Zeigt this->p_server auch ganz sicher auf gültigen Speicher (schau mal im Debugger)?

Ist this bzw. das connection-Objekt bei dem delete connection Aufruf auch nur wirklich existent?

Auch wenn (theoretisch) keine 2 Zugriffe auf die map statt finden können, pack die Zugriffe da doch zur Sicherheit mal in ne critical section um auch ganz sicher zu gehen, dass es das wirklich nicht ist.

]]>https://www.c-plusplus.net/forum/post/1260577https://www.c-plusplus.net/forum/post/1260577Fri, 06 Apr 2007 11:55:50 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Fri, 06 Apr 2007 12:05:08 GMT]]>Verdammt bin ich blöd! 😮
Ich hab in jeder den beiden Klassen(Connection und Instance Template) folgendes stehen:

friend IServer;
    IServer* p_Server;

Hab aber den Pointer für p_Server nur beim Instance Template mitgegeben und bei der Connection völlig vergessen.
Vielen Dank für den Denkanstoß!

Nochmal wegen der Synchronisierung: werd ich noch machen, nicht dass ich dann wieder mit so nem dummen Problem hierher gerannt komme 😃 .

]]>https://www.c-plusplus.net/forum/post/1260585https://www.c-plusplus.net/forum/post/1260585Fri, 06 Apr 2007 12:05:08 GMT<![CDATA[Reply to Ruft map::erease den Destruktor auf? on Fri, 06 Apr 2007 18:27:00 GMT]]>Dr. C++ schrieb:

Die Map sieht so hier aus:

map<unsigned long, IServerConnection*> mConnections

...

Ok, das wusste ich nicht.

Gruß,

Simon2.

]]>https://www.c-plusplus.net/forum/post/1260837https://www.c-plusplus.net/forum/post/1260837Fri, 06 Apr 2007 18:27:00 GMT