"PHP" / Говнокод #1799 Ссылка на оригинал

0

  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 gorky, (Updated )

Комментарии (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'а.
        Ответить
  • Реальный говнокод. Из-за такого правописания потом приходится перелопатить пол скрипта в поисках того, что выполняет эта функция(
    Ответить
    • Т.е. для Вас удобнее перелопатить весь проект, чтобы поменять формат даты, чем изменить это в одном месте?

      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. Почему - читайте мануалы.
    Ответить

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

Где здесь C++, guest?!

    А не использовать ли нам bbcode?


    8