bin noch relativ neu in den Welten des C++, hoffe deshalb ihr könnt mir weiterhelfen.

Ich möchte in einer Klasse CWelt mehrere Objekte der Klasse CMensch dynamisch anlegen. Dazu nutze ich eine kleine for-Schleife, in welcher ich die Objekte initialisiere.

Allerdings hab ich das Problem, dass ich auf die übergebenen Werte (Name, Größer, Alter) in der Klasse CMensch nicht zugreifen kann, obwohl ich sie im Konstruktor der Klasse CMensch korrekt übergebe.

Kann mir jemand weiterhelfen?

Grüße,
klanc

CMensch* PointerMensch;
PointerMensch = new CMensch[anzahl];

for(int i=0;i<anzahl; i++){

PointerMensch[i] = Mensch(Name, Größe, Alter);

}

PointerMensch[i] = Mensch(Name, Größe, Alter);

sieht nicht sehr korrekt aus. Sollte das nicht lieber

PointerMensch[i] = CMensch(Name, Groesse, Alter);

heissen?

Wie sieht denn Deine Klasse CMensch aus?

PS: Die Speicherverwaltung solltest du zurückstellen, bis du die Grundlagen verstanden hast. Nimm lieber einen vector<CMensch> anstelle deines Pointers, der ist einfacher zu handhaben.

Nach dem Aufruf von PointerMensch = new CMensch[anzahl]; gibt diese Objekte schon, der Konstruktor wird hier schon aufgerufen. In der for-Schleife könntest du dann entsprechende Funktionen von den Objekten aufrufen, um den Namen, das Alter und die Größe zu setzen.

edit: laaaaaaaaaaaangsam...

Also die Zeile

PointerMensch[i] = Mensch(Name, Größe, Alter);

sieht nicht sehr korrekt aus. Sollte das nicht lieber

PointerMensch[i] = CMensch(Name, Groesse, Alter);

heissen?

Hast natürlich recht, heißt bei mir auch

PointerMensch[i] = CMensch(Name, Groesse, Alter);

Der Konstruktor sieht folgendermaßen aus:

CMensch::CMensch(char Name, int Groesse, int Alter)
{

	age_person = Alter;
        name_person = Name;
        height_person = Groesse;

}

Die Variablen age_person, name_person und height_person werden im Header der Klasse CMensch deklariert. Aber wenn ich dann von der Klasse CWelt auf eine Funktion der Klasse CMensch zugreife, sind diese Variablen nicht initialisiert. Als ob das Objekt die Werte vergisst, oder nicht behält. Sollten die Objekte nicht solange auf dem Heap liegen, bis ich sie mit delete wieder zerstöre?

@Badestrand: hab es auch schon über entsprechende Funktionen: set_alter, set_name und set_groesse versucht, aber da tritt dasselbe Problem auf.

@CStoll: Wie kann ich das mit einem vector<CMensch> umsetzen?

PS: Bist du sicher, daß du mit genau dem Zeiger gearbeitet hast, hinter dem du dein Array versteckt hast? Zeig doch mal den Code im größeren Zusammenhang - inklusive der Versuche, die Objekte wieder auszulesen.

@Badestrand: hab es auch schon über entsprechende Funktionen: set_alter, set_name und set_groesse versucht, aber da tritt dasselbe Problem auf.

CMensch* PointerMensch;
PointerMensch = new CMensch[anzahl];

for(int i=0;i<anzahl; i++){
    PointerMensch[i].set_alter( Alter );
    PointerMensch[i].set_name( Name );
    PointerMensch[i].set_groesse( Größe );
}

void CMensch::set_alter( int Alter )
{
    age_person = Alter;
}

Das funktioniert nicht? Dann stimmt irgendwas aber gewaltig nicht! Und ja, die Objekte sollten eigentlich nix vergessen

Die Werte im Konstruktor zu übergeben ist aber sowieso eleganter/besser. Nichtsdestotrotz scheint da irgendwas ja nicht ganz so zu funktionieren...

//Header
CMensch* pMensch1;
CMensch* pMensch2;

//CPP

pMensch1 = new CMensch(Wili, 180, 25);
pMensch2 = new CMensch(Anne, 170, 22);

pMensch1->SetzeGewicht();
...

Damit hat das auch gut funktioniert. Die Initialisierung der Variablen im Konstruktor und der Zugriff darauf innerhalb der SetzeGewichts-Funktion. Aber mit dem Klassenarray funzt das nicht so, weiß auch nicht wieso.
Ausgelesen hab ich die Variablen im Debug-Modus. Da waren die Variablen nicht belegt.

Name sollte auch kein char sondern ein std::string sein es sei denn alle deine Namen haben nur einen Buchstaben.

Hab ich wieder falsch angegeben, sind in Wirklichkeit auch Vom Typ string. Sorry

Anfangs hab ich zwei Objekte der Klasse CMensch angelegt....

Mag sein ... aber warum ?

Geht doch viel einfacher (sicherer, stabiler, ...) mit

//CPP

CMensch Mensch1("Wili", 180, 25);
CMensch Mensch2("Anne", 170, 22);

Mensch1.SetzeGewicht(5);
...

...
und wenn man einen Container mit denen haben will, ist der vector-Vorschlag von CStoll immer noch der bessere:

CStoll schrieb:

vector<CMensch> Menschen;
for(int i=0;i<anzahl;++i)
  Menschen.push_back(CMensch(n,g,a));

Gruß,

Simon2.

Zeig doch mal deine Klassendeklaration.

class CMensch

{
public:
	CMensch(void);

	CMensch(string Name, int Groesse, int Alter);

public:
	~CMensch(void);

private:
	int age_person, height_person, name_person;

public:

	void SetzeGewicht();
}

Dies ist die Deklaration der Klasse CMensch.

Braunstein schrieb:

Zeig doch mal deine Klassendeklaration.

class CMensch
	
{
public:
	CMensch(void);
	
	CMensch(string Name, int Groesse, int Alter);

public:
	~CMensch(void);

private:
	int age_person, height_person, name_person;
	
	

public:
	
	void SetzeGewicht();
}

Dies ist die Deklaration der Klasse CMensch.

Wofür brauchst Du denn den Destruktor ?
Ach ja: Und möchtest Du Gewicht auch auf irgendetwas setzen ? Der Funktion täte ein Parameter vielleicht ganz gut.

Gruß,

Simon2.

P.S.: Die mehreren public's da drin lassen die Java-Herkunft noch ein wenig wahrscheinlicher wirken ... (zusammen mit den überflüssigen news).

Danke euch allen. Habs den fehler gefunden, der lag woanders....

Dann zeig doch mal das Ergebnis (interessiert mich ja jetzt doch) .. und am Besten noch, wo der Fehler lag (man lernt nie aus).

Gruß,

Simon2.