Po pierwsze, twoja kolejność oceny jest nieprawidłowa. Nie powinieneś ustawiać zmiennej z wartością POST PRZED sprawdzeniem, czy wartość POST w ogóle istnieje. Powinieneś sprawdzić jego istnienie, a następnie ustawić go w zmiennej tylko wtedy, gdy istnieje.
$id = $conn->lastInsertId(); // obtained from above (*)
if (!isset($_POST['services'])) {
echo 'Nothing Selected';
} else {
$services = $_POST['services']; // array(0 => 1, 1 => 2, ...)
Po drugie, zakładam, że masz już połączenie sprzed (*) - więc nie ma potrzeby ponownego łączenia. Ponieważ Twoje zapytanie jest krótkie, możesz użyć ?
aby powiązać parametry, jak pokazano w Przykład #3
.
try {
$stmt = $conn->prepare('INSERT IGNORE INTO marketing_lookup SET domain_id = ?, service_id = ?');
foreach ($services as $serviceId) {
$stmt->execute(array($id, $serviceId));
}
} catch (PDOException $e) {
echo $e->getMessage();
}
}
$conn = null; // pointless
Możesz zajrzeć do transakcji jak robisz wiele wstawek.