If else statement

[r0ry]

Hello just started learning and I don't understand what I'm doing wrong here , so here is the problem, when I input the string and choose 1 it continues but then when I want the program to exit with 2 I have to input it couple of times why is that happening?

#include <iostream>
#include <string>
using namespace std;

class nameClass{
public:
    void setName(string x){
        cout << "Enter a name :";
        cin >> x;
        name = x;


    }
    string getName(){
        return name;
    }
private:
    string name;
};

int rep(){

    nameClass object;
    object.setName("");
    cout << "Hello " << object.getName() << endl;
    cout << "Press 1 to continue, or 2 to exit!\n";
    int b;
    cin >> b;
    if (b==1){
        rep();
    }else if (b==2){
        return 1;
    }
}

int main()
{
    while (rep()!=true){
        rep();

    }
    return 0;
}

[Akshit]

try debugging on your own for these types.

But anyways here is the problem

in main()
firstly rep is called in while (rep()!=true)
then you enter name and then 1
rep is called again
this time you enter name and then 2
it returns 1 and get back to previous rep which ends without any return statement
so 0 is returned.
Hence we enter in while loop where rep is called again.It will be called once more while cond. is checked again

Now if you press 2 after name then 1 is returned and condition get false and it exits else it goes again

[jyU4izwU]

#include <iostream>
#include <conio>
using namespace std;

int main()
{
  clrscr();
  int name,ans,end;
  cout << "\n Enter Name:";
  cin >> name;
  cout << "\n Hi " << name;
  cout << "\n Do Want To Stay <Y/N>";
  cin >> ans;
  if ( ans == Y )
  {
    cout << "\n yay Your Staying!";
    // rest of your code or you can make a loop
  }
  else
  {
    cout << "\n Bye...";
    cin >> end;
  }
}

[r0ry]

I see , but then again i did another exercise ,and I don't see any difference in the way it works but it does exactly what I wanted it to

#include <iostream>

using namespace std;

int func(){
    cout << "Compare some numbers \n";
    int a;
    cout << "Enter a number";
    cin >> a;
    int b;
    cout << "Enter a second number \n";
    cin >> b;

        if (a==b){
            cout <<  a <<" is equal to " << b;
        }else if (a>b){
            cout << a << " is bigger than " << b ;
        }else if (a<b){
            cout << a << " is less than " << b;
        }

    cout << endl;
    cout << "Go Again? Yes=1 No=2\n";
    int e;
    cin >> e;

        if (e==1){
            func();
        }else if(e==2){
            return 1;
        }

}

int main()
{
    while (func()!=true){
        func();

    }
    return 0;
}

[Chervil]

That example looks a bit odd as well. Function func() has only one return statement, return 1; so it must always return a logical true result.

Because of that, this code:

    while (func()!=true){
        func();
    }

will call the function func(), the result will be true, so the while () condition is false and the body of the loop is never executed.

Therefore, main() could be simplified to just:

int main()
{
    func();

    return 0;
}

As for the option to go again, it's usually better to use a loop rather than having the function call itself. Notice the boolean variable again and the while loop.

void func()
{
    bool again = true;

    while (again)
    {
        cout << "Compare some numbers \n";
        int a;
        cout << "Enter a number        ";
        cin >> a;
        int b;
        cout << "Enter a second number ";
        cin >> b;

            if (a==b)
                cout <<  a <<" is equal to " << b;
            else if (a>b)
                cout << a << " is bigger than " << b ;
            else
                cout << a << " is less than " << b;

        cout << endl;

        cout << "Go Again? Yes=1 No=2\n";
        int e;
        cin >> e;

        if (e!=1)
            again = false;
    }
}

alternatively,

void func()
{
    char again = '1';
    int a, b;

    while (again == '1')
    {
        cout << "Compare some numbers \n";
        cout << "Enter a number        ";
        cin >> a;
        cout << "Enter a second number ";
        cin >> b;

            if (a==b)
                cout <<  a <<" is equal to " << b;
            else if (a>b)
                cout << a << " is bigger than " << b ;
            else
                cout << a << " is less than " << b;

        cout << endl;

        cout << "Go Again? Yes=1 No=2\n";
        cin >> again;
    }
}

[r0ry]

Thanks for the quick answers , helped a lot :)