DaniWeb Forum Index > Software Development > C++

Problems With Simple C++ Battle System; Need Help

Please support our C++ advertiser: Programming Forums - DaniWeb Sister Site

•

Join Date: Nov 2009

Posts: 3

Reputation: djnstuff is an unknown quantity at this point

Solved Threads: 0

djnstuff

Offline

Newbie Poster

Problems With Simple C++ Battle System; Need Help

Nov 2nd, 2009

Okay, what I'm trying to do here is set up a some-what simple battle system for a text based game... I'm having some trouble getting it to work correctly though. I want the Player's attack and the Monster's defense to be random each time the program loops (which is also not working), but it keeps giving the same numbers.

I don't have much experience, so any help you can offer would be appreciated, thanks.

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <string>
#include <cstdlib>
#include <ctime>
using namespace std;
 
// Variables
int charLevel, Floor;
int Hp, Atk, Def;
int monLevel, monHp, monAtk, monDef;
string action, charName, monName;
int damageDealt;
 
// Prototypes
void battlePhase();
 
int main()
{
	// Defining the character
	cout << "What is your name? ";
	cin >> charName;
	cout << "\nWhat level are you? ";
	cin >> charLevel;
	cout << "\nWhat floor are you on? ";
	cin >> Floor;
	cout << "\nWhat monster do you wish to fight? ";
	cin >> monName;
 
	// Player Stats
		// charLevel defined earlier
	srand((unsigned)time(0));	
	Hp = (15 + ( charLevel * 5 ));
	Atk = (((rand() % 6) + 0) + (charLevel - 1));
	Def = (((rand() % 4) + 0) + (charLevel - 1));
 
	// Monster Stats
	monLevel = (1/2) * Floor + (charLevel - 2);
	if ( monLevel < 0 )
		{
			monLevel == 1;
		}
 
	monHp = (10 + (monLevel * 2));
	monAtk = (((rand() % 4) + 1) + (monLevel - 1));
	monDef = (((rand() % 3) + 1) + (monLevel - 1));
 
	while (Hp > 0 ||  monHp > 0)
	{
		battlePhase();
	}
 
	return 0;
 
}
 
void battlePhase()
{	 
	// Actual Battle
	cout << endl << "\nWhat do you want to do? ";
	cin >> action;
 
	if ( action == "Attack" || action == "attack" )
		{
			cout << endl << charName << " attacked " << monName << " for " << Atk << endl;
			cout << monName << " defended " << charName << " for " << monDef << endl;
			damageDealt = Atk - monDef;
 
				if ( damageDealt < 0 )
					{
						damageDealt = 0;
					}
 
				if ( damageDealt == 0 )
					{
						cout << "\nNo damage!";
					}
 
				if ( damageDealt > 0 )
					{
						cout << monName << " took " << damageDealt << " damage!";
						monHp = monHp - damageDealt;
					}
		}
	else
		{
			cout << "\nThat isn't an action! Try typing 'attack'.";
			battlePhase();
		}
 
}
#include <iostream>
#include <string>
#include <cstdlib>
#include <ctime>
using namespace std;

// Variables
int charLevel, Floor;
int Hp, Atk, Def;
int monLevel, monHp, monAtk, monDef;
string action, charName, monName;
int damageDealt;

// Prototypes
void battlePhase();

int main()
{
	// Defining the character
	cout << "What is your name? ";
	cin >> charName;
	cout << "\nWhat level are you? ";
	cin >> charLevel;
	cout << "\nWhat floor are you on? ";
	cin >> Floor;
	cout << "\nWhat monster do you wish to fight? ";
	cin >> monName;

	// Player Stats
		// charLevel defined earlier
	srand((unsigned)time(0));	
	Hp = (15 + ( charLevel * 5 ));
	Atk = (((rand() % 6) + 0) + (charLevel - 1));
	Def = (((rand() % 4) + 0) + (charLevel - 1));

	// Monster Stats
	monLevel = (1/2) * Floor + (charLevel - 2);
	if ( monLevel < 0 )
		{
			monLevel == 1;
		}
	
	monHp = (10 + (monLevel * 2));
	monAtk = (((rand() % 4) + 1) + (monLevel - 1));
	monDef = (((rand() % 3) + 1) + (monLevel - 1));

	while (Hp > 0 ||  monHp > 0)
	{
		battlePhase();
	}
	
	return 0;

}
	
void battlePhase()
{	 
	// Actual Battle
	cout << endl << "\nWhat do you want to do? ";
	cin >> action;

	if ( action == "Attack" || action == "attack" )
		{
			cout << endl << charName << " attacked " << monName << " for " << Atk << endl;
			cout << monName << " defended " << charName << " for " << monDef << endl;
			damageDealt = Atk - monDef;
			
				if ( damageDealt < 0 )
					{
						damageDealt = 0;
					}
				
				if ( damageDealt == 0 )
					{
						cout << "\nNo damage!";
					}
				
				if ( damageDealt > 0 )
					{
						cout << monName << " took " << damageDealt << " damage!";
						monHp = monHp - damageDealt;
					}
		}
	else
		{
			cout << "\nThat isn't an action! Try typing 'attack'.";
			battlePhase();
		}
	
}

•

Join Date: Oct 2009

Posts: 54

Reputation: Fbody is an unknown quantity at this point

Solved Threads: 6

Fbody

Offline

Junior Poster in Training

Nov 2nd, 2009

If you want the Atk, Def, monAtk, and monDef variables to randomize for each iteration of the loop, you will have to calculate them inside the loop. Currently, you calculate them before the loop starts and you have no way to get back to them to re-calculate them.

Last edited by Fbody; Nov 2nd, 2009 at 5:32 pm.

•

Join Date: Nov 2009

Posts: 3

Reputation: djnstuff is an unknown quantity at this point

Solved Threads: 0

djnstuff

Offline

Newbie Poster

Nov 2nd, 2009

Thanks for the feedback... Hm, I tried what you said and moved:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
Atk = (((rand() % 6) + 0) + (charLevel - 1));
Def = (((rand() % 4) + 0) + (charLevel - 1));
 
monAtk = (((rand() % 4) + 1) + (monLevel - 1));
monDef = (((rand() % 3) + 1) + (monLevel - 1));
Atk = (((rand() % 6) + 0) + (charLevel - 1));
Def = (((rand() % 4) + 0) + (charLevel - 1));

monAtk = (((rand() % 4) + 1) + (monLevel - 1));
monDef = (((rand() % 3) + 1) + (monLevel - 1));

Into the if statement to make:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
void battlePhase()
{	 
	// Actual Battle
	cout << endl << "\nWhat do you want to do? ";
	cin >> action;
 
	if ( action == "Attack" || action == "attack" )
		{
			// Randomizing the attack/defend values for player and monster
			Atk = (((rand() % 6) + 0) + (charLevel - 1));
			Def = (((rand() % 4) + 0) + (charLevel - 1));
 
			monAtk = (((rand() % 4) + 1) + (monLevel - 1));
			monDef = (((rand() % 3) + 1) + (monLevel - 1));
 
			cout << endl << charName << " attacked " << monName << " for " << Atk << endl;
			cout << monName << " defended " << charName << " for " << monDef << endl;
			damageDealt = Atk - monDef;
 
				if ( damageDealt < 0 )
					{
						damageDealt = 0;
					}
 
				if ( damageDealt == 0 )
					{
						cout << "\nNo damage!";
					}
 
				if ( damageDealt > 0 )
					{
						cout << monName << " took " << damageDealt << " damage!";
						monHp = monHp - damageDealt;
					}
		}
	else
		{
			cout << "\nThat isn't an action! Try typing 'attack'.";
			battlePhase();
		}
 
}
void battlePhase()
{	 
	// Actual Battle
	cout << endl << "\nWhat do you want to do? ";
	cin >> action;

	if ( action == "Attack" || action == "attack" )
		{
			// Randomizing the attack/defend values for player and monster
			Atk = (((rand() % 6) + 0) + (charLevel - 1));
			Def = (((rand() % 4) + 0) + (charLevel - 1));
			
			monAtk = (((rand() % 4) + 1) + (monLevel - 1));
			monDef = (((rand() % 3) + 1) + (monLevel - 1));

			cout << endl << charName << " attacked " << monName << " for " << Atk << endl;
			cout << monName << " defended " << charName << " for " << monDef << endl;
			damageDealt = Atk - monDef;
			
				if ( damageDealt < 0 )
					{
						damageDealt = 0;
					}
				
				if ( damageDealt == 0 )
					{
						cout << "\nNo damage!";
					}
				
				if ( damageDealt > 0 )
					{
						cout << monName << " took " << damageDealt << " damage!";
						monHp = monHp - damageDealt;
					}
		}
	else
		{
			cout << "\nThat isn't an action! Try typing 'attack'.";
			battlePhase();
		}
	
}

It still doesn't seem to work however, and when I tried this the first result for the monDef was -1... Which shouldn't be able to happen.

I'm rather stumped.

•

Join Date: Oct 2009

Posts: 140

Reputation: restrictment is on a distinguished road

Solved Threads: 10

restrictment restrictment is offline

Offline

Junior Poster

Nov 2nd, 2009

You need to use srand, with integrated time, so the random integer will change(seed).

Example:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
				srand ((unsigned)time(0));
				random3 = (rand() % 4) + 1;
				srand ((unsigned)time(0));
				random3 = (rand() % 4) + 1;

Edit: Also include the header: #include<time.h> if you do not already have it.

Also, if you don't mind, please send me the source code once you are done so I can check it out. I love checking out others' RPGs that they make.

Last edited by restrictment; Nov 2nd, 2009 at 6:24 pm.

•

Join Date: Nov 2009

Posts: 3

Reputation: djnstuff is an unknown quantity at this point

Solved Threads: 0

djnstuff

Offline

Newbie Poster

Nov 3rd, 2009

Thanks, but it still doesn't work correctly... Hmm...

•

Join Date: Oct 2009

Posts: 54

Reputation: Fbody is an unknown quantity at this point

Solved Threads: 6

Fbody

Offline

Junior Poster in Training

Nov 3rd, 2009

I didn't see it before...

In your original post, Line 40, change that line (whatever line number it is now) from the equality comparison operator to the assignment operator.

monLevel == 1;
needs to be:
monLevel = 1;

Last edited by Fbody; Nov 3rd, 2009 at 4:16 pm.

•

Join Date: Nov 2008

Posts: 29

Reputation: Shinedevil is an unknown quantity at this point

Solved Threads: 0

Shinedevil Shinedevil is offline

Offline

Light Poster

Nov 3rd, 2009

Apparently using "goto x;" is some kind of ungodly error that i seem to have picked up. But I have been able to create some really successful programs with it.

You can create different type of modules within your program using goto.

For instance:

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
{
HPSTASIS:
hptot = Health - EnDamage;
Health = hptot;
if (Health <= 0) {cout<<"You have died.";system("PAUSE");goto endgame;}
else if (Health > 0){
goto CHECKPOINT;
}
}
{
HPSTASIS:
hptot = Health - EnDamage;
Health = hptot;
if (Health <= 0) {cout<<"You have died.";system("PAUSE");goto endgame;}
else if (Health > 0){
goto CHECKPOINT;
}
}

In which after each attack against you, it makes sure your health is correct and checks to see if you are dead. In which the game would end, otherwise if you are not dead, it goes to the checkpoint module that continues the battle.

Thread Tools	Search this Thread
Show Printable Version Email this Page

DaniWeb Message

Cancel Changes