При использовании оператора 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;
}

@easybudda · 14.10.2011, 12:14

Не по теме:

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

те, кто реально писал сложный реальный код...

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

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";
        }

Ну такое-то действительно найдёшь разве-что в реальнопрогерском реальном коде... ;)

@Напильнег · 14.10.2011, 12:14

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

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

Это как? У нас что, общие знакомые есть вообще? Не говорю уже о том, что тут нужны известные обоим сторонам коллеги по цеху.

Мдя, сразу виден профессиональный подход... И ни разу не удивляет - анти-гоутушники они такие...

@Nameless One · 14.10.2011, 12:20

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

Это как? У нас что, общие знакомые есть вообще? Не говорю уже о том, что тут нужны известные обоим сторонам коллеги по цеху.

мне бы подошла ссылка на высказывание какого-нибудь достаточно известного программиста, либо ссылка на coding-style guide крупного проекта на С/С++.

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

И ни разу не удивляет - анти-гоутушники они такие...

ну почему сразу "анти-"? Есть пару ситуаций, в которых, как считаю лично я, использование goto оправдано

@Напильнег · 14.10.2011, 12:35

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

Ну такое-то действительно найдёшь разве-что в реальнопрогерском реальном коде...

Жалкая попытка передернуть - даже если бы тут и был какой криминал, это минимальная правка исходного кода для доведения его до работоспособности в соответствии с установками на задачу - ИМХО, именно профессиональный подход. Через cin.get() сделано для того, чтобы показать, что именно происходит - cin.ignore(...) такого представления не дает. А вот твой вариант применительно к ситуации - интеллектуальное пижонство.

@yariknarik · 14.10.2011, 12:58

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

@easybudda · 14.10.2011, 14:45

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

ИМХО, именно профессиональный подход.

В смысле - абы как, лишь бы заработало? Тот факт, что среди преподов (судя по постам на форуме) тоже "реальные профессионалы" встречаются, ещё не означает, что вот так и надо делать. Помимо goto там ещё числа double с целочисленной константой сравниваются, что само по себе тоже реально профессиональная затея. Ну ладно try/catch - пижонство, хотя по-моему это как-раз по стилю к С++ гораздо ближе, чем конструкции вроде if ( ! cin )... В любом случае ничего плохого в том, что человек увидит, как програмка может выглядеть. А быдлокодить его, видимо, и без нас с вами научат...

@greeezz · 14.10.2011, 16:35

Не по теме:

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

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

Вопрос риторический. Я что сказал что у него код не работает из за goto?

По моему мнению для новичка использование оператора goto, усложняет поиск ошибок и плюс в обсуждаемой программе он совершенно не нужен. К примеру здесь нет глубоко вложенной структуры, выполнение которой надо прервать. Однако даже в такой ситуации можно обойтись и без goto.

P.S. Я был бы вам очень признателен, если бы вы пояснили как правильно применять cin.ignor(), в каких случаях это лучше всего делать и почему я использую ее неграмотно. Заранее спасибо.

p.s.2: Напильнег если для вас такие авторы как Керниган и Ритчи имеют хоть какой-то незначительный авторитет то откройте их книгу. Глава 3.9. Оператор GOTO и метки

@ProgressM · 14.10.2011, 21:05 **[ТС]**

Спасибо!!!!

@Напильнег · 19.10.2011, 13:12

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

Я был бы вам очень признателен, если бы вы пояснили как правильно применять cin.ignor(), в каких случаях это лучше всего делать и почему я использую ее неграмотно

Длина буфера ввода гораздо больше 100 символов, так что Ваш вызов cin.ignore(100, ...) не гарантирует его очистку. А уж сколько надо для Вашей системы, определяйте сами. Я ставлю 32k-1 - чтобы и запас на будущее был, и Борман при необходимости это тоже скушал.

@greeezz · 19.10.2011, 16:22

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

Длина буфера ввода гораздо больше 100 символов, так что Ваш вызов cin.ignore(100, ...) не гарантирует его очистку. А уж сколько надо для Вашей системы, определяйте сами. Я ставлю 32k-1 - чтобы и запас на будущее был, и Борман при необходимости это тоже скушал.

istream& ignore ( streamsize n = 1, int delim = EOF );
Extracts characters from the input sequence and discards them.
The extraction ends when n characters have been extracted and discarded or when the character delim is found, whichever comes first. In the latter case, the delim character itself is also extracted.
источник

В связи с этим использование мной cin.ignore(100, '\n') обосновано. т.к. в случе обсуждаемой программы так или иначе символ конца строки имеет основной смысл и нет надобности игнорировать буфер от начала и до конца.

@Напильнег · 19.10.2011, 16:38

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

В связи с этим использование мной cin.ignore(100, '\n') обосновано.

По-моему, у тебя проблемы с английским языком - переведи, что отквотил, для контроля.

@greeezz · 19.10.2011, 18:30

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

По-моему, у тебя проблемы с английским языком - переведи, что отквотил, для контроля.

Не по теме:

урок английского что ли ?

Extracts characters from the input sequence and discards them.
Извлекает сиволы из входящей последовательности и "сбрасывает" их.

The extraction ends when n characters have been extracted and discarded or when the character delim is found, whichever comes first. In the latter case, the delim character itself is also extracted.
Извлечение заканчивается когда Н символов было извлечено и "сброшено" ИЛИ когда найдет разграничеительный символ, что бы ни настало первым.

еще что-то перевести ?

@easybudda · 19.10.2011, 18:36

greeezz, видимо, смысл в том, что если вместо числа отправить 101+ букву, конструкция

C++
1
cin.ignore(100, '\n');

не сработает и '\n' останется в потоке. Вот по этому и

C++
1
std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

@Напильнег · 19.10.2011, 20:06

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

еще что-то перевести ?

Достаточно, переводишь ты приемлемо - ты просто не можешь понять и применить, чего прочитал.

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

видимо, смысл в том,

Почему видимо? Так и есть.

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

Уважительно относитесь к участникам форума. На первый раз устное предупреждение.

@greeezz · 19.10.2011, 20:24

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

greeezz, видимо, смысл в том, что если вместо числа отправить 101+ букву, конструкция

C++
1
cin.ignore(100, '\n');

не сработает и '\n' останется в потоке. Вот по этому и

C++
1
std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

Мне это элементарно понятно. Именно по этому мной было написано несколько постов назад :

В связи с этим использование мной cin.ignore(100, '\n') обосновано. т.к. в случе обсуждаемой программы так или иначе символ конца строки имеет основной смысл и нет надобности игнорировать буфер от начала и до конца.

Не по теме:

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

... ты просто глуп ...

можно сделать вывод о "необходимости" данной дискусии

@kravam · 19.10.2011, 20:30

Не по теме:

Не связывайтесь с напильником, он оскорблятель

@Напильнег · 19.10.2011, 21:32

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

мне бы подошла ссылка на высказывание какого-нибудь достаточно известного программиста

Представь себе - ни одного не знаю, в области IT больше известны менеджеры. В любом случае ориентироваться на чьи либо ВЫСКАЗЫВАНИЯ при решении задачи не буду - спрос то будет с меня и за результат, а не с, скажем, Вирта, за его болтовню.

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

либо ссылка на coding-style guide крупного проекта на С/С++.

Вот это матерая попытка перевести стрелки! Но не пройдет - это ты предъяви Coding-style, где использование goto запрещено - посмотреть, как это вообще можно технически осуществить.

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

ну почему сразу "анти-"? Есть пару ситуаций, в которых, как считаю лично я, использование goto оправдано

Второй раз - при разборе ввода такие ситуации возникают нередко, и делать goto на хорошо видимый в листинге повторный запрос ввода с осмысленным названием метки более удобно, чем лепить безликий continue, который к тому же не всегда возможен.

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

В смысле - абы как, лишь бы заработало?

Это ты совсем не по адресу - я как раз постоянно грызусь с быдлокодерами. В данном случае мой код функционально отличается от твоего только тем, что ошибки не сортирует - так и не надо. Зато корень проблемы показывает лучше!

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

Тот факт, что среди преподов (судя по постам на форуме) тоже "реальные профессионалы" встречаются, ещё не означает, что вот так и надо делать.

В данном случае препод как раз нормальные требования предъявляет - рискну предположить, что имеет большую практику решения вычислительных задач в ходе научной работы. Быдлопреподы как раз и goto как огня боятся.

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

Помимо goto там ещё числа double с целочисленной константой сравниваются

И чего? Приведение типов для констант еще на этапе компиляции делается, насколько мне известно. Хотя я для красоты точку обычно рисую, но ее отсутствие не криминал ни фига.

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

ничего плохого в том, что человек увидит, как программка может выглядеть.

Ничего плохого. Но надо было так и сказать. А сначала - причину ошибки разъяснить и незамутненный способ ее устранения продемонстрировать.

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

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

Именно по этому мной было написано несколько постов назад :

В связи с этим использование мной cin.ignore(100, '\n') обосновано. т.к. в случе обсуждаемой программы так или иначе символ конца строки имеет основной смысл и нет надобности игнорировать буфер от начала и до конца.

Неизлечим...

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

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

Вопрос риторический. Я что сказал что у него код не работает из за goto?

Вы для неработающей [по другой причине] программы порекомендовали заменить goto на цикл - как это по другому понять?

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

здесь нет глубоко вложенной структуры, выполнение которой надо прервать. Однако даже в такой ситуации можно обойтись и без goto.

Можно, но это как раз

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

усложняет поиск ошибок

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

если для вас такие авторы как Керниган и Ритчи имеют хоть какой-то незначительный авторитет

Книжка ихняя - дрянь.

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

откройте их книгу. Глава 3.9. Оператор GOTO и метки

Начинается с двух стандартных страшилок (вот он где первоисточник, оказывается), кончается хорошим примером, который, при рассмотрении его с точки зрения здравого смысла, показывает как раз то, что с goto бороться с целью побороть его во что бы то ни стало, не надо

@Nameless One · 20.10.2011, 05:13

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

Представь себе - ни одного не знаю, в области IT больше известны менеджеры. В любом случае ориентироваться на чьи либо ВЫСКАЗЫВАНИЯ при решении задачи не буду - спрос то будет с меня и за результат, а не с, скажем, Вирта, за его болтовню.

вот тогда и не выдавай свое мнение за истину в последней инстанции

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

Вот это матерая попытка перевести стрелки!

это не я тут кричал про "тех, кто реально писал сложный код"

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

Но не пройдет - это ты предъяви Coding-style, где использование goto запрещено - посмотреть, как это вообще можно технически осуществить.

еще раз, где я говорил про "запрещено"? могу дать ссылку, по которой использование goto поощряется, но совсем в другом случае

@easybudda · 20.10.2011, 08:54

Не по теме:

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

...могу дать ссылку, по которой использование goto поощряется, но совсем в другом случае

Use basic, Luke! ;)

@Nameless One · 20.10.2011, 16:54

Не по теме:

easybudda, я имел в виду Linux Kernel Coding Style, 7 главу :D

Новые блоги и статьи Все статьи Все блоги /
После бана на Пикабу не работает авторизация аккаунта с Яндекса. Hrethgir 09.07.2026 В частности https:/ / auto. ru/ logbook/ zapret-na-vozhdenie-posle-65-let_1cf28789-07b4-4c75-a2f7-4cd4edf9deeb тут хотел оставить комментарий такой Зрите в корень - монополизация на разработку программ. . .	GitDOS: Когда я решил скрестить MS-DOS, GitHub и Claude MaGz GoLd 08.07.2026 Привет, Киберфорум! Хочу поделиться своим проектом, который, надеюсь, вас заинтересует. GitDOS — это веб-приложение, которое позволяет запускать DOS в браузере с тремя крутыми фичами: диски живут в. . .	Учебный мини "цифровой двойник" = (Сервер = Java + Flask_питон )+(3D = Гугловскаая 4T nvideo + google colab) +( WMS = Odoo на Docker) anaschu 07.07.2026 Увидел в требованиях к вакансии создателя имитационных моделей связку (REst + Api + WMS + Anylogic + питон + 3D Nvideo), и решил посмотреть, что это такое. Из этого всего я работал более менее. . .	Море в объективе kumehtar 06.07.2026 Хотел написать много проникновенных слов, но передумал. Оно само - говорит. Кто слышит тот слышит.
Doom для терминала без стрельбы и монстров. 3D Raycasting на ascii. dcc0 05.07.2026 Попросил нейронную сеть deepai. org написать рейкастинг 3D с библиотекой ncurses для Linux. Чтобы можно было ходить на стрелочки. Чтобы стены были отрисованы символами. Справилась. Первый вариант. . .	Установка статуса документа по условию Maks 05.07.2026 Алгоритм из решения ниже реализован на нетиповом документе "НарядПутевка" разработанного в КА2. Задача: в табличной части "Материалы" документа при записи автоматически устанавливать статус. . .	Сезонность и суточность закисления почв anaschu 04.07.2026 200 часов это все равно моловато. Есть ситуации, но нестандартные, когда смена происходит за 5 лет. Но обычно это 50 лет и более. Наверное, закисление почвы происходит сезонно в средней. . .	В чем ценность человеческого опыта в глобальном смысле? kumehtar 03.07.2026 Возможно, ценность человека не в том, что он однажды достигает мудрости, а в том, что он становится носителем карты пути. Он знает не только истину, но и последовательность внутренних изменений,. . .

@yariknarik 0 / 3 / 2 Регистрация: 02.06.2010 Сообщений: 36
	14.10.2011, 12:58
	Ну так правельно, пока весит цикл пока програма будет зацикливаться... Убери цыкл и поставь оператор условия и все будет работать... Все зависит от того сколько раз ты хочешь прогнать работу программы и что получить на выходе, при ее разных значениях пераметров, при запусках... 0

@ProgressM 1 / 1 / 0 Регистрация: 13.10.2011 Сообщений: 5
	14.10.2011, 21:05 [ТС]
	Спасибо!!!! 0

@kravam
	19.10.2011, 20:30
	Не по теме: Не связывайтесь с напильником, он оскорблятель 0

@Nameless One
	20.10.2011, 16:54
	Не по теме: easybudda, я имел в виду Linux Kernel Coding Style, 7 главу :D 0