Ваше отношение к goto

@__Corey · Регистрация: 09.11.2019

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

Всем привет.

Обычно оператор безусловного перехода goto хейтится.
Данным постом я не хочу поднимать очередной холивар, хочу просто узнать мнение окружающих - как относитесь к практике применения 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
25
   private Func<Data, bool> GetSearchPredicate()
        {
            Func<Data, bool> predicate;
 
            switch (_currentState)
            {
                case SelfState.Improve:
                {
                    // Внутри SomeMethod заложена нужная логика
                    predicate = x=> SomeMethod(_field, x);
                    break;
                }
                case SelfState.Replannig:
                {
                    predicate = x=> SomeMethod(_field, x);
                    break;
                }
                default:
                {  
                    goto case SelfState.Improve;   
                }
            }
 
            return predicate;
        }

Какие выходы, не используя goto:
1. Либо сделать присвоение predicate = null(или ещё что) и избавиться от блока default целиком. Хотя лично мне спокойнее, когда default в коде имеется.
2. В блоке default скопипастить кейс, на который ссылаюсь(тогда при изменениях надо помнить, что менять нужно в 2 местах).

В свитче я оставил goto(мой первый goto в жизни

)

Другой пример:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
            foreach (var group in groups)
            {
                foreach (var answer in group)
                {
                    if(statement)
                    {
                        goto next;
                    }
                }
 
            }
            next:
            {
 
            }

Какие выходы, не используя goto:
1. Применить SelectMany к groups и оставить один цикл foreach.
2. Повозиться с флагами и break'ами.

Здесь мне готушить не хочется, не знаю почему.

Т.е. можно и без goto жить в обоих приведенных(и ещё n неприведённых) примерах.

@kolorotur · 11.08.2021, 00:49

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

Когда Вы в последний раз прибегали к использованию goto?

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

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

Жду явный пример этого.

Там несколькими сообщениями выше ссылка на почти тысячу примеров с указанием причин.

@wizard41 · 11.08.2021, 01:00

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

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

А я как раз разрабатываю. Правда, не в c#.

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

Там несколькими сообщениями выше ссылка на почти тысячу примеров с указанием причин.

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

@kolorotur · 11.08.2021, 01:12

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

А я как раз разрабатываю.

Тем более странно от вас слышать подобные заявления.

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

не очень убедительно

Убеждать в чем-то цели не стояло.

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

Там не объясняется тех причин, по которым следовало бы воздержаться от применения этого оператора.

Дык были бы причины — воздержались бы.

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

goto является устаревшим оператором и в современном программировании находит все меньше места.

Бесспорно.
Но когда-таки находит — не стоит от него шарахаться, как бес от ладана.

@wizard41 · 11.08.2021, 01:20

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

Тем более странно от вас слышать подобные заявления.

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

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

Но когда-таки находит — не стоит от него шарахаться, как бес от ладана.

Согласен. Никого не отговариваю.
Вопрос вам, kolorotur,
Некая функция F2 принимает в свои аргументы результат предстоящей выше функции F1, которая, в свою очередь, работает в зависимости от возвращаемых значений функции F3, стоящей после функции F2.
Вы тут тоже про goto подумали? Думаю, нет.

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

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

Дык были бы причины — воздержались бы.

А эти причины им были нужны?

@kolorotur · 11.08.2021, 02:02

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

Вы тут тоже про goto подумали?

Нет, я про него вообще не думаю.
Этот оператор не предназначен для структурирования кода.
Вы выше сделали утверждение, что наличие goto в коде — признак его плохого структурирования. Из того, что в плохо структурированном коде может присутствовать оператор goto вовсе не следует, что наличие в коде goto — автоматически означает его плохую структурированность. Могут быть и другие причины для его наличия, вполне осознанные и продуманные.

Кстати, вот выше есть примеры использования этого ужасного оператора в нашем любимом фреймворке. Фреймворк с открытыми исходниками, поддерживатели активно принимают PR-ы.
Попробуйте ради эксперимента разместить PR, где в каком-нибудь классе эта мерзость отрефакторена под нормальную структуру — может и правда код практиканта проскользнул мимо ревью, а может оказаться что там люди по-умнее нас с вами сидят и прокомментируют что-нибудь познавательное.

@wizard41 · 11.08.2021, 02:27

__Corey,

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
            foreach (var group in groups)
            {
                foreach (var answer in group)
                {
                    if(statement)
                    {
                        goto next;
                    }
                }
 
            }
            next:
            {
 
            }

Тут уже религия как кому позволяет. В данном случае ваш next явно ведет к какому-то методу, который просто можно вызвать из цикла, при условии if(statement), без использования goto. Т.е. в данном случае это goto совершенно бесполезно.

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

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

Могут быть и другие причины для его наличия, вполне осознанные и продуманные.

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

Из того, что в плохо структурированном коде может присутствовать оператор goto вовсе не следует, что наличие в коде goto — автоматически означает его плохую структурированность.

Мда... kolorotur, это означает, что не все надо воспринимать так, как подается в "правильных" источниках информации. Нет, я не спорю, что вы воспринимаете это так, а не иначе, и у вас некое свое суждение по этому поводу. С высоты, так сказать, полета опыта. А вот меня это ввело в ступор. Знаете почему? Потому что я несколько углубился в суть понятий, объясняющихся на каждом шагу, вроде бы, но, не тут то было.

Добавлено через 5 минут
А обычным сишарпистам, которые крестики-нолики рисуют - понятно дело, это не интересно. Там до goto еще Go.......Tooooo....

@ViterAlex · 11.08.2021, 02:41

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

Какие выходы, не используя goto:

избавиться от него. И default имеется и дублирования кода нет.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
private Func<Data, bool> GetSearchPredicate()
{
    Func<Data, bool> predicate;
 
    switch (_currentState)
    {
        case SelfState.Replannig:
        {
            predicate = x=> SomeMethod(_field, x);
            break;
        }
        case SelfState.Improve:
        default:
       {
            // Внутри SomeMethod заложена нужная логика
            predicate = x=> SomeMethod(_field, x);
            break;
        }
    }
 
    return predicate;
}

@Usaga · 11.08.2021, 05:38

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

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

Авторы .NET знают о нюансах кодогенерации и JIT'а на порядок больше, чем все мы тут вместе взятые. На этот процесс влияет вообще всё: от порядка следования инструкций и объявлений переменных, до выбранных операторов. Это - раз. Два: от базовой библиотеки классов .NET требуется максимальная производительность. Поэтому её очень жёстко тюнят и оптимизируют. Так жёстко, что Саша Грей бы удивилась. А он того кода, что вы на различных ресурсах видели, такого не требуется. Как правило, от пользовательского кода требуется именно производительность труда разработчика и только в последнюю очередь от кода.

Оттуда и разница в том, что вы видите. Не делайте резких выводов о том, что не знаете. Интереса ради посмотрите на обсуждения PR'ов в том же .NET. Там малые баталии могут разворачиваться из-за буквально одного каста неуместного или проверки на null не в том месте. Эти люди внатуре знают, что они делают.

Если вас поставить в их положение, когда от кода требуется 146% производительности, то вы тоже начнёте озадачиваться тем, а что же JIT будет генерировать в той или иной ситуации. Тогда, вдруг, окажется, что плохие практики, типа goto, не такие уж и плохие, когда дело доходит до оптимизаций. Оптимальный код и так уродлив.

@wizard41 · 11.08.2021, 10:42

Usaga, плюсую вам. Вы много написали, я прочел. И так, мой вопрос - вы часто используете goto? В своей практике?

Добавлено через 8 минут
Знаете что интересно? Просто, как только поднимается некая тема, касающаяся неких вещей, неожиданно встретившихся, так от куда ни возьмись вылезает куча (утрированно) специалистов, готовых пояснить миру - вы ж что? как вы этого не знали? Это ж круто (правильно)... ну все в таком духе.
Usaga, kolorotur, покажите мне реальный документ, в котором будет сказано о том, что применение goto обосновано. Даже не нужно, просто обосновано. В рамках строения программ современного времени, даже пусть не ООП.
На всякий случай напомню вопрос ТС:

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

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

@ashsvis · 11.08.2021, 10:47

Можно спросить экспертов, чем заменить это, без goto:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
            foreach (var group in groups)
            {
                foreach (var answer in group)
                {
                    if(statement)
                    {
                        goto next;
                    }
                }
 
            }
            next:
            {
 
            }

@Рядовой · 11.08.2021, 10:50

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

Можно спросить экспертов, чем заменить это, без goto:

вынести в метод и сделать ретурн. Можно в лямбду и сделать ретурн, код внешне будет практически идентичен вашему.

@wizard41 · 11.08.2021, 11:42

Раз уж воцарилось некоторое затяжное молчание, то, с позволения так сказать, прокомментирую:

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

Авторы .NET знают о нюансах кодогенерации и JIT'а на порядок больше, чем все мы тут вместе взятые

Это естественно. Мы пользователи творения их умов. Но кто нам запретил использовать с пользой (простите за тавтологию) плоды их творчества?

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

На этот процесс влияет вообще всё: от порядка следования инструкций и объявлений переменных, до выбранных операторов.

Хм, все да не все. Вынужден не согласиться с этим утверждением.

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

Как правило, от пользовательского кода требуется именно производительность труда разработчика и только в последнюю очередь от кода.

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

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

Не делайте резких выводов о том, что не знаете.

Вы не допускаете мысли, что я, возможно, что-то знаю, раз так говорю? Тут, как у евреев - встречный вопрос: с чего вы так решили, что я не знаю о чем говорю? Вы явно не считаете себя истиной в последней инстанции. Так от чего это?

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

Эти люди внатуре знают, что они делают.

Эти люди никому не сказали, что все вы тут лохи, потому что не используете goto. Они это в натуре знают. Смешно.

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

Тогда, вдруг, окажется, что плохие практики, типа goto, не такие уж и плохие, когда дело доходит до оптимизаций

Не спорю. Оптимизировали все, кто что только мог. Ни одного goto за 12 лет. Ни одного. Может быть только вы таким образом оптимизируете, но никому не показываете?

@Usaga · 11.08.2021, 12:40

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

И так, мой вопрос - вы часто используете goto? В своей практике?

Ни разу не доводилось)

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

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

Вам уже ссылку на исходники .NET привели. Там каждое применение goto - результат обдуманный и взвешенный, с консультацией с командой Core (ответственной за JIT) и внимательным исследованием машинного кода.

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

Это естественно. Мы пользователи творения их умов. Но кто нам запретил использовать с пользой (простите за тавтологию) плоды их творчества?

Кто и что вам запретил использовать?

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

Хм, все да не все. Вынужден не согласиться с этим утверждением.

Можете не соглашаться. Команда разработки .NET не согласна с вашим несогласием.

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

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

Из какого моего тезиса вы вывели "бесполезность" кода?

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

Вы не допускаете мысли, что я, возможно, что-то знаю, раз так говорю? Тут, как у евреев - встречный вопрос: с чего вы так решили, что я не знаю о чем говорю? Вы явно не считаете себя истиной в последней инстанции. Так от чего это?

Слишком однозначные и резкие выводы о чём-либо всегда являются признаком поверхностного познания темы. Да себя вы явно поставили выше ребят из MS. Тоже сомнения это вызывает.

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

Эти люди никому не сказали, что все вы тут лохи, потому что не используете goto. Они это в натуре знают. Смешно.

Ещё раз, для упёртых: они используют goto как необходимость, для того, чтобы подстроиться под особенности JIT'а, который будет генерировать максимально производительный код. Они никому ничего не говорят и говорить не должны. Сами придумали чушь, сами над ней смеётесь?

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

Не спорю. Оптимизировали все, кто что только мог. Ни одного goto за 12 лет. Ни одного. Может быть только вы таким образом оптимизируете, но никому не показываете?

Я так ничего не оптимизирую. От меня такого не требуется. Мой код ещё ни разу не упирался в возможности CLR или JIT. Узким местом всегда выступают другие вещи. И то, что вы там за 12 лет нигде goto не использовали значит, что вам тоже не доводилось обходить вездесущие проверки, что JIT вставляет. Из этого какой вывод можно сделать? Что вообще никому это не нужно, а те, кто используют goto - обезьяны без права на помилование?

@wizard41 · 11.08.2021, 14:10

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

Команда разработки .NET не согласна с вашим несогласием.

А зря. Но, не о них речь.

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

Из какого моего тезиса вы вывели "бесполезность" кода?

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

Как правило, от пользовательского кода требуется именно производительность труда разработчика и только в последнюю очередь от кода.

Из этого. Что вы этим хотели сказать?

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

Ещё раз, для упёртых: они используют goto как необходимость, для того, чтобы подстроиться под особенности JIT'а,

Сколько много бесполезных слов. Мы с вами JIT разрабатываем? Прочтите еще раз вопрос темы.

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

Да себя вы явно поставили выше ребят из MS

От куда это следует? Вы сами себя слышите то?

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

Я так ничего не оптимизирую. От меня такого не требуется. Мой код ещё ни разу не упирался в возможности CLR или JIT.

Отлично. Теперь еще раз перечитайте вопрос и скажите: насколько оправданно применение goto в повседневной жизни.

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

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

И так, мой вопрос - вы часто используете goto? В своей практике?

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

Ни разу не доводилось)

Так что вы мне тут доказываете, в итоге?

@Wolfdp · 11.08.2021, 14:43

На какой странице можно писать модераторам, что тему пора переносить в священные войны?

@Элд Хасп · 11.08.2021, 14:56

Wolfdp, эта тема имеет непосредственное отношение именно к этому разделу.
Переносить не следует.

@wizard41 · 11.08.2021, 15:42

Wolfdp, Надо срочно открыть раздел "Войны". Возможно, священные

@HF · 11.08.2021, 15:46

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

Можно спросить экспертов, чем заменить это, без goto:

Знать бы что подразумевается под "statement", тогда скорее всего и варианты будут другие. Предположу что это что-то вроде канселейшенТокена. Мол выходим если условие успешное. Но такая обработка крайне подозрительна, если мы не выходим, а продолжаем. Не обработав ничего. Если "statement" где-то внутри циклов меняет состояние, то тоже вариант выхода может быть другим.

Сообщение от Рядовой

вынести в метод и сделать ретурн. Можно в лямбду и сделать ретурн, код внешне будет практически идентичен вашему.

Хороший вариант. Сначала не понял о чём речь.
Но заметил минус такой реализации - такие методы приходится нагружать параметрами. Так как уже видно что извне мы 2 параметра используем. А часто бывает что побольше. Сталкивался не раз.

@ashsvis · 11.08.2021, 15:59

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

Знать бы что подразумевается под "statement"

Пример кода взят у __Corey, и что он подразумевает под "statement", известно только ему.
От меня вопрос был про выход из циклов глубокой вложенности (необязательно 2, как в примере)

@HF · 11.08.2021, 16:10

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

Если вас поставить в их положение, когда от кода требуется 146% производительности, то вы тоже начнёте озадачиваться тем, а что же JIT будет генерировать в той или иной ситуации. Тогда, вдруг, окажется, что плохие практики, типа goto, не такие уж и плохие, когда дело доходит до оптимизаций. Оптимальный код и так уродлив.

Две интересные темы про использование goto и оптимизацию.
https://aakinshin.net/posts/inlining-and-starg/
https://ayende.com/blog/183553-A/using-goto-in-c

Результаты опроса: Используете ли Вы оператор goto?
Программирую исключительно на goto	4	10.00%
Использую, если имеет смысл	22	55.00%
Категорически не использую	13	32.50%
Использую, но никто об этом не знает	1	2.50%
Голосовавшие: 40. Вы ещё не голосовали в этом опросе

Новые блоги и статьи Все статьи Все блоги /
Саморегулирующийся социальный контракт для сервера cross-section. Hrethgir 14.08.2026 С кодом конечно таких глубоких размышлений пока не было, впрочем я уже привык к алгоритмизации. Суть предмета записи: снова в диалоге с нейросетью (я взял пока себе ник для учётки админа - Rector). . . .	Часы электронные Uhbif79 12.08.2026 Выкладываю программу часов. Программа позволяет: 1. Использовать системное время и дату, 2. Есть возможность вводить время и дату вручную. 3. Реализованы 2 будильника: начало и конец рабочего дня. . . .	Часы с будильником на основе класса QLCDNumber Uhbif79 12.08.2026 Всем добрый день, выкладываю программу часов с будильником на основе класса QLCDNumber. Здесь я пробовал самостоятельно создавал классы, впервые столкнулся с видимостью переменной одного класса из. . .	Установка MinGW GCC 16.2 и CMake 8Observer8 10.08.2026 VK Видео: https:/ / vkvideo. ru/ video-240781534_456239017 YouTube: eY5-5PyI9NM Текстовая версия
Неделя из жизни имитационной модели склада: мои кривые руки растут, откуда надо anaschu 10.08.2026 Неделя из жизни имитационной модели склада: как я почти написал неправильную логику и что с этим делать Работаю сейчас над учебно-рабочим проектом: строю в AnyLogic имитационную модель процессов. . .	Калькулятор для расчета родства russiannick 07.08.2026 1. Задача: Создать калькулятор для расчета родства. Родственных связей существует 8 ступеней, такие как: p - отец P - мать q - муж Q - жена b - брат B - сестра s - сын S - дочь	Мир по моей воле kumehtar 07.08.2026 Когда-то кажется, что всё просто. Ты весь такой светлый. Причиняешь добро. Борешься за справедливость в этом тёмном мире. Потом начинаешь замечать одну неприятную вещь. Почти каждый хороший. . .	Кредитный калькулятор Maks 05.08.2026 Решение задачи по прикладной информатике средствами 1С. Задача: Напишите приложение-калькулятор, которое помогает рассчитывать параметры кредита для аннуитетного и дифференцированного видов. . .

@Wolfdp 3790 / 1767 / 371 Регистрация: 15.06.2012 Сообщений: 6,543 Записей в блоге: 3
	11.08.2021, 14:43
	На какой странице можно писать модераторам, что тему пора переносить в священные войны? 1

@Элд Хасп Модератор 16165 / 11285 / 2891 Регистрация: 21.04.2018 Сообщений: 33,174 Записей в блоге: 2
	11.08.2021, 14:56
	Wolfdp, эта тема имеет непосредственное отношение именно к этому разделу. Переносить не следует. 1

@wizard41 3461 / 2782 / 575 Регистрация: 04.09.2018 Сообщений: 8,746 Записей в блоге: 3
	11.08.2021, 15:42
	Wolfdp, Надо срочно открыть раздел "Войны". Возможно, священные 0