souci avec else Fermé

Question

Bonjour, J'ai fait un formulaire qui ne fonctionne qu'en partie, je m'explique 3 champs: nom, adresse mail, commentaire Je souhaite que si l'un des champs nom ou adresse mail soit vide, un message d'erreur apparaisse sinon message indiquant que l'inscription est bien prise en compte. Bref, à l'heure actuelle, si l'adresse mail est bonne et même si le champ nom est vide, il s'affiche le double message "merci de remplir tous les champs et félicitations". Il semblerait qu'il y ait une erreur avec else. Bref, ci-dessous mon code php. Quelqu'un aurait-il la solution? D'avance merci : '; } if(preg_match("#^[a-z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$#",$_POST['email'])) $db=mysql_connect('localhost','root','') or die ('Erreur de connection'.mysql_error()); mysql_select_db('exo2',$db) or die ('Merci de renseigner votre adresse mail'.mysql_error()); $sql = "insert into exo2(id, nom, email, commentaire) values ('','$nom', '$email', '$commentaire')"; mysql_query($sql) or die ('Erreur d\'écriture'.mysql_error()); else { echo'Tous les champs doivent être renseignés'; } { echo'Félicitations, vous venez de nous rejoindre'; } mysql_close(); ?>

Zephirr · Answer

Des accolades après ton if?

ElementW · Answer

'lut, vires-moi ces fonctions mysql_*... Je suis sérieux, débarrasses-toi en, car elles induisent la gestion d'erreur que tu utilise, qui est atroce: or die c'est le mal.
Apprends a utiliser les Exceptions, et construit ta gestion d'erreur à partir de ça (en supplément d'être le moyen de gestion d'erreur de PDO ou MySQLi).

Xavierdu34 · Answer

Voila une possibilité : 
<?php 
ini_set('display_errors', 'off');

function ifisset($value, $default=null) {
$val = (isset($_GET[$value])) ? $_GET[$value] : $default;
$val = (isset($_POST[$value])) ? $_POST[$value] : $default;
return $val;
}

$nom = ifisset('nom');
$email = ifisset('email'); 
$commentaire = ifisset('commentaire');

if($nom != null && $email != null && preg_match("#^[a-z0-9._-]+@[a-z0-9._-]{2,}\.[a-z]{2,4}$#",$email)) { 
	$db=mysql_connect('localhost','root','') or die ('Erreur de connection'.mysql_error()); 
	mysql_select_db('exo2',$db) or die ('Merci de renseigner votre adresse mail'.mysql_error()); 
	$sql = "insert into exo2(id, nom, email, commentaire) values ('','$nom', '$email', '$commentaire')"; 
	mysql_query($sql) or die ('Erreur d\'écriture'.mysql_error()); 
} 
elseif ($nom != null && $email != null) { 
	echo 'L\'email renseigné n\'est pas valide ! '; 
} elseif($nom == null && $email == null) {
	echo 'Veuillez remplir les champs nom et email';
} elseif($nom == null) {
	echo 'Veuillez remplir le champ nom';
} else {
	echo 'Veuillez remplir le champ email';
}
 
mysql_close();

?>

Souci avec else

3 réponses

Discussions similaires