1. Pascal / Говнокод #3125

    +97

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    /// До этой строчки еще больше 1000 строк кода (И это всё в ОДНОЙ процедур)
 finally
    if Main.RecordCount > 0
    then Main.First;
    Main.EnableControls;
    Panel2.Color:=clLime;
    lbTimeSpend.Caption:='Âðåìÿ çàòðà÷åíî:'+TimeToStr(time()-TimeWork);
  end;

  Exit;

  try
    if TmpCollect_Skd.IsSelectAll then
    begin
      Askd_count := TmpCollect_Skd.RxDBGrid1.DataSource.DataSet.RecordCount;
    end
 /// После этой строки еще более 1000 строк кода всё в этой же процедуре!

    Продолжим. Вот так люди используют Exit;

    Запостил: goodron, 28 Апреля 2010

    Комментарии (36) RSS

    • ava guest 28.04.2010 17:09 # +1

      Соль нужна. Напостил тут блин с три короба.
      Слабо было написать так? :
      //код...
      Exit;
      //никогда не выполняющийся код...
      Ответить

      • ava guest 28.04.2010 17:16 # 0

        Так не интересно, если всё выкинуть, то будет слишком банально.
        Ответить

      • ava murz 28.04.2010 17:18 # +2

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

        • ava guest 28.04.2010 17:23 # 0

          Так это одно когда какое-то условие верно и там выход, а тут в любом случае выход.
          Ответить

          • ava murz 28.04.2010 17:48 # 0

            В моих случаях выход был всегда, просто таким образом отрезал старый/не нужный код
            Ответить

        • ava turdman 28.04.2010 17:25 # 0

          Но как только такое попадает в production, оно становится 100% говном.
          Ответить

    • ava guest 28.04.2010 18:08 # −12

      показать все, что скрытонаконец таки запостили с++
      Ответить

      • ava guest 28.04.2010 20:37 # 0

        :p
        Ответить

      • ava guest 28.04.2010 21:01 # +2

        А ты невьебенно плохой мальчик.
        Ответить

        • ava guest 28.04.2010 23:13 # −1

          Хорошие мальчики пишут «невъебенно» с твёрдым знаком.
          Ответить

          • ava guest 28.04.2010 23:44 # −1

            Ну я не такой уж и хороший... Чуть чуть хороший. :)
            Ответить

            • ava guest 28.04.2010 23:49 # −1

              хоръоший.
              Ответить

              • ava guest 28.04.2010 23:49 # −1

                Косишь под хорошего?
                Ответить

                • ava guest 29.04.2010 15:43 # −2

                  Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?Где здесь С++?
                  Ответить

            • ava guest 28.04.2010 23:49 # −1

              Вы сама скромность.
              Ответить

    • ava guest 29.04.2010 01:53 # +1

      Самое главное же здесь Âðåìÿ çàòðà÷åíî:!
      Именем Âðåìÿ çàòðà÷åíî: объявляю говнокод удачным!
      Ответить

      • ava murz 29.04.2010 07:04 # 0

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

        • ava guest 29.04.2010 08:46 # +2

          Не совсем. Это официальный глюк броланда, продукции броланда, и софта, написанного на продукции броланда. Эти чумовые сосиски что-то намудрили, поэтому:
          - из продукции броланда (цбуилдер, делфы) во всё, что угодно, кроме продукции броланда русский текст вставляется вот таким образом.
          - из продукции броланда и софта, написанного на продукции броланда в аналогичные им продукты вставляется просто зашибись.
          В этом случае автор при копировании открыл окно с дельфёй, а магии не делал.
          Если нужно вставить из продукции броланда в приличное приложение, нужно выполнить следующие магические действия:
          - переключить раскладку в окне на "русский";
          - скопировать из нужного окна нужный текст, выделив его;
          - можно безопасно вставлять в любое приложение, кодировка верная.

          Инфа 100%, я гарантирую это как цбуилдерник со стажем.
          Ответить

          • ava murz 29.04.2010 09:54 # 0

            За способ с переключением раскладки огромное спасибо.

            Но это встречается не только в продукции борланда. Например, есть такая программка Homesite для верстки сайтов, при копирование русского текста, например, в ворд наблюдается точно такой же глюк
            Ответить

            • ava guest 29.04.2010 11:09 # 0

              > и софта, написанного на продукции броланда
              Macromedia Homesite 5.5, что подвернулась под руку:
              Borland Delphi 4.0 - 5.0
              Ответить

          • ava cfdev 29.04.2010 19:06 # 0

            "продукции броланда" - это эвфемизм? :)

            а я знал, что в борланде говнокодеры сидят
            Ответить

          • ava guest 04.05.2010 16:24 # 0

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

            • ava guest 05.05.2010 18:56 # 0

              В этом есть магия броланда, которые не осилили юникот.
              Ответить

              • ava guest 31.05.2010 20:36 # 0

                procedure TForm1.FormCreate(Sender: TObject);
                var
                s: string;
                begin
                s := 'редиска';
                end;

                end.

                Может быть давайте обсудим еще почему в TPfW не было поддержки юникода?
                Ответить

        • ava guest 03.05.2010 10:55 # +1

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

    • ava ganja_boy 04.05.2010 15:22 # 0

      не факт, что человек открывал исходник именно в Дельфях, и еще неизвестно каким браузером он пользуется. по-этому глюк кодировок вполне объясним. а по поводу кода, то ниче нет плохого в использовании exit, было интереснее посмотреть на несколько строк из той тысячи сверху. что-то мне подсказывает, что там обработка базы данных вручную. если так, то похоже на гавнокод
      Ответить

    • ava guest 04.05.2010 16:29 # 0

      Это не говнокод, это говнокамент...
      Лучше бы обернули в {$IFDEF NOT_DEFINED}/{$ENDIF}, а еще лучше юзать системы контроля версий и вытирать атавизмы из исходников без боязни невозвращения
      Ответить

    • ava guest 18.05.2010 22:17 # −1

      это не говнокод)
      с Exit просто косяк.
      А вообще, говнокод должен быть краток, суров, безжалостен и беспощаден.
      Ответить

    Добавить комментарий