taille d'un champ input

**rimbaut** · 17/05/2010, 08h13

Bonjour à tous,
Dans le code ci après, j'ai mis l'attribut maxlength afin que tous mes champs aient la même dimension.
Cependant, je constate que maxlength n'est pas pris en compte.

De plus, mon bouton submit mis après la boucle while ne renvoie pas sur la page escomptée ; le navigateur dit ne pas trouver la page !

D'où viennent ces erreurs ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 while($var=mysql_fetch_array($result)){
    extract($var);
	echo '<form action="questionMaj.php" method="post">';
	echo '<table border=\"0\">';
	echo '<tr>';
	   echo '<td bgcolor=\"#CCCCCC\">'.$mot.'<input type=\"text\" name=\"score[idMot]\" maxlength=\"20\" size=\"2\"</td>';
	echo '</tr>';
	}
    echo '<td colspan=2><br>';
	   echo '<input type="submit" value="Envoyer">';
	echo '</td>';
 
	echo '<table>';
	echo '</form>';

Je remercie vivement la ou les personnes qui m'apporteront leur aide.

Bonne journée

**RunCodePhp** · 17/05/2010, 08h40

Salut

Code :

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

<input type=\"text\" name=\"score[idMot]\" maxlength=\"20\" size=\"2\"

Il manque la fermeture de la balise du input : />

Code :

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

<input type=\"text\" name=\"score[idMot]\" maxlength=\"20\" size=\"2\" />

Il manque la fermeture de la ligne TR, juste avant celle qui ferme le table. Donc il manque un </tr>

Une erreur aussi sur la balise qui ferme le tableau. <table> au lieu de </table>

Tu indique un colspan="2" (une fusion de 2 cellules/colonnes) alors que le tableau comporte seulement 1 colonne.

Si ton DOCTYPE c'est du XHTML, alors il est préférable de faire ceci : <br /> au lieu de ceci <br>. C'est valable pour tout ce genre de balise, comme les <input /> etc ...

Le code HTML que tu fais est trop approximatif, ça déboussole complètement le navigateur. Faux être plus pointilleux sur ce point.

Aide toi des outils, comme le plugin "webdeveloppeur" qui s'intègre à FireFox (par exemple), ou d'un éditeur plus évolué.

Petite parenthèse qui n'a rien avoir

A titre personnel, la fonction extract() est loin d'être la recette miracle, surtout dans le cadre où tu l'utilise.
Pourquoi ne pas exploiter directement le tableau ?
Comme $var['nom_du_champ'], c'est tout aussi bien, enfin, c'est même mieux.
Si tu généralise cette pratique, un de ces 4 il va t'arriver des trucs pas clair, à en perdre la boule.

Comme ça, juste pour info, j'utilise jamais extract(), jamais eu besoin.
M'enfin, à chacun sa façon de faire, cela va de soit

**sabotage** · 17/05/2010, 08h41

maxlength c'est le nombre de caractère qu'on peut saisir dans le champ.
Pour définir la taille, tu peux utiliser du CSS.

Pour la page de destination, si tu as comme retour qu'elle n'existe, sous toute reserve, c'est ce c'est vrai.
Fait attention aux majuscules / miniscules.

**rimbaut** · 19/05/2010, 20h10

Merci pour vos réponses RunCodePhp et sabotage. J'aimerai comprendre pourquoi la fonction extract n'est pas la recette miracle ? Quelles peuvent être les conséquences d'utiliser cette superglobale ?

Peux tu m'en dire un peu plus car ça m'intéresse énormément et comme je débute autant adopter de suite les bonnes pratiques.

Merci encore et bonne soirée

**RunCodePhp** · 20/05/2010, 07h14

Disons qu'en 1er extract() n'est pas une super global, mais juste une fonction.
Ca extrait les valeurs d'un tableau pour en transformant en variables.
Il cependant en 2ème argument (par défaut il vaut EXTR_OVERWRITE) qui fait que si une variable du même nom existe, elle sera écrasée. Rien que ça suffit pour se poser les bonnes questions.

Ensuite, j'ai précisé que SI tu généralise cette façon de procéder, tu prends le risque que le extract() vienne un jour écraser une variable déjà existante et cela par simple oubli, accident ou non précaution.
Il faudra alors bien connaitre son projet pour en déduire que la cause viendrait de cette fonction.
Donc cette pratique demande de connaitre le nom de chaque champ de toutes tes tables pour éviter de créer une variable du même nom, sinon, il y aura un risque, c'est certain.

Idem, si tu applique un extract() pour tous les tableaux de ton application (comme le SESSION, GET, POST, et aute ...), ça va faire tellement de noms de variables à faire attention que cela deviendra tout simplement ingérable.

Bref ... toujours est il que j'ai vu dans un autre de tes topic (forum Php) que tu l'utilisais aussi pour le tableau SESSION, je te déconseille fortement de ne pas le faire ici. Au moins ça.
De plus, (et si tu avais lu la doc sur cette fonction) il aurait au moins fallut mettre comme valeur EXTR_REFS au 2ème paramètre.

Code :

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

extract($_SESSION, EXTR_REFS)

Le plus simple, c'est d'exploiter ce tableau, fait référence directement aux éléments, comme $_SESSIONS['mon_element'], c'est plus simple, nettement plus sécurisé, et un simple coup d'oeil dans son code on sait à quoi ça correspond.
A l'inverse, un $mon_element qui a été extrait de la session devient une banale variable (saut si on précise EXTR_REFS), mais là ça demande quand même de savoir que cette variable est particulière (une variable de session).
Genre de truc qui au bout d'1 an on oubli, et puis ce jour là on risque de faire une tonne de bêtise, genre la renommer, etc ...
Ingérable, tout simplement.

Ici, cette pratique ne fait que rendre les choses plus compliquée, autant l'abandonner.
-> Un peu de lecture sur la Doc Php Officielle : http://fr2.php.net/manual/fr/function.extract.php
Tu constateras qu'il y a pas mal de recommandations, d'avertissement sur l'usage de cette fonction.

Et puis parcourt le forum, regarde comment les autres font, tu constateras que très peu de codeurs utilisent cette fonction, autant dire personne tellement il y a peu.
Tout ça pour dire que cette fonction ressemble beaucoup à une recette miracle, mais c'est une illusion, elle ne fait que rajouter de la complexité.
Franchement, ça sert à rien, mais c'est un conseil.