Ich habe eine Klasse namens Auto von der ich viele Instanzen erzeugen möchte.
Mithilfe eines Arrays ginge das natürlich, aber da Autos auch jederzeit wieder entfernt bzw. neue hinzugefügt werden müssen, habe ich einen std::vector genommen.
Mittels push_back() kann ich dann über eine bereits vorhandene Instanz Kopien davon erzeugen. Das ganze funktioniert aber nicht wie ich es mir vorgestellt habe.
Die Instanzen werden anscheinend nicht erzeugt, sondern nur kopiert (Instanzenzähler bleibt auf 1).
Kurz gesagt: Wie erzeuge ich eine große Anzahl Instanzen richtig und kann dann noch weitere hinzufügen und entfernen?

#include <iostream>
#include <vector>

int AnzAutos = 0;

int main()
{
	class Auto
	{
		public:
			int Geschwindigkeit;

			Auto()
			{
				AnzAutos++;
				Geschwindigkeit = 200;
			}
	};

	Auto Auto1;

	std::vector<Auto> Autosammlung;

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

	std::cout << "AnzAutos = " << AnzAutos; // = 1
	std::cout << "\nAutosammlung[87].Geschwindigkeit = " << Autosammlung[87].Geschwindigkeit; // = 200

	return 0;
}

#include <iostream>
#include <vector>

int AnzAutos = 0;

int main()
{
    class Auto
    {
        public:
            int Geschwindigkeit;

            Auto()
            {
                AnzAutos++;
                Geschwindigkeit = 200;
            }
    };

    Auto Auto1;

    std::vector<Auto> Autosammlung;

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

    std::cout << "AnzAutos = " << AnzAutos; // = 1
    std::cout << "\nAutosammlung[87].Geschwindigkeit = " << Autosammlung[87].Geschwindigkeit; // = 200

//Sieh hier:
   std::cout << "\nAnzahl: " << Autosammlung.size() << '\n';
   for(int i = 0; i < Autosammlung.size(); ++i)
      Autosammlung[i].Geschwindigkeit = i;

   std::cout << "\nAuto 5 Geschwindigkeit: " << Autosammlung[5].Geschwindigkeit << '\n';
   std::cout << "\nAuto 87 Geschwindigkeit: " << Autosammlung[87].Geschwindigkeit << '\n';

    return 0;
}

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

Dann wird AnzAutos auch deiner Anzahl hinzugefüger Autos entsprechen. Allerdings ist das wenig sinnvoll sich so die Anzahl der Autos zu merken (wenn der überhaupt Ziel ist). Der std::vector speichert sich die Anzahl der hinzugefügten Objekte schon selbst.

Kinemeier schrieb:

aber da Autos auch jederzeit wieder entfernt bzw. neue hinzugefügt werden müssen, habe ich einen std::vector genommen.

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.

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.

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!

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.

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

schreiben.

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

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.