Поломка программы из-за символа - C++

@Corax111 · Регистрация: 16.06.2013

Вод код. Программа работает нормально, но если ввести туда символ, или знак типа * или / то она летит... как сделать так, чтобы нельзя было вводить символы, либо такие знаки? Точнее, чтобы при вводе их был просто выход из программы, или вообще ничего не происходило?...

C++

#include <iostream>
 
using namespace std;
 
int main ()
{
    setlocale(LC_ALL, "Russian");
 
    int x;
    int y;
    int o;
 
 
 
    while (true) 
    {
 
        cout << "1 - умножение" << endl;
        cout << "2 - деление" << endl;
        cout << "3 - сложение" << endl;
        cout << "4 - вычитание" << endl;
        cout << "5 - возведение в квадрат" << endl;
        cout << "6 - возведение в куб" << endl;
        cout << "7 - остаток от числа" << endl;
        cin >> o;
        if (o == 1) 
        {
        cout << "Введите два числа," << "которые Вы хотите перемножить." << endl;
        cin >> x;
        cin >> y;
        cout << "Ответ: ";
        cout << x*y << endl;
        
        }
        if (o == 2) 
        {
        cout << "Введите два числа," << "которые Вы хотите разделить." << endl;
        cin >> x;
        cin >> y;
        cout << "Ответ: ";
        cout << x/y << endl;
        }
 
        if (o == 3) 
        {
        cout << "Введите два числа," << "которые Вы хотите сложить." << endl;
        cin >> x;
        cin >> y;
        cout << "Ответ: ";
        cout << x+y << endl ;
        }
    
        if (o == 4) 
        {
        cout << "Введите два числа," << "которые Вы хотите вычесть." << endl;
        cin >> x;
        cin >> y;
        cout << "Ответ: ";
        cout << x-y << endl;
        }
        if (o == 5) 
        {
        cout << "Введите число, которое хотите возвести в квадрат." << endl;
        cin >> x;
        cout << "Ответ: ";
        cout << x*x << endl;
        }
        if (o == 6) 
            {
            cout << "Введите число, которое хотите возвести в куб." << endl;
            cin >> x;
            cout << "Ответ: ";
            cout << x*x << endl;
            }
        if (o == 7) 
        {
        cout << "Введите числа, остаток от которых вам нужен." << endl;
        cin >> x;
        cin >> y;
        cout << "Ответ: ";
        cout << x%y << endl;
        }
    
    }
        
}

Добавлено через 2 минуты
Такая же беда, если ввести слишком много цифр...

alsav22 · 30.08.2013, 01:38

Сообщение от Corax111

чтобы при вводе их был просто выход из программы,

C++

cin >> o;
if (!cin)
{
    system("pause");
    return 0;
}

@kvadro · 30.08.2013, 01:48

C++

1	if ( !( cin >> o ) )

Добавлено через 6 минут

или вообще ничего не происходило?

Можно просто валидировать:

C++

int getNumb()
{
    int n;
 
    while ( !( cin >> n ) )
    {
        cout << "Введите число! ";
        cin.clear();
        cin.ignore( numeric_limits<streamsize>::max(), '\n' );
    }
 
    return n;
}
 
// ...
 
x = getNumb();
y = getNumb();

@isaew999 · 30.08.2013, 09:50

Я бы посоветовал переделать с if на switch case, тогда и проблема отпадет сама собой. И можно сделать обработку исключения средствами самого switch с помощью default.

@albert! · 30.08.2013, 13:17

Так здесь же ошибка в другом, по-моему. Символы * / и т.п не являются числами. А также, если ты вводишь большое число, то оно может быть слишком большим для значения типа int, используй long int

@isaew999 · 30.08.2013, 14:36

Сообщение от Corax111

но если ввести туда символ, или знак типа * или / то она летит...

Вопрос тут как раз во вводе данных символов а не черезмерной величине числа. А с помощью switch case можно сделать все как надо, что все мат операции будут соответствовать своемусимвольному значению, а не цыфрам 1, 2, 3, ...... и т.д.

@Ermak_nk · 30.08.2013, 15:17

Когдато делал нечто подобное, может поможет

C++ (Qt)

#include <iostream>
#include <conio.h>
#include <math.h>
using namespace std;
int main(void)
{
    double a, c;
    char b;
    cout << "Enter the example: ";
    cin >> a >> b >> c;
    switch (b)
    {
    case '+':
        cout << a << "+" << c << "=" << a + c << endl;
        break;
    case '-':
        cout << a << "-" << c << "=" << a - c << endl;
        break;
    case '*':
        cout << a << "*" << c << "=" << a * c << endl;
        break;
    case '/':
        cout << a << "/" << c << "=" << a / c << endl;
        break;
    case 'sin':
        cout << "sin" << c << " = " << sin(c) << endl;
        break;
    case 'cos':
        cout << "cos" << c << " = " << cos(c) << endl;
        break;
    case 'tg':
        cout << "tg" << c << " = " << sin(c)/cos(c) << endl;
        break;
    default:
        break;
    }
    getch ();
    return 0;
}

Еще есть вариант все вводить в символьном виде в строку, а дальше преобразовывать строку в числа и операции.

Добавлено через 27 минут
Только что обратил внимание - по поводу sin, cos, tg - там несколько символов, так что эти строки не совсем правильные, делал когда только начинал изучать язык

@albert! · 30.08.2013, 20:02

Я не знаю, надо ли выкладывать готовую программу, но короче как-то так.

Кликните здесь для просмотра всего текста

C++

#include <cstdlib>
#include <iostream>
#include <conio.h>
 
 
 
using namespace std;
 
int main ()
{
    setlocale(LC_ALL, "Russian");
 
    int x;
    int y;
    char o;
    o = getch();
    
    
 
 
 
    while (true) 
    {
 
        cout << "1 - ГіГ¬Г*Г®Г¦ГҐГ*ГЁГҐ" << endl;
        cout << "2 - Г¤ГҐГ«ГҐГ*ГЁГҐ" << endl;
        cout << "3 - Г±Г«Г®Г¦ГҐГ*ГЁГҐ" << endl;
        cout << "4 - ГўГ»Г·ГЁГІГ*Г*ГЁГҐ" << endl;
        cout << "5 - ГўГ®Г§ГўГҐГ¤ГҐГ*ГЁГҐ Гў ГЄГўГ*Г¤Г°Г*ГІ" << endl;
        cout << "6 - ГўГ®Г§ГўГҐГ¤ГҐГ*ГЁГҐ Гў ГЄГіГЎ" << endl;
        cout << "7 - Г®Г±ГІГ*ГІГ®ГЄ Г®ГІ Г·ГЁГ±Г«Г*" << endl;
        cin >> o;
        switch(o)
        
        {
                 case '1':
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г¤ГўГ* Г·ГЁГ±Г«Г*," << "ГЄГ®ГІГ®Г°Г»ГҐ Г‚Г» ГµГ®ГІГЁГІГҐ ГЇГҐГ°ГҐГ¬Г*Г®Г¦ГЁГІГј." << endl;
        cin >> x;
        cin >> y;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x*y << endl;
        break;
        
        
        case '2':
        
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г¤ГўГ* Г·ГЁГ±Г«Г*," << "ГЄГ®ГІГ®Г°Г»ГҐ Г‚Г» ГµГ®ГІГЁГІГҐ Г°Г*Г§Г¤ГҐГ«ГЁГІГј." << endl;
        cin >> x;
        cin >> y;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x/y << endl;
        break;
        
 
        case '3':
        
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г¤ГўГ* Г·ГЁГ±Г«Г*," << "ГЄГ®ГІГ®Г°Г»ГҐ Г‚Г» ГµГ®ГІГЁГІГҐ Г±Г«Г®Г¦ГЁГІГј." << endl;
        cin >> x;
        cin >> y;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x+y << endl ;
        break;
    
        case '4': 
        
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г¤ГўГ* Г·ГЁГ±Г«Г*," << "ГЄГ®ГІГ®Г°Г»ГҐ Г‚Г» ГµГ®ГІГЁГІГҐ ГўГ»Г·ГҐГ±ГІГј." << endl;
        cin >> x;
        cin >> y;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x-y << endl;
        
        case '5':
        
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г·ГЁГ±Г«Г®, ГЄГ®ГІГ®Г°Г®ГҐ ГµГ®ГІГЁГІГҐ ГўГ®Г§ГўГҐГ±ГІГЁ Гў ГЄГўГ*Г¤Г°Г*ГІ." << endl;
        cin >> x;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x*x << endl;
        break;
        case '6': 
            
            cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г·ГЁГ±Г«Г®, ГЄГ®ГІГ®Г°Г®ГҐ ГµГ®ГІГЁГІГҐ ГўГ®Г§ГўГҐГ±ГІГЁ Гў ГЄГіГЎ." << endl;
            cin >> x;
            cout << "ГЋГІГўГҐГІ: ";
            cout << x*x << endl;
            break;
            
        case '7':
        
        cout << "Г‚ГўГҐГ¤ГЁГІГҐ Г·ГЁГ±Г«Г*, Г®Г±ГІГ*ГІГ®ГЄ Г®ГІ ГЄГ®ГІГ®Г°Г»Гµ ГўГ*Г¬ Г*ГіГ¦ГҐГ*." << endl;
        cin >> x;
        cin >> y;
        cout << "ГЋГІГўГҐГІ: ";
        cout << x%y << endl;
        break;
        default: cout << "ГЌГҐГЇГ°Г*ГўГЁГ«ГјГ*Г»Г© ГўГўГ®Г¤" <<endl;
        break;
       
 
}
    }
}

alsav22 · 30.08.2013, 20:22

Сообщение от albert!

но короче как-то так.

Как насчёт этого, в вашей программе?

Сообщение от Corax111

Программа работает нормально, но если ввести туда символ,

@isaew999 5 / 13 / 2 Регистрация: 19.09.2012 Сообщений: 203
	30.08.2013, 09:50	#4
	Я бы посоветовал переделать с if на switch case, тогда и проблема отпадет сама собой. И можно сделать обработку исключения средствами самого switch с помощью default. 0

@albert! 3 / 3 / 0 Регистрация: 23.01.2013 Сообщений: 45
	30.08.2013, 13:17	#5
	Так здесь же ошибка в другом, по-моему. Символы * / и т.п не являются числами. А также, если ты вводишь большое число, то оно может быть слишком большим для значения типа int, используй long int 0

@isaew999 5 / 13 / 2 Регистрация: 19.09.2012 Сообщений: 203
	30.08.2013, 14:36	#6
	Сообщение от Corax111 но если ввести туда символ, или знак типа * или / то она летит... Вопрос тут как раз во вводе данных символов а не черезмерной величине числа. А с помощью switch case можно сделать все как надо, что все мат операции будут соответствовать своемусимвольному значению, а не цыфрам 1, 2, 3, ...... и т.д. 0

alsav22 5434 / 4829 / 442 Регистрация: 04.06.2011 Сообщений: 13,587
	30.08.2013, 20:22	#9
	Сообщение от albert! но короче как-то так. Как насчёт этого, в вашей программе? Сообщение от Corax111 Программа работает нормально, но если ввести туда символ, 0

Опции темы

С++ для начинающих

Поломка программы из-за символа - C++