Bij het inserten van email adressen lukt alles goed.
Maar als er een email aanwezig is met een foutje vb naam"@gmail.com krijg ik een foutmelding.
Door dat " aanwezig is.
Weten jullie raad ?

PHP Code:

 <?php
  if ($_POST['mail'] == "") { 


} else {
$input = $_POST["mail"];
$input = explode(";",$input);

if (count($input) > 0) {
    $output = "('" . implode("'), ('", $input) . "')";
    $sql = "INSERT INTO mailusers (email) VALUES $output";
    mysql_query($sql);
    echo"De volgende mails zijn toegevoegd: ";
    echo $_POST["mail"];
} else {
    // Dit is gegeven de case onwaarschijnlijk, maar moet je wel rekening mee houden - anders gaat de boel kapot
}

} 
 ?>

Je moet je variabelen ook 'escapen' voordat je ze in de database opslaat. De code die je nu hebt is kwetsbaar voor SQL injections (Google maar als je weet wat het niet is). Een oplossing is door "mysql_escape_string($_POST['mail'])" te gebruiken.

Of als je het echt save wilt doen PDO gebruiken.

En de input vooraf ook controleren, daarna pas in de database wegschrijven.
In geval van een email adres kan dat bijvoorbeeld met:

Code:

if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
echo 'ongeldig emailadres';
} else {
//wegschrijven naar database
}

bedankt voor de hulp.
maar als ik mysql_escape_string($_POST['mail']) gebruik plaatst deze alm de adressen na elkaar in plaats van verschillende velden.

ipv

Code:

mysql_escape_string($_POST['mail'])

probeer eens

Code:

$output = "('" . implode("'), ('", mysql_escape_string($input)) . "')";

Het is sowieso een beetje onhandig in elkaar gezet.
Doordat alle emailadressen met een ';' gescheiden zijn is het nog veel vervelender wanneer iemand per ongeluk een ; in zijn emailadres zet.

Probeer het eens zo;

Code:

if (isset($_POST['mail'])){
  $mail = explode(';',$_POST['mail']);
  foreach ($mail as $address){
    if (filter_var($address, FILTER_VALIDATE_EMAIL)){
      mysql_query("INSERT INTO `mailusers` (mail) VALUES ('".$address."')");
    }
  }
}

Met de trim functie kan je alle foute input er toch uit halen ;-)?

Oh ja? En wat als iemand per ongeluk voornaam.achternaam;@gmail.com invult?
Dan krijg jij 2 objecten in je array die daarna pas door FILTER_VAR gecontroleerd moeten worden.

Origineel gepost door Lars Velthuis

Met de trim functie kan je alle foute input er toch uit halen ;-)?

Met trim haal je enkel spaties, tabs etc weg (bron: http://www.php.net/manual/en/function.trim.php)

Origineel gepost door Remo in het V

Oh ja? En wat als iemand per ongeluk voornaam.achternaam;@gmail.com invult?
Dan krijg jij 2 objecten in je array die daarna pas door FILTER_VAR gecontroleerd moeten worden.

Exact. Met filter_var kun je controleren of het e-mail adres geldig is. (voorbeeldje: http://stackoverflow.com/questions/1...address-in-php) In het geval van voornaam.achternaam;@gmail.com maakt hij er voornaam.achternaam EN @gmail.com van. Dit zijn beide los geen geldige e-mailadres.