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

    +160

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if($login){
	$sql="DELETE FROM orders WHERE login='$login' AND id=$id";
}else{
	$sql="DELETE FROM orders WHERE id=$id";
}

    $id конечно же PRIMARY

    Запостил: Paradoks, 07 Апреля 2011

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

    • ava Int 07.04.2011 17:12 # 0

      Это на тот случай если нужна будет проверка ещё и логина. Вдруг ты не тот логин укажешь при удалении или просто захочешь удалить что-то чужое. Например, первый пункт срабатывает для пользователя, а второй - для админа, которому всё равно что удалять.
      Ответить

      • ava rO_ot 07.04.2011 23:18 # −1

        В первом пункте login лишний, если уж есть условие с id, то оно однозначно идентифицирует запись
        Ответить

        • ava Int 11.04.2011 10:01 # 0

          Идентифицирует, но вдруг ты хочешь удалить чужую запись? А так удалишь только ту, автором которой ты сам являешься. Логичнее использовать userID вместо login, но вдруг это тяжело сделать
          Ответить

          • ava rO_ot 11.04.2011 18:28 # 0

            Пардон, я посчитал что id это в данном случае и есть userId :)
            Ответить

    • ava rO_ot 07.04.2011 23:17 # 0

      $id = "1 OR 1"
      Ответить

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