1. C# / Говнокод #5268

    +134

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    private static void GetUserData(User user)
        {
            if (user == null)
                return;
    
            var firm = user.Firm;
    
            if (firm == null)
                return;
        }

    Бессмысленно и беспощадно.

    Запостил: Gizz, 13 Января 2011

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

    • кажется, что-то пропущено
      Ответить
      • возвращаемый тип
        Ответить
        • спасибо КО
          Ответить
        • void же
          Ответить
          • Здесь имеется в виду отсутствие типа с семантической точки зрения, а не с синтаксической. Функции с именами Get*() должны возвращать значение, чего в данном случае не наблюдается.
            Ответить
            • почему же? функция GetVoid скажем ;)
              Ответить
              • угу. чтоб ещё она умела разные значения возвращать.
                Ответить
    • Где говнокод? Просто результат рефакторинга. Сейчас функция больше ничего не делает. Просто её забыли удалить.
      Ответить
    • Вот к чему приводит автоматический рефакторинг.
      Ответить
    • Эта история произошла со мной, когда мне едва исполнилось 18 лет. Я рос без отца, наверное, поэтому мне не хватало твердости в характере. Я был очень податливым и легко поддавался чужому влиянию.
      Ответить

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