Es ce que mon code est propre

Je code un formulaire pour faire un trie dans ma requete, mais ce que j'ai fais fonctionne mais je pense que j'en fais trop :/

quelqu'un peu me conseiller Merci

voici mon code

<form method="POST" name="critereform" action="index.php?mod=recette&ac=recette">
 
<?
// déclaration requete sql
    $query=mysql_query("SELECT distinct  niveau categorie FROM test");
   
    echo('<select name="niveau" onchange="document.critereform.submit();">');
if($niveau==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>');
}
else
{
echo('<option value="'.'">'.'</option>');}
    while ($ligne1=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne1->niveau.'</option><option value=\"$_POST["niveau"]\" selected=\"selected\">$_POST["niveau"]</option>'); }
    echo('</select>');
   
   
  echo('<select name="temps" onchange="document.critereform.submit();">');
if($temps==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>');
}
else
{
echo('<option value="'.'">'.'</option>');}
    while ($ligne2=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne2->temps.'</option><option value=\"$_POST["temps"]\" selected=\"selected\">$_POST["temps"]</option>'); }

    echo('</select>'); 
   
echo('<select name="categorie" onchange="document.critereform.submit();">');
if($categorie==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>');
}
else
{
echo('<option value="'.'">'.'</option>');}
    while ($ligne3=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne3->categorie.'</option><option value=\"$_POST["categorie"]\" selected=\"selected\">$_POST["categorie"]</option>'); }
    echo('</select>'); 
    echo('<select name="saison" onchange="document.critereform.submit();">');
if($saison==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>');
}
else
{
echo('<option value="'.'">'.'</option>');}
    while ($ligne4=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne4->saison.'</option><option value=\"$_POST["saison"]\" selected=\"selected\">$_POST["saison"]</option>'); }
    echo('</select>');   
// fermeture de la base pour eviter les problèmes
mysql_close();

oups me suis trompé de code

// déclaration requete sql
    $query=mysql_query("SELECT distinct  niveau FROM test");    
    echo('<select name="niveau" onchange="document.critereform.submit();">');
if($niveau==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>
');
}
else
{
echo('<option value="'.'">'.'</option><option value='.$test .' selected=\"selected\">' .$test . '</option>');}
    while ($ligne1=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne1->niveau.'</option>'); }
    echo('</select>');
// fermeture de la base pour eviter les problèmes

$query=mysql_query("SELECT distinct temps FROM test");    
    echo('<select name="temps" onchange="document.critereform.submit();">');
if($temps==""){
echo('<option value="'.'">'.'< Choisir >'.'</option>
');
}
else
{
echo('<option value="'.'">'.'</option><option value='. $_POST['temps'] .' selected=\"selected\">'. $_POST['temps'] . '</option>');}
    while ($ligne1=mysql_fetch_object($query)){
//la concatenation des champs se fait ici
echo('<option>'. $ligne1->temps.'</option>'); }
    echo('</select>');
// fermeture de la base pour eviter les problèmes
mysql_close();
 
?>

Oups j'avais pa vu ça non plus ^^ les erreurs.  merci de ta reponse a bientot

Salut,

fais réponse accèptée...


In a dream, I saw me, drop dead... U was there, U cried... It was just a dream, if I die, U won't cry, maybe, U'll be happy

Mon site (articles sur la programmation et programmes)