Tom: SQL - Delete function

Beitrag lesen

Hallo,

meistens ist es die Faulheit, die eine bestraft ;-)

mysql_query("DELETE FROM tuspgatour WHERE id=".$_GET['nummer'])or die("Fehler in Abfrage.  Fehlercode: ".mysql_errno().": ".mysql_error());

1. alles das, was Tobias Dir auch schon geschrieben hat...
2. Übernimm nie Werte direkt aus externen Quellen (GRT, POST, COOKIE, ...)
3. teile auf in mehrere Schritte.

$id = intval($_GET['nummer']);  ## dadurch ist sichergestellt,
                                ## dass in $id wirklich nur eine
                                ##Nummer steht

$sql = "DELETE FROM tuspgatour WHERE id = $id";
$ok  = mysql_query($sql, $con):
if (!$ok)
{
  ## "die()" ist keine Funktion für fertige Scripte!
  ## Only fpr debug
  die("Fehler in Abfrage. Fehlercode: ".mysql_errno().": ".mysql_error());
}

Die C-Faule-Schweine-Syntax ist es immer wieder, die PHP-Programmierer zum Absturz bringt. Das, was man an Tipparbeit bei der Code-Ersterfassung spart, brauchtg man ungefähr 10-100fach, um die Fehler darin zu suchen.

Gewöhn Dir einen lockeren und ausführlichen Schreibstil an. Das Parsen dauert selten länger, denn die gleichen Funktionen müssen aufgelöst werden.

Du hattest übrins nur ein Anführungszeichen vergessen, aber das sieht man beim C-Faule-Schweine-Stil ( :-)) ) nicht.

Grüße

Tom