Удаление определенной записи, сделав функцию удаления

Question

One мою страницу я называю это функция удаления

$Sql=mysql_query("SELECT Id FROM products"); 
    $Result=mysql_fetch_row($Sql); 
    $Id=$Result[0]; 
    DeleteRecords("products","Id",$Id) 

на functions.php я сделал эту функцию ниже

function DeleteRecords($Table,$Exp,$Value) 
{ 
if(isset($_GET['delete'])) 
{ 
$Sql="DELETE FROM $Table WHERE $Exp=$Value"; 
$Result=mysql_query($Sql); 
if($Result) 
    { 
     echo "<script>alert('Record Deleted')</script>"; 
     echo "<script>window.location='products.php';</script>";  
    } 
} 
} 

Теперь, когда я нажав на ссылку удалить:

<a href='".$_SERVER['PHP_SELF']."?delete=$Id'>Delete</a> 

Его удалить первый ряд. кажется, есть некоторая проблема с $ Id

Пожалуйста, помогите мне, как удалить выбранную мной строку;

Лучшее было бы все, что должно быть сделано в функциях, которые мне нужно просто вызвать функцию на моей странице!

Answer 1

Вы выбираете все продукты:

$Sql=mysql_query("SELECT Id FROM products"); 

Затем вы назначаете идентификатор первого элемента $Id:

$Id=$Result[0]; 

Затем вы проходите $Id к вашей функции - то же самое значение называется $Value там :

DeleteRecords("products","Id",$Id) 

И наконец, в вас r вы проверяете, установлен ли $_GET['delete']. И если да, то вы используете $Id от первого элемента результата, прежде чем в удалении запроса (вместо переданного значения в $_GET['delete']:

$Sql="DELETE FROM $Table WHERE $Exp=$Value"; 

Таким образом, вы никогда не использовать значение $_GET['delete']

---

.

Вы могли бы написать:

DeleteRecords("products","Id", $_GET['delete']); 

и функции:

$Value = intval($Value); 

if (0 != $Value) { 
    $Sql="DELETE FROM $Table WHERE $Exp=$Value"; 
    $Result=mysql_query($Sql); 
    […] 

Это, по крайней мере, предотвращение инъекции SQL. Не подходит для всех ситуаций, но является отправной точкой. Есть хороший сайт о "Bobby Tables", который может дать вам больше информации об этом.