A Bit Of A Problem Here...

I have a function in which the user can type in a number:

int Input()
{
int b;
cin >> b;
return b;
}

If the user inputs 1, this function is run:

void RunStats()
{
cout << "\nYour HP: ";
cout << mos.GetHP() << endl;
cout << "Your Money: ";
cout << sat.GetMoney() << " GP " << endl;
cout << "Tired Level: ";
cout << sat.GetTired() << " Points" << endl;
int Input(); //This returns to the function where you input a number
}

I want it to show your stats, and then return to the input function, where you can type your commands once again. However, the program ends once you return to the input function. It doesn't allow you to input your selection again. How can I fix this problem?

'int Input();' is a declaration and should be used only once
to tell the computer that a function with this name exists,
if you want to call the function use 'Input();'.

That sorta helps, but after I go back to the Input() function and type in something again, the program ends.

so why not encase the program in a while loop?


maybe like while(input != 999)

How about the entire code?

In your main function, you should be using a loop like so:

Code:

---

while (choice!=2)
{
  cout<<"1. Run stats"<<endl;
  cout<<"2. Exit"<<endl;
  cin>>choice;
  if (choice==1)
    RunStats();
}

---

The way you have it, it calls input somewhere else (I assume main) runs the RunStats function which calls input again, but after it does that, then RunStats is done and it goes back to where it left off in main - which is probably at the end of the file. Make sense?

Normally, it would, but my code is a bit strange. I'm getting a run-time error with that code, and no matter what I type in, it runs your stats. It might take some time for me to locate the source of the problem:cool:

If you post your code whe can help too

HEADER:
// This class sets Sebba's stats.

class Sebba
{
public:
int GetHP() ;
void SetHP(int x);
private:
int HP;
};
void Sebba::SetHP(int x)
{
HP = x;
}
int Sebba::GetHP ()
{
return HP;
}

//This class sets the Stats of your money, and how tired you are.

class Stats
{
public:
void SetMoney(int y);
int GetMoney();
void SetTired(int z);
int GetTired();
private:
int Money;
int Tired;
};
void Stats::SetMoney(int y)
{
Money = y;
}
int Stats::GetMoney()
{
return Money;
}
void Stats::SetTired(int z)
{
Tired = z;
}
int Stats::GetTired()
{
return Tired;
}
Sebba mos;
Stats sat;


AND THE C++ SOURCE CODE:

#include <iostream.h>
#include "header.h"
int GAExp();
void RunStats()
{
cout << "\nYour HP: ";
cout << mos.GetHP() << endl;
cout << "Your Money: ";
cout << sat.GetMoney() << " GP " << endl;
cout << "Tired Level: ";
cout << sat.GetTired() << " Points" << endl;
GAExp();
}

//Run Exp is the function that runs the 'Exploration' Function

void RunExp()
{
cout << "\nWow. What a beautiful place!";
//This is another exploration function:
GAExp();
}

//Main Code:

int main()
{
cout << "\n\nSetting Statistics...\n";
//These are Hit Point, Money, and Tired Values:
int x = 5;
int y = 100;
int z = 10;
//These set Hit Point, Money, and Tired Values:
sat.SetMoney(y);
mos.SetHP(x);
sat.SetTired(z);

//Information:

cout << "\nYour HP has been set to " << mos.GetHP();
cout << "\nYour money has been set to " << sat.GetMoney() << " GP";
cout << "\nYour tired level has been set to " << sat.GetTired();
cout << "\n\nDone Loading Stats";

//These is your choices. Enjoy

cout << "\nWhat do you want to do?\n1.)ViewStats\n2.)Explore\n3.)Use Item\n";

//This is your imput command, and tells the program where to go:

int a = GAExp();
while (a == 1)
{
cout<<"1. Run stats"<<endl;
cout<<"2. Exit"<<endl;
RunStats();
}

//If you type in 1, You'll open up your menu.

return 0;
}

//This is currently a sample function:

int GAExp()
{
int b;
cin >> b;
return b;
}

Code:

---

Sebba.h

class Sebba
{
 public:
    Sebba() : HP(0) {}
    Sebba(int x) : HP(x) {}
    int GetHP() const {return HP;}
    void SetHP(int x) { HP = x;}
 private:
    int HP;
};

---

Code:

---

Stats.h

class Stats
{
 public:
    Stats() : Money(0),Tired(0) {}
    Stats(int money,int tired) : Money(money),Tired(tired) {}
    void SetMoney(int y) { Money = y;}
    int GetMoney()const { return Money;}
    void SetTired(int z) { Tired=z;}
    int GetTired()const { return Tired;}
 private:
    int Money;
    int Tired;
};

---

Code:

---

main.cpp

#include<iostream>
#include "Stats.h"
#include "Sebba.h"
using namespace std;

int GAExp()
{
    int b;
    cin >> b;
    return b;
}

void RunStats()
{
    cout << "\nYour HP: ";
    cout << mos.GetHP() << endl;
    cout << "Your Money: ";
    cout << sat.GetMoney() << " GP " << endl;
    cout << "Tired Level: ";
    cout << sat.GetTired() << " Points" << endl;
    GAExp();
}



int main()
{
    cout << "\n\nSetting Statistics...\n";
    Sebba mos(5);
    Stats sat(100,10);
    cout << "\nYour HP has been set to " << mos.GetHP();
    cout << "\nYour money has been set to " << sat.GetMoney() << " GP";
    cout << "\nYour tired level has been set to " << sat.GetTired();
    cout << "\n\nDone Loading Stats";
    bool exit = false;
    while (1) //infinite loop
    {
        cout << "\nWhat do you want to do?\n1.)ViewStats\n2.)Explore\n3.)Use Item\n4)Exit\n";
        int input = GAExp();
        switch (input)
        {
            case 1:  RunStats();
                          break;
            case 2:  // do whatever to explore
                          break;
            case 3:  // do whatever to use item
                          break;
            case 4:  exit = true;
                          break;
            default : cout << "Not valid option"<<endl;
        }
        if (exit) break;
    }
return 0;
}

---

Heres one way of doing what you wanted. Code cleaned up a little too.

Quote:

---

Originally posted by LordVirusXXP
//These are Hit Point, Money, and Tired Values:
int x = 5;
int y = 100;
int z = 10;

---

Then why don't you call them hp, money, and tired? You want to get into the habit of using descriptive identifiers. x, y, and z generally aren't too descriptive.