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

    +170

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    $order = strtolower($order);
    if ($order=='id' OR $order=='created_by' OR $order=='created_at' OR
    	$order=='action')
    {
    	$order = $order;
    }
    else
    {
    	$order = 'id';
    }

    Надо крепче спать и внимательней писать.

    Запостил: azzz, 29 Апреля 2010

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

    • Говнокод — эталонный.
      Ответить
    • Тут разве что $order=$order; убрать надо
      Ответить
      • Достаточно было написать...
        $order = strtolower($order);
        if(!in_array($order,array('id','created_by','created_at','action')))
          $order = 'id';
        Ответить
        • Умный самый?
          Ответить
        • А вот тут не соглащусь... Зачем добавлять проверку в массив, если это только замедляет работу... Через OR вполно нормально, т.к. условий мало идёт.
          Ответить
          • На вкус и цвет. Не известно сколько потом будет условий.
            Ответить
          • Фигня. На порядок медленнее что ли будет?
            Ответить
      • а еще лучше
        else
        {
        	$order = 'id';
                $order = $order;
        }
        Ответить
    • Говнокод!
      Ответить
    • а вы не думали что $order может идти от $_POST или от $_GET и это чтото типа аля у switch default ... можно было через switch сделать ...
      Ответить
      • Именно так и есть. Делал сортировку говна($order потом в SQL-запросе использовался) и, чтобы лишнее в гете не пришло, так извратнулся. Конечно же, отрефакторил потом :)
        Ответить
    • показать все, что скрытоА мне нравится... Я бы, правда, так сделал:

      $order = strtolower($order);
      if (strpos(' id created_by created_at action ', " $order ") === false)
      ...
      Ответить
      • Ню-ню. И туда попадет order = 'id created_by', 'created_by created_at', 'created_at action' и т.д.
        Ответить
        • чё-та не понял. Как оно туда попадёт? Т.е. откуда ОНО может прийти - $order = 'id created_by'?. В говнокоде обычно всё строго и однозначно.
          Ответить
    • '
      Ответить

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