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

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

      Как говорил наш лектор по Си: "У Microsoft всё, что системное, с одним подчерком, что жутко системное - с двумя. Если придумают что-то вообще супер-пупер-страшное, оно будет с тремя подчерками!"
      Ответить
    • Помню, у меня в универе многие преподы подписывали комментами окончания блоков)
      Ответить
      • Большому блоку - большой комментарий
        Ответить
      • Ну, если там высрали функцию на 100500 строк, в которой на 10 уровней вложены циклы и условия, то лишним явно не будет ;)
        Ответить
        • Сама такая функция будет лишней… >:-P
          Ответить
          • 100500 чистого, непорочного говна не могут быть лишними
            Ответить
        • - Если функция умещается на экране - визуально проще определить где она кончается по оступам, без чтения коммента в конце
          - Поменяли название функции авторефакторингом - комментарий пошли менять руками или не пошли и он начал вводить в заблуждение
          - От того что я вижу что это конец такой-то функции мне не на много легче
          - Даже Notepad++ подсвечивает скобочки - проще поставить курсор и посмотреть - где подсветилась открывающая скобка - чем писать и читать этот мусор
          - Зачем вообще добавлять "лепнину" к стандарным ключевым словам и скобкам? Это так же упрощает кодирование, как тонирование стёкол машины упрощает вождение.
          Ответить
          • > Если функция умещается на экране
            Безусловно. Функция больше экрана должна вызывать подозрения, хотя и вполне допустима.

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

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

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

          Вот это настоящий ад. У меня без преувеличений ушло три часа на то, чтобы найти где if заканчивается. (А еще, как оказалось, в VBS иф можно начать в одном инклюде, а закончить - в другом).
          Ответить
      • Ну как, окончил универ?
        Ответить
    • Есть такой редактор для 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>
      Ответить
      • Уж не знаю с какой целью они этот генератор мусора делали.
        Видимо в силу славных традиций генерации документации для либ, которая полностью повторяет названия методов и параметров и ни слова больше.
        Ответить

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