Уважаемые программисты.Скажите пожалуйста все ли правильно в моем коде.

[Paul_AG]

Уважаемые программисты.Скажите пожалуйста все ли правильно в моем коде.Не допустил ли я какихто ошибок при написании программы.
вот код :

#include <iostream>
#include <math.h>
#include <conio.h>
using namespace std;
int main(){
    double n1,n2;
    int inp;
    cout << "Enter number :\n\n\n";
    cout << "\tNumber 1 :\n";
    cin >> n1;
    cout << "\t\tNumber 2 :\n";
    cin >> n2;
       if(!cin){
             cout << "\n\t- Error! -\n";
             cin.clear();
             cin.ignore();
             getch();
             return -1;
             }
             cout << "Enter [1] for  [+]\n";
             cout << "Enter [2] for  [-]\n";
             cout << "Enter [3] for  [/]\n";
             cout << "Enter [4] for[*]\n\n\n";
             cin >> inp;
           
              switch(inp)
             
             {
                         case 1:
                              cout << " Summa :\n";
                              cout << "\n\n" << n1+n2 << "\n\n\t";
                              
                              getch();
                              break;
                              case 2:
                                   cout << " Summa :\n";
                                   cout << "\n\n" << n1-n2 << "\n\n\t";
                                   getch();
                                   break;
                                   case 3:
                                        cout << " Summa :\n";
                                        cout << "\n\n" << n1/n2 << "\n\n\t";
                                        getch();
                                        break;
                                        case 4:
                                             cout << " Summa :\n";
                                             cout << "\n\n" << n1*n2 << "\n\n\t";
                                             getch();
                                             break;
                                           
                                                       
                                                     default:
                                                             cout << "\tonly numbers 1 2 3 4\t\n";
                                                             }
                                                             return 0;
                                                             }

[Volchara]

Вроде нет, но у тебя в switchе всегда печатается Summa, хотя результат выводишь как суммы так нет.
Я имею ввиду, что ты пишешь

case 3:
cout << " Summa :\n";
cout << "\n\n" << n1/n2 << "\n\n\t";
getch();
break;

ты пишешь summa а результат деления

[Sazary]

Вроде все так.
Но есть пара замечаний:
Во-первых, вы на любое действие выводите "Сумма".
Во-вторых, разве не удобнее ли вводить не цифру, обозначающую действие, а сам знак действия?

[Paul_AG]

Спасибо огромное!

[Paul_AG]

Под словом Summa я подразумевал результат.
спасибо.

[Paul_AG]

а как это сделать?тоесть вместо цифры знак соответствующий действию.

[Sazary]

Изменить тип inp:

char inp;

и дальше:

switch (inp)

{
case '+':

и так далее..