"delete" Frage

[Firefighter]

Gehen wir mal davon aus wir haben folgenden Fall vorliegen:

class EineKlasse
{
}

class Test
{
   private:
       std::vector<EineKlasse*> zeigers;

  public:
    void einefunktion(int wert)
    {
        for(int i =0;i<wert;i++)
        {
            zeigers.push_back(new EineKlasse);
        }
    }
  ~Test()
   {
       //muss ich hier wieder jedes element im vector abbräumen? ála
       for(int i =0;i<zeigers.size();i++)
       {
           delete zeigers[i];
       }
    }

};

Muss ich wie oben beschrieben, die Elemente nach dem "Gesetzt: Für jedes new ein delete" auch so wieder abräumen? Oder übernimmt das der Vector für mich?

[Simon2]

Firefighter schrieb:

...
Muss ich wie oben beschrieben, die Elemente nach dem "Gesetzt: Für jedes new ein delete" auch so wieder abräumen? ...

Ja.

Firefighter schrieb:

...
Oder übernimmt das der Vector für mich?

Nein.
Der vector destruiert nur die Objekt, die sich in ihm befinden ... in diesem Fall Zeiger.

Gruß,

Simon2.

[Fellhuhn]

Der Vector löscht nur die Pointer selbst, nicht die Objekte auf die sie zeigen. Musst du selbst übernehmen.

[Firefighter]

Ok alles klar, und der Dtor hat auch so seine richtigkeit, ja?

[David_pb]

Firefighter schrieb:

Ok alles klar, und der Dtor hat auch so seine richtigkeit, ja?

Ja!

[Firefighter]

Ok danke, hab ichs mir doch richtig vorgestellt.

[Firefighter]

std::vector<std::auto_ptr<Kontobewegung> > bewegungen;

Wenn ich das so gestalte(deklaration scheint noch fehlerhaft zu sein, da ich fehler bekomme) brauche ich mich später nicht über das aufräumem kümmern richtig`?

[David_pb]

Theoretisch hast du da recht!

[MFK]

Firefighter schrieb:

Wenn ich das so gestalte(deklaration scheint noch fehlerhaft zu sein, da ich fehler bekomme) brauche ich mich später nicht über das aufräumem kümmern richtig`?

Falsch. auto_ptr ist wegen seiner Kopiersemantik nicht für die Container der Standardbibliothek geeignet.

[Firefighter]

EDIT: Ok MFK ich danke dir, das erklärt einige Fehler.

Mit den shared_ptr, hab ich recht? aber mal ne andere Frage, wenn meine deklaration wie oben aussieht, wie könnte dann ein pushback eines möglichen kontobewegung-zeiger aussehen?Die Referenz ist mir dabei gerade nicht sehr hilfreich

[Fellhuhn]

Kontenbewegung *bestechung = new Kontenbewegung();
bewegungen.push_back(bestechung);

[Firefighter]

Öhmm...nee ich glaube das meinte ich nicht...sondern ich meinte eher, wie es mit shared_ptr ausehen müsste?Weil mit den burschen hab ich noch net so viel erfahrung.

[David_pb]

So?

bewegungen.push_back( shared_ptr<Kontenbewegung>( new Kontenbewegung() ) );

[Firefighter]

Ok das hilft mir schonmal weiter, und die Deklaration vom vector wäre dann so hier?:

std::vector< shared_ptr<Irgendwas> > vieles_irgendwas;

?

[David_pb]

Sieht ok aus.

[Firefighter]

Und auf die deklaration kann ich dein push_back anwenden? Kann doch eigentlich bei deinem push_back das führende shared<> zeugs weglassen und nur ein normales new machen oder nicht?

[Firefighter]

OK hat sich erledigt, habs alleine hinbekommen. Aber mal ne andere Frage, worin besteht nun der Underschied zwischen auto_ptr und shared_ptr?also aufjedenfall schonmal die Containerkompatiblität, und das shared_ptr sich ums aufräumen kümmert, aber was noch?

[Firefighter]

Ok hab mir auch das ebend selbst beantwortet, danke für eure Beiträge.

[Nexus]

Vielleicht kannst du dir auch mal Boost.Pointer Container anschauen, da werden die Zeiger auch automatisch freigegeben.

[Firefighter]

Alles klar Nexus, danke für den Hinweis, kannste eventuell kurz erklären wo der signifikanteste Unterschied liegt?