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

    +121

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    /// <summary>
            /// Returns current UTC time
            /// </summary>
            /// <returns></returns>
            public static DateTime GetCurrentTimeUtc()
            {
                DateTime dt = DateTime.UtcNow;
                return dt;
            }

    Индус-стайл метод

    Запостил: sem13, 17 Августа 2010

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

    • Расчет мог быть на то, что в этом вызове с dt произведутся действия извращенного характера.
      Ответить
      • Правда название функции и комментарий говорит об обратном.
        Хотя вполне может статься, что о DateTime.UtcNow сначала не знали и использовали какие-то хитрожопые вычисления.
        А потом случился рефакторинг.
        Ответить
        • Кстати да, тоже.
          А что с названием и комментом? Мало ли, какую-нить хрень с часовыми поясами мутить может. Хотя может это что-то шарповое, я даже DateType в глаза не видел)
          Ответить
          • >А что с названием и комментом?
            GetCurrentTimeUtc, Returns current UTC time - текущее UTC время,
            стало быть никаких "действия извращенного характера" не планируется.
            >DateType в глаза не видел
            А во что видел ? :))
            Ответить
    • Обычное дело после рефакторинга: заменили кусок говнокода вызовом стандартной функции. Но саму функцию убирать нельзя, ибо интерфейс нарушится и клиенты ассембли огребут. Такого добра кстати довольно много если код писался под старые дотнеты, а потом мигрировался в новые: свой говнокод меняли на стандартные функции, а интерфейсы не меняли.
      Ответить
      • Только вот всё равно не дорефакторили.
        Ответить
      • Какие интерфейсы? Метод статический! D
        Ответить
        • Есть библиотека, у библиотеки есть классы, у классов есть методы. Набор методов и классов библиотеки есть её интерфейс, его используют стронние продукты. Если убрать функцию, то сторонний продукт может тупо не заработать, ибо вызов останется.
          Если я слаб в терминологии, тогда поправьте меня.
          Ответить

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