viele Instanzen erzeugen

SeppJ

sebi707 schrieb:

Das entfernen von Autos ist mit std::vector in etwa genauso umständlich wie bei Arrays. Zumindest wenn du mitten drin Autos entfernen willst weil du dann alle Autos die noch folgen umkopieren musst.

Anscheinend ist hier die Reihenfolge nicht wichtig, daher kommt man auch mit einer einzelnen Verschiebung aus.

Was Du u.U. noch nicht weißt: es gibt neben dem Konstruktor, den Du implementiert hast noch weitere.
Dein Konstruktor - der ohne Argumente wird default-constructor genannt.

push_back() in Deinem Code fügt allerdings eine Kopie von auto1 an den vector an. Um die Kopie zu erstellen benutzt es den copy-constructor.
Da Du keinen copy-c'tor geschrieben hast, wird einer vom Compiler generiert. Der sieht ungefähr so aus:

class Auto
    {
        .... wie zuvor
            Auto(const Auto& other)  // copy-ctor
            {
                Geschwindigkeit = other.Geschwindigkeit;
            }
    };

Wie Du siehst: da wird anzAutos nicht inkrementiert. Klar, eigentlich - der Compiler kann ja nicht Deine Gedanken lesen! (Aber er kann doof jedes member kopieren...)

Spendier Auto eine Kopierkonstruktor, der anzautos inkrementiert: et voilà.

"Et voilà" with a twist: Du wirst feststellen, dass AnzAuto enorm groß wird - im Gegensatz zur Größe des Vektors.

Das liegt daran, dass im Hintergrund fröhlich Autos hin und her kopiert werden, wenn der Vektor sich vergrößert, weil er mehr Speicherplatz braucht.
In diesem Moment alloziert der vector neuen (und größeren) Speicherplatz, kopiert jedes Element an seinen neuen Platz und löscht schließlich die alten Elemente. Dafür wird deren destructor aufgerufen.
Dementsprechend musst Du AnzAutos noch im destructor von Auto um 1 verringern.

Viel zu lernen...

Es lohnt sich!

Nathan

Du kannst auch übrigens einfach Autosammlung.size() nehmen um die Anzahl der Autos innerhalb des vectors zu erfragen.

Kinemeier

Danke für Eure zahlreichen Antworten.

sebi707 schrieb:

for(int i = 0; i < 100; i++)
    {
        Autosammlung.push_back(Auto());
    }

Genau so habe ich mir das vorgestellt.

Ziel der ganzen Sache ist es, dass ich erst einmal x Instanzen anlege,
geänderte Werte (Lackierung, Kilometerzähler etc.) aus einer Datei einlese,
dann während des Spiels Autos entferne, hinzufüge und verändere und am Ende alles wieder abspeichere.
Das sollte ich nun hinbekommen.

Braunstein

Man könnte statt dessen auch einfach.

std::vector<Auto> Autosammlung(100);

schreiben.

Kinemeier

Oh ja, das läuft sogar noch viel schneller.
(Habe allerdings alle Compileroptimierungen deaktiviert.)

patrick246

std::vector<Auto> autosammlung;
autosammlung.reserve(100);
for(std::size_t i = 0; i < 100; i++)
{
    autosammlung.push_back(Auto());
}

Dürfte ähnlich schnell laufen und so kannst du sogar unterschiedliche Autos im vector haben, falls du mal einen anderen Konstruktor hinzufügst.

Kinemeier schrieb:

Oh ja, das läuft sogar noch viel schneller.
(Habe allerdings alle Compileroptimierungen deaktiviert.)

Stoppst du mit einem extra Programm die Zeit oder hast du mit mehreren Elementen Millionen Elementen gestestet? Bei 100 Elementen dürfte selbst mit Optimierungen deaktiviert das Programm sofort fertig sein.

Kinemeier

Ja, habe es mit 100 Millionen getestet.
Die Zeiten schwanken zwar enorm, aber das Ergebnis ist trotzdem eindeutig.

SeppJ

Niemanden interessiert, wie schnell unoptimierter Code läuft.