Der Martin: Formular

Beitrag lesen

Hallo,

Wo ist hier der Fehler?

soll das ein Ratespiel sein?
Es kann nicht sein, dass du einfach nur ein paar Zeilen Code hinschmeißt und fragst, was da falsch ist. Das Mindeste ist, dass du den Fehler _beschreibst_. Wie äußert er sich, was sollte passieren, was passiert tatsächlich?

Nichtsdestotrotz enthält dein Code eine Reihe von kleinen Fehlern - teils nur "sinnlos, aber unschädlich", teils aber gravierende Sicherheitslücken.

//Variablen initialisieren
$vorname = $_POST['vorname'];
$nachname = $_POST['nachname'];

Das Umkopieren ist so ein Beispiel von "sinnlos, aber unschädlich". Wozu soll das gut sein? Und was machst du, wenn gar keine POST-Daten vorliegen - beispielsweise dann, wenn erstmal das leere Formular abgerufen wird?

//Anfrage erstellen
$einfuegen = "INSERT INTO tbl_fs09_namen VALUES ('$vorname', '$nachname')";

//Anfrage senden
mysql_query($einfuegen);

Autsch. Spätestens hier ist deine DB potentiell gefährdet. Denn du übergibst Eingaben an die DB, ohne sie auf Plausibilität oder Gültigkeit zu überprüfen, und ohne sie kontextgerecht zu codieren.

//Datensätze werden übergeben
while ($row = mysql_fetch_object($ergebnis))
{
echo "<tr><td>$row->vorname</td><td>$row->nachname</td></tr>";
}
echo "</table>";

Auch hier fehlt die kontextspezifische Behandlung.

Ciao,
 Martin

--
Um mit einem Mann glücklich zu werden, muss eine Frau ihn sehr gut verstehen und ein bisschen lieben.
Um mit einer Frau glücklich zu werden, muss ein Mann sie sehr lieben und darf gar nicht erst versuchen, sie zu verstehen.
Selfcode: fo:) ch:{ rl:| br:< n4:( ie:| mo:| va:) de:] zu:) fl:{ ss:) ls:µ js:(