Discussion :

[maub85]

Bonjour à tous,

Voici le bout de code qui me fait des misères ...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
		echo "<form>";
		echo '<form action="Menu_personnes_HV.php" method="POST"> ';
		echo ' DOYENNE sélectionné : <td> <select name="id_doyenne">';   echo liste_doyennes("").' </select> </td> </tr>';
 
		?>
		<select id="mon_choix">
  			<option value="valeur1"> AVEC les Photos </option> 
  			<option value="valeur2"> SANS les Photos </option>
  	        </select>
		<?php
 
		echo ' <input type="submit" value=" Valider CE(s) Choix " > ';
 
		echo "</form>";
 
		if (isset($_POST['mon_choix']) and $_POST['mon_choix'] <> "" ) {		echo "mon_choix : ".$mon_choix; exit; }
 
		//  echo "mon_choix : ".@$mon_choix; exit;
 
		//  dump_post();

Je voudrais récupérer (en php) la valeur résultant du choix fait dans l'option "AVEC ou SANS photo" pour m'en servir plus loin dans le code pour rajouter (éventuellement) une clause where à ma requête ...

Je n'y arrive pas !

Merci de votre aide.
Cordialement
maub

[Invité]

Bonjour,

1- tu as 2 balises <form>
2- c'est <select name="...">

"Formulaire PHP"

[CosmoKnacki]

À ta place j'utiliserai les balises courtes <?= $variable ?> plutôt que <?php echo $variable; ?> ainsi que la syntaxe alternative des structures de contrôle if, for, foreach, etc. pour éviter de t'enmêler les pinceaux avec tes balises html (en couplant le tout à une indentation rigoureuse). Par exemple:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
<form action="Menu_personnes_HV.php" method="POST">
    DOYENNE sélectionnée :
    <select name="id_doyenne"><?= liste_doyennes("") ?></select>
 
    <select name="mon_choix">
        <option value="valeur1"> AVEC les Photos </option> 
        <option value="valeur2"> SANS les Photos </option>
    </select>
    <input type="submit" value=" Valider ce(s) choix " >
</form>
<?php if ( !empty($_POST['mon_choix']) ): ?>
mon_choix : <?= $_POST['mon_choix'] ?>
<?php endif; ?>

(J'ai viré les td et tr qui étaient perdus dans l'espace.)

[laurentSc]

À ta place j'utiliserai les balises courtes <?= $variable ?> plutôt que <?php echo $variable; ?> ainsi que la syntaxe alternative des structures de contrôle if, for, foreach, etc.

Effectivement, j'y avais pas pensé mais ça améliore la lisibilité (et les commentaires que j'avais pris l'habitude de rajouter, du style //fin de if (isset($name)), deviendraient moins nécessaires). Par contre, cet exemple qu'on trouve dans la doc pour la syntaxe alternative :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
<?phpif ($a == 5):
    echo "a égale 5";
    echo "...";
elseif ($a == 6):
    echo "a égale 6";
    echo "!!!";
else:
    echo "a ne vaut ni 5 ni 6";
endif;
?>

m'ayant surpris, je l'ai réécrit comme ça :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
<?php if ($a == 5): ?>
    a égale 5
    ...
<?php elseif ($a == 6): ?>
    a égale 6
    !!!
<?php else: ?>
    a ne vaut ni 5 ni 6
<?php endif; ?>

et ça marche aussi. Et ça semble plus conforme au principe ; non ?

[CosmoKnacki]

Par contre, cet exemple qu'on trouve dans la doc pour la syntaxe alternative :
...
m'ayant surpris, je l'ai réécrit comme ça:

Cette page de la documentation a juste pour but de présenter la syntaxe alternative des structures de contrôle, or on ne les utilise pas seulement pour faire de l'affichage en html (même si c'est dans ce contexte qu'elles sont utilisées la plupart du temps). C'est donc normal qu'elles soient présentées de cette manière avec des instructions simples comme echo dans un manuel, et ce, sans introduire d'autres notions, histoire d'embrouiller le moins de monde possible. Mais effectivement on peut très bien l'écrire comme tu l'as fait. Par contre gare aux espaces entre les echos successifs lors du passage d'une écriture à l'autre:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
<?php if (a == 5): ?>
a égale 5...
<?php elseif (a == 6): ?>
a égale 6!!!
<?php else: ?>
a ne vaut ni 5 ni 6
<?php endif; ?>

(Il n'y a pas d'espace entre 5 et ... ni entre 6 et !!!.)

Maintenant pour ce qui est de ce type de commentaires: //fin de if (isset($name)), que ce soit avec l'une ou l'autre syntaxe, ils ne devraient jamais être nécessaires, le repère pour savoir à qui appartient la fermeture d'un bloc de code c'est l'indentation.

C'est sûr que la syntaxe alternative apporte un plus en précisant l'instruction qu'elle vient fermer, mais ça ne solutionne pas certains problèmes de fond: si un bloc de code devient tellement long qu'on en vient à utiliser ce genre de commentaire, c'est le signe qu'une partie du code du bloc en question doit être déporté dans une fonction ou une méthode, ou bien écrit d'une autre manière. Même topo lorsqu'on se retrouve avec trop de niveaux d'imbrication de blocs, c'est un signe, il faut agir!

[laurentSc]

Si un bloc de code devient tellement long qu'on en vient à utiliser ce genre de commentaire, c'est le signe qu'une partie du code du bloc en question doit être déporté dans une fonction ou une méthode, ou bien écrit d'une autre manière. Même topo lorsqu'on se retrouve avec trop de niveaux d'imbrication de blocs, c'est un signe, il faut agir!

Oui, t'as raison, et ça rejoint la maxime de rawsrc :"diviser pour mieux régner".

[Zoki_Marciano]

Bonjour,

Il y a aussi la ligne 4 qui contient un <td></td> et </tr> mal placé ( je dirais même qu'ils n'ont rien à faire là ).


Edit : Oups! CosmoKnacki m'a doublé...