PHP, mon code est t'il correct.

sff · 29 Novembre 2006

Je voudrais avoir votre avis sur le code php suivant. En fait trouve juste idiot de devoir faire :

Code:

if (!isset($_POST['previsualiser'])) {
	
	$_POST['message']= null;
	$bbcode = null;
	
	}else{
	
	$texte = new texte();
	$bbcode = $texte->ms_format($_POST['message']);
	
	}

Pour qu'il ne me donner pas une erreur comme quoi $bbcode et Post message sont inconnus. Avez vous une autre solution?

Code:

   function news_villes() {

  $page ='<div><img src="images/jeu/actions/administration.jpg" width="580" height="82" alt="Administration"/><br /><br /></div>';
	
	if (isset($_POST['submit'])&&!($_POST['previsualiser'])) {
        
        extract($_POST);
        $errors = 0;
        $errorlist = "";
        if ($message == "") { $errors++; $errorlist .= "- Vous devez écrire une nouvelle.<br />"; }
        if (preg_match("/[\^*+<>#]/", $texte)==1) { $errors++; $errorlist .= "- La nouvelle doit être écrit en caractères alphanumériques.<br />"; }

        if ($errors == 0) { 
            $update = doquery("INSERT INTO {{table}} SET id='',postdate=NOW(),content='".addslashes($message)."'", "news");
         $page .='La news vient d\'être posté dans toutes les villes!<br /><br />Maintenant vous pouvez:<br /><br /><a href="index.php">» retourner au jeu</a><br />» Sélectionner une autre rubrique à administrer';  
        } else {
         $page .= '<div>La news n\'a pas pu être posté, car les erreur(s) suivante(s) se sont produite(s):<br /><br /><span class="alerte">'.$errorlist.'</span><br /><br />Maintenant vous pouvez:<br /><br /><a href="?page=news_villes">» retourner et réessayer</a><br /><a href="index.php">» retourner au jeu</a></div>'; 
        }            
    }else{
	
	if (!isset($_POST['previsualiser'])) {
	
	$_POST['message']= null;
	$bbcode = null;
	
	}else{
	
	$texte = new texte();
	$bbcode = $texte->ms_format($_POST['message']);
	
	}
	       
$page .= '
<img src="images/jeu/puce4.gif" alt=""/> <span class="mauve2"><b>Ajouter une news de villes:</b></span><br /><br />

<form enctype="multipart/form-data" action="" method="post" name="post">
<table width="580" border="0" cellspacing="0" cellpadding="0">
<tr valign="top"><td style="width:110px">BBcode:</td><td><div><select class="taille2" onchange="bbfontstyle(\'[color=\' + this.form.couleur.options[this.form.couleur.selectedIndex].value + \']\', \'[/color]\');this.selectedIndex=0;" name="couleur"><option style="color: black;" value="#000000">Couleur</option><option style="color: red;" value="#FF0000">Rouge</option><option style="color: orange;" value="#FFA500">Orange</option><option style="color: yellow;" value="#FFFF00">Jaune</option><option style="color: green;" value="#008000">Vert</option><option style="color: violet;" value="#EE82EE">Violet</option><option style="color: blue;" value="#0000FF">Bleu</option><option style="color: indigo;" value="#4B0082">Indigo</option></select> <select onchange="bbfontstyle(\'[size=\' + this.form.taille.options[this.form.taille.selectedIndex].value + \']\', \'[/size]\')" name="taille"> <option value=1>Très petit</option> <option value=2>Petit</option> <option value=3 selected>Normal</option> <option value=4>Grand</option> <option value=5>Très grand</option></select><input onclick="bbstyle(0)" type="button" value="G" class="taille2" style="font-weight: bold;"> <input onclick="bbstyle(2)" type="button" value="I" class="taille2" style="font-style: italic;"> <input onclick="bbstyle(4)" type="button" value="U" class="taille2" style="text-decoration: underline;"> <input onclick="bbstyle(6)" type="button" value="Url" class="taille2"> <input onclick="bbstyle(8)" type="button" value="Image" class="taille2"></div></td></tr>
<tr valign="top"><td style="width:110px"></td><td style="height:4px"></td></tr>
<tr valign="top"><td style="width:110px"></td><td><div><a href="javascript:emoticon(\':D\')"><img src="images/jeu/blog/smileys/sourire.gif" style="border:0"  alt=""></a> <a href="javascript:emoticon(\';\)\')"><img src="images/jeu/blog/smileys/clin.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':\(\')"><img src="images/jeu/blog/smileys/triste.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':surpris:\')"><img src="images/jeu/blog/smileys/yeuxrond.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':o\')"><img src="images/jeu/blog/smileys/etonne.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':confus:\')"><img src="images/jeu/blog/smileys/confus.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':lol:\')"><img src="images/jeu/blog/smileys/lol.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':fire:\')"><img src="images/jeu/blog/smileys/flame.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':splif:\')"><img src="images/jeu/blog/smileys/petard.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':bigsmile:\')"><img src="images/jeu/blog/smileys/green.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':x\')"><img src="images/jeu/blog/smileys/mad.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':roll:\')"><img src="images/jeu/blog/smileys/rolleyes.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':bigcry:\')"><img src="images/jeu/blog/smileys/crying.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':colere:\')"><img src="images/jeu/blog/smileys/colere.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':P\')"><img src="images/jeu/blog/smileys/razz.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\'8\)\')"><img src="images/jeu/blog/smileys/lunettes.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':\)\')"><img src="images/jeu/blog/smileys/sourire2.gif" style="border:0" alt=""></a> <a href="javascript:emoticon(\':oops:\')"><img src="images/jeu/blog/smileys/redface.gif" style="border:0" alt=""></a><br /><br /></div></td></tr>
<tr valign="top"><td style="width:110px">Message :</td><td><textarea name="message" rows="5" cols="50">'.$_POST['message'].'</textarea><br /><br /></td></tr>
<tr valign="top"><td style="width:110px"></td><td><input type="submit" name="submit" value="Envoyer" /> <input type="reset" name="reset" value="Annuler" /> <input type="submit" name="previsualiser" value="Prévisualiser" /><br /><br /></td></tr>
<tr valign="top"><td style="width:110px"></td><td style="width:340px"><div>'.$bbcode.'</div></td></tr>
</table>
</form>';
 
}    
  echo $page; 
}

sonikbuzz · 29 Novembre 2006

sff a dit:
Pour qu'il ne me donner pas une erreur comme quoi $bbcode et Post message sont inconnus. Avez vous une autre solution?

C'est pas un warning dans ce cas ?

sff · 29 Novembre 2006

non une simple Notice indefined bbcode et message

creationfr · 29 Novembre 2006

essaye avec :

Code:

   $_POST['message']= "";
   $bbcode = "";

au lieu de

Code:

   $_POST['message']= null;
   $bbcode = null;

sff · 29 Novembre 2006

justement j'ai mi ces lignes

(!isset($_POST['previsualiser'])) {

$_POST['message']= null;
$bbcode = null;

}else{

$texte = new texte();
$bbcode = $texte->ms_format($_POST['message']);

}

pour éviter ces erreur, mais je trouve que ca fait pas propre, c'est pour cela que je demande qi il y a pas possibilité de faire autrement et surtout éviter ces 2 lignes de clodo :

$_POST['message']= null;
$bbcode = null;

sonikbuzz · 29 Novembre 2006

sff a dit:
non une simple Notice indefined bbcode et message

Donc ce n'est pas une erreur, Je m'embêterai pas avec ca alors :wink: . (que les intégristes me pardonnent :lol: )