1. Java / Говнокод #12589

    +81

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public static void trustAllHttpsCertificates() {
    // Is the deprecated protocol setted?
    if (isDeprecatedSSLProtocol()) {
        __trustAllHttpsCertificates();
    } else {
        _trustAllHttpsCertificates();
    } // else
} // trustAllHttpsCertificates

    Captain Obvious поучаствовал?
    P.S. Похоже, это писал старый программист, закалённый в борьбе с VisualBasic.

    Запостил: wissenstein, 14 Февраля 2013

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

    • ava someone 14.02.2013 17:04 # +10

      Подчерки умилили.

      Как говорил наш лектор по Си: "У Microsoft всё, что системное, с одним подчерком, что жутко системное - с двумя. Если придумают что-то вообще супер-пупер-страшное, оно будет с тремя подчерками!"
      Ответить

    • ava body90 15.02.2013 05:59 # +2

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

      • ava absolut 15.02.2013 06:38 # +2

        Большому блоку - большой комментарий
        Ответить

      • ava bormand 15.02.2013 06:45 # 0

        Ну, если там высрали функцию на 100500 строк, в которой на 10 уровней вложены циклы и условия, то лишним явно не будет ;)
        Ответить

        • ava wissenstein 15.02.2013 14:19 # +5

          Сама такая функция будет лишней… >:-P
          Ответить

          • ava Lure Of Chaos 17.02.2013 17:40 # +2

            100500 чистого, непорочного говна не могут быть лишними
            Ответить

        • ava guest 23.02.2013 14:33 # +1

          - Если функция умещается на экране - визуально проще определить где она кончается по оступам, без чтения коммента в конце
          - Поменяли название функции авторефакторингом - комментарий пошли менять руками или не пошли и он начал вводить в заблуждение
          - От того что я вижу что это конец такой-то функции мне не на много легче
          - Даже Notepad++ подсвечивает скобочки - проще поставить курсор и посмотреть - где подсветилась открывающая скобка - чем писать и читать этот мусор
          - Зачем вообще добавлять "лепнину" к стандарным ключевым словам и скобкам? Это так же упрощает кодирование, как тонирование стёкол машины упрощает вождение.
          Ответить

          • ava bormand 23.02.2013 14:59 # +1

            > Если функция умещается на экране
            Безусловно. Функция больше экрана должна вызывать подозрения, хотя и вполне допустима.

            > проще поставить курсор и посмотреть - где подсветилась открывающая скобка
            Да, а еще проще поставить курсор и нажать "прыжок на противоположную скобку".

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

            P.S. Ну неужели не видно было, что моя фраза про "лишним явно не будет" это сарказм?
            Ответить

        • ava wvxvw 23.02.2013 15:06 # 0

          Мегавложенность - не такая уж и проблема по сравнению с if'ами в шаблонах. Я вот недавно обнаружил такую картинку:
          <% If Request("something") Then %>
 2115 строк месива из HTML + VBS + JavaScript
<% Else %>
 еще строк 500 аналогичного месива
<% End If %>

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

      • ava gEKA6PbCKuu_nemyx 22.12.2021 12:14 # 0

        Ну как, окончил универ?
        Ответить

    • ava guest 23.02.2013 14:14 # +1

      Есть такой редактор для C# - MonoDevelop - так там КО-генератор встроен в систему:
      1. Набираем public bool IsValue() { return false };
      2. Ставим перед этой строчкой ///
      3. Наслаждаемся:
      /// <summary>
      /// Determines whether this instance is value.
      /// </summary>
      /// <returns>
      /// <c>true</c> if this instance is value; otherwise, <c>false</c>.
      /// </returns>
      Ответить

      • ava guest 23.02.2013 14:18 # +1

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

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