bonsoir  
 
j'aimerais savoir si mon code contient des erreurs car cela beug !!!  
merci d'avance
pour expliquer:
dans ma base de donnée j'ai des offres de promo en %
et je souhaite qu'il s'affiche sur la page mais cela s'affiche s'en rester en place!!
  <?php
    if($row_sejours['reduction']!='')
    {
     $row_sejours['reduction']=explode("\r\n",$row_sejours['reduction']);
     foreach($row_sejours['reduction'] as $key => $value)
     {
     if($value!='')
     {
    $value=explode(";",$value);
    if($value[0]!=0)
     echo'<option value="f'.$value[0].'" onClick="document.getElementById(\'reduction_name\').value=\''.addslashes($value[2]).'\';">'.$value[2].'</option>';
    elseif($value[1]!=0)
     echo'<option value="p'.$value[1].'" onClick="document.getElementById(\'reduction_name\').value=\''.addslashes($value[2]).'\';">'.$value[2].'</option>';
    }
    }  
    }
    ?>
 
merci du coup de pouce

ta requete SQL est surement mal faite.
Ton code est à chier: plein de boucles, mélange de php/html, simple quote, double quote, multiple explode, des index nommés avec des chiffres, des variables qui écrasent des variables du même nom, mal indenté, test de condition mal fait, non utilisation de fonctions php.

merci pour ta reponse franche!
mais il y a surement plus simple, n'est ce pas?
peut tu me suggerer un code, please.
merci

Pour les simple quotes double quotes il a raison d'utiliser les doubles pour le HTML et les simples pour la concaténation et délimitation de chaine, donc pas la peine de l'engueuler.
 
Ensuite les clef numérique des tableaux sont dues aux explodes donc encore une fois, c'est pas la peine de l'engueuler.
 
Maintenant l'utilisation de fonction et une bonne indentation simplifirait certainement la découverte de ce qui plante ton code.
 
Ton premier explode, tu souhaite découper ton texte par ligne, puis ensuite le deuxieme par ";", il existe des fonction CSV dans php qui allegerait surement cette phase de ton code.

merci pour ta reponse
je vais m'y pencher dessus tres vite

 
 
ok, il code bien et son code et lisible

Euh je préfère tout séparer perso :
 

<?php
if(!empty($row_sejours['reduction']))
{
    $row_sejours['reduction'] = explode("\r\n",$row_sejours['reduction']);
    foreach($row_sejours['reduction'] as $key => $value)
    {
        if(!empty($value))
        {
            $value = explode(';',$value);
            if($value[0] != 0) {
                ?>
                <option value="p<?php echo $value[0]; ?>" onClick="document.getElementById('reduction_name').value='<?php echo addslashes($value[2]) ?>';"><?php echo $value[2] ?></option>
                <?php
            } elseif($value[1] != 0) {
                ?>
                <option value="p<?php echo $value[1]; ?>" onClick="document.getElementById('reduction_name').value='<?php echo addslashes($value[2]) ?>';"><?php echo $value[2] ?></option>
                <?php
            }
        }
    }
}
?>

 
La tu évites tout pb d'échappement et de guillemets