Удалить несколько записей с помощью флажков

Question

Я пытаюсь выяснить, как я могу удалить несколько записей, используя флажки. У меня есть таблица на моей странице, которая получает данные из базы данных. Первый столбец для каждой строки имеет флажок, который выглядит следующим образом:

<input type="checkbox" name="checked[]" value='.$row['UserId'].' class="checkbox" />

Мой код выглядит следующим образом:

<?php
// get required includes
require_once(ROOT_PATH.'connections/mysql.php');
require_once(ROOT_PATH.'admin/controls/users_az/error_messages.php');

// declare variables
$msg = '';

// ------------------------------------------------------------------
// DELETE SELECTED USERS
// ------------------------------------------------------------------
if(isset($_POST['btnDeleteSelected']) && isset($_POST['checked']))
{
$checked = array_map(mysqli_real_escape_string($conn, $_POST['checked']));
$list = "'" . implode("','", $checked) . "'";

$delete_selected = mysqli_query($conn, "DELETE FROM users WHERE UserId IN ($list)")
or die($dataaccess_error);

if($delete_selected)
{
    $msg = mysqli_affected_rows($delete_selected).' '.$msg_success;
}
}
elseif(isset($_POST['btnDeleteSelected']) && !isset($_POST['checked']))
{
$msg = $msg_error;
}
?>

Проблема: Естественно, это не работает. Это первый раз, когда я пытаюсь это сделать.

Вопрос: я на правильном пути с этим? Как мне нужно изменить это, чтобы оно работало?

Answer 1

Это приведет к ошибке:

$checked = array_map(mysqli_real_escape_string($conn, $_POST['checked']));

Так как array_map хочет обратный вызов в качестве первого и массив в качестве второго аргумента. Остальная часть настройки довольно хорошая, но как mysqli_real_escape_string требуется соединение, array_map или array_walk не так удобны, вы можете попробовать нормальный foreach петля. В качестве альтернативы, если идентификаторы всегда целые:

$checked = array_map('intval',$_POST['checked']);

... и поместите кавычки вокруг значений в операторе implode (попробуйте ввести целые числа mysql для целочисленных столбцов, строки для других столбцов (char/date/blob/text)).

Answer 2

Просто убедитесь, что каждый элемент вашего массива является целым числом (например, ctype_digit()), и тогда вы можете использовать implode без кавычек, например так:

$list = implode(", ", $checked);

Answer 3

С PHP >= 5.3 вы можете использовать анонимную функцию:

$checked = array_map(function($s) use ($conn) { return mysqli_real_escape_string($conn, $s); }, $_POST['checked']);

В противном случае вы можете вызвать обратный вызов пользовательской функции:

$checked = array_map('myescape', $_POST['checked']);
function myescape($s)
{
    global $conn;
    return mysqli_real_escape_string($conn, $s);
}

или просто вручную зацикливать.