Delete mittels Checkboxen

[Tom]

Hi,

BITTESCHÖÖN:

<?php

if($_POST['submit']) {
$db=mysql_connect("localhost","root","");
$select=mysql_select_db("ConferenceSystem",$db);
// Wenn der Submit-Button geklickt wurde ...
//Die Anzahl der markierten Datensaetze ermitteln
$y=count($_POST['loeschen']);
 // Für jeden markierten Datensatz
// den delete-Befehl aufrufen.

Es ist eleganter, alle zu löschenden IDs in einer Query zu übergeben.
Das geht entweder mit mehreren ORs oder mit "ID IN( Liste )"

for($x=0;$x<$y;$x++) {
$datensatz_id = $_POST['loeschen'][$x];
mysql_query("delete from reviewer where person_ID='$datensatz_id'");
mysql_query("delete from person where ID='$datensatz_id'");
}
}     ?>
 .

Du hast keinerlei Sicherheit. Jedermann kann Dir die ganze DB löschen.

Wenn kein "submit" dann gibts keine Connection zur DB

$query = "SELECT surname, email, comments, person.ID, reviewer.person_ID FROM person, reviewer WHERE person.ID = reviewer.person_ID";
  $result=mysql_query($query,$db);

if(!$result)
  {
  die("Fehler bei Abfrage!");
  }
  while ($row = mysql_fetch_assoc($result))

{
  $ID=$row["ID"];
  $surname=$row["surname"];
  $name=$row["name"];
  $email=$row["email"];
  $comments=$row["comments"];

echo "<tr bgcolor=#DFE2DE><td><input type=checkbox name=loeschen[] value=$ID ></td>";

den Datenfeldern braucht man eigentlich gar keine Namen zu geben. Und Input-Objekte müssen das ja auch nicht sein.

echo"<td><input type=text name=Surname[] value=$surname size=30></td>";
  echo"<td><input type=text name=Email[] value=$email size=30></td>";
  echo "<td><input type=text name=Comments[] value=$comments></td>";
  echo "<td><input type=text name=id[] value=$ID></td>";

}
?>
 .
 .
 .
<input type="submit" name="submit" value="Löschen">

</table>

table ist nicht geöffnet
form ist nicht geöffnet
form ist nicht geschlossen

Mhh. Nörgeln kann jeder. Wenn Du es selber gemacht hast, schon ganz gut. Aber eben nicht perfekt. Musst noch ein bisschen dran schnitzen. Und solche Aufgaben sollte man nie ohne Session (Autentifizierung) lösen.

Grundregel: Nur was rausgeschickt wurde, darf auch wieder reinkommen.
            (frei übersetzt)

Grüße

Tom