richtiges Script

[Matze]

Hallo!

## email richtig speichern
    $data[3] = strtolower(data[3]);
    $data[3] = str_replace("\r", "", $data[3]); // \r entfernen (Zeilenende)
    $data[3] = str_replace("\n", "", $data[3]); // \n entfernen (Zeilenende)

Wieso strtolower()? E-mailadressen sind nicht case-sensitive.
Und wieso speicherst du \r und \n bei E-mailadressen?
\r und \n bei der Ausgabe zu entfernen scheint mir überflüssig.

Wieso strtolower()? E-mailadressen sind nicht case-sensitive.

## schon vorhanden?
    $result = mysql_query("SELECT email FROM ".DB_TABLE." WHERE email='".addslashes($data[3])."'");

Wieso addslashes()? Die richtige Funktion für den MySQL-Kontext wäre mysql_real_escape_string().

if(mysql_num_rows($result)) {
      echo addslashes($data[3])." ist bereits eingetragen<br>";
    }

Wieso addslashes()? Die richtige Funktion für den HTML-Kontext wäre htmlspecialchars().

"VALUES ('".addslashes($data[0])."', '".
                  addslashes($data[1])."', '".
                  addslashes($data[2])."', '".
                  [...])";

Warum addslashes()? Siehe oben!

Grüße, Matze