DaniWeb Forum Index > Software Development > C++

My fighting Game

Please support our C++ advertiser: Intel Parallel Studio Home

•

Join Date: Apr 2007

Posts: 8

Reputation: cjwenigma is an unknown quantity at this point

Solved Threads: 0

cjwenigma

Offline

Newbie Poster

My fighting Game

Sep 19th, 2007

OK so here is a fighter game I made..except my fighters power won't decrease and I can't get my fighters names to show up....*sigh*... any idea.. My functions could be wrong.. but I always thought it went above int main()..

 Help with Code Tags 
 C++ Syntax (Toggle Plain Text) 
<span class="ad_notxt"><code class="inlinecode">
#include<iostream>
using namespace std;
int fighterOne (char fighter1Attack){
string fighter1Name;
// char fighter1Attack; 
int fighter2Power=1000;
string fighter2Name;
cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
cin >> fighter1Attack;
if (fighter1Attack == 'P' || fighter1Attack == 'p') {
fighter2Power -= 50;
cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
}
else if (fighter1Attack == 'K' || fighter1Attack == 'k') {
fighter2Power -= 100;
cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
}
else {
cout << "Invalid attack, you lose a turn!" << endl;
}
}
int fighterTwo (char fighter2Attack) {
string fighter2Name;
//char fighter2Attack; 
int fighter1Power=1000;
string fighter1Name;
cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
cin >> fighter2Attack;
if (fighter2Attack == 'P' || fighter2Attack == 'p') {
fighter1Power -= 50;
}
else if (fighter2Attack == 'K' || fighter2Attack == 'k') {
fighter1Power -= 100;
cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
}
else {
cout << "Invalid attack, you lose a turn!" << endl;
}
}
int main() {
string fighter1Name = "Greg"; 
string fighter2Name = "Casey"; 
int fighter1Power = 1000; 
int fighter2Power = 1000; 
char fighter1Attack;
char fighter2Attack;
 
do {
fighterOne (fighter1Attack);
fighterTwo (fighter2Attack);
}while((fighter1Power > 0) && (fighter2Power > 0));
if (fighter1Power > fighter2Power) {
cout << "Fighter 1 is the winner!" << endl;
}
else if (fighter2Power > fighter1Power) {
cout << "Fighter 2 is the winner!" << endl;
}
else {
cout << "The game ended in a draw!" << endl;
}
return 0;
}
 
</code></span>
<span class="ad_notxt"><code class="inlinecode">
#include<iostream>
using namespace std;
int fighterOne (char fighter1Attack){
string fighter1Name;
// char fighter1Attack; 
int fighter2Power=1000;
string fighter2Name;
cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
cin >> fighter1Attack;
if (fighter1Attack == 'P' || fighter1Attack == 'p') {
fighter2Power -= 50;
cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
}
else if (fighter1Attack == 'K' || fighter1Attack == 'k') {
fighter2Power -= 100;
cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
}
else {
cout << "Invalid attack, you lose a turn!" << endl;
}
}
int fighterTwo (char fighter2Attack) {
string fighter2Name;
//char fighter2Attack; 
int fighter1Power=1000;
string fighter1Name;
cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
cin >> fighter2Attack;
if (fighter2Attack == 'P' || fighter2Attack == 'p') {
fighter1Power -= 50;
}
else if (fighter2Attack == 'K' || fighter2Attack == 'k') {
fighter1Power -= 100;
cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
}
else {
cout << "Invalid attack, you lose a turn!" << endl;
}
}
int main() {
string fighter1Name = "Greg"; 
string fighter2Name = "Casey"; 
int fighter1Power = 1000; 
int fighter2Power = 1000; 
char fighter1Attack;
char fighter2Attack;
 
do {
fighterOne (fighter1Attack);
fighterTwo (fighter2Attack);
}while((fighter1Power > 0) && (fighter2Power > 0));
if (fighter1Power > fighter2Power) {
cout << "Fighter 1 is the winner!" << endl;
}
else if (fighter2Power > fighter1Power) {
cout << "Fighter 2 is the winner!" << endl;
}
else {
cout << "The game ended in a draw!" << endl;
}
return 0;
}
 
</code></span>

•

Join Date: May 2006

Posts: 3,121

Reputation: WaltP has much to be proud of

Solved Threads: 282

WaltP

Offline

Posting Sensei

Re: My fighting Game

Sep 20th, 2007

This information will help us help you if you follow the suggestions therein.

The 3 Laws of the Procrastination Society:
1) Never do today that which can be put off until tomorrow
2) Tomorrow never comes

•

Join Date: Dec 2005

Posts: 5,850

Reputation: Salem has a reputation beyond repute

Solved Threads: 751

Salem

Offline

Void main'ers are DOOMed

Re: My fighting Game

Sep 20th, 2007

It looks as badly indented here as it does on other forums.

•

Join Date: Sep 2007

Posts: 1

Reputation: kim165 is an unknown quantity at this point

Solved Threads: 0

kim165

Offline

Newbie Poster

Re: My fighting Game

Sep 20th, 2007

1. when using string you also need

Help with Code Tags

C++ Syntax (Toggle Plain Text)

#include <string>

2. for char values try char name = ' ';

Help with Code Tags

C++ Syntax (Toggle Plain Text)

example char fighter1Attack = ' ';

•

Join Date: Aug 2005

Posts: 5,273

Reputation: iamthwee is a splendid one to behold

Solved Threads: 378

iamthwee

Offline

Posting Expert

Re: My fighting Game

Sep 20th, 2007

>cin >> fighter2Attack;
if (fighter2Attack == 'P' || fighter2Attack == 'p')

Seeing as fighter2attack is a string that won't work.

*Voted best profile in the world*

•

Join Date: Nov 2006

Posts: 202

Reputation: n.aggel is an unknown quantity at this point

Solved Threads: 11

n.aggel

Offline

Posting Whiz in Training

Re: My fighting Game

Sep 22nd, 2007

Here is a better indented code {i used astyle to do this...}

 Help with Code Tags 
 c++ Syntax (Toggle Plain Text) 
#include<iostream>
 
using namespace std;
 
int fighterOne (char fighter1Attack)
{
 
  string fighter1Name;
// char fighter1Attack;
  int fighter2Power=1000;
  string fighter2Name;
 
  cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter1Attack;
 
  if (fighter1Attack == 'P' || fighter1Attack == 'p')
    {
      fighter2Power -= 50;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else if (fighter1Attack == 'K' || fighter1Attack == 'k')
    {
      fighter2Power -= 100;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
 
}
 
 
int fighterTwo (char fighter2Attack)
{
 
  string fighter2Name;
//char fighter2Attack;
  int fighter1Power=1000;
  string fighter1Name;
 
  cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter2Attack;
 
  if (fighter2Attack == 'P' || fighter2Attack == 'p')
    {
      fighter1Power -= 50;
    }
  else if (fighter2Attack == 'K' || fighter2Attack == 'k')
    {
      fighter1Power -= 100;
      cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
}
 
 
int main()
{
 
 
  string fighter1Name = "Greg";
  string fighter2Name = "Casey";
  int fighter1Power = 1000;
  int fighter2Power = 1000;
  char fighter1Attack;
  char fighter2Attack;
 
  do
    {
      fighterOne (fighter1Attack);
      fighterTwo (fighter2Attack);
    }
  while ((fighter1Power > 0) && (fighter2Power > 0));
  if (fighter1Power > fighter2Power)
    {
      cout << "Fighter 1 is the winner!" << endl;
    }
  else if (fighter2Power > fighter1Power)
    {
      cout << "Fighter 2 is the winner!" << endl;
    }
  else
    {
      cout << "The game ended in a draw!" << endl;
    }
 
 
  return 0;
}
#include<iostream>

using namespace std;

int fighterOne (char fighter1Attack)
{

  string fighter1Name;
// char fighter1Attack;
  int fighter2Power=1000;
  string fighter2Name;

  cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter1Attack;

  if (fighter1Attack == 'P' || fighter1Attack == 'p')
    {
      fighter2Power -= 50;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else if (fighter1Attack == 'K' || fighter1Attack == 'k')
    {
      fighter2Power -= 100;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }


}


int fighterTwo (char fighter2Attack)
{

  string fighter2Name;
//char fighter2Attack;
  int fighter1Power=1000;
  string fighter1Name;

  cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter2Attack;

  if (fighter2Attack == 'P' || fighter2Attack == 'p')
    {
      fighter1Power -= 50;
    }
  else if (fighter2Attack == 'K' || fighter2Attack == 'k')
    {
      fighter1Power -= 100;
      cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }

}


int main()
{


  string fighter1Name = "Greg";
  string fighter2Name = "Casey";
  int fighter1Power = 1000;
  int fighter2Power = 1000;
  char fighter1Attack;
  char fighter2Attack;

  do
    {
      fighterOne (fighter1Attack);
      fighterTwo (fighter2Attack);
    }
  while ((fighter1Power > 0) && (fighter2Power > 0));
  if (fighter1Power > fighter2Power)
    {
      cout << "Fighter 1 is the winner!" << endl;
    }
  else if (fighter2Power > fighter1Power)
    {
      cout << "Fighter 2 is the winner!" << endl;
    }
  else
    {
      cout << "The game ended in a draw!" << endl;
    }


  return 0;
}

Two roads diverged in a wood, and I— I took the one less traveled by, and that has made all the difference.

by Robert Frost the "The Road Not Taken"

•

Join Date: Nov 2006

Posts: 202

Reputation: n.aggel is an unknown quantity at this point

Solved Threads: 11

n.aggel

Offline

Posting Whiz in Training

Re: My fighting Game

Sep 22nd, 2007

here is something that works:

 Help with Code Tags 
 c++ Syntax (Toggle Plain Text) 
#include <iostream>
#include <string>
 
using namespace std;
 
/*although i didn't understand why you wanted to pass string fighter1Attack as an argument, i passed it as a reference*/
int fighterOne (string& fighter1Attack)
{
 
  string fighter1Name = "Subzero"; /*i added the names*/
  // char fighter1Attack;
  int fighter2Power=1000;
  string fighter2Name;
 
  cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter1Attack;
 
/*here i made the comparisons from a_string=='char' to a_string=="another_string"*/
  if (fighter1Attack == "P" || fighter1Attack == "p")
    {
      fighter2Power -= 50;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else if (fighter1Attack == "K" || fighter1Attack == "k")
    {
      fighter2Power -= 100;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
 
}
 
 
int fighterTwo (string& fighter2Attack)
{
 
  string fighter2Name = "Scorpio";
  //char fighter2Attack;
  int fighter1Power=1000;
  string fighter1Name;
 
  cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter2Attack;
 
  if (fighter2Attack == "P" || fighter2Attack == "p")
    {
      fighter1Power -= 50;
    }
  else if (fighter2Attack == "K" || fighter2Attack == "k")
    {
      fighter1Power -= 100;
      cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
}
 
 
int main()
{
 
 
  string fighter1Name = "Greg"; /*you don't pass the names to the function, so this is why you see subzero and scorpion when the program is running! Hint:: try to this like i did the string fighterXattack, if you can't i 'll help you.*/
  string fighter2Name = "Casey";
  int fighter1Power = 1000;
  int fighter2Power = 1000;
  string fighter1Attack;   /*i changed this part*/
  string fighter2Attack;   /*i changed this part */
 
  do
    {
      fighterOne (fighter1Attack);
      fighterTwo (fighter2Attack);
    }
  while ((fighter1Power > 0) && (fighter2Power > 0));
  if (fighter1Power > fighter2Power)
    {
      cout << "Fighter 1 is the winner!" << endl;
    }
  else if (fighter2Power > fighter1Power)
    {
      cout << "Fighter 2 is the winner!" << endl;
    }
  else
    {
      cout << "The game ended in a draw!" << endl;
    }
 
 
  return 0;
}
#include <iostream>
#include <string>
 
using namespace std;
 
/*although i didn't understand why you wanted to pass string fighter1Attack as an argument, i passed it as a reference*/
int fighterOne (string& fighter1Attack)
{
 
  string fighter1Name = "Subzero"; /*i added the names*/
  // char fighter1Attack;
  int fighter2Power=1000;
  string fighter2Name;
 
  cout << fighter1Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter1Attack;
 
/*here i made the comparisons from a_string=='char' to a_string=="another_string"*/
  if (fighter1Attack == "P" || fighter1Attack == "p")
    {
      fighter2Power -= 50;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else if (fighter1Attack == "K" || fighter1Attack == "k")
    {
      fighter2Power -= 100;
      cout << fighter2Name << "'s health reduced to " << fighter2Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
 
}
 
 
int fighterTwo (string& fighter2Attack)
{
 
  string fighter2Name = "Scorpio";
  //char fighter2Attack;
  int fighter1Power=1000;
  string fighter1Name;
 
  cout << fighter2Name << " please choose an attack (P) = Punch, (K) = Kick => ";
  cin >> fighter2Attack;
 
  if (fighter2Attack == "P" || fighter2Attack == "p")
    {
      fighter1Power -= 50;
    }
  else if (fighter2Attack == "K" || fighter2Attack == "k")
    {
      fighter1Power -= 100;
      cout << fighter1Name << "'s health reduced to " << fighter1Power << endl;
    }
  else
    {
      cout << "Invalid attack, you lose a turn!" << endl;
    }
 
}
 
 
int main()
{
 
 
  string fighter1Name = "Greg"; /*you don't pass the names to the function, so this is why you see subzero and scorpion when the program is running! Hint:: try to this like i did the string fighterXattack, if you can't i 'll help you.*/
  string fighter2Name = "Casey";
  int fighter1Power = 1000;
  int fighter2Power = 1000;
  string fighter1Attack;   /*i changed this part*/
  string fighter2Attack;   /*i changed this part */
 
  do
    {
      fighterOne (fighter1Attack);
      fighterTwo (fighter2Attack);
    }
  while ((fighter1Power > 0) && (fighter2Power > 0));
  if (fighter1Power > fighter2Power)
    {
      cout << "Fighter 1 is the winner!" << endl;
    }
  else if (fighter2Power > fighter1Power)
    {
      cout << "Fighter 2 is the winner!" << endl;
    }
  else
    {
      cout << "The game ended in a draw!" << endl;
    }
 
 
  return 0;
}

Basically the error that i foud was that you were using c++ string facilities with chars...this is why nothing worked....

Check the comments i made on your code.

Also you are starting object oriented programming maybe you should check about encapsulation
1 and 2

Another thing is we don't you implement the fighters as one class,and just instantiate 2 objects fighter1 and fighter2

Last but not least maybe you should check for a tutorial on strings:: http://www.cppreference.com/cppstring/index.html and http://www.cprogramming.com/tutorial/string.html

PS::it is the first time i help someone else, in this fourm, and i am relative noob so i don't promise that this the best thing you could do!

Two roads diverged in a wood, and I— I took the one less traveled by, and that has made all the difference.

by Robert Frost the "The Road Not Taken"