par
Ryle » 23 janv. 2007, 17:21
Je ne suis pas fan des @ dans le code... je trouve dommage de devoir cacher une erreur ou un avertissement, alors qu'il suffit d'un peu de rigueur pour la corriger.
Et le jour où la page ne fonctionne plus pour une raison ou pour une autre, il y aura tellement d'arobase dans le code que l'on passera 3 jours à trouver laquelle il faut enlever pour savoir ce qui fait planter le script...
Dans ton cas, le test est simple. Le message d'erreur signal que l'index "pseudo" que tu essayes d'utiliser dans $_POST n'est pas défini. Il te suffit donc de vérifier s'il existe avant de l'utiliser, par exemple :
// si la variable n'est pas définie OU si la variable est vide
if ( !isSet($_POST['pseudo']) || empty($_POST['pseudo']) ) {
echo "vous n'avez pas choisi de pseudo";
}
else {
...
}
Je ne suis pas fan des @ dans le code... je trouve dommage de devoir cacher une erreur ou un avertissement, alors qu'il suffit d'un peu de rigueur pour la corriger.
Et le jour où la page ne fonctionne plus pour une raison ou pour une autre, il y aura tellement d'arobase dans le code que l'on passera 3 jours à trouver laquelle il faut enlever pour savoir ce qui fait planter le script...
Dans ton cas, le test est simple. Le message d'erreur signal que l'index "pseudo" que tu essayes d'utiliser dans $_POST n'est pas défini. Il te suffit donc de vérifier s'il existe avant de l'utiliser, par exemple :
[php]// si la variable n'est pas définie OU si la variable est vide
if ( !isSet($_POST['pseudo']) || empty($_POST['pseudo']) ) {
echo "vous n'avez pas choisi de pseudo";
}
else {
...
}[/php]