Discussion :

[rico the hobbit]

Bonjour à tous,

Je suis en train de construire un formulaire en PHP à partir d'une base de donnée MySQL. L'objectif est de proposer un formulaire de création de personnage pour un jeu de rôle (Shadowrun plus exactement). Les données du jeu sont sur la base de donnée (races, caractéristiques, compétences, capacités spéciales). Le joueur effectue ses choix et le formulaire calcule à chaque validation le nombre de points qu'il lui reste pour faire son personnage. Les données saisies par le joueur ne sont pas envoyées à la base de données mais sont envoyées de nouveau dans le formulaire par le biais de $_GET ce qui permet de faire les calculs des points restants avant d'envoyer à la base de donnée.

Mon problème, car il y a un problème, se situe au niveau des compétences. Les compétences sont divisées en groupes. Chaque groupe est divisé en domaine. Ex : dans le domaine 'Combat', nous avons deux groupes : 'Armes à feu' et 'Combat rapproché', dans le domaine : 'Physique', nous avons trois groupes : 'Athlétisme', 'Furtivité' et 'Plein Air'. Dans le groupe 'Armes à feu', nous avons trois compétences : 'Armes automatiques, 'Fusils' et 'Pistolet'.

Dans le formulaire, je mets en place un classement pour lister les groupes de compétences en fonction de leur domaine. Mais lorsque je veut récupérer les données saisies par le joueur, je n'y arrive pas car le code n'arrive pas à distinguer les groupes selon leur domaine. Le groupe 1 du domaine 1 aura la même valeur que le groupe 1 du domaine 2 et du domaine 3. J'avais pensé mettre un nom spécifique au formulaire pour chaque domaine Ex :

Code :

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

name=\"groupe$iddomaine[]\"

mais le problème est que lorsque je veut récupérer les données

Code :

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

$groupe = $_GET['groupe$iddomaine'];

ça ne marche pas, il ne reconnait pas $iddomaine comme étant une variable.

Si mes explications sont claires, que puis-je faire pour obtenir une saisie unique pour chaque groupe ?

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
22
23
24
25
26
27
28
29
<?php
$groupe = $_GET['groupe'];
echo "<fieldset><h2>Comp&eacute;tences</h2>";
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;
while ($compteur < mysql_numrows ($req))
	{
		$nomdomaine = mysql_result ($req, $compteur, "nom_domaine");
		$idomaine = mysql_result ($req, $compteur, "id_domaine");
		echo "<p><b>$nomdomaine</b></p><p><table>";
		$reqgroupe = mysql_query ("SELECT * FROM shad_groupe WHERE id_domaine='$idomaine' ORDER BY nom_groupe");
		$compteurgroupe = 0;
		while ($compteurgroupe < mysql_numrows ($reqgroupe))
			{
				$nomgroupe = mysql_result ($reqgroupe, $compteurgroupe, "nom_groupe");
				$idgroupe = mysql_result ($reqgroupe, $compteurgroupe, "id_groupe");
				$valeurgroupe = $groupe[$compteurgroupe];
				$valeurgroupegen = $valeurgroupe * 4;
				$gengroupe = $gengroupe + $valeurgroupegen;
				echo "<tr><td><label><b>$nomgroupe</b> (<i>GC</i>)</label></td>";
				echo "<td><input type=\"text\" name=\"groupe[]\" id=\"$idgroupe\" value=\"$valeurgroupe\" size=\"1\"/></td></tr>";
				$compteurgroupe++;
			}
		echo "</table></p>";
		$compteur++;
	}
echo "</fieldset>";
?>

[asdf007]

Pas compris grand chose, et très sceptique sur la façon de faire les choses, mais tente $_GET["id$domaine"], avec des guillemets.

[rico the hobbit]

Pas compris grand chose, et très sceptique sur la façon de faire les choses, mais tente $_GET["id$domaine"], avec des guillemets.

Merci pour ton commentaire. Il se peut que ma façon de programmer ne soit pas très académique. N'ayant aucune formation en informatique, le codage est totalement empirique. Pour l'instant, il fonctionne et me permet d'arriver à mes fins. Cependant, actuellement, je coince, d'où ma question sur la récupération de variables.
Pourrais-tu préciser tes incompréhensions afin que j'essaye de les éclairer (si j'y arrive )

Je vais tester ta suggestion, merci

[asdf007]

Bon, après relecture, j'ai un peu mieux compris le truc. Je retire donc ce que j'ai dit plus haut : même si c'est pas un code idéal (mélange du PHP et du HTML, entre autres), pour quelqu'un qui n'a pas de formation, tu te débrouilles mieux et avec davantage de bon sens que beaucoup de gens postant par ici...

Bref, tente donc le coup des guillemets (les variables ne sont reconnues qu'à l'intérieur de guillemets). Voir http://fr.php.net/manual/fr/language.types.string.php vers "Analyse des variables".

Sinon, quitte à utiliser la bibliothèque mysql de base, pense aux fonctions mysql_fetch_*, par exemple http://fr2.php.net/manual/fr/functio...tch-object.php.

[rico the hobbit]

même si c'est pas un code idéal (mélange du PHP et du HTML, entre autres), pour quelqu'un qui n'a pas de formation, tu te débrouilles mieux et avec davantage de bon sens que beaucoup de gens postant par ici...

J'avoue personnellement mieux m'y retrouver dans mon code en passant le html directement dans le code php.

Code :

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

1
2
3
4
5
6
<?php
$groupe = $_GET['groupe'];
echo "<fieldset><h2>Comp&eacute;tences</h2>";
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?php
$groupe = $_GET['groupe'];
?>
<fieldset>
<h2>
Comp&eacute;tences
</h2>
<?php
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;

Après, si le premier code est peu académique, je m'en cogne du moment que je m'y retrouve et que j'arrive à faire des applications. Par contre pour les guillemets, au lieu de mettre des " ", je mets des ' ' et il me semble que ça marche autant à moins que... je vérifie quand même. Si c'est le cas, tu viens de m'enlever une belle épine du pied

[rico the hobbit]

J'ai tenté un nouveau code :

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
22
23
24
25
26
27
28
29
<?php
echo "<fieldset><h2>Comp&eacute;tences</h2>";
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;
while ($compteur < mysql_numrows ($req))
	{
		$nomdomaine = mysql_result ($req, $compteur, "nom_domaine");
		$idomaine = mysql_result ($req, $compteur, "id_domaine");
		echo "<p><b>$nomdomaine</b></p><p><table>";
		$reqgroupe = mysql_query ("SELECT * FROM shad_groupe WHERE id_domaine='$idomaine' ORDER BY nom_groupe");
		$compteurgroupe = 0;
		while ($compteurgroupe < mysql_numrows ($reqgroupe))
			{
				$nomgroupe = mysql_result ($reqgroupe, $compteurgroupe, "nom_groupe");
				$idgroupe = mysql_result ($reqgroupe, $compteurgroupe, "id_groupe");
				$groupe = $_GET["groupe$iddomaine"];
				$valeurgroupe = $groupe[$compteurgroupe];
				$valeurgroupegen = $valeurgroupe * 4;
				$gengroupe = $gengroupe + $valeurgroupegen;
				echo "<tr><td><label><b>$nomgroupe</b> (<i>GC</i>)</label></td>";
				echo "<td><input type=\"text\" name=\"groupe$idgroupe[]\" value=\"$valeurgroupe\" size=\"1\"/></td></tr>";
				$compteurgroupe++;
			}
		echo "</table></p>";
		$compteur++;
	}
echo "</fieldset>";
?>

Et voici la réponse :

Code :

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

Parse error: syntax error, unexpected ']', expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/rico/www/private/jdr/shadow/pj/competences.php on line 27

La ligne 27 étant la suivante :

Code :

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

echo "<td><input type=\"text\" name=\"groupe$idgroupe[]\" value=\"$valeurgroupe\" size=\"1\"/></td></tr>";

[asdf007]

J'avoue personnellement mieux m'y retrouver dans mon code en passant le html directement dans le code php.

Code :

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

1
2
3
4
5
6
<?php
$groupe = $_GET['groupe'];
echo "<fieldset><h2>Comp&eacute;tences</h2>";
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;

Code :

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

1
2
3
4
5
6
7
8
9
10
11
<?php
$groupe = $_GET['groupe'];
?>
<fieldset>
<h2>
Comp&eacute;tences
</h2>
<?php
$valeur = 0;
$req = mysql_query ("SELECT * FROM shad_domaine");
$compteur = 0;

Je parle d'une séparation complète : tout le PHP de "contrôle" (récupération de l'input, etc), puis ensuite un fichier de template HTML qui affiche le résultat de façon basique. M'enfin bon.

Pour les guillements/ simples/doubles, si tu as lu le lien que je t'ai passé tu auras compris la différence : les variables ne sont analysées qu'à l'intérieur des doubles.

En ce qui concerne l'erreur de parsing, je te laisse relire ce même bout de doc, tout y est et avec des exemples.

(Tu n'as manifestement pas non plus regardé les mysql_fetch_*...)

[rico the hobbit]

Je parle d'une séparation complète : tout le PHP de "contrôle" (récupération de l'input, etc), puis ensuite un fichier de template HTML qui affiche le résultat de façon basique. M'enfin bon.

OK, il faudra que j'y songe

Pour les guillements/ simples/doubles, si tu as lu le lien que je t'ai passé tu auras compris la différence : les variables ne sont analysées qu'à l'intérieur des doubles.

D'accord mais je n'arrive pas à comprendre d'où vient l'erreur sur cette ligne car si j'essaye de mettre une variable à la place du nom, il m'indique une erreur. Value prend bien la variable $valeurgroupe par contre name ne prend la variable que si l'on n'ajoute pas les crochets $nomdomaine et non $nomdomaine[]:

Code :

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

echo "<td><input type=\"text\" name=\"$nomdomaine[]\" value=\"$valeurgroupe\" size=\"1\" /></td></tr>";

Code :

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

Parse error: syntax error, unexpected ']', expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/rico/www/private/jdr/shadow/pj/competences.php on line 27

Je vais continuer de consulter la documentation que tu m'as laissé.

(Tu n'as manifestement pas non plus regardé les mysql_fetch_*...)

Pour mysql_fetch, je remplace mysql_numrows ($req) par un code avec mysql_fetch ? Si c'est le cas, qu'est-ce que cela m'apporte comme intérêt ?

Merci pour tes infos

[rico the hobbit]

J'ai réussi à obtenir une valeur unique par groupe en nommant par l'id du groupe. Mon erreur venait des guillemets simples que je mettais au $_GET

Code :

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

1
2
3
4
5
$valeurgroupe = $_GET["groupe$idgroupe"];
				$valeurgroupegen = $valeurgroupe * 10;
				$gengroupe = $gengroupe + $valeurgroupegen;
				echo "<tr><td><label><b>$nomgroupe</b> (<i>GC</i>)</label></td>";
				echo "<td><input type=\"text\" name=\"groupe$idgroupe\" value=\"$valeurgroupe\" size=\"1\" /></td></tr>";

[asdf007]

Pour mysql_fetch, je remplace mysql_numrows ($req) par un code avec mysql_fetch ? Si c'est le cas, qu'est-ce que cela m'apporte comme intérêt ?

Ne pas avoir à faire de compteur numérique, ni à appeler mysql_result pour chaque colonne. Tu boucles juste sur le résultat de mysql_fetch_* et tu obtiens directement les valeurs. Ce que tu fais fonctionne, ceci dit, c'est juste lourd.

De toute façon, pour quelque chose de vraiment propre, il vaudrait mieux passer à mysqli, ou encore mieux à PDO. Mais tant que tu protèges tes requêtes, si c'est pour un petit site ça devrait bien se passer.

[rico the hobbit]

Ne pas avoir à faire de compteur numérique, ni à appeler mysql_result pour chaque colonne. Tu boucles juste sur le résultat de mysql_fetch_* et tu obtiens directement les valeurs. Ce que tu fais fonctionne, ceci dit, c'est juste lourd.

De toute façon, pour quelque chose de vraiment propre, il vaudrait mieux passer à mysqli, ou encore mieux à PDO. Mais tant que tu protèges tes requêtes, si c'est pour un petit site ça devrait bien se passer.

Merci pour tes conseils.

Pour la sécurisation du formulaire, pour l'instant, mon formulaire n'est pas en ligne mais uniquement en local, du coup, je ne me suis pas trop penché sur la question.

Le formulaire n'est toutefois pas terminé, j'aurais donc quelques petites autres questions.