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

    +155

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    // Отладочная информация
if(1)
{
	print "<!--\r\n";
	$time_end = microtime(true);
	$exec_time = $time_end-$time_start;
  
  	if(function_exists('memory_get_peak_usage'))
		print "memory peak usage: ".memory_get_peak_usage()." bytes\r\n";  
	print "page generation time: ".$exec_time." seconds\r\n";  
	print "-->";
}

    Simpla CMS, красавцы! :D

    Запостил: volter9, 05 Сентября 2014

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

    • ava bormand 05.09.2014 20:35 # +1

      А что не так?
      Ответить

      • ava inkanus-gray 05.09.2014 20:46 # +1

        Говно не в этом коде, говно в php: http://php.net/manual/en/function.microtime.php#101628

        Если во время исполнения скрипта подвести часы, то в $exec_time будет нереальное значение.
        Ответить

        • ava bormand 05.09.2014 20:49 # +1

          А во время перевода часов назад - даже отрицательное.
          Ответить

          • ava inkanus-gray 05.09.2014 20:58 # +2

            А разве microtime() от местного времени зависит? Оно же вроде по UTC. Или замечание на случай, когда админы не парятся с часовыми поясами и ставят серверное время по местному?
            Ответить

            • ava bormand 05.09.2014 21:06 # 0

              Резонно. Не должно зависеть.

              По умолчанию microtime() возвращает string в формате "msec sec", где sec представляет собой текущее время, прошедшее с начала Эпохи Unix (1 января 1970 0:00:00 GMT) в секундах, а msec - это количество микросекунд, прошедших после sec.
              Ответить

        • ava Arbatan_Pokrovan 28.10.2020 23:48 # 0

          Не понял, поясни.
          Что значит «подвести»?
          Ответить

      • ava guest 05.09.2014 22:39 # 0

        If (1) вам ничего не говорит?
        Ответить

        • ava bormand 05.09.2014 22:42 # 0

          Так говно только в этой мелочи?
          Ответить

          • ava guest 05.09.2014 22:45 # 0

            Ну еще несколько print'ов в строчку, вам вообще видимо не угодить )
            Ответить

            • ava bormand 05.09.2014 22:53 # +2

              > несколько print'ов в строчку
              Где? По одному принту в строке. Каждый выводит одну строку в stdout.

              > вам вообще видимо не угодить
              Х.з. От кода на пыхе я обычно ожидаю чего-то ужасного и щекочущего нервы. А тут всё вполне прилично и читаемо.
              Ответить

            • ava bormand 05.09.2014 22:56 # +1

              P.S. Ну разве что сама идея сомнительна. Я бы в лог писал эту статистику, а не отдавал браузеру. Ибо юзеру она нахер не сдалась.

              А из лога ее потом всегда можно посмотреть, может быть даже что-то оптимизнуть.
              Ответить

              • ava volter9 05.09.2014 22:59 # +1

                Это да. Проблема еще в Симплы это то что движок коряво поддерживает PHP5.5, не имеет логов и не показывает никаких ошибок, а их там полно, особенно в админке!
                Ответить

          • ava kegdan 06.09.2014 09:10 # +1

            я думаю це не говно, а типа флаг отладочной инфы. потом просто руками ставится 0

            хотя проще было написать переменную isDebug... так что все таки говно
            Ответить

      • ava guest8 28.10.2020 20:33 # −999

        показать все, что скрытоvanished
        Ответить

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