Проблема с удалением пользователя из БД

[arakula]

В общем я написал код который выводит пользователей и кнопку DELETE напротив них,
проблема в том что по циклу в переменную $login я пытаюсь положить нужный мне параметр а именно логин пользователя напротив кнопки,но переменная $логин постоянно перезаписывается и поэтому удаляется не тот пользователь который нужен а последний.
Заранее благодарю за помощь.
if(!empty($otvet)){
while($kek=mysqli_fetch_assoc($otvet)){
echo "</br>";
echo $kek['login'];
echo $knopka;
$login=$kek['login'];
}
}
if(isset($_REQUEST['delete'])){
$query=('DELETE FROM users WHERE login = "'.$login.'"');
mysqli_query($link,$query);
}

 

[c0dex]

Ну, мы сразу все поняли...

 

[WMix]

вероятная проблема, не понимание как работает php

index.php

while($kek=mysqli_fetch_assoc($otvet)){
echo $kek['login'].' <a href="delete.php?login='.$kek['login'].'>удалить</a><br />";
}

delete.php

if(isset($_REQUEST['login'])){
$query=('DELETE FROM users WHERE login = "'.$_GET['login'].'"');
mysqli_query($link,$query);
}

 

[arakula]

вероятная проблема, не понимание как работает php

index.php

while($kek=mysqli_fetch_assoc($otvet)){
echo $kek['login'].' <a href="delete.php?login='.$kek['login'].'>удалить</a><br />";
}

delete.php

if(isset($_REQUEST['login'])){
$query=('DELETE FROM users WHERE login = "'.$_GET['login'].'"');
mysqli_query($link,$query);
}

Да я полный новичок,спасибо за решение!

 

[scorpion-ds]

Удаление чего либо, а тем более пользователей GET запросами преступление против человечества. К примеру какой-то "умный" браузер, который делает пред загрузку страниц сайта, удалит сразу все что попадется ...

 

[arakula]

А как бы было правильно это сделать?

Удаление чего либо, а тем более пользователей GET запросами преступление против человечества. К примеру какой-то "умный" браузер, который делает пред загрузку страниц сайта, удалит сразу все что попадется ...[/QUOTE

 

[scorpion-ds]

А как бы было правильно это сделать?

В идеале DELETE, но можно и POST запросом.

 

[vvm]

Я бы написал микропроцедурку, которая получает данные от клиента и процедуру удаления.
Тогда получится раздельный независимый код.

 

[scorpion-ds]

Лучше реализовать отдельный микросервис на отдельном VPS который будет принимать единственную команду DELETE, сам сервис при помощи RabbitMQ передает команду основному сревису на удаление пользователя, сложно, но модно.

 

[michelle]

@scorpion-ds, сударь, вы забыли что этот микросервис должен работать в докере, как и раббит и покрыт селениум тестами минимум на 200%.

 

[nefone]

А как бы было правильно это сделать?

Выполнение действий лучше POST запросом и проверять:

<input type="submit" name="action" value="Delete">

if (isset($_POST['action']) and $_POST['action'] == 'Delete') {
//удаляем запись
}