Почему переменная int не работает внутри этой функции? - PullRequest
0 голосов
/ 02 июля 2010

Я сделал функцию для создания переменной коров, затем позже я сделал другую функцию, чтобы убить их.Эти две функции являются методами в классе.Когда я использую функцию создания коровы, она работает, и я получаю ценность коров.Однако когда я обращаюсь к функции мясника, значение коров равно нулю.

Это как две разные переменные;Я не думал, что функции могли бы иметь локальные переменные, если бы они были пустыми функциями.

Как я могу это исправить?

#include <string>
using namespace std;

class Cheif
{
    int points;
    string name;

protected:
    int NoodleS;
    int RosemaryS;
    int ParcleyS;
    int PepperS;
    int Field;
    int Water;

public:
    int star; 
    int foodPrep; 
    int ingrPrep;     
    int endOfPrep;
    int money;
    int ingr1;
    int ingr2;
    int ingr3;
    int Noodle;
    int Rosemary;
    int Parcley;
    int Pepper;
    int chickMeat;
    int beef;
    int chickens;
    int cows;

// constructor 
    Cheif()
    {
        money = 1000;
        points = 0;
        star = 10;    
        endOfPrep = 0; 
        ingr1 = 0;
        ingr2 = 0;
        ingr3 = 0;
        Noodle = 0;
        Rosemary = 0;
        Parcley = 0;
        Pepper= 0 ;
        cows = 0;
        chickens = 0;
        beef = 0;
        chickMeat = 0;
    }

// method

//////////////////////////////////////////////////
//                                              //
//            ask user for their name           //
//                                              //
//////////////////////////////////////////////////
    void FindName()
    {

        cout << "what is your name? " << endl;;
        cin >> name;
        cout << endl << " Welcome " << name
         << " let us begin... " <<endl;   


    }

//////////////////////////////////////////////////
//                     END                      //
//////////////////////////////////////////////////

//////////////////////////////////////////////////
//                                              //
//         Buy animal live stock                //
//                                              //
//////////////////////////////////////////////////
    void Buy_Cow()
    {
        if(money > 199)
        {
        money -= 200;
        cows += 1;
        cout <<" you now have " << cows << " cows" << endl;
        }
    }

    void Buy_Chick()
    {
        if(money > 99)
        {
        money -= 200;
        chickens += 1;
        } 
    }

//////////////////////////////////////////////////
//                     END                      //
//////////////////////////////////////////////////


//////////////////////////////////////////////////
//                                              //
//         if user goes to open store           //
//                                              //
//////////////////////////////////////////////////

    void GOTO_Store()
    {
        // while food is not prepared yet
        while(endOfPrep == 0){
        PREPAREMEAL:
        // show menu
        cout << "<1> Make Food \n"
             << "<2> Collect Ingridents \n"        
             << "<3> Butcher Animal \n"
             << "<4> Go Back... \n" << endl;


        // create variable to hold choice
        string OS_Choice;

        cin >> OS_Choice;   

        /////////// if user decides to "make food" /////////////
        if (OS_Choice == "1")
        {
            if(foodPrep == 1)
            {
            cout << "you've already prepared the meal..." << endl;         
            }else{          
            if(ingr1 <= 0 ||ingr2 <= 0 || ingr3 <= 0)
            {
                goto PREPAREMEAL;
            }else{          
                cout << "your using" << ingr1 << " " << ingr2<< " " << ingr3 << endl;
            } // end of ingredient check

            cout << " and how shall this mean be prepared? " << endl;

            int prepMethod;

            cout << "<1> Baked \n "
                 << "<2> boiled \n "
                 << "<3> Fried \n "
                 << "<4> mixed \n ";

            cin >> prepMethod;

            cout << " And what kind of sides would you like to add? " << endl;

            int sideChoice;

            cout << "<1> bread Roll \n " 
                 << "<2> Rice \n "
                 << "<3> Beans \n" << endl;

            cin >> sideChoice;

            foodPrep = 1;
            }// end of food choice 

            //begin food compare. 
            /////////// if user decides to get ingrediants /////////////
        }else if(OS_Choice == "2"){
            if (ingrPrep == 1){
            cout << " you have already collected the ingredients " << endl;  

            }else{     
            cout << "what 3 ingridents will you use? " << endl;
            cin >> ingr1;
            cin >> ingr2;
            cin >> ingr3;
            }// end of ingrident prep
            /////////// if user decides to get ingrediants /////////////
        }else if(OS_Choice == "3")
        {
            cout << " you have " << cows << " cows \n "
             << " you have " << chickens << " Chickens \n ";

            cout << "what would you like to butcher? " << endl;
            cout << "<1> Cows    : " << cows << endl;
            cout << "<2> Chicken : " << chickens << endl;
            int B_Choice;

            cin >> B_Choice;

            if(B_Choice == 1){
            if(cows == 0){
                cout << " sorry you dont have any cows " << endl;
            }else{
                cows = cows - 1;
                beef = beef + 5;
                cout << " you now have " << beef << "peices of cow meat" << endl;
                ingrPrep = 1;
            }//end of cow check
            }else if(B_Choice == 2){
            if(chickens == 0){
                cout << " sorry you dont have any chickens " << endl;
            }else{
                chickens = chickens - 1;
                chickMeat = chickMeat + 2;
                cout << " you now have " << chickMeat << "peices of Chicken meat" << endl;
                ingrPrep = 1;
            } // end chicken Check
            }else {
            cout << "invalid Choice" << endl;
            }// end of b choice
        }else if(OS_Choice == "4") {
            endOfPrep = 1;      
            foodPrep = 0;
            ingrPrep = 0;
            ingr1 = 0;
            ingr2 = 0;
            ingr3 = 0;
        }// end of ingr prep. 
        }//end of while loop  
    }
//////////////////////////////////////////////////
//                     END                      //
//////////////////////////////////////////////////
};

Ответы [ 2 ]

1 голос
/ 02 июля 2010

Нет ничего плохого в построении вашего класса, который мог бы вызвать проблему, которую вы описываете.(Суть размотки заключается в том, что обычно не рекомендуется помещать реализации ваших методов непосредственно в объявление класса, но это не является причиной вашей проблемы.)

На самом деле, если я добавлю простоеmain () до конца кода, который вы опубликовали:

int main() {

   Cheif c;

   c.Buy_Cow();
   c.GOTO_Store();
}

и скомпилируйте и запустите его, он даст ожидаемые результаты (значение коров = 1).

Итак, это говоритпроблема не в этом классе, а в том, как вы вызываете его из остальной части вашей программы.

Чтобы отследить это, применяются обычные методы отладки.Например:

  • попытаться определить конкретную последовательность событий, вызывающих проблему
  • использовать отладчик и / или операторы print, чтобы выяснить, где что-то идет не так
1 голос
/ 02 июля 2010

Независимо от того, имеют ли методы возвращаемый тип void или нет, не имеет значения;если они являются частью одного класса, они будут использовать переменные экземпляра класса.

Но ваши методы не похожи на методы;нет имени класса перед их именами.Они определены в объявлении класса?

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...