Text Spiel

[Danielnoel]

Kann mir einer helfen. Ich möchte ein Text basiertes Spiel programmieren, bekomme aber immer so ein Fehler.


Danke schonmal.

[Madd Eye]

So wie das ausschaut wird das mit C++ gemacht oder??
Also die Programmiersprachen angeben ist immer Gut

Und evtl. den Codeteil in denen du die besagten variablen defenierst

[Danielnoel]

Also erst mal danke, dass du mir hilfst. Ich habe das Programm in VB geschrieben.

#ifndef _SPIELER_H_
#define _SPIELER_H_

class Spieler
{
private:
	char name[32];
	int gold;
	int experience;
	int health;
	int maxhealth;
	int mana;
	int maxmana;
	int str;
	int inti;
public:
	Spieler();
	bool SetName(char* newname);
	char* GetName();
	void SetGold(int newgold);
	void AddGold(int amount);
	void SpendGold(int amount);
	int GetGold();
	void SetExperience(int newexperience);
	void AddExperience(int amount);
	int GetExperience();
	void SetHealth(int newhealth);
	void AddHealth(int amount);
	void LoseHealth(int amount);
	int GetHealth();
	void SetMaxHealth(int newmaxhealth);
	int GetMaxHealth();
	void SetMana(int newmana);
	void AddMana(int amount);
	void LoseMana(int amount);
	int GetMana();
	void SetMaxMana(int newmana);
	int GetMaxMana();
};

#endif]

#include "Bibliotek.h"

Spieler::Spieler()
{
	SetName("Hero");
	str = 1;
	inti = 1;
	SetGold(500000);
	SetExperience(0);
	SetMaxHealth(25);
	SetHealth(25);
	SetMaxMana(10000);
	SetMana(10000);
}

void Spieler::AddExperience(int amount)
{
	SetExperience(experience + amount);
}

void Spieler::AddGold(int amount)
{
	SetGold(gold + amount);
}

void Spieler::AddHealth(int amount)
{
	SetHealth(health + amount);
}

void Spieler::AddMana(int amount)
{
	SetMana(mana + amount);
}

int Spieler::GetExperience()
{
	return experience;
}

int Spieler::GetGold()
{
	return gold;
}

int Spieler::GetHealth()
{
	return health;
}

int Spieler::GetMana()
{
	return mana;
}

int Spieler::GetMaxHealth()
{
	return maxhealth;
}

int Spieler::GetMaxMana()
{
	return maxmana;
}

char*Spieler::GetName()
{
	return name;
}

void Spieler::SetExperience(int newexperience)
{
	experience = newexperience;
}

void Spieler::SetGold(int newgold)
{
	gold = newgold;
}

void Spieler::SetHealth(int newhealth)
{
	health = newhealth;

	if (health > maxhealth)
		health = maxhealth;

	if (health < 0)
		health = 0;
}

void Spieler::SetMana(int newmana)
{
	mana = newmana;

	if (mana > maxmana)
		mana = maxmana;

	if (mana < 0)
		mana = 0;
}

void Spieler::SetMaxHealth(int newmaxhealth)
{
	maxhealth = newmaxhealth;
}

void Spieler::SetMaxMana(int newmaxmana)
{
	maxmana = newmaxmana;
}

bool Spieler::SetName(char *newname)
{
	if (strlen(newname) == 0)
		return false;

	if (strlen(newname) > 32)
		return false;

	strcpy_s(name,newname);

	return true;
}
void Spieler::LoseHealth(int amount)
{
	SetHealth(health - amount);
}

void Spieler::LoseMana(int amount)
{
	SetMana(mana - amount);
}

void Spieler::SpendGold(int amount)
{
	SetGold(gold - amount);
}

void deinZimmer(Spieler* spieler)
{
	cout << "Du bist in deinem Zimmer und siehst ein gemuetliches Bett.\n";
	char antwort;
	antwort = 'm';
	while(antwort != 'y' || antwort != 'n')
	{
		cout << "Moechtest du ne Runde schlafen? (y/n)\n";
		cout << ">";
		cin >> antwort;
		cout << endl;
		if(antwort = 'y')
		{
			cout << "Du legst dich hin und schlaefst bis zum Sonnenaufgang.";
			spieler->SetHealth(spieler->GetMaxHealth());
		}
		if(antwort = 'n')
		{
			atlantis(spieler);
		}

	}


}

Beim untersten möchte ich die Hp wieder auf voll setzten.
Genau dort erscheint auch der Fehler.

[Madd Eye]

Erstmal ist das niemals VB sonder entweder C# oder C++ (Vom Code her)
Und der Fehler CXX0030 gibts nur bei C++ also xD

Naja gehen wir das mal durch

void Spieler::SetHealth(int newhealth)
{
	health = newhealth;

[COLOR="Red"]	if (health > maxhealth)
		health = maxhealth;[/COLOR]

	if (health < 0)
		health = 0;
}

Was bringt es dir die HP auf Maximum zu stellen wenn man nicht über das Maximum kommen kann?

void deinZimmer(Spieler* spieler)
{
	cout << "Du bist in deinem Zimmer und siehst ein gemuetliches Bett.\n";
	char antwort;
	antwort = 'm';
	[COLOR="Red"]while(antwort != 'y' || antwort != 'n')
	{[/COLOR]
		cout << "Moechtest du ne Runde schlafen? (y/n)\n";
		cout << ">";
		cin >> antwort;
		cout << endl;
		if(antwort = 'y')
		{
			cout << "Du legst dich hin und schlaefst bis zum Sonnenaufgang.";
			spieler->SetHealth(spieler->GetMaxHealth());
		}
		if(antwort = 'n')
		{
			atlantis(spieler);
		}

[COLOR="Red"]	}[/COLOR]


}

Hier kannst du dir die while schleife sparen da mit cin>> antwort; Das Programm wartet bis der Spieler eine Eingabe macht


Das rote ist also eigentlich unnötig xD

Zum eigentlichen Fehler:

Versuch es mal mit:

int maxh = spieler->GetMaxHealth();
spieler->SetHealth(maxh);

statt

spieler->SetHealth(spieler->GetMaxHealth());

Der Fehler ist nämlich ein Windows spezifischer Fehler, für den es so direkt keine Lösung gibt

[Danielnoel]

Zum ersten: Sry hab da was verwechselt meinte natülich Visual C++.
Zum zweiten: Wenn ich des weg machen würde, würde er über das Maximum gehen.
Zum dritten: Stimmt hast recht. xD
Zum vierten: Ne geht net. Er zeigt den Fehler hier an.

int Spieler::GetMaxHealth()
{
	return maxhealth;
}

bei return maxhealth

[Madd Eye]

Ersetz dann mal

class Spieler
{
private:

mit

class Spieler
{
public:

[Danielnoel]

Ich hab hier doch beides stehen.

#ifndef _SPIELER_H_
#define _SPIELER_H_

class Spieler
{
[COLOR="Red"]private:[/COLOR]
	char name[32];
	int gold;
	int experience;
	int health;
	int maxhealth;
	int mana;
	int maxmana;
	int str;
	int inti;
[COLOR="Red"]public:[/COLOR]
	Spieler();
	bool SetName(char* newname);
	char* GetName();
	void SetGold(int newgold);
	void AddGold(int amount);
	void SpendGold(int amount);
	int GetGold();
	void SetExperience(int newexperience);
	void AddExperience(int amount);
	int GetExperience();
	void SetHealth(int newhealth);
	void AddHealth(int amount);
	void LoseHealth(int amount);
	int GetHealth();
	void SetMaxHealth(int newmaxhealth);
	int GetMaxHealth();
	void SetMana(int newmana);
	void AddMana(int amount);
	void LoseMana(int amount);
	int GetMana();
	void SetMaxMana(int newmana);
	int GetMaxMana();
};

#endif]

[Madd Eye]

Ja und ersetz das
private:

mal mit einem public:

das beugt in manchen fällen diesem Fehler vor
aber eben nicht immer

[Danielnoel]

Geht nicht er zeigt immernoch den Fehler hier.

int Spieler::GetMaxLeben()
{
	return maxleben;
}

[Madd Eye]

Oh man bin ich Blöd xD

schau mal genau hin

in den Definitionen steht:

int max[COLOR="Red"][B][U]health[/U][/B][/COLOR];

und bei unten dann:

int Spieler::GetMaxLeben()
{
	return max[COLOR="Red"][B][U]leben[/U][/B][/COLOR];
}

xD du hast da leben statt health stehen

[Danielnoel]

Oh!!
Ist mir gar net aufgefallen.
Es funktioniert aber immer noch nicht.

[Madd Eye]

welche fehler kommen den noch?

Immer noch die selben?

[Danielnoel]

Ich geb dir mal den ganzen Code und du schaust mal OK??
Wenn du es startest, dann nach Atlantis gehst, dann Mein Zimmer und wenn man y eingibt kommt der Fehler.

[xNopex]

@TS: Das Verwenden von dem this-Zeiger schafft Übersicht. So als Randbemerkung.

if(antwort = 'y') [...] if(antwort = 'n')

Fail...

Hier kannst du dir die while schleife sparen da mit cin>> antwort; Das Programm wartet bis der Spieler eine Eingabe macht

Die while-Schleife hat den Sinn, dass die Eingabe wiederholt wird, solange die Eingabe nicht 'y' oder 'n' ist. Das was du da von dir gibst hat damit gar nichts zu tun.

Versuch es mal mit:
Code:

int maxh = spieler->GetMaxHealth();
spieler->SetHealth(maxh);

statt

Code:

spieler->SetHealth(spieler->GetMaxHealth());

Beide Ausdrücke sagen das gleiche Aus, dasselbe in Grün.

Ersetz dann mal
Code:

class Spieler
{
private:

mit
Code:

class Spieler
{
public:

Noch nie was von Datenkapselung gehört, oder? @TS: Bitte so lassen, wie es war, der Vorschlag deines Helfers war nicht klug. Innerhalb der Klasse hat man eh auf jedes Attribut und jeder Methode Zugriff.. Nur so am Rande bemerkt.

Oh man bin ich Blöd xD

Joa...

Wenn du es startest, dann nach Atlantis gehst, dann Mein Zimmer und wenn man y eingibt kommt der Fehler.

Bei mir kommt da kein Fehler, sondern das Verhalten, das man wegen dem Quellcode auch zu erwarten hat.

[Madd Eye]

Versuch es mal mit:

int maxh = spieler->GetMaxHealth();
spieler->SetHealth(maxh);

statt

Code:

spieler->SetHealth(spieler->GetMaxHealth());

Beide Ausdrücke sagen das gleiche Aus, dasselbe in Grün.

Das die beiden das selbe aussagen weiß ich, aber es ist die schreibmethode die VC++ oftmals nicht korrekt ausführt

Ersetz dann mal

class Spieler
{
private:

mit
Code:

class Spieler
{
public:

Noch nie was von Datenkapselung gehört, oder? @TS: Bitte so lassen, wie es war, der Vorschlag deines Helfers war nicht klug. Innerhalb der Klasse hat man eh auf jedes Attribut und jeder Methode Zugriff.. Nur so am Rande bemerkt.

Das hatte den selben Sinn
Manchmal schluckt VC++ diese Befehle in dieser Form nicht und dann taucht der Fehler CXX030 auf

Ich weiß wovon ich rede hatte ihn schon selber oft genug -.-

Joa...

Und abfällig musst du auch nicht werden