Hello, this is kinda my first program. I would like to know how to clean the screen so only this code would stay in the console

cout<<"Welcome to calculator!\n\n";
    cout<<"[*] - a * b\n[/] - a / b\n[-] - a - b\n[+] - a + b\n\n";

I m also having trouble that when i make wrong ccalculation suchs as 5+5+, the program is looping the 5 + 5 calculation and ignores any input.

#include <iostream>

using namespace std;

int main ()
{
    double a;
    double b;
    char oper;
    int exit = 0;

    cout<<"Welcome to calculator!\n\n";
    cout<<"[*] - a * b\n[/] - a / b\n[-] - a - b\n[+] - a + b\n\n";

    do {

    cout<<"Calculator is now ready to calculate\n\n";

    cin>>a >>oper >>b;
    cout<<endl;

    if (oper == '+')
        {
            cout<<a<<" + "<<b<<" = "<<a+b<<endl<<endl;
            cout<<"Press 1 to exit or 0 to continue: ";
            cin>>exit;
            cout<<endl;
        }

    else if (oper == '-')
        {
            cout<<a-b;
        }

    else if (oper == '*')
        {
            cout<<a*b;
        }

    else if (oper == '/')
        {
            cout<<a/b;
        }

    else
        {
            cerr<<"Not a valid operation. Try again\n\n";
        }

    }while (exit < 1);

    cout<<"Thank you for using...Press any key.... \n";

    return 0;
}

To clear your screen you can use system("CLS") - keep in mind this makes your program less portable though.

The reason you continuously loop when you enter bad input is because you're trying to accept a number, but are receiving a char, and your input buffer is overflowing.

Try flushing the input stream after you get the data you want.

Also, I would recommend using a switch rather than multiple IFs.

Edited 5 Years Ago by Duki: n/a

Comments
no reason to downvote, you can just say that in the command below him WaltP!
He didn't say he was on a Windows system.

The reason you continuously loop when you enter bad input is because you're trying to accept a number, but are receiving a char, and your input buffer is overflowing.

It's not because of an input buffer overflow. When you try to read a non-digit into an integer, the read fails. The input buffer remains the same. Next time you try to read, since that character is still there, the input fails again. And again. And ... you get the idea.

1) Test your read to see if it was successful. Don't ask How? Look it up. It's part of cin error checking.
2) If it fails, clear the input buffer as Duki's link suggests. Be sure to process the error in your code. Don't just go on your merry way and execute the code as if nothing happened.

Also, I would recommend using a switch rather than multiple IFs.

So would I, but only if you've learned them.

Comments
Thanks for the correction. :)

I have learned the switch but how could i use it in this case? My variable "oper" gets the operator from user input and uses it to detect what operation to do. But if i m correct then the switch need numerical variable.

And isnt there any other way to clear screen other than involving system in it. Cause i would like my program to be as independent as possible.

Thanks for all answers

EDIT: Nvm about the switch, i was wrong obviously. Did some researching and found useful info :)

My code looks like this now

#include <iostream>

using namespace std;

int main ()
{
    double a;
    double b;
    char oper;
    int exit = 0;

    cout<<"Welcome to calculator!\n\n";
    cout<<"[*] - a * b\n[/] - a / b\n[-] - a - b\n[+] - a + b\n\n";

    do{
    cout<<"Calculator is now ready to calculate\n\n";
    cin>>a >> oper >> b;

        switch (oper)
            {
                case '+':
                cout<<a<<" + "<<b<<" = "<<(a+b)<<endl;
                cout<<"Press 1 to exit\n";
                cin>>exit;
                break;

                case '-':
                cout<<a<<" - "<<b<<" = "<<(a-b)<<endl;
                cout<<"Press 1 to exit\n";
                cin>>exit;
                break;

                case '/':
                cout<<a<<" / "<<b<<" = "<<(a/b)<<endl;
                cout<<"Press 1 to exit\n";
                cin>>exit;
                break;

                case '*':
                cout<<a<<" * "<<b<<" = "<<(a*b)<<endl;
                cout<<"Press 1 to exit\n";
                cin>>exit;
                break;
            }
        }while(exit != 1);


    cout<<"Thank you for using...Press any key.... \n";

    return 0;


}

Should i make function for exiting rather than typing it in all the time (Hardcoding is the expression i think ?) And i m still working on the loop problem.

Edited 5 Years Ago by Martin C++: n/a

And isnt there any other way to clear screen other than involving system in it. Cause i would like my program to be as independent as possible.

Output '\n' 25 to 50 times. Depending on how many lines in your console.

Should i make function for exiting rather than typing it in all the time (Hardcoding is the expression i think ?)

No. Just remove the lines

cout<<"Press 1 to exit\n";
    cin>>exit;

from your switch and put them once at the bottom of the loop.

Do '+' signs work as SPACES in user input. For instance when i have code

cout<<"Enter two numbers ";
cin>>a >>b

and then when i run it and am asked to enter two numbers then SPACE between words works like enter.

Input to console: Hello World

Then the HEllo would be assigned to variable a and World to variable b. So the question is do operators like +,-,/,* have the same effect ?

This article has been dead for over six months. Start a new discussion instead.