1. PHP / Говнокод #1799

    +150.9

    1. 1
    2. 2
    3. 3
    4. 4
    function get_current_time() //get current date and time as a string
    {
    	return strftime("%Y-%m-%d %H:%M:%S", time());
    }

    апстену... про функцию date() люди не знали...
    старая версия WebAsyst Shop-Script

    Запостил: gorky, 09 Сентября 2009

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

    • Обоснуйте, почему этот код не имеет права на жизнь.
      Ответить
      • да потому что есть универсальная функция date.
        вся эта лабуда записывается как date('Y-m-d H:i:s');
        Ответить
        • Чем strftime не универсальная?
          Она даже по-универсальней будет, чем дате. Парочку аргументов:
          Во-первых, во внутрь можно вставлять произвольный текст
          Во-вторых, точно такой синтаксис поддерживается, например, в MySQL при выборке, да и в других системах тоже
          Ответить
          • Поддерживаю!
            Ответить
          • strftime универсальная. а get_current_time() - нет.
            >Во-вторых, точно такой синтаксис поддерживается, например, в MySQL при выборке, да и в других системах тоже
            не имеет здесь смысла, вот если автор написал
            define('date_format', '%Y-%m-%d %H:%M:%S');
            тогда да
            Ответить
        • gorky, вы не правы. Повторяться почему не буду, поддержу guest'а.
          Ответить
        • всегда вместо date юзаю такой код. Низачот.
          Ответить
    • Реальный говнокод. Из-за такого правописания потом приходится перелопатить пол скрипта в поисках того, что выполняет эта функция(
      Ответить
      • Т.е. для Вас удобнее перелопатить весь проект, чтобы поменять формат даты, чем изменить это в одном месте?

        P.S. Используйте Zend. Тогда с помощью простой комбинации клавиш Ctrl+Shift+G найдете любую функцию в файлах, которые добавлены в проект.
        Ответить
        • ну если уж неохота перелопачивать весь код, то проще написать
          define('date_format', 'Y-m-d H:i:s');
          а потом юзать
          date('date_format');
          >Во-вторых, точно такой синтаксис поддерживается, например, в MySQL при выборке, да и в других системах тоже
          и что из этого?
          Ответить
          • не знаю конечно, может дело привычки, но когда разбираешься в чужом коде, приятнее видеть знакомые слова, чем однострочные самописные функции, которых легко избежать.
            зы пахнет холиваром
            Ответить
    • что тут спорить что лучше, date() или это, а вам не кажется господа что при вызове date() ядро PHP проделает те-же две операции - sprintf и time
      Ответить
      • ну если для компилируемых языков переписывание имеющихся функций не так и страшно (вдруг лучше выйдет даже ;-) ), то для интерпретатора это все же более критично.
        Ответить
    • PHP и универсальные функции несовместимы:)
      Ответить
    • strftime использовать лучше чем date. Почему - читайте мануалы.
      Ответить

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