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

    +155

    1. 1
    2. 2
    3. 3
    $pageController = new PageController();
    $photo = $pageController->getPhoto($pageRecord->id);
    $photo =  $photo ;

    А вдруг ПХП с первого раза не понял что в переменной должно быть фото? Повторение мать учения.

    Запостил: Kos, 19 Июня 2011

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

    • По лишним пробелам явно видно, что там были какие-то операторы.

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

      P.S. если сильно изъебнутся, можно написать такой PageController, что это присваивание действительно будет что-то делать.
      Ответить
      • > P.S. если сильно изъебнутся, можно написать такой PageController, что это присваивание действительно будет что-то делать.
        каким, простите, образом? это же пхп.
        Ответить
        • Да через tick-и например.
          Ну или вернуть ссылку на несуществующее свойство объекта с __get. Получим возможность выполнять код при каждом обращении к переменной.
          Ответить
          • __get вызывается только при обращении к несуществующему полю. в данном случае - вызовется максимум __toString и то, это невероятно, потому что в этом выражении переменная интерпретируется не как строка.
            Ответить
      • Всё куда прозаичнее: в проект взяли стажера)
        Ответить
    • > А вдруг ПХП с первого раза не понял что в переменной должно быть фото? Повторение мать учения.

      Kos, где там повторение?
      Ответить
      • вона, тама!

        вдруг не все триггеры открылись
        Ответить
    • Да опечатался чувак, должно было быть что-то вроде:
      $this->photo =$photo;
      Ответить

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