При использовании оператора GOTO зацикливается программа при введении букв.

@ProgressM · Регистрация: 13.10.2011

Студворк — интернет-сервис помощи студентам

Приветствую всех новых знакомых в этом храме программирования. Я новичек, пытаюсь программировать в С++ недавно. У меня возник вопрос такого плана. При использовании оператора GOTO зацикливается программа при введении букв. Не могу узреть корень проблемы. Очень надеюсь на вашу помощь. Заранее благодарен

.
Сама по себе программа простая:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
#include <iostream>
#include <conio.h>
using namespace std;
int main()
{
    double a, b;
l1:     
        cout << "Vvedi a b\n";
        cin >> a;
        cin >> b;
        
    if (cin.fail() || cin.bad() || a < 0 || b < 0 || a==0 || b==0)
    {
        //cin.clear();
        cout << "Error\n";
        goto l1;
    }
    else 
        {
            cout <<"S=" << a*b << "\n";
        }
        _getch();   
    return 0;
}

@Чистый · 13.10.2011, 21:58

что программа должна делать? Использование go to не очень рекомендуется тем более что на вашем уровне знаний... при этом у меня код отработал если вводить а и b >0 как и требует условие

@Hi4ko · 13.10.2011, 22:02

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
               
#include <iostream>
#include <conio.h>
using namespace std;
int main()
{
         double a, b;            
                cout << "Vvedi a b\n";
                cin >> a;
                cin >> b;        
while(cin.fail() || cin.bad() || a < 0 || b < 0 || a==0 || b==0)
        {
                //cin.clear();
                cout << "Error\n";
                cin >> a;
                cin >> b; 
        }
                        cout <<"S=" << a*b << "\n";           
                _getch();       
        return 0;
}

"качество программного кода обратно пропорционально количеству операторов GOTO в нём"(с) Дейкстра

Байт · 13.10.2011, 22:18

Hi4ko, Ну, то что твоя программа будет безнадежно циклиться, это очевидно.
Несмотря на отсутствие "goto"
Хотя по Дейкстре качество твоего кода равно бесконечности

@Hi4ko · 13.10.2011, 22:20

Сообщение от Байт

Hi4ko, Ну, то что твоя программа будет безнадежно циклиться, это очевидно.
Несмотря на отсутствие "goto"
Хотя по Дейкстре качество твоего кода равно бесконечности

Только что её откомпилировал и запустил - всё отлично

Байт · 13.10.2011, 22:20

Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.

@Hi4ko · 13.10.2011, 22:21

Сообщение от Байт

Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.

Не по теме:

Да с кем не бывает)
Я сам нуб, на меня можно наезжать)

@ProgressM · 13.10.2011, 23:17 **[ТС]**

Сообщение от Байт

Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.

Он работает нормально только для чисел (в смысле для чисел проверяет исходные данные нормально) Что же касается символов, то происходит зацикливание. Проблема в правильности проверки исходных данных.

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

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

Только что её откомпилировал и запустил - всё отлично

Попробуйте ввести какую либо букву или иной символ, не относящийся к числам. У меня она начинает циклиться.

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

Сообщение от Чистый

что программа должна делать? Использование go to не очень рекомендуется тем более что на вашем уровне знаний... при этом у меня код отработал если вводить а и b >0 как и требует условие

Сама по себе программа вычисляет произведение и все. Но загвоздка в правильности исходных данных, в ее проверке. Да как вы и заметили для a, b>0 она работает, но при введении символов отличных от числовых, возникает вышеописанная проблема.

@easybudda · 13.10.2011, 23:22

ProgressM, чтобы уж совсем на С++

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include <iostream>
#include <stdexcept>
#include <limits>
 
int main(){
    double a, b;
    bool correct;
    
    std::cin.exceptions(std::istream::failbit | std::istream::badbit);
    correct = false;
    
    while ( ! correct ){
        try {
            std::cout << "A = ";
            std::cin >> a;
            std::cout << "B = ";
            std::cin >> b;
            
            if ( a <= 0.0 || b <= 0.0 )
                throw std::runtime_error("A and B must be greater than zero.");
            
            std::cout << "S = " << ( a * b ) << std::endl;
            correct = true;
        }
        catch ( std::istream::failure & e ){
            std::cin.clear();
            std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            std::cerr << "Wrong input! Only numbers allowed." << std::endl;
        }
        catch ( std::exception & e ){
            std::cerr << e.what() << std::endl;
        }
        catch ( ... ){
            std::cerr << "Unknown error!" << std::endl;
            return 1;
        }
    }
    
    return 0;
}

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

на меня можно наезжать

Запрещено использовать нецензурные выражения в любом виде, оскорблять других участников форума, умышленно использовать выражения, противоречащие правилам русского языка.

Правила форума

@ProgressM · 13.10.2011, 23:29 **[ТС]**

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
               
#include <iostream>
#include <conio.h>
using namespace std;
int main()
{
         double a, b;            
                cout << "Vvedi a b\n";
                cin >> a;
                cin >> b;        
while(cin.fail() || cin.bad() || a < 0 || b < 0 || a==0 || b==0)
        {
                //cin.clear();
                cout << "Error\n";
                cin >> a;
                cin >> b; 
        }
                        cout <<"S=" << a*b << "\n";           
                _getch();       
        return 0;
}

"качество программного кода обратно пропорционально количеству операторов GOTO в нём"(с) Дейкстра

Благодарю за альтернативу программного кода, но если есть какая возможность все таки с GOTO, а то преподаватель уж совсем придирается. Но думаю такой вариант сойдет, еще раз спасибо

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

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

ProgressM, чтобы уж совсем на С++

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include <iostream>
#include <stdexcept>
#include <limits>
 
int main(){
    double a, b;
    bool correct;
    
    std::cin.exceptions(std::istream::failbit | std::istream::badbit);
    correct = false;
    
    while ( ! correct ){
        try {
            std::cout << "A = ";
            std::cin >> a;
            std::cout << "B = ";
            std::cin >> b;
            
            if ( a <= 0.0 || b <= 0.0 )
                throw std::runtime_error("A and B must be greater than zero.");
            
            std::cout << "S = " << ( a * b ) << std::endl;
            correct = true;
        }
        catch ( std::istream::failure & e ){
            std::cin.clear();
            std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            std::cerr << "Wrong input! Only numbers allowed." << std::endl;
        }
        catch ( std::exception & e ){
            std::cerr << e.what() << std::endl;
        }
        catch ( ... ){
            std::cerr << "Unknown error!" << std::endl;
            return 1;
        }
    }
    
    return 0;
}

Правила форума

Благодарю за альтернативу программного кода, достаточно интересный вариант

@Hi4ko · 13.10.2011, 23:40

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

ProgressM, чтобы уж совсем на С++

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
#include <iostream>
#include <stdexcept>
#include <limits>
 
int main(){
    double a, b;
    bool correct;
    
    std::cin.exceptions(std::istream::failbit | std::istream::badbit);
    correct = false;
    
    while ( ! correct ){
        try {
            std::cout << "A = ";
            std::cin >> a;
            std::cout << "B = ";
            std::cin >> b;
            
            if ( a <= 0.0 || b <= 0.0 )
                throw std::runtime_error("A and B must be greater than zero.");
            
            std::cout << "S = " << ( a * b ) << std::endl;
            correct = true;
        }
        catch ( std::istream::failure & e ){
            std::cin.clear();
            std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
            std::cerr << "Wrong input! Only numbers allowed." << std::endl;
        }
        catch ( std::exception & e ){
            std::cerr << e.what() << std::endl;
        }
        catch ( ... ){
            std::cerr << "Unknown error!" << std::endl;
            return 1;
        }
    }
    
    return 0;
}

Правила форума

насчёт русского языка - понял, не буду так больше
А так мне самому было интересно, как такое решать, спасибо)
Я, действительно, неправильно условие прочитал. Буквы я не тестил

@ProgressM · 13.10.2011, 23:44 **[ТС]**

Прошу прощения за то, что не мог поучаствовать в моей же проблеме. Всех участников от всей души благодарю за драгоценное время потраченное на меня. Думаю у меня будет много вопросов. Надеюсь когда нибудь и я помогу вам в чем нибудь когда наберусь опыта в программировании. Всего доброго

@easybudda · 13.10.2011, 23:57

Не по теме:

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

насчёт русского языка - понял, не буду так больше

Ну я вообще-то про наезды... Нельзя тут ни на кого наезжать, правилами запрещено... :)

@Напильнег · 14.10.2011, 01:29

Значит так: дело тут в том, что функция clear только сбрасывает поток из ошибочного состояние в нормальное, но не очищает его, а в C++ ввод устроен так, что неформатные символы, вызвавшие ошибку, остаются в потоке и участвуют в следующей операции ввода, опять вызывают ошибку, но из потока опять не удаляются и создают таким образом зацикливание. Чтобы этого не происходило, надо после ошибки чистить поток вручную.

Вот работающий фрагмент:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        double a, b;
l1:             
        cout << "Vvedite a b\n";
        cin >> a;
        cin >> b;
                
        if (!cin || a < 0 || b < 0 || a==0 || b==0)
        {
                cin.clear();
                cin >> noskipws; 
                while (cin.get()!='\n') ;
                cin >> skipws;
                cout << "Error\n";
                goto l1;
        }
        else 
        {
                cout << "S = " << a*b << "\n";
        }

- как видно, goto тут абсолютно не причем.

@greeezz · 14.10.2011, 02:09

Решение предложенное easybudda на мой взгляд самое правильное с точки зрения обработки ошибок. Однако для новичка оно может показаться сложноватым.
Уважаемый ProgressM. Попробуйте использовать циклы вместо go to. Например в вашем случае цикл while достаточно уместен.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
#include <iostream>
#include <conio.h>
using namespace std;
 
int main()
{
    double a, b;
    cout << "Vvedi A\n";
    
    while(!(cin >> a)) { //даем возможность пользователю исправить свою ошибку.
    cout << "Please enter only numbers\n";
    cin.clear();
    cin.ignore(100,'\n');  //игнорируем следующие в потоке 100 символов 
                               //либо символ конца строки. (то, что наступит раньше.)
    }
 
    cout << "Vvedi B\n";
    while(!(cin >> b)) {
    cout << "Please enter only numbers\n";
    cin.clear();
    cin.ignore(100,'\n');       
    }
 
     cout <<"S=" << a*b << "\n";
 
      _getch();       
      return 0;
}

@Nameless One · 14.10.2011, 05:17

Не по теме:

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

При использовании оператора GOTO зацикливается программа при введении букв.

использование оператора goto оправдано только в некоторых случаях. Но вот организовывать с его помощью циклические конструкции - это однозначно плохо

@Напильнег · 14.10.2011, 11:19

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

Уважаемый ProgressM. Попробуйте использовать циклы вместо go to. Например в вашем случае цикл while достаточно уместен.

Уважаемый, повторяю - исходный код не работает не из-за goto. И ваш код работает не из-за применения while, а из-за того, что вы чистите поток функцией ignore (кстати, неграмотно Вы ее используете, но для практического применения сойдет). И оператор goto пишется слитно, кстати.

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

Сообщение от Nameless One

использование оператора goto оправдано только в некоторых случаях

Ввод данных со сложной поверкой - это как раз тот случай, когда использование goto зачастую предпочтительнее, чем while(1) - continue - именно из соображений простоты и наглядности кода.

Комментарий модератора

Следите за выражениями. Цитата из правил несколько постов назад была.

@Nameless One · 14.10.2011, 11:24

Сообщение от Напильнег

именно из соображений простоты и наглядности кода.

Сообщение от Напильнег

наглядности

а вот это уже кому как

@Напильнег · 14.10.2011, 11:52

Сообщение от Nameless One

а вот это уже кому как

Угу - те, кто реально писал сложный реальный код, и, кстати, возвращался к нему для правки через год-полтора, прекрасно осведомлены, что "гоу-то-боязнь" - штука весьма вредная. А у экспертов по решению задач из букваря мнение, как правило, другое.

@Nameless One · 14.10.2011, 12:01

Сообщение от Напильнег

Угу - те, кто реально писал сложный реальный код

ну например (себя в пример можно не приводить)?

Новые блоги и статьи Все статьи Все блоги /
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .	Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .

@Чистый 2733 / 1429 / 89 Регистрация: 08.09.2011 Сообщений: 3,746 Записей в блоге: 1
	13.10.2011, 21:58
	что программа должна делать? Использование go to не очень рекомендуется тем более что на вашем уровне знаний... при этом у меня код отработал если вводить а и b >0 как и требует условие 1

Байт Диссидент 27714 / 17332 / 3810 Регистрация: 24.12.2010 Сообщений: 38,978
	13.10.2011, 22:18
	Hi4ko, Ну, то что твоя программа будет безнадежно циклиться, это очевидно. Несмотря на отсутствие "goto" Хотя по Дейкстре качество твоего кода равно бесконечности 1

Байт Диссидент 27714 / 17332 / 3810 Регистрация: 24.12.2010 Сообщений: 38,978
	13.10.2011, 22:20
	Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально. 1

@ProgressM 1 / 1 / 0 Регистрация: 13.10.2011 Сообщений: 5
	13.10.2011, 23:44 [ТС]
	Прошу прощения за то, что не мог поучаствовать в моей же проблеме. Всех участников от всей души благодарю за драгоценное время потраченное на меня. Думаю у меня будет много вопросов. Надеюсь когда нибудь и я помогу вам в чем нибудь когда наберусь опыта в программировании. Всего доброго 1