Discussion :

[kerozen77]

Bonjour,

Je cherche à incrémenter les variables (position) des joueurs d'une même room. Pour cela je sélectionne le premier joueur trouvé dans la liste des joueurs de la room et je lui donne pour position 0 puis je selectionne le second trouvé et je lui donne pour position 1 ect...

Voici mon code qui ne fonctionne pas

Code :

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

1
2
3
4
5
6
7
8
9
10
 
//$roomjoueur le nombre de joueurs dans la room
for ($compteur=0; $compteur<$roomjoueur; $compteur++) {
$reqjoueurn = $bdd->prepare('SELECT id_joueur FROM joueurs WHERE id_room = ? ORDER BY id_joueur LIMIT ?,1');
$reqjoueurn->execute(array($_SESSION['id_room'], $compteur));
$joueurn = $reqjoueurn->fetch();
 
$insertmbr = $bdd->prepare("UPDATE joueurs SET position = ? WHERE id_joueur = ?");
$insertmbr->execute(array($compteur, $joueurn['id_joueur']));
}

Si je remplace le 2eme ? par un 0,1,2... cela marche mais pas avec une variable. :/

Il y aurait il une autre solution ?

[AS_PTT]

je ne pense pas que mettre une requête dans une boucle soit une bonne idée, mais je peux me tromper.
D'habitude on fait une boucle pour trier les données.

Et aussi je crois que tu fais un 'prepare' avec une requête 'select' ce qui n'as pas de sens à mon goût. Je pense que c'est plutot réservé le prepare à un 'insert'. Et je peux aussi me tromper.

[Invité]

Je répète :
tant que tu ne sauras pas exprimer clairement, avec des mots, le but que tu veux atteindre, tu ne sauras pas comment l'atteindre.

La LOGIQUE de ton code est totalement fausse.


Et vu que tu n'expliques pas à quoi ça doit servir, on ne peux pas t'orienter vers une meilleure solution.

[Tsilefy]

Je ne me pronnce pas sur la quesiton originale, qui n'est pas claire du tout en effet, mais sur ceci:

je ne pense pas que mettre une requête dans une boucle soit une bonne idée, mais je peux me tromper.
D'habitude on fait une boucle pour trier les données.

Tu te trompes. On peut tout mettre dans une boucle, y compris une requête. Dans le cas qui nous intéresse, la partie prepare() ne change pas, donc il faut la mettre en dehors de la boucle. La partie execute() a bien sa place dans une boucle.

Et aussi je crois que tu fais un 'prepare' avec une requête 'select' ce qui n'as pas de sens à mon goût. Je pense que c'est plutot réservé le prepare à un 'insert'. Et je peux aussi me tromper.

Effectivement, tu te trompes, et je corrige pour les débutants qui risquent de tomber sur tes "conseils". On doit faire une requête préparée à chaque fois qu'on envoie des variables issues de l'extérieur vers la base des données. La préparation (prepare + execute) sert à protéger la requête contre une injection, et l'injection peut survenir dans toutes les formes de requête MYSQL.

[AS_PTT]

ok, merci ces des questions que je me posais en fait. Merci pour tes réponses

[kerozen77]

Finalement j'ai opté pour incrémenter la variable position lors de l'entré d'un joueur dans une room, donc mon problème est résolu.
Merci quand même ^^