Форум программистов, компьютерный форум CyberForum.ru

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

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 21, средняя оценка - 5.00
ProgressM
1 / 1 / 0
Регистрация: 13.10.2011
Сообщений: 5
#1

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

13.10.2011, 21:43. Просмотров 2725. Ответов 39
Метки нет (Все метки)

Приветствую всех новых знакомых в этом храме программирования. Я новичек, пытаюсь программировать в С++ недавно. У меня возник вопрос такого плана. При использовании оператора 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;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
13.10.2011, 21:43     При использовании оператора GOTO зацикливается программа при введении букв.
Посмотрите здесь:
Вычислить функцию при использовании оператора for C++
C++ Выход за пределы массива при использовании оператора new
Выдается ошибка в std при использовании оператора for C++
При введении больше 20 чисел программа выдает странный результат C++
C++ Ошибка при использовании оператора for в задаче с символьными строками
C++ Программа зацикливается при изменении всех 2 на 2
C++ Ошибка при использовании оператора delete в Visual Studio 2010
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Чистый
Автор FAQ
2704 / 1400 / 73
Регистрация: 08.09.2011
Сообщений: 3,732
Записей в блоге: 1
13.10.2011, 21:58     При использовании оператора GOTO зацикливается программа при введении букв. #2
что программа должна делать? Использование go to не очень рекомендуется тем более что на вашем уровне знаний... при этом у меня код отработал если вводить а и b >0 как и требует условие
Hi4ko
74 / 74 / 4
Регистрация: 21.10.2010
Сообщений: 376
13.10.2011, 22:02     При использовании оператора GOTO зацикливается программа при введении букв. #3
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 в нём"(с) Дейкстра
Байт
Эксперт C
15841 / 10168 / 1522
Регистрация: 24.12.2010
Сообщений: 19,177
13.10.2011, 22:18     При использовании оператора GOTO зацикливается программа при введении букв. #4
Hi4ko, Ну, то что твоя программа будет безнадежно циклиться, это очевидно.
Несмотря на отсутствие "goto"
Хотя по Дейкстре качество твоего кода равно бесконечности
Hi4ko
74 / 74 / 4
Регистрация: 21.10.2010
Сообщений: 376
13.10.2011, 22:20     При использовании оператора GOTO зацикливается программа при введении букв. #5
Цитата Сообщение от Байт Посмотреть сообщение
Hi4ko, Ну, то что твоя программа будет безнадежно циклиться, это очевидно.
Несмотря на отсутствие "goto"
Хотя по Дейкстре качество твоего кода равно бесконечности
Только что её откомпилировал и запустил - всё отлично
Байт
Эксперт C
15841 / 10168 / 1522
Регистрация: 24.12.2010
Сообщений: 19,177
13.10.2011, 22:20     При использовании оператора GOTO зацикливается программа при введении букв. #6
Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.
Hi4ko
74 / 74 / 4
Регистрация: 21.10.2010
Сообщений: 376
13.10.2011, 22:21     При использовании оператора GOTO зацикливается программа при введении букв. #7
Цитата Сообщение от Байт Посмотреть сообщение
Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.

Не по теме:

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

ProgressM
1 / 1 / 0
Регистрация: 13.10.2011
Сообщений: 5
13.10.2011, 23:17  [ТС]     При использовании оператора GOTO зацикливается программа при введении букв. #8
Цитата Сообщение от Байт Посмотреть сообщение
Hi4ko, извините. Что-то не разобрал. Конечно, Ваш код работает нормально.
Он работает нормально только для чисел (в смысле для чисел проверяет исходные данные нормально) Что же касается символов, то происходит зацикливание. Проблема в правильности проверки исходных данных.

Добавлено через 1 минуту
Цитата Сообщение от Hi4ko Посмотреть сообщение
Только что её откомпилировал и запустил - всё отлично
Попробуйте ввести какую либо букву или иной символ, не относящийся к числам. У меня она начинает циклиться.

Добавлено через 4 минуты
Цитата Сообщение от Чистый Посмотреть сообщение
что программа должна делать? Использование go to не очень рекомендуется тем более что на вашем уровне знаний... при этом у меня код отработал если вводить а и b >0 как и требует условие
Сама по себе программа вычисляет произведение и все. Но загвоздка в правильности исходных данных, в ее проверке. Да как вы и заметили для a, b>0 она работает, но при введении символов отличных от числовых, возникает вышеописанная проблема.
easybudda
Эксперт CЭксперт С++
9470 / 5483 / 927
Регистрация: 25.07.2009
Сообщений: 10,505
13.10.2011, 23:22     При использовании оператора GOTO зацикливается программа при введении букв. #9
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
1 / 1 / 0
Регистрация: 13.10.2011
Сообщений: 5
13.10.2011, 23:29  [ТС]     При использовании оператора GOTO зацикливается программа при введении букв. #10
Цитата Сообщение от 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
74 / 74 / 4
Регистрация: 21.10.2010
Сообщений: 376
13.10.2011, 23:40     При использовании оператора GOTO зацикливается программа при введении букв. #11
Цитата Сообщение от 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
1 / 1 / 0
Регистрация: 13.10.2011
Сообщений: 5
13.10.2011, 23:44  [ТС]     При использовании оператора GOTO зацикливается программа при введении букв. #12
Прошу прощения за то, что не мог поучаствовать в моей же проблеме. Всех участников от всей души благодарю за драгоценное время потраченное на меня. Думаю у меня будет много вопросов. Надеюсь когда нибудь и я помогу вам в чем нибудь когда наберусь опыта в программировании. Всего доброго
easybudda
13.10.2011, 23:57
  #13

Не по теме:

Цитата Сообщение от Hi4ko Посмотреть сообщение
насчёт русского языка - понял, не буду так больше
Ну я вообще-то про наезды... Нельзя тут ни на кого наезжать, правилами запрещено...

Напильнег
480 / 120 / 10
Регистрация: 30.09.2010
Сообщений: 473
14.10.2011, 01:29     При использовании оператора GOTO зацикливается программа при введении букв. #14
Значит так: дело тут в том, что функция 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 тут абсолютно не причем.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
14.10.2011, 02:09     При использовании оператора GOTO зацикливается программа при введении букв.
Еще ссылки по теме:
C++ Ошибка при использовании перегруженного оператора вывода в поток: no match for 'operator<<'
При вводе в переменную нетипичного ей символа программа зацикливается C++
При вводе слишком большого числа программа зацикливается C++
Задание на ряды: программа зацикливается при вводе некоторых значений C++
Программа с массивами при использовании подпрограммы C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
greeezz
272 / 165 / 4
Регистрация: 10.07.2011
Сообщений: 441
14.10.2011, 02:09     При использовании оператора GOTO зацикливается программа при введении букв. #15
Решение предложенное 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;
}
Yandex
Объявления
14.10.2011, 02:09     При использовании оператора GOTO зацикливается программа при введении букв.
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru