Matti Mäkitalo: Register

Beitrag lesen

Hi,

[code]

du musst noch die Sprache angeben, damit du Syntax-highlighting bekommst.

$submit = $_POST['submit'];

das Umkopieren ist unnötig und verschleiert später die Herkunft der Variablen, was zu einem ernsten Sicherheitsproblem führt.

if ($submit)

ich würde dir empfehlen, die komplette Überprüfung in eine Funktion auszulagern. Du kannst wesentlich Einrückungstiefe sparen, da du dann sofort bei negativem Ergebnis die Funktion verlassen kannst.

$queryreg = mysql_query("

INSERT INTO users VALUES ('','$username','$vorname','$nachname','$password','$birthday','$birthmonth,'$birthyear','$email')
     ");

a) überprüf den Rückgabewert von mysql_query.
b) behandle deine Daten kontextgerecht. Du erlaubst hier deinen Usern beliebigen Zugriff auf deine Daten.

Bis die Tage,
Matti