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

    +147

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    // ...
    // Количество распиленных досок
    $spl = 0;
    
    // Количество использованных пакетов
    $clpz = 0;
    if ($spl === 0) {
        for ($ix = 1; $ix <= $cll; $ix++) {
            $dsk[$ix] = $dd;
        }
    }
    // ....

    Встретился вот такой код. Условие по моему тут лишнее :)

    Запостил: Joo, 19 Мая 2011

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

    • че-то вообще муть бессвязная какая-то
      Ответить
      • Присваивают переменной $spl 0 и тут же проверяют равна ли переменная нулю, вероятно не доверяют интерпретатору, вдруг он все таки вместо нуля что-то другое туда записал.
        Ответить
        • на этапе отладки и я так делаю
          Ответить
          • На этапе отладки??? По моему если не хочешь чтобы выполнилась какая-то часть кода "на этапе отладки" то можно написать if(true)...,if(false)..., вместо того чтобы городить $flag=0;if($flag === 0)... Я понимаю если есть одна переменная которая отключает/включает разом несколько блоков для отладки, но тут в коде куча таких мест, и везде своя переменная, которая используется всего один раз и устанавливается непосредственно перед условием.
            Ответить
    • > Количество распиленных досок
      O_o
      Ответить
    • показать все, что скрытоvanished
      Ответить
    • показать все, что скрытоvanished
      Ответить

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