also ich habe folgende Klassen erstellt(aufs wesentliche reduziert):

string CURRENTDIRECTORY_SAVE; 
CEntitySortCount *pEntities;
XXX *pXXX;
class XXX
{
private:
    CBEntity **pEntity;
    const string CURRENTDIRECTORY_XXX;
public:
    XXX(const string &name);
    void CreateEntities();
    string GetCURRENTDIRECTORY_XXX();
};
//-----------------------Definitionen--------------------
XXX::XXX(const string &name)          //(2)
         : (...), CURRENTDIRECTORY_XXX(CURRENTDIRECTORY_SAVE + name) 
{
    pEntity = new CBEntity*[pEntities->Entities_total + 1];
    (...)
    CreateEntities();       //(3)
}

void XXX::CreateEntities()    //(4)
{
    pEntity[0] = new EntityRoot(pEntities->pName[0]);   //(5)
    (...)
}

string XXX::GetCURRENTDIRECTORY_XXX(){
	return CURRENTDIRECTORY_XXX;}   //(8)

//=======================================================
class CBEntity
{
protected:
	(...)
	CBClass **pClass;
	(...)
	string CURRENTDIRECTORY_ENTITY;
public:
	CBEntity(const string &name, const string &currentdirectory);
};

//------------------Definitionen-------------------------
CBEntity::CBEntity(const string &name, const string &currentdirectory ="")
    : (...), CURRENTDIRECTORY_ENTITY(pXXX->GetCURRENTDIRECTORY_XXX() + currentdirectory)   //(7)
{}

//=======================================================
class EntityRoot: public CBEntity
{
public:
    EntityRoot(const string &name);
};

//--------------------Definitionen-----------------------
EntityRoot::EntityRoot(const std::string &name): CBEntity(name, "/0_RootEntity")  //(6)
{ (...) }

//=======================================================
class CEntitySortCount
{
public:
    //(...)
    unsigned short Entities_total;
    string *pName;
    CEntitySortCount(unsigned short npc, unsigned short monster,
	  unsigned short item, unsigned short chest): (...)  
	  {
	      Entities_total = //Initialisierung ; 
	      pName = new string[Entities_total + 1]; 
	  }
};

//==================Startfunktion=========================
void CreateXXXPrototype()
{
	//CEntitySortCount *pEntities initialisieren!
	pEntities = new CEntitySortCount(0,0,0,0);
	pEntities->pName[0] = "RootEntity";
	//XXX-Prototyp Erstellen!
	pXXX = new XXX("String");      //(1)
}

In der main() wird CreateXXXPrototype aufgerufen. Danach habe ich den "Problemweg" durch die Nummerierung //(1) - //(2) - ... - //(8) gekennzeichnet. In Markierung //(8) (hier: Zeile 29) bekomme ich folgende Fehlermeldung:

Unhandled exception at 0x104a0fb6 (msvcp90d.dll) in haupt.exe: 0xC0000005: Access violation reading location 0x000000b4.

Irgendwas stimmt mit der Übergabe des Strings an //(7) nicht, aber ich versteh es nicht.

(der mit new angeforderte Speicherplatz wird natürlich später wieder freigegeben)

Schonmal vielen Dank für die Hilfe

Furchtbarer Stil, übrigens. Globale Variablen und unsägliches Zeigergehampel. Kein Wunder, dass du nicht mehr durchsteigst.

Furchtbarer Stil, übrigens. Globale Variablen und unsägliches Zeigergehampel. Kein Wunder, dass du nicht mehr durchsteigst.

Das wird wohl stimmen. Allerdings wüsste ich nicht, wie ich das Problem anders lösen sollte. Nebenbei bemerkt bin ich noch relativ am Anfang mit C++.

Wie soll ich denn diese globalen Variablen umgehen? Zumindest
[/cpp]CEntitySortCount *pEntities;
XXX *pXXX;[cpp]

werden sehr oft gebraucht. Ich könnte sie natürlich "thematisch" in einer extra Klasse zusammenfassen, allerdings fände ich das auch nicht gerade übersichtlich.

Das mit dem "Zeigergehampel" kann ich verstehen. Aber das ist alles ziemlich abhängig von den Eingaben des benutzers, desswegen habe ich es so gemacht.
Ich mach das halt nur hobbymäßig .

Trotzdem danke für die Hilfe.

Eine Möglichkeit, die ich sehe wäre den Aufruf von "XXX:CreateEntities" aus dem Konstruktor zu entfernen und CBEntity **pEntity; erstmal auf NULL setzen. Erst wenn das Objekt erstellt ist "XXX:CreateEntities" über pXXX aufrufen.
Das dürfte doch klappen, oder?

Möglich, wäre aber genauso übler Stil.

Don Quijote schrieb:

Das wird wohl stimmen. Allerdings wüsste ich nicht, wie ich das Problem anders lösen sollte. Nebenbei bemerkt bin ich noch relativ am Anfang mit C++.

Ich würde dir dringend raten, dir fürs Erste eine einfachere Aufgabe zu suchen, bis du das Handwerkszeug drauf hast.

Don Quijote schrieb:

Wie soll ich denn diese globalen Variablen umgehen? Zumindest
[/cpp]CEntitySortCount *pEntities;
XXX *pXXX;[cpp]

werden sehr oft gebraucht. Ich könnte sie natürlich "thematisch" in einer extra Klasse zusammenfassen, allerdings fände ich das auch nicht gerade übersichtlich.

Ich sehe ja ein, dass man manche Dinge global verfügbar haben möchte. Dafür gibt's das Singleton-Pattern. Aber wenn schon eine globale Variable, warum dann auch noch als Zeiger? Damit man die extra noch initialisieren muss? Damit noch mehr schiefgehen kann?

Don Quijote schrieb:

Das mit dem "Zeigergehampel" kann ich verstehen. Aber das ist alles ziemlich abhängig von den Eingaben des benutzers, desswegen habe ich es so gemacht.

Schau dir bitte mal std::vector und die anderen Container der Standardbibliothek an. Auf Zeiger auf irgendwelche dynamisch alloziierten Arrays kann man eigentlich immer verzichten, und statt dessen einen Container nehmen.

Schwierig wird es immer dann, wenn die Besitzverhältnisse der Klassen nicht geklärt sind, aber dann ist schon beim Design etwas schiefgelaufen.

Möglich, wäre aber genauso übler Stil.

Naja, dann muss ich da wohl noch einiges verbessern.

Schau dir bitte mal std::vector und die anderen Container der Standardbibliothek an. Auf Zeiger auf irgendwelche dynamisch alloziierten Arrays kann man eigentlich immer verzichten, und statt dessen einen Container nehmen.

Ich wollte eigentlich erstmal auf diese eingebauten Strukturen verzichten und erstmal das Grundlegende lernen. Diese Container bauen doch darauf auf, oder? Und ich möchte ungern irgendwelche vorgefertigten Strukturen verwenden, wo ich gar nicht richtig weis, wie das alles hintendran abläuft.(Ich bin überzeugt, dass diese sehr praktisch sind)

Edit: Also diese globalen Zeigervariablen sind wirklich unnötig riskant. Diese kann ich relativ leicht umgehen. Die sonstigen globalen Variablen werde ich wahrscheinlich anderweitig verpacken. Die restliche Struktur gefällt mir eigentlich ganz gut, ich werde deine Ratschläge aber beherzigen.

Ich sehe ja ein, dass man manche Dinge global verfügbar haben möchte. Dafür gibt's das Singleton-Pattern.

Das Singleton-Pattern ist eher dazu gedacht, eine Klasse höchstens einmal zu instanziieren, als als Ersatz für globale Variablen.

MFK schrieb:

Schwierig wird es immer dann, wenn die Besitzverhältnisse der Klassen nicht geklärt sind, aber dann ist schon beim Design etwas schiefgelaufen.

Muss nicht sein. Wenn sich mehrere Objekte einen Besitz teilen, rät sich zum Beispiel std::tr1::shared_ptr .

Don Quijote schrieb:

Ich wollte eigentlich erstmal auf diese eingebauten Strukturen verzichten und erstmal das Grundlegende lernen. Diese Container bauen doch darauf auf, oder? Und ich möchte ungern irgendwelche vorgefertigten Strukturen verwenden, wo ich gar nicht richtig weis, wie das alles hintendran abläuft.(Ich bin überzeugt, dass diese sehr praktisch sind)

Für den Anfang ist es sicher nicht schlecht, wenn man selber ein wenig herumexperimentiert. Allerdings würde ich dann das als einzelne Teilaufgabe angehen, die Thematik ist komplex genug.

Mit der Zeit kannst du aber ohne schlechtes Gewissen die Standardbibliothek einsetzen. Bis man besser als die STL programmiert, braucht es schon etwas. Es gehören ja nicht nur Container selbst dazu, da kommen die ganzen Iteratoren- und Algorithmengeschichten, und so weiter. Dazu kommt, dass diese bereits getestet, weitgehend fehlerfrei und auch relativ gut optimiert sind. Du tust dir wirklich keinen Gefallen, wenn du dir alles selbst programmieren willst.

Aber wie gesagt: Um die Sache zu verstehen, ist das kein Problem. Nur sollte man erkennen, wann es Zeit ist, umzusteigen...:)

Aber wie gesagt: Um die Sache zu verstehen, ist das kein Problem. Nur sollte man erkennen, wann es Zeit ist, umzusteigen...:)

So sollte es auch sein. Ich möchte mir auch nicht alles selbst programmieren, aber da dies mein erstes etwas umfangreicheres Projekt sein wird, möchte ich hier versuchen, das ein oder andere umzusetzen. Ob das jetzt an der einen oder anderen Stelle nicht ganz so professionell hin haut, ist natürlich klar. Aber als Übung sit es doch bestimmt nicht falsch. Dann merkt man auch, wann man was machen bzw. besser lassen sollte. (siehe globale Zeigervariable...habe ich heute was dazu gelernt)

Die Container werde ich mir an späterer Stelle bestimmt anschauen und dann auch verwenden.