Discussion :

[Kakon]

Bien le bonjour à toutes et tous,

Voilà je viens vers vous car notre ami Google m'a lâché sur ce coup-là.
Je suis actuellement en train de monter de mes petites mains un site pour une association de Jeux et j'ai un souci avec une donnée que je souhaiterais stocker/affiché, il s'agit des genres de jeux. Alors, vous allez me dire, ce n’est qu’une donnée parmi d’autres rien de bien compliqué.

Eh bien si, je souhaite tout d’abord, pour faciliter l’utilisation par les usagers, présenter cela sous forme de checkbox pour les différents genres sachant que ça peut être, par exemple, un jeu de cartes, un jeu de stratégie ou un jeu de cartes ET de stratégie. Afin de stocker ce type de données, il était dit à de multiples reprises sur le net que le type le plus optimisé sur mysql pour ce genre de stockage (point de vue place mémoire et sécurité sur les valeurs entrantes) était le type SET (comme un ENUM mais pouvant prendre plusieurs des valeurs données). Or j’ai souffert comme un damné déjà pour réussir à obtenir dynamiquement avec une fonction php/sql les valeurs contenues dans le SET de façon à générer les checkbox cependant maintenant je bloque car :

1. je ne vois pas comment rentrer l’info dans la base de données vu que le principe du SET est de juste stocker un binaire avec un bit pour chaque valeur du SET, et donc je ne vois pas comment faire le parallèle avec les checkbox
2. j’ai du mal à visualiser comment je pourrais récupérer « aisément » les genres d’un jeu de façon à pouvoir gérer des affichages thématiques plus tard.

Alors, d’aucuns diront que le plus simple serait de changer de type de stockage de l’info ou de la travailler différemment et ils n'auront peut-être pas tort, cependant j’aimerais par pure envie de savoir qu’on m’aide à trouver une solution, et ensuite que l’on me conseille sur la manière la plus optimisé de faire ça.

J’espère avoir été clair, si jamais vous avez besoin de certains de mes codes pour expliciter certains points, n’hésitez pas à me dire ce qu’il vous faut.

Par avance merci.

[RunCodePhp]

Salut

Alors, d’aucuns diront que le plus simple serait de changer de type de stockage de l’info ou de la travailler différemment et ils n'auront peut-être pas tort, cependant j’aimerais par pure envie de savoir qu’on m’aide à trouver une solution, et ensuite que l’on me conseille sur la manière la plus optimisé de faire ça.

D'après tes explications, un champ de type SET (ou ENUM) s'y prêterait mal, voir pas du tout.

Ce que tu souhaites c'est de représenter des combinaisons, donc un ou plusieurs voire tous parmi une liste.
Genre : bleu, vert, rouge, orange, jaune comme choix.
Et avoir : orange
ou : bleu, rouge, jaune
... etc ... toutes combinaisons possibles.

Un champ SET (ou ENUM) est bien plus fait pour un bouton radio (coté HTML), c'est à dire 1 seul et unique choix parmi plusieurs choix (ou propositions).
C'est radicalement pas la même chose.


Ceci dit, faut voir si dans ton cas c'est possible, s'il n'y a pas trop de combinaisons en faite, de tricher un peu (si on peu dire), de faire en sorte qu'il y ait qu'un seul choix.
Si on prend ton exemple : cartes, stratégie, cartes et stratégie
On peu peu faire (insérer) ainsi : cartes, strategie, cartes_strategie
Donc de créer les 3 combinaisons (uniques) possibles, et le tour est joué.
Coté formulaire, il suffit de créer un groupe de boutons radios, de proposer les mêmes choix (3 ici), et 1 seul sera choisi.
Faut voir.


Attention, il faut bien comprendre que ce type de champs fait que les données sont figées, il sera alors très difficile (très compliqué) de les faire évoluées, comme modifier une ou plusieurs valeurs (combinaisons).
Ceci peut être un énorme inconvénient (selon le cas).
Ce type de champ est bon pour des données comme :
Mr, Mme, Mlle
ou encore Homme, Femme, Enfant
Ce genre de données n'évolueront pour ainsi dire jamais.


Si on perçois qu'au fil du temps cela peu évoluer, ce type de champ ne sera pas adapté.
Il serait nettement mieux de le faire proprement, c'est à dire de créer une table "activites" par exemple et d'insérer tous les type d'activité.
Exemple :
table "activites"
activite_id | nom
(liste des activites)

table "activites_usagers"
usagers_id | activites_id
(liste des associations/combinaisons des activites des usagers)

Ici, le nom de l'activité à (presque) aucune importance car le principe du fonctionnement repose sur des identifiants (donc jointures de tables, etc ...).


Tout cela sauf erreur (comme d'hab)

[RunCodePhp]

Tout cela sauf erreur (comme d'hab)

Il me semble avoir dit une bêtise.

Dans la doc de MySQL on peu voir ceci pour champ de type SET :
La doc : Le type SET

FIND_IN_SET() :

mysql> SELECT * FROM nom_de_table WHERE set_col LIKE '%value%';
mysql> SELECT * FROM nom_de_table WHERE FIND_IN_SET('value',set_col)>0;

Mais ce qui suit fonctionnera aussi :

mysql> SELECT * FROM nom_de_table WHERE set_col = 'val1,val2';
mysql> SELECT * FROM nom_de_table WHERE set_col & 1;

Donc la différence entre le type Enum et SET, c'est que le type ENUM permet d'insérer qu'un seul ET unique choix parmi plusieurs mais le type SET permet d'insérer plusieurs choix (théoriquement tous) parmi plusieurs.

On peu alors insérer comme ceci :

Code :

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

1
2
INSERT INTO usagers (usagers_id, activites)
VALUES (10, 'bleu,vert,jaune')

je ne vois pas comment rentrer l’info dans la base de données vu que le principe du SET est de juste stocker un binaire avec un bit pour chaque valeur du SET, et donc je ne vois pas comment faire le parallèle avec les checkbox

Normalement non, rien obligerait à stocker un binaire.
C'est comme le champ ENUM, on insère une chaine séparée par une virgule des différents choix possibles à la création du champ :
Exemple : bleu, vert, rouge, orange, jaune
Après on insère les différentes combinaisons qu'on aura récupéré, une chaine séparé par une virgule (exemple plus haut).

Ce qui fait qu'à la récupération de la donnée, ce sera aussi une chaine séparée par une virgule s'il y a plusieurs choix d'effectués.
Un simple explode() en Php suffit pour obtenir un tableau des différents choix.


J'espère cette fois ne pas dire trop de bêtises.

[Kakon]

Tout d'abord, merci pour tes réponses

Ensuite, pour le champs SET, j'avais déjà lu la doc et c'est justement ce qui m'avait conforté dans les différentes infos trouvées sur les autres forums, autrement dit que l'on pouvait stocker un élément du SET ou une combinaison de ceux-ci. Pour ce qui est du stockage en binaire dont je parle, c'est parce que sur ces fameux site/forum où j'ai entendu parler de SET, ils disaient que le stockage se faisait sous forme indexé, exemple:

un set défini tel que: set('carte',strategie','plateau'); sera fait l'association: 'carte'=00000001, 'strategie'=00000010, 'plateau'=00000100; ensuite la donnée 'carte' sera stocké sous la forme: 00000001, quant à la combinaison carte+plateau ce serai: 00000101.

Après, si ce que tu dis sur le fait de rentrer une chaine avec des virgules, j'avoue que ça simplifierai bien les choses, je vais tenter de faire un test de suite après ce post ^^

Enfin, pour récupérer les valeurs des checkbox, qui sont comme on peut s'en douter dans un formulaire, j'utilise un tableau de checkbox, donc obtenir la chaine devrai pas être compliqué avec

Code :

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

$chaine=implode(', ', $montableau)

cependant dans la fonction de mise à jour de ma table je me contente de mettre $chaine dans mon UPDATE au niveau du champs des genre de jeux ou je dois faire une manip alambiquée?

[Kakon]

Alors bonne nouvelle, en effet ça à l'aire de prendre les chaines de caractères. Dans le code ci-après permettant l'action de mon formulaire (pour les valeurs des checkbox et ce qui s'y rapporte, il faut suivre la variable $type) j'arrive à obtenir une chaine de caractère cohérente (par exemple: carte, plateau) cependant seul le premier mot (ici 'carte') est pris par la base de données...

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
				if (isset($_POST['nom_jeu']) AND isset($_POST['description_jeu']) AND isset($_POST['nombre_joueur']) AND isset($_POST['temps_moyen']) AND isset($_POST['type']) AND isset($_POST['disponibilite']) )
				{
					$nom_jeu = addslashes($_POST['nom_jeu']);
					$description_jeu = addslashes($_POST['description_jeu']);
					$nombre_joueur = addslashes($_POST['nombre_joueur']);
					$temps_moyen = addslashes($_POST['temps_moyen']);
					$note = addslashes($_POST['note']);
					$type=$_POST['type'];
					$type=implode(', ', $type);
					echo $type;
					$disponibilite = addslashes($_POST['disponibilite']);
					$id_visuel = addslashes($_POST['id_visuel']);
					// On vérifie si c'est une modification d'un jeu ou pas
					if ($_POST['id'] == 0)
					{
						// Ce n'est pas une modification, on crée une nouvelle entrée dans la table
						mysql_query("INSERT INTO jeux VALUES('" . $nom_jeu . "', '" . $description_jeu . "', '" . $nombre_joueur . "', '" . $temps_moyen . "', '" . $note . "', '". $type ."', '" . $disponibilite . "', '" . $id_visuel . "', '')");
					}
					else
					{
						// On protège la variable "id" pour éviter une faille SQL
						$_POST['id'] = addslashes($_POST['id']);
						// C'est une modification, on met à jour l'affichage
						mysql_query("UPDATE jeux SET nom_jeux='" . $nom_jeu . "', description_jeu='" . $description_jeu . "', nombre_joueur='" . $nombre_joueur . "', temps_moyen='" . $temps_moyen . "', note='" . $note . "', type='" . $type . "', disponibilite='" . $disponibilite . "', id_visuel='" . $id_visuel . "' WHERE id='" . $_POST['id'] . "'");
					}
				}

Par la suite j'ai essayé d'autre implode pour voir si cela pouvait venir de la formulation de la chaine de caractère mais pour tout ceux ci-après, je n'ai même plus l'entré qui ce fait dans la BDD...

Code :

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

1
2
3
4
5
$type='\''. implode(', ', $type).'\''	génère $type='carte, plateau'
$type='\''. implode(',', $type).'\''	génère $type='carte,plateau'
$type='\''. implode('\',\'', $type).'\''	génère $type='carte','plateau'
$type='\''. implode('\', \'', $type).'\''	génère $type='carte', 'plateau'
$type='"\''. implode('\',\'', $type).'\'"'	génère $type="'carte','plateau'"

[RunCodePhp]

Code :

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

if (isset($_POST['nom_jeu']) AND isset($_POST['description_jeu']) AND isset($_POST['nombre_joueur']) AND isset($_POST['temps_moyen']) AND isset($_POST['type']) AND isset($_POST['disponibilite']) )

La fonction isset() accepte n éléments, ce qui éviter les répétitions de code inutiles.
La doc : isset()

Exemple :

Code :

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

1
2
3
if (isset($_POST['nom_jeu'], $_POST['description_jeu'], $_POST['nombre_joueur'], $_POST['temps_moyen'], $_POST['type'], $_POST['disponibilite'], $_POST['disponibilite'])) {
    // ... etc ...
}

Aussi, il ne faut pas utiliser addslashes(), mais dans ton cas : mysql_real_escape_string().
Cette fonction est dédiée pour cela : échapper la chaine et éviter une éventuelle injection SQL.
La doc : http://fr2.php.net/manual/fr/functio...ape-string.php
Exemple :

Code :

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

$nom_jeu = mysql_real_escape_string($_POST['nom_jeu']);

Code :

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

$_POST['id'] = addslashes($_POST['id']);

Et lorsque qu'un type de donnée DOIT être un identifiant (donc 1, 10, 200, etc ...), c'est nettement plus efficace de caster (ou typer) la donné que de l'échapper.



J'ai commis une erreur dans l'exemple de code que j'ai mis pour l'insertion.
Il ne faut pas mettre d'espace entre chaque valeur.
En rectifiant :

Code :

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

1
2
INSERT INTO usagers (usagers_id, activites)
VALUES (10, 'bleu,vert,jaune')

Au niveau de ton code ça donnerait ceci :

Code :

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

1
2
3
$type = $_POST['type'];
$type = implode(',', $type);
$type = mysql_real_escape_string($type);

Qu'est-ce qui te pousses à utiliser un type SET au lieu de concevoir cela plus proprement, comme l'exemple que j'avais donné, c'est à dire avec 2 tables (tables des activités et tables des associations des activités des usagers).

En faite, perçois tu que la gestion de ces types sera plus facile à gérer avec cette solution ci-dessus qu'avec un champ de type SET ?