nicht angemeldet
echo "<p>Query = ".$query."</p>";
$result = mysql_query($query);
dberror();
echo "Ihre Eingabe war korrekt und wurde in die Datenbank eingetragen!";
$db = mysql_close();
dberror();
>
> dberror() ist definiert wie folgt:
>
> ~~~php
function dberror()
> {
> if (mysql_errno())
> {
> switch(mysql_errno())
> {
> case 1062:
> die ("Ein Eintrag mit diesen Daten existiert bereits! Bitte korrigieren Sie Ihre Eingabe!<br>");
> break;
> default:
> die (mysql_errno().": ".mysql_error()."<br>");
> }
> }
> }
>
Review:
> $db = mysql_close();
Du schließt also die Verbindung bevor Du
> dberror();
aufrufst und darin dann mysql_errno() und mysql_error(). Was soll das bringen? In der Dokumentation steht zu den Parametern:
Zitat:
Verbindungs-Kennung:
Die MySQL-Verbindung. Wird die Verbindungskennung nicht angegeben, wird die letzte durch mysql_connect() geöffnete Verbindung angenommen. Falls keine solche Verbindung gefunden wird, wird versucht, eine Verbindung aufzubauen, wie es beim Aufruf von mysql_connect() ohne Angabe von Argumenten der Fall wäre. Falls zufällig keine Verbindung gefunden oder aufgebaut werden kann, wird eine Warnung der Stufe E_WARNING erzeugt.
Wenn die Verbindung gelöst wird, dann ist auch die letzte Fehlernummer und ~Meldung im Orkus.
Korrektur:
$result = mysql_query($query) or dberror( mysql_errno(), mysql_error(), $query );
wäre besser, denn dann kannst Du in Deiner Funktion alles auswerten.
function dberror($mysqlErrNo, $mysqlError, $query)
{
if ($mysqlErrNo)
{
switch($mysqlErrNo)
{
case 1062:
die ("Ein Eintrag mit diesen Daten existiert bereits! Bitte korrigieren Sie Ihre Eingabe!<br>");
break;
default:
die ($mysqlErrNo.": ".$mysqlError."<br>Query war: <pre>".$query."<pre>");
}
}
}
Besser wäre statt des 'die ($mysqlErrNo.": ".$mysqlError."<br><pre>".$query."<pre>");' einen Fehler zu triggern.
trigger_error('Von Dir gebauter Text', E_USER_ERROR);
... wäre Dein Freund. Kombiniere das mit mit meiner use-strict-Lösung.
Jörg Reinholz
Auge