0

Есть форма

    <h1>Вы уверены что хотите удалить этого пользователя!</h1>
    <form action="" method="post" style="text-align:center;">
        <label><input type="radio" name="yn" value="yes"/>Да</label>
        <label><input type="radio" checked name="yn" value="no"/>Нет</label>    </br>
        <input type="submit" name="vse" value="Подтвердить"/>
    </form>

вот PHP:

    if(isset($_POST['vse'])){
    if(isset($_POST['yn'])=='yes'){
        mysql_query("DELETE FROM student WHERE id='".$_GET['id']."'");
        header("location:home.php?id=".$id);
    }elseif(isset($_POST['yn'])=='no'){
        echo "asd";
    }
}

Проблема в том что при нажатии на радиокнопку НЕТ, пользователь всеравно удаляется(

1
  • 2
    Понимаю, буквы платные, но уж раскошельтесь в местах типа ключей параметров, а то при тимлид при CodeReview Вас шваброй погонит. Как и за "vse".
    – user207618
    22 янв 2017 в 8:13

4 ответа 4

5

Верным будет следующее решение:

if(isset($_POST['vse']) && isset($_POST['yn'])){
    if($_POST['yn']=='yes'){
        mysql_query("DELETE FROM student WHERE id='".$_GET['id']."'");
        header("location:home.php?id=".$id);
    }elseif($_POST['yn']=='no'){
        echo "asd";
    }
}

А теперь самое интересное:
Давайте разберёмся почему у вас срабатывало условие!

if(isset($_POST['yn'])=='yes'){}

Скорее всего Вы хотели чтобы условие сработало если в массиве $_POST есть значение с ключом "yn" и что оно равно "yes".

По факту это две проверки и результатом должно было логичкое сложение двух булевых значений.
Для этого используется оператор &&, он будет возвращать true когда значение справа и слева тоже равны true.
Верным способом сделать это было бы получить поочередно два булевыых значения, а потом уже получить их сумму и её подставлять в условие.
Для понимания я вывел по одной операции на одн строку:

$is_set = isset($_POST['yn']); 
$is_positive = ($_POST['yn']=='yes');
$result = $is_set && $is_positive;
if($result){ }

Или всё тоже самое, но только без временных переменных и в одну строку

if(isset($_POST['yn']) && ($_POST['yn']=='yes')){ }

Я для того, чтобы не писать два раза проверку isset($_POST['yn']) вынес её на уровень выше.

С верным решением то всё понятно - почему тем не менее срабатывало условие в примере?!

Также пошагово разберём Ваш пример:

if(isset($_POST['yn'])=='yes'){}
  1. Сначала Вы проверяете наличие поля в массиве( isset($_POST['yn']) ).
    При проверке через данную форму оно в любом случае будет равно true
  2. Далее Вы сравниваете полученное значение true со строкой
    И как странно это не прозвучит, но сравнение true со строкой 'yes' в результате вернёт true
    И условие срабатывает

Разберём подробнее сравнение булева значения со строкой
на самом деле там происходит не одна операция а две!
Так как изначально сравниваемые значения имеют разные типы данных, то они в итоге сводятся к одному типу.
php является (слабо-) или динамическитипизированным языком и это надо учитывать.

Перед тем как сравнить строку "yes" с булевым значением интерпретатор php приводит её к булеву типу.
И тут кроется ответ: любая НЕПУСТАЯ строка приводится к true!
false даёт при приведении только "" или другими словами пустая строка!

Вот и получается, что Вы сравниваете

if(true == true){}

Что и даёт соответствующий результат!

0
if( $_POST['yn'] == 'yes' ){
...
} else {
0
 if(isset($_POST['vse'])){
    if($_POST['yn']=='yes'){
        mysql_query("DELETE FROM student WHERE id='".$_GET['id']."'");
        header("location:home.php?id=".$id);
    }else{
        echo "asd";
    }
}
-1

Перепише второй if как

if (isset($_GET['smth'] ) && $_GET['smth'] == 'yes')
3
  • Не помогло, все равно удаляет даже при выборе "нет"
    – As Lan
    22 янв 2017 в 7:35
  • Просто в примере указан массив $_GET, а не $_POST 22 янв 2017 в 8:11
  • Да, я писал Post но что то тоже
    – As Lan
    22 янв 2017 в 8:32

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.