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

    • ava Altravert 18.08.2010 08:07 # 0

      Расчет мог быть на то, что в этом вызове с dt произведутся действия извращенного характера.
      Ответить

      • ava absolut 18.08.2010 09:53 # +1

        Правда название функции и комментарий говорит об обратном.
        Хотя вполне может статься, что о DateTime.UtcNow сначала не знали и использовали какие-то хитрожопые вычисления.
        А потом случился рефакторинг.
        Ответить

        • ava Altravert 18.08.2010 09:57 # 0

          Кстати да, тоже.
          А что с названием и комментом? Мало ли, какую-нить хрень с часовыми поясами мутить может. Хотя может это что-то шарповое, я даже DateType в глаза не видел)
          Ответить

          • ava absolut 18.08.2010 10:24 # 0

            >А что с названием и комментом?
            GetCurrentTimeUtc, Returns current UTC time - текущее UTC время,
            стало быть никаких "действия извращенного характера" не планируется.
            >DateType в глаза не видел
            А во что видел ? :))
            Ответить

    • ava musuk 27.08.2010 07:38 # +1

      Обычное дело после рефакторинга: заменили кусок говнокода вызовом стандартной функции. Но саму функцию убирать нельзя, ибо интерфейс нарушится и клиенты ассембли огребут. Такого добра кстати довольно много если код писался под старые дотнеты, а потом мигрировался в новые: свой говнокод меняли на стандартные функции, а интерфейсы не меняли.
      Ответить

      • ava absolut 27.08.2010 09:46 # 0

        Только вот всё равно не дорефакторили.
        Ответить

      • ava LostPain 09.10.2010 00:06 # 0

        Какие интерфейсы? Метод статический! D
        Ответить

        • ava musuk 09.10.2010 03:45 # +1

          Есть библиотека, у библиотеки есть классы, у классов есть методы. Набор методов и классов библиотеки есть её интерфейс, его используют стронние продукты. Если убрать функцию, то сторонний продукт может тупо не заработать, ибо вызов останется.
          Если я слаб в терминологии, тогда поправьте меня.
          Ответить

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