Slet flere MySQL-poster via Submit knap

Jeg tro vi skal bruge lidt kode fra dig af hvis akl kunne hjælpe dig

<ole>

Hvis dine checkbox'e hedder 'foo[]', kan du i princippet skrive noget à la:

$a = array();
for ($i=0,$j=count($_POST["foo"]); $i<$j; $i++) {
    $a[$i] = "`id`='".$_POST["foo"][$i]."'";
}
$sql = "DELETE FROM `tabelnavn` WHERE ".implode(" OR ", $a);

- men du bør jo som altid tjekke al brugerinput, før det kommer nær databasen  ;o)

/mvh
</bole>

*Jeg tro vi skal bruge lidt kode fra dig af hvis vi skal kunne hjælpe dig

Min nuværende kode ser sådan her ud, men er godt klar over, at det selvfølgelig ikke virker. Jeg kan sagtens liste posterne, og jeg mener også, at navngivningen af chekcboksene er OK. Men det er mere, når jeg skal slette, det kniber.

<?php
if(isset($_GET['update']) && $_GET['update'])
{
    $id    = mysqlRealEscapeString($_POST['name']);
    $sql = "DELETE FROM kundeliste WHERE id = '".$id."'";
    database($sql);

    echo "Kunderne er nu slettet.<br>";
}

echo "<form action='".$_SERVER['PHP_SELF']."?update=true' method=post>";

  $sql = "SELECT * FROM kundeliste";
  $res = database($sql);
  $row = mysql_fetch_array($res);

while($row = mysql_fetch_array($res))
{
    echo "<input type='checkbox' value='1' name='slet".$row['id']."'>";
    echo $row['navn']."<br>";
}

echo "<input type='submit' value='Slet kunder'>
</form>";
?>

Prøv denne kode:

<?php
if(isset($_POST['context']) && $_POST['context']=='update')
{
    $a = array();
    for ($i=0,$j=count($_POST["foo"]); $i<$j; $i++) {
        $id    = mysqlRealEscapeString($_POST['slet'][$i]);
        $a[$i] = "`id`='".$id."'";
    }
    $sql = "DELETE FROM `kundeliste` WHERE ".implode(" OR ", $a);
    database($sql);

    echo "Kunderne er nu slettet.<br>";
}

echo "<form action='".$_SERVER['PHP_SELF']."' method=post>";
echo "<input type='hidden' name='context' value='update'>";

  $sql = "SELECT * FROM `kundeliste`";
  $res = database($sql);
  $row = mysql_fetch_array($res);

while($row = mysql_fetch_assoc($res))
{
    echo "<input type='checkbox' value='".$row['id']."' name='slet[]'>";
    echo $row['navn']."<br>";
}

echo "<input type='submit' value='Slet kunder'>
</form>";
?>

Damned ... en copy/paste fejl! Der skal stå:
    for ($i=0,$j=count($_POST["slet"]); $i<$j; $i++) {

Hmmm ... flere fejl. Sådan:

<?php
if(isset($_POST['context']) && $_POST['context']=='update')
{
    $a = array();
    for ($i=0,$j=count($_POST["slet"]); $i<$j; $i++) {
        $id    = mysqlRealEscapeString($_POST['slet'][$i]);
        $a[$i] = "`id`='".$id."'";
    }
    $sql = "DELETE FROM `kundeliste` WHERE ".implode(" OR ", $a);
    database($sql);

    echo "Kunderne er nu slettet.<br>";
}

echo "<form action='".$_SERVER['PHP_SELF']."' method='post'>";
echo "<input type='hidden' name='context' value='update'>";

  $sql = "SELECT * FROM `kundeliste`";
  $res = database($sql);

while($row = mysql_fetch_assoc($res))
{
    echo "<input type='checkbox' value='".$row['id']."' name='slet[]'>";
    echo $row['navn']."<br>";
}

echo "<input type='submit' value='Slet kunder'>
</form>";
?>

- og for bedre performance burde du nok specificere/minimere din select:
    $sql = "SELECT `id`, `navn` FROM `kundeliste`";

Hej Olebole. Din sidste kommentar er jeg helt med på.
Jeg har også afprøvet dit forslag til kode, og det virker præcis som ønsket. Så tusind tak for denne hjælp.
Jeg vil bare også gerne forstå, hvad der sker.
Jeg forstår nemlig ikke, hvordan du sletter flere poster på én gang, når DELETE kommandoen ikke er en del af "for"-løkken.
Måske fordi de valgte poster er smidt i et Array ($a)??
Send bare et svar.:-))

Hvis du i stedet for at køre sql-sætningen mod databasen, prøver at skrive den ud med echo, vil den se sådan noget lignende ud:

    DELETE FROM `kundeliste` WHERE `id`='4' OR `id`='33' OR `id`='42' OR `id`='67'

Array'et ser sådan ud:
    $a = array("`id`='4'", "`id`='33'", "`id`='42'", "`id`='67'");

Udtrykket:
    implode(" OR ", $a);

- skaber strengen:
    `id`='4' OR `id`='33' OR `id`='42' OR `id`='67'

http://dk2.php.net/manual/en/function.implode.php

Det er en langt mere effektiv måde, end at rette 4 separate queries mod databasen  =)

Super forklaring - så er jeg endnu engang blevet meget klogere.

- Nemlig!  ;D
Jeg ville nok også foretrække at udskrive dine felter på følgende måde:

$aHTML = array();
while($row = mysql_fetch_assoc($res))
{
    $aHTML[] = "<input type='checkbox' value='".$row['id']."' name='slet[]'>".$row['navn'];
}
echo implode("<br>", $aHTML);

Det er somregel en del hurtigere. Tak for points  ;o)

Læg i øvrigt mærke til mine andre ændringer:

Jeg bruger mysql_fetch_assoc i stedet for mysql_fetch_array, der som default henter både et talindekseret array og et associativt array. Vi skal kun bruge det associative:
    http://dk2.php.net/manual/en/function.mysql-fetch-array.php
    http://dk2.php.net/manual/en/function.mysql-fetch-assoc.php

Jeg har ændret koden, så du kun bruger post-variabler (det skjulte felt 'context'). Dels er det mere tro mod idéen bag HTTP metoderne get og post, ikke at blande dem sammen - og dels er det rart at vide præcis, hvor alle ens variabler kommer fra.

Derudover backtick'er jeg alle felt- og tabelnavne i MySQL kald af hensyn til evt. reserverede navne. Ordet 'by' har f.eks. en betydning i MySQL, hvorfor du enten må kalde feltet for 'city' - eller bruge backticks

Ok - tak for de udførlige informationer.

Selvtak  ;o)

Øh - jeg har lige en lille udfordring mere.
Hvis der nu ikke er markeret nogle checkbokse og der klikkes på knappen, så får jeg en MySQL-fejl.
Jeg vil derfor lave et tjek på om Arrayet er tomt, og hvis ja, så skriver jeg en fejltekst.
Men hvad er syntax for om et Array er tomt??

Jeg ville nu:

if(isset($_POST["context"],$_POST["slet"]) && $_POST["context"]== "update" && is_array($_POST["slet"]))
{

$sql = "DELETE FROM `kundeliste` WHERE `id` IN (".implode(",", $_POST["slet"]).")";
database($sql);

}

Showsource: Skal jeg forstå dit forslag som en løsning til at undgå MySQL fejl, hvis der ikke er markeret nogen checkbokse eller mener du, det er bedre kode, end det olebole har leveret eller?????
Og brugerinput bliver vist nok ikke tjekket inden der skrives i databasen. Dette sker jo i den anden kode med funktionen "mysqlRealEscapeString"

Jeg løste problemet med Mysql-fejlen ved at lave et tjek på variablen $j. Hvis den er 0, så springes over DELETE og der gives en fejlmeddelelse.

Nu var jeg ikke klar over andre kunne poste også.
Bedre kode ? næhh

Hvis ingen checkbox er markeret, er $_POST["slet"] ikke sat.
Og for at sikre det er tal som er i array'et "slet"

if(isset($_POST["context"],$_POST["slet"]) && $_POST["context"]== "update" && is_array($_POST["slet"]))
{

$delete = (int)$_POST["slet"][0];
$antal = count($_POST["slet"]);

    for($i = 1; $i < $antal; $i++) {
    $delete .= ",".(int)$_POST["slet"][$i];
    }


$sql = "DELETE FROM `kundeliste` WHERE `id` IN (".$delete.")";

database($sql);

}