Calculator

[Dee Hood]

I've looked through the history and haven't seen this particular problem. here is the code

#include <iostream>
#include <iomanip>

using namespace std; 

int main ()
{
	double num1;
	double num2;
	char op;

	cout << fixed << showpoint;
	cout << setprecision (2);

	cout << "Enter first number: ";
	cin >> num1;
	cout << endl;
	
	cout << "Enter operator: ";
	cin >> op;
	cout << endl;

	cout << "Enter second number: ";
	cin >> num2;
	cout << endl;

	cout << num1 << op << num2;

	switch (op)
	{
	case '+': 
			cout << "=" << num1 + num2 << endl;
			break;
	case '-':
			cout << "=" << num1 - num2 << endl;
			break;
	case '*': 
			cout << "=" << num1 * num2 << endl;
		break;
	case '/':
			cout << "=" << num1 / num2 << endl;
	if (num2 == 0)
	cout << "ERROR" << endl;
	cout << "Cannot divide by 0" << endl;
	}
	system ("Pause");

	return 0;
}

My problem being whenever I use the calculator to divide by 0 instead of it 'num1 / num2 = ERROR' and then next line "Cannot divide by 0" I get
'num1 / num2 =1.#J'
Any help or push in the right direction would be greatly appreciated, Thanks!

[CJC0117]

You have it divide by zero first, and THEN it checks to see if number 2 is zero. try this instead:

#include <iostream>
#include <iomanip>

using namespace std; 

int main ()
{
	double num1;
	double num2;
	char op;

	cout << fixed << showpoint;
	cout << setprecision (2);

	cout << "Enter first number: ";
	cin >> num1;
	cout << endl;
	
	cout << "Enter operator: ";
	cin >> op;
	cout << endl;

	cout << "Enter second number: ";
	cin >> num2;
	cout << endl;

	cout << num1 << op << num2;

	switch (op)
	{
	case '+': 
			cout << "=" << num1 + num2 << endl;
			break;
	case '-':
			cout << "=" << num1 - num2 << endl;
			break;
	case '*': 
			cout << "=" << num1 * num2 << endl;
		break;
	case '/':
			if (num2 == 0)
                        {
	                    cout << "=ERROR" << endl;
	                    cout << "Cannot divide by 0" << endl;
                        }
                       else
                            cout << "=" << num1 / num2 << endl;
                       break;
	}
	system ("Pause");

	return 0;
}

[Dee Hood]

Ah that makes much more sense! Thanks a lot for your help! =)