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

    • ava mrbig66 29.04.2010 14:17 # 0

      Говнокод — эталонный.
      Ответить

    • ava Death 29.04.2010 15:46 # 0

      Тут разве что $order=$order; убрать надо
      Ответить

      • ava mrbig66 29.04.2010 16:04 # 0

        Достаточно было написать...
        $order = strtolower($order);
if(!in_array($order,array('id','created_by','created_at','action')))
  $order = 'id';
        Ответить

        • ava guest 29.04.2010 16:07 # 0

          Умный самый?
          Ответить

        • ava Death 29.04.2010 17:33 # −2

          А вот тут не соглащусь... Зачем добавлять проверку в массив, если это только замедляет работу... Через OR вполно нормально, т.к. условий мало идёт.
          Ответить

          • ava mrbig66 30.04.2010 08:11 # 0

            На вкус и цвет. Не известно сколько потом будет условий.
            Ответить

          • ava Klesk 05.05.2010 08:21 # 0

            Фигня. На порядок медленнее что ли будет?
            Ответить

      • ava shmaltorhbooks 29.04.2010 16:24 # −3

        а еще лучше
        else
{
	$order = 'id';
        $order = $order;
}
        Ответить

    • ava DanxilLs 29.04.2010 21:00 # 0

      Говнокод!
      Ответить

    • ava guest 30.04.2010 02:29 # 0

      а вы не думали что $order может идти от $_POST или от $_GET и это чтото типа аля у switch default ... можно было через switch сделать ...
      Ответить

      • ava azzz 04.05.2010 12:14 # 0

        Именно так и есть. Делал сортировку говна($order потом в SQL-запросе использовался) и, чтобы лишнее в гете не пришло, так извратнулся. Конечно же, отрефакторил потом :)
        Ответить

    • ava guest 30.04.2010 10:43 # −5

      показать все, что скрытоА мне нравится... Я бы, правда, так сделал:

      $order = strtolower($order);
      if (strpos(' id created_by created_at action ', " $order ") === false)
      ...
      Ответить

      • ava murz 30.04.2010 22:26 # 0

        Ню-ню. И туда попадет order = 'id created_by', 'created_by created_at', 'created_at action' и т.д.
        Ответить

        • ava guest 04.05.2010 11:03 # 0

          чё-та не понял. Как оно туда попадёт? Т.е. откуда ОНО может прийти - $order = 'id created_by'?. В говнокоде обычно всё строго и однозначно.
          Ответить

    • ava guest 30.04.2010 22:57 # −4

      '
      Ответить

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