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

    • Это на тот случай если нужна будет проверка ещё и логина. Вдруг ты не тот логин укажешь при удалении или просто захочешь удалить что-то чужое. Например, первый пункт срабатывает для пользователя, а второй - для админа, которому всё равно что удалять.
      Ответить
      • В первом пункте login лишний, если уж есть условие с id, то оно однозначно идентифицирует запись
        Ответить
        • Идентифицирует, но вдруг ты хочешь удалить чужую запись? А так удалишь только ту, автором которой ты сам являешься. Логичнее использовать userID вместо login, но вдруг это тяжело сделать
          Ответить
          • Пардон, я посчитал что id это в данном случае и есть userId :)
            Ответить
    • $id = "1 OR 1"
      Ответить

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