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

    +151

    1. 1
    $o = ($o < 0) ? 0 : $o;

    Оооо, как меня задолбал такой код

    Запостил: sogekoder, 08 Августа 2010

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

    • ava Kornew 08.08.2010 11:34 # +3

      могло быть так:
      $o=$o<$O?$O:$o;

      или так:
      $o=$o<o?o:$o;

      зы: байан -)
      Ответить

    • ava WGH 08.08.2010 12:10 # +2 

      $o = max(0, $o);
      Ответить

      • ava Kornew 08.08.2010 14:58 # +2

        имхо, это будет хуже, чем сабж:
        ибо, вызов функции
        Ответить

        • ava Dalim 08.08.2010 15:20 # −3

          Экономия на спичках - это лучше удобства?
          Ответить

          • ava Kornew 08.08.2010 18:38 # +3

            не совсем уж и на спичках;
            вот результат теста:
            простой цикл:
$c = ($a<$b)?$a:$b;

стандартная функция:
$c = max($a, $b);

пользовательская функция:
function max_ab($a, $b) { return ($a<$b)?$a:$b; }
$c = max_ab($a, $b);

количество тестов: 10 
количество циклов: 1000000 

среднее время выполнения:
time[простой цикл]: 0.1905001c
time[вызов стандартной функции]: 0.4586536c
time[вызов пользовательской функции]: 0.4634484c


            как видно:
            - сам вызов функции занимает довольно много времени
            (в данном тесте, на вызов функции тратится времени больше, чем на выполнение тела этой же функции);

            конечно, для мелочных проектов, которые рассчитаны на маленькую нагрузку - это не играет роли;
            а если какой-то крупный проект, с большим количество запросов -- то такая вот мелочь может вызвать довольно большие задержки;

            (хотя если это работа фрилансера, то после сдачи проекта - дальнейшая судьба проекта фрилансера не волнует)
            Ответить

          • ava ZX_Spectrum 09.08.2010 04:59 # +2

            да ну его, такое удобство... мах-ом элементарные операции заменять - изврат.
            Ответить

          • ava 3.14159265 09.08.2010 10:05 # 0

            блин надо еще для присваивания и инкремента ввести функции - ну так чисто для удобства.
            let($o,0)
            inc($o_0)
            Ответить

            • ava 3.14159265 09.08.2010 10:07 # 0

              и для сравнения тож
              cmpEq($o,0)
              cmpLower($o,0)
              удобно, хуле
              Ответить

        • ava Анонимус 10.08.2010 16:15 # 0

          Мне всегда так нравится, когда в PHPшных проектах в серьез говорят об оптимизации
          Ответить

          • ava Lure Of Chaos 10.08.2010 21:41 # +1

            (чуть перефразируя)
            мне всегда так нравится, когда в серьезных проектах говорят о PHP
            Ответить

    • ava paranoid 09.08.2010 18:19 # 0

      смайлокодер ))
      Ответить

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