PHP

Simplification du code

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Simplification du code

akalaan

Bonjour à tous,

J'aurais souhaité avoir un conseil pour simplifier un code...(enfin si on peut appeler ca un code).

J'ai créer un formulaire dans lequel l'utilisateur doit rentrer le jour,moi,années d'un rendez-vous ainsin que l'heure début et l'heure de fin.

Voila la chose

Code :

echo "Date du rendez-vous : jour <select name='jour'>
<option value=1>1
<option value=2>2
<option value=3>3
<option value=4>4
<option value=5>5
<option value=6>6
<option value=7>7
<option value=8>8
<option value=9>9
<option value=10>10
<option value=11>11
<option value=12>12
<option value=13>13
<option value=14>14
<option value=15>15
<option value=16>16
<option value=17>17
<option value=18>18
<option value=19>19
<option value=20>20
<option value=21>21
<option value=22>22
<option value=23>23
<option value=24>24
<option value=25>25
<option value=26>26
<option value=27>27
<option value=28>28
<option value=29>29
<option value=30>30
<option value=31>31
</select>
mois <select name='mois'>
<option value=1>Janvier
<option value=2>Fevrier
<option value=3>Mars
<option value=4>Avril
<option value=5>Mai
<option value=6>Juin
<option value=7>Juillet
<option value=8>Aout
<option value=9>Septembre
<option value=10>Octobre
<option value=11>Novembre
<option value=12>Decembre
</select>
annees <select name='annees'>
<option value=1>2005
<option value=2>2006
<option value=3>2007
</select><br>";

C'est laid n'est-ce pas ! :cry:

Vu que je suis nul en prog je voulais savoir si il y avait moyen de mettre ca sous forme de variable ou je sais pas trop quoi, indiquer moi juste une piste.

Merci par avance, :jap:

Publicité

PepMat

Tu peux faire une boucle for :

Code :

echo "Date du rendez-vous : jour <select name='jour'>";
for ($i = 1; $i <= 31; $i++) echo "<option value=$i>$i";

j'ai pas testé, fait longtemps que j'ai pas retouché au PHP,
c'est ptet pas la bonne syntaxe mais t'as la logique.

akalaan

Je te remercie infiniment car c'est exactement ça.

akalaan

Le seul probleme c'est que je n'arrive pas à rajouter les mois à la suite des jours, car je pense laisser le code comme ca pour les mois ?

akalaan

oups dsl je n'ai rien dit je vienx de trouver.

Mille excuse,

The-Shadow

Développeur
T'as été voir dans ton profil?

Code :

$mois=array(1=>'Janvier','Février','Mars','Avril','Mai','Juin','Juillet','Août','Septembre','Octobre','Novembre','Décembre');
for ($i=1;$i<13;$i++) {
echo '<option value="'.$i.'">'.htmlentities($mois[$i]).'</option>'.chr(13);
}

akalaan

Je te remercie, effectivement c'est beaucoup plus simple (quoique)

Je manque de logique pour ce genre de chose, je suis pas pres d'etre developpeur

akalaan

j'ai regardé sur http://fr2.php.net/manual/fr/function.htmlentities.php et je dois bien avouer que je n'est pas tout a fait saisi le sens de

Code :

.htmlentities($mois[$i])

Pourrais tu m'expliquer stp

The-Shadow

Développeur
T'as été voir dans ton profil?

htmlentities sert à transcoder tes caractères en langage HTML, par exemple, il transforme

Code :

é par é
ê => &circ;
& => &
Février devient Février

Je le mets d'office quand je ne connais pas l'encodage de ta page même si ce n'est pas indispensable.

Message édité par The-Shadow le 27-04-2006 à 15:21:51

akalaan

ok je vois un peu mieux, c'est une maniere d'adapter quelque soit le type de code.

Publicité

akalaan

je voudrais juste poser un derniere question (enfin j'espere) j'ai utilisé à la fois ton code et celui de PepMat, le souci c'est que à l'affichage, ça me (concatene) le reste du formulaire

Code :

echo "Date du rendez-vous : <select name='jour'>";
for ($j = 1; $j <= 31; $j++)
echo "<option value=$j>$j";
echo "<select name='mois'>";
$mois=array(1=>'Janvier','Février','Mars','Avril','Mai','Juin','Juillet','Août','Septembre','Octobre','Novembre','Décembre');
for ($i=1;$i<13;$i++)
echo '<option value="'.$i.'">'.htmlentities($mois[$i]).'</option>'.chr(13);
echo "<select name='annees'>";
for ($a = 2006; $a <= 2050; $a++)
echo "<option value=$a>$a"; //a ce niveau, ca m'affice "Heure du début du rendez-vous" dans la meme case en meme temp que les dates, et c'est pareil pour le reste
echo "<br>Heure du début du rendez-vous : <select name='heure'>";
for ($h = 0; $h <= 23; $h++)
echo "<option value=$h>$h";
echo "<select name='minute'>";
for ($m = 0; $m <= 59; $m++)
echo "<option value=$m>$m";
echo "<select name='secondes'>";
for ($s = 0; $s <= 59; $s++)
echo "<option value=$s>$s";
echo "Heure de fin du rendez-vous : <input type='text' name='heure_fin'><br>";
echo "<br><input type='submit' value='Enregistrer (INSERT)'>";
echo "<input type='hidden' name='ajout' value='1'>";
echo "</form>";

Exite t il des séparateur à mettre ?

The-Shadow

Développeur
T'as été voir dans ton profil?

Bah normalement, .chr(13) à la fin des lignes.

Les options du mois sont concaténés aussi ?

Message édité par The-Shadow le 27-04-2006 à 15:41:42

akalaan

en faite, dans la liste pour les années, ca va jusqu'a 2050 et juste à coté de 2050 il m'affiche le nom du prochain select, ici c'est "Heure du début du rendez-vous". c'est pareil pour "Heure de fin du rendez-vous" qui est affiché dans la liste des secondes.

Mais j'ai remarqué que lorsque je ferme mon <select name='année'> par </select> après echo "<option value=$a>$a" ;(ce qui n'est pas le cas ici), cela ne le fait plus, mais il ne m'affiche plus qu'une seule année, en l'occurence, la valeur de base de $a. et ilm'affiche toutes les autres années comme du texte

The-Shadow

Développeur
T'as été voir dans ton profil?

Prends quelques bonnes habitudes.

Code :

for (...) {
la boucle à exécuter dans des accolades.
}

Ce n'est pas indispensable, mais c'est préférable, c'est plus facile de s'y retrouver dans le code et de plus, ça résoudra ton problème ici présent.

akalaan

Oui dsl, je suis d'accord avec toi ça fait plus propre mais cela ne résout pas le probleme

Message cité 1 fois

The-Shadow

Développeur
T'as été voir dans ton profil?

akalaan a écrit :

Oui dsl, je suis d'accord avec toi ça fait plus propre mais cela ne résout pas le probleme

T'es sûr ? :whistle:

akalaan

Et bien je pense les avoirs correctement mis...

Code :

function Ajouter() {
echo "<form action='action_agenda.php' method='post'>";
echo "<br>Client visité : <input type='text' name='client_visite'><br>";
echo "Date du rendez-vous : <select name='jour'>";
for ($j = 1; $j <= 31; $j++) {
echo "<option value=$j>$j";
}
echo "<select name='mois'>";
$mois=array(1=>'Janvier','Février','Mars','Avril','Mai','Juin','Juillet','Août','Septembre','Octobre','Novembre','Décembre');
for ($i=1;$i<13;$i++) {
echo '<option value="'.$i.'">'.htmlentities($mois[$i]).'</option>'.chr(13);
}
echo "<select name='annees'>";
for ($a = 2006; $a <= 2050; $a++) {
echo "<option value=$a>$a</select>";
}
echo "Heure du début du rendez-vous : <select name='heure'>";
for ($hd = 0; $hd <= 23; $hd++) {
echo "<option value=$hd>$hd";
}
echo "<select name='minute'>";
for ($md = 0; $md <= 59; $md++) {
echo "<option value=$md>$md";
}
echo "<select name='secondes'>";
for ($sd = 0; $sd <= 59; $sd++) {
echo "<option value=$sd>$sd</select>";
}
echo "Heure de fin du rendez-vous : <select name='heure'>";
for ($hf = 0; $hf <= 23; $hf++) {
echo "<option value=$hf>$hf";
}
echo "<select name='minute'>";
for ($mf = 0; $mf <= 59; $mf++) {
echo "<option value=$mf>$mf";
}
echo "<select name='secondes'>";
for ($sf= 0; $sf <= 59; $sf++) {
echo "<option value=$sf>$sf</select>";
}
echo "<br><input type='submit' value='Enregistrer (INSERT)'>";
echo "<input type='hidden' name='ajout' value='1'>";
echo "</form>";
}

The-Shadow

Développeur
T'as été voir dans ton profil?

Penses à fermer les <option> également.

The-Shadow

Développeur
T'as été voir dans ton profil?

Non, ils sont mal fermés.
Pense à fermer les <option> par </option> et regarde où tu fermes tes <select>. :whistle:

Ce n'est pas indispensable, mais ça évite de genre de déconvenu.

Message édité par The-Shadow le 27-04-2006 à 16:06:30

akalaan

Dsl, ça ne change rien

akalaan

Code :

echo "<select name='annees'>";
for ($a = 2006; $a <= 2050; $a++) {
echo "<option value=$a>$a</option></select>";
}

akalaan

Ok je crois avoir compris

Les select se ferme juste avant le commencement d'un nouveau select et non a la fin de celui-ci

The-Shadow

Développeur
T'as été voir dans ton profil?

Le select doit se fermer après la liste d'<option>, en aucun cas on ne doit trouver un </select> dans une boucle.

Code :

echo "<select name='annees'>";
for ($a = 2006; $a <= 2050; $a++) {
echo "<option value=$a>$a</option>";
}
echo '</select>';

akalaan

Je te remercie pour tes précieux conseil, maintenant je vais m'attaquer à la récupération de ces données vers une bdd MySQL.

Merci,

The-Shadow

Développeur
T'as été voir dans ton profil?

Si tu veux un autre conseil, prend l'habitude de bien séparer les variables du texte, ex
Au lieu de :

Code :

echo "<option value=$a>$a</option>";

Tu mets :

Code :

echo '<option value='.$a.'>'.$a.'</option>';

C'est plus rapide pour le PHP à traiter et avec un éditeur à coloration syntaxique, tu t'y retrouveras mieux sur des gros projets.

Edit : Rien qu'avec la coloration sur HFR, la deuxième ligne est plus claire.

Message édité par The-Shadow le 27-04-2006 à 16:26:31

akalaan

Oui c'est vrai tu as raison, je vais essayer d'en prendre l'habitude

Publicité

FORUM HardWare.fr

Programmation

PHP

Simplification du code

Sujets relatifs
[Resolu] Appel de javascript depuis code .Net	comment inserer du code html avec javascript ?
Balise div dans le code .NET	Numérotater les lignes de code VBA dans le VBAProject
[divers] Mettre en forme son code sur word	[Java]Modification de code[Résolu]
[Aide] Code source toolbar ....	Récupérer le code html d'une page web
code d'impression	Redirection différente pour le même code !
Plus de sujets relatifs à : Simplification du code

Page générée en 0.085 secondes