PHP для ошибки цикла Неопределенное смещение

Question

У меня есть код ниже, который проходит через выбранные флажки и выполняет и оператор SQL.

if (isset($_POST['delete']))
{
   for($i=0;$i<$recordcount;$i++)                       
   {                        
       $deleteid = isset($_POST['checkbox'][$i]);
       echo "ID = ".$deleteid."<br />"; //Error checking
       $sqldelete = "DELETE FROM customer WHERE cus_ID = ".$deleteid."";
       echo $sqldelete."<br />"; //Error checking
       $deleters = $conn->Execute($sqldelete);                  
   }
}

Это запускает запрос SQL, но он пытается выполнить его еще раз, чем требуется, то есть я отмечаю два флажкаsql работает 3 раза.Так что это приводит к ошибке, так как идентификатор пуст.Я также получаю неопределенное смещение 1.

Любой совет?Благодарю.

Answer 1

Эта строка определенно неверна:

$deleteid = isset($_POST['checkbox'][$i]);

Может, ты это имел в виду?

$deleteid = isset($_POST['checkbox'][$i]) 
    ? intval($_POST['checkbox'][$i]) : false;
if ($deleteid === false) {
    continue;
}

// rest of the code follows

Кроме того, вы, вероятно, могли бы избавиться от $recordcount и сделать foreach над $_POST['checkbox'].

Answer 2

if (isset($_POST['delete']))
{
   for($i=0;$i<$recordcount;$i++)                       
   {                        
       $deleteid = isset($_POST['checkbox'][$i]);
       if ($deleteid) {
          $sqldelete = "DELETE FROM customer WHERE cus_ID = ".$_POST['checkbox'][$i];
          echo $sqldelete."<br />"; //Error checking
          $deleters = $conn->Execute($sqldelete);                  
       }
   }
}

И избежать вашего ввода для безопасности

Answer 3

попробуйте

if(isset($_POST['delete']))
    {
       $i=0;
        foreach($_POST['checkbox'] as $user)
        {
            $sqldelete = "DELETE FROM customer WHERE cus_ID = ".$user."";
           echo $sqldelete."<br />"; //Error checking
         $deleters = $conn->Execute($sqldelete); 
            $i++;
        }

    }