Sélection aléatoire et update

Version imprimable

Bonjour,voici ce que je désire réaliser.

J'ai une table round(id_round int ,joueur1 int ,joueur2 int,resultat int) qui est censé représenter le déroulement d'un match d'échec.

Je dois piocher au hasard deux joueurs dans la tablejoueur_licencie(id_joueur,id_club,numero_licence,nom_joueur,prenom_joueur,adresse_joueur,ville_joueur,elo,classment_championnat).

En fait je sélectionne deux champs numero_licence(ils sont unique)ce qui me permet de différencier les joueurs entre eux.

et je les insére dans ma table round dans les champs joueur1 et joueur2.

Ensuite je génère un nombre aléatoire(0 1 ou 2) et je l'insère dans le champ résultat:

0: match nul
1: victoire joueur 1
2: victoire joueur2

Ainsi je modélise une rencontre et le résultat.

Voici mon code il y a un problème au niveau de l'insertion des numéros de licence des joueurs.
Code:

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 30 31 32 33 34 35 36 37 38 39 40 41 <?php include("connection_base.php"); $query_joueur1="SELECT numero_licence FROM joueur_licencie ORDER BY RAND( )LIMIT 1" ; $result_joueur1=mysql_query($query_joueur1) or die("Echec de la sélection du premier joueur"); $query_joueur2="SELECT numero_licence FROM joueur_licencie WHERE numero_licence <> ".$joueur1."LIMIT 1"; $result_joueur2=mysql_query($query_joueur2) or die ("Echec de la sélection du second joueur"); $resultat=mt_rand(0,2); while(($insertion_joueur1=mysql_fetch_assoc($result_joueur1)) &&($insertion_joueur2=mysql_fetch_assoc($result_joueur2))) { $query_insertion="INSERT INTO round(joueur1,joueur2,resultat) VALUES ('".$insertion_joueur1['numero_licence']."','".$insertion_joueur2['numero_licence']."','".$_resultat."');" $result_insertion=mysql_query($query_insertion) or die("Echec de l'insertion des paramètres du round"); } switch($resultat) { case 0: echo'Les deux joueurs ont fait match nul'; $nul_joueur1="UPDATE joueur_licencie SET classement_championnat=(classement_championnat+0,5) WHERE numero_licence='".$_joueur1."''"; $result_nul_joueur1=mysql_query($nul_joueur1); $nul_joueur2="UPDATE joueur_licencie SET classement_championnat=(classement_championnat+0,5) WHERE numero_licence='".$_joueur2."''"; $result_nul_joueur2=mysql_query($nul_joueur2); break; case 1: echo'Le joueur 1 a remporté la partie'; $victoire_joueur1="UPDATE joueur SET classement_championnat=classement_championnat+0,5 WHERE numero_licence='".$_joueur1."''"; mysql_query($victoire_joueur1); break; case 2: echo'Le joueur 2 a remporté la partie'; $victoire_joueur2="UPDATE joueur SET classement_championnat=classement_championnat+0,5 WHERE numero_licence='".$_joueur2."''"; mysql_query($victoire_joueur2); break; default: break; }

05/04/2010, 02h20
sabotage

Pourquoi tu ne fais pas

Code:

SELECT numero_licence FROM joueur_licencie ORDER BY RAND( )LIMIT 2

?

Exact ^^.

Jvais tester !

Mais comment utiliser les 2 numéros par la suite ?
Code:

1 2 3 4 5 6 $query_joueurs="SELECT numero_licence FROM joueur_licencie ORDER BY RAND( )LIMIT 2" ; $result_joueurs=mysql_query($query_joueur1) or die("Echec de la sélection des joueurs"); while(($insertion_joueurs=mysql_fetch_assoc($result_joueurs))) { ...... }
J'utilise $insertion_joueur[0] et $insertion_joueurs[1]?

Voilà ce que j'ai fait.Il y a une erreur.

Je pense que ça vient de l'utilisation de résultat du select dans la requête d'insertion !

Code:

".$insertion_joueurs[0].",".$insertion_joueurs[1]."

Code:

1
2
3
4
5
6
7
8
9
10
11
12
include("connection_base.php");
 
$query_joueurs="SELECT numero_licence FROM joueur_licencie ORDER BY RAND( )LIMIT 2" ;
$result_joueurs=mysql_query($query_joueur1) or die("Echec de la sélection des joueurs");
 
$resultat=mt_rand(0,2);
 
while(($insertion_joueurs=mysql_fetch_assoc($result_joueurs)))
{
	$query_insertion="INSERT INTO round(joueur1,joueur2,resultat) VALUES (".$insertion_joueurs[0].",".$insertion_joueurs[1].",".$_resultat.");"
	$result_insertion=mysql_query($query_insertion) or die("Echec de l'insertion des paramètres du round");
}

Salut

Je ne vois pas pourquoi tu utilise la fonction ROUND() de MySQL pour l'insertion des 2 joueurs.
round() c'est pour arrondir un nombre décimal.
Sauf erreur, ll me semble pas qu'il soit permis de faire un ROUND() après un INSERT INTO, ça n'a pas de sens.

Faudrait voir la structure de cette table (qui n'a pas de nom d'ailleurs) pour les résultats, mais à priori ça devrait être simple.
Par contre, il ne faut pas faire l'insertion dans la boucle car la boucle te retourne 1 seul joueur par boucle, alors qu'il te faut les 2 pour l'insertion.
Code:

1 2 3 4 5 6 while(($insertion_joueurs = mysql_fetch_assoc($result_joueurs))) { $joueurs[] = $insertion_joueurs['numero_licence']; } // $query_insertion = "INSERT INTO rencontre (joueur1, joueur2, resultat) VALUES (".(int)$joueurs[0].", ".(int)$joueurs[1].", ".(int)$_resultat.")";
(le ; à la fin était mal placé aussi)
Je parts du principe aussi que le "numero_licence" est un nombre numérique (un entier). Si ce n'est pas le cas, faut enlever les (int) et rajouter des quote -> '' entre les valeurs.

Modifie aussi ceci : ... RAND( )LIMIT...
par : ... RAND() LIMIT ...

05/04/2010, 09h54
asdf007

Ben justement sa table s'appelle round... c'est sûr que ça va pas aller, c'est un mot-clé MySQL (à moins de l'escaper tout le temps). Renomme ça en "rounds" par exemple, il est de bonne pratique de mettre les noms de tables au pluriel.
05/04/2010, 12h39
RunCodePhp

Citation:

Envoyé par asdf007

Ben justement sa table s'appelle round...

Ben zut alors, j'ai pas percuté 1 seconde. Décidément :mrgreen:

Le piège dans cette histoire, c'est surement le fait d'utiliser PhpMyAdmin pour créer ses tables, car ce soft échappe justement tous les noms (table, champs) dans le cas où ce serait un nom réservé.

Pour éviter le piège, faut faire au moins un petit essai dans son code (un SELECT champs, ... FROM la_table), sans échapper évidemment.
Si erreur il y a, on le saura de suite, car il est difficile d'avoir en tête tous les noms réservés.

Enfin, c'est ce que je fais pour éviter les noms réservés ...

Un truc que je viens de voir, c'est ceci : ... classement_championnat+0,5
Si c'est un nombre décimal, faut faire comme ceci : ... classement_championnat+0.5
La virgule est le séparateur Français, mais en Anglais c'est un point.

L'insertion dans la table "rounds" fonctionne.Mais pas l'update du classement dans la table joueur_licencie.

A quoi sert le int dans ce code :

Code:

(int)$joueurs[0]

Code:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
<?php
 
include("connection_base.php");
 
$query_joueurs="SELECT numero_licence FROM joueur_licencie ORDER BY RAND() LIMIT 2";
$result_joueurs=mysql_query($query_joueurs) or die("Echec de la sélection des numéros de licence des joueurs");
 
$resultat=mt_rand(0,2);
 
while($insertion_joueurs=mysql_fetch_array($result_joueurs))
{
	$joueurs[] = $insertion_joueurs['numero_licence'];
}
	$query_insertion="INSERT INTO rounds(joueur1,joueur2,resultat) VALUES (".(int)$joueurs[0].",".(int)$joueurs[1].",".(int)$resultat.")";
	$result_insertion=mysql_query($query_insertion) or die("Echec de l'insertion des paramètres du round");
 
	switch($resultat)
	{
		case 0:
		echo'Les deux joueurs ont fait match nul';
		$nul_joueur1="UPDATE joueur_licencie SET classement_championnat=(classement_championnat+0.5) WHERE numero_licence='".$joueurs[0]."'";
		$result_nul_joueur1=mysql_query($nul_joueur1) or die("Echec de l'insertion du résultat nul pour le joueur 1");
		$nul_joueur2="UPDATE joueur_licencie SET classement_championnat=(classement_championnat+0.5) WHERE numero_licence='".$joueurs[1]."'";
		$result_nul_joueur2=mysql_query($nul_joueur2) or die("Echec de l'insertion du résultat nul pour le joueur 2");
		break;
 
		case 1:
		echo'Le joueur 1 a remporté la partie';
		$victoire_joueur1="UPDATE joueur SET classement_championnat=classement_championnat+0.5 WHERE numero_licence='".$joueurs[0]."'";
		mysql_query($victoire_joueur1) or die("Echec de l'insertion de la victoire du joueur 1");
		break;
 
		case 2:
		echo'Le joueur 2 a remporté la partie';
		$victoire_joueur2="UPDATE joueur SET classement_championnat=classement_championnat+0.5 WHERE numero_licence='".$joueurs[1]."'";
		mysql_query($victoire_joueur2) or die("Echec de l'insertion de la victoire du joueur 2");
		break;
 
		default:
		break;
	}
mysql_close($conn);
?>

05/04/2010, 16h42
guipe

C'est bon ça marche.Maintenant il faut que je modélise le fait que deux joueurs ne peuvent s'affronter qu'une seule fois !

J'ai modifié ce que je voulais faire.Je veux juste sélectionner 2 numéros de licence au hasard.Ca ça marche.Puis je sélectionne un nombre entre 0 1 et 2 au hasard.Et j'affiche les valeurs que j'obtiens !

C'est à dire les 2 joueurs sont : .....

Le résultat est : ....

Voici le code :

Code:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
<?php
 
include("connection_base.php");
 
$query_joueurs="SELECT numero_licence FROM joueur_licencie ORDER BY RAND() LIMIT 2";
$result_joueurs=mysql_query($query_joueurs) or die("Echec de la sélection des numéros de licence des joueurs");
 
$resultat=mt_rand(0,2);
 
while($insertion_joueurs=mysql_fetch_array($result_joueurs))
{
	$joueurs[] = $insertion_joueurs['numero_licence'];
}
 
echo'Le round oppose le joueur '.$joueurs[0].' et le joueur '.$joueurs[1].'<br>';
 
switch($resultat)
{
	case 0:
	echo'Les deux joueurs '.$joueurs[0].' et '.$joueurs[1].' ont fait match nul';
	$nul_joueur1="UPDATE joueur_licencie SET classement_championnat=((classement_championnat)+1) WHERE numero_licence='".$joueurs[0]."'";
	$result_nul_joueur1=mysql_query($nul_joueur1) or die("Echec de la modification du classement du  joueur 1(nul)");
	$nul_joueur2="UPDATE joueur_licencie SET classement_championnat=((classement_championnat)+1) WHERE numero_licence='".$joueurs[1]."'";
	$result_nul_joueur2=mysql_query($nul_joueur2) or die("Echec de la modification du classement du joueur 2(nul)");
	break;
 
	case 1:
	echo'Le joueur '.$joueurs[0].' a remporté la partie';
	$victoire_joueur1="UPDATE joueur_licencie SET classement_championnat=((classement_championnat)+2) WHERE numero_licence='".$joueurs[0]."'";
	mysql_query($victoire_joueur1) or die("Echec de la modification du classement du joueur 1(victoire)");
	break;
 
	case 2:
	echo'Le joueur '.$joueurs[1].' a remporté la partie';
	$victoire_joueur2="UPDATE joueur_licencie SET classement_championnat=((classement_championnat)+2) WHERE numero_licence='".$joueurs[1]."'";
	mysql_query($victoire_joueur2) or die("Echec de la modification du classement du joueur 2(victoire)");
	break;
 
	default:
	break;
}
mysql_close($conn);
?>

Ca marche quasiment tout le temps sauf 1 fois sur 50 ou j'ai une option "or die" qui se déclenche.Je ne sais pas pourquoi !

05/04/2010, 20h22
asdf007

Teste les mysql_error.
05/04/2010, 20h45
guipe

J'ai jamais fait !
05/04/2010, 22h32
asdf007

Dans les die :
mysql_query(...) or die(mysql_error())
06/04/2010, 00h46
guipe

OK c'est bon.Ce que je n'arive pas à faire là c'est éviter que deux joueurs peuvent se rencontrer 2 fois !
06/04/2010, 07h51
RunCodePhp

Citation:

Ce que je n'arive pas à faire là c'est éviter que deux joueurs peuvent se rencontrer 2 fois !

La clé primaire doit être double : PRIMARY KEY (joueur1, joueur2)

Cependant, il sera possible de créer les 2 possibilités des 2 mêmes joueurs (match allé et retour si on peu dire).
Si l'appli ne doit pas l'autoriser, faudra alors le vérifier avant insertion, faire une requête quoi.
Voir même ne pas donner la possibilité de sélectionner ces 2 joueurs si une rencontre a déjà été faite entre eux (requête toujours).
06/04/2010, 15h28
guipe
Les joueurs ne peuvent se rencontrer qu'une seule fois ! Pas de notion de match aller-retour !

Donc si je met le champ joueur 1 et 2 en clé primaire ça suffit ?

Je n'arrive pas à définir les 2 champs en tant que clé primaire.C'est quoi la manipulation pour y arriver? Ou le bout de code que je dois écrire ?

J'ai fait ça et ça ne fonctionne pas !
Code:

1 2 3 4 5 6 7 8 CREATE TABLE rounds ( id_round INT NOT NULL AUTO_INCREMENT, joueur1 INT NOT NULL, joueur2 INT NOT NULL, resultat INT NOT NULL, CONSTRAINT pk_joueurs PRIMARY KEY(joueur1,joueur2) );
06/04/2010, 19h40
RunCodePhp

Non, faut surtout pas rajouter un champ auto_increment, pas de id_round, juste joueur1 et joueur2 en clé primaire, comme tu as fais.

Après, il ne me semble pas qu'il existe un moyen d'éviter les 2 possibilités (les aller/retour).

Les 2 couples comme ceci par exemple : 123 - 789 et 789 - 123 sont différents, et une clé double ne pourra pas l'éviter, ça évite juste la duplication de ces 2 couples d'IDs.

En somme, je n'est pas connaissance d'une telle instruction en SQL.
Si ça existe, je l'apprendrais.

A défaut, et pour le faire (du moins pour l'éviter), je ne vois que 2 possibilités :
- Soit les procédures stockées.
On fait une requête qui vérifiera si le couple d'IDs (joueur1/joueurs2) est déjà dans la Bdd. On insère ou pas selon le cas.
- Soit plus simplement, on effectue cette requête dans sa page php, dans sont code (comme toutes requêtes), ensuite on insère ou pas selon le cas.
06/04/2010, 21h21
guipe

C'est bon j'ai modifié le schéma de ma base !

Mais comment faire pour vérifier que le couple existe déjà dans la base ?
07/04/2010, 06h38
RunCodePhp

Citation:

Mais comment faire pour vérifier que le couple existe déjà dans la base ?

A ben là ça va plus.
C'est qui qui est en charge de créer ce projet dis moi ? ;)

Personnellement, je te retourne la question.
Comment ferais tu pour le vérifier ?
Sous entendu, que tu as tous les éléments en mains maintenant, et que si tu ne parviens pas à trouver comment le faire, c'est tout le reste de ton projet qui risque de ne pas avancer, parce que ici c'est trop simple.

C'est pas un reproche, mais il faut chercher un peu, non ?
07/04/2010, 19h52
guipe

Je mets mes champs joueur1 et joeuur2 en clé primaire et dans le or die de la requete j'explique que si l'insertion marche pas c'est que le match a déjà eu lieu !
08/04/2010, 06h51
RunCodePhp
Citation:

Je mets mes champs joueur1 et joeuur2 en clé primaire et dans le or die de la requete j'explique que si l'insertion marche pas c'est que le match a déjà eu lieu !

Pourquoi pas, encore que là c'est un peu radical.

Mais ceci n'empêchera pas de créer le match retour, chose où tu dis qu'il ne devrait pas être possible.

Exemple : (table rounds)
joueur1 | joueur2
123 | 789
789 | 123
123 | 123
789 | 789

Ici, les 2 joueurs auront fait 2 parties d'échec entre eux (aller/retour), et même les faire jouer contre eux même. La base de donnée ne l'empêchera pas, il n'y aura pas de retour d'erreur.

Si 1 match a déjà eu lieu entre ces 2 joueurs :
joueur1 | joueur2
123 | 789
Et qu'on tente de les faire à nouveau jouer entre eux, comme 789 contre 123, le seul moyen de l'éviter c'est de faire une requête pour vérifier si une ligne existe :
Code:

1 2 3 4 5 6 SELECT COUNT(*) AS total FROM rounds WHERE (joueur1 = 123 AND joueur2 = 789) OR (joueur1 = 789 AND joueur2 = 123) OR (joueur1 = 123 AND joueur2 = 123) OR (joueur1 = 789 AND joueur2 = 789)
Cette requête retourne qu'1 seule ligne, et la valeur de "total" retournera 0 ou 1
Si c'est 1 -> Erreur, la rencontre a déjà eu lieu ou non autorisée : Pas d'insertion
Si c'est 0 -> Ok, on effectue l'insertion

Ici, on évite 4 choses potentiellement possible en 1 opération.

Les valeurs de joueurs1 et joueur2 tu les as, c'est le tableau $joueurs.

De plus, si on raisonne un peu, faire la requête avec le rand() (table joueur_licencie) n'est géantissime, car obtenir un duo au tout début d'un tournoi, ça va être facile, mais après, par la suite, et pour les derniers joueurs très particulièrement, ça risque d'être compliqué d'obtenir des joueurs qui ne se seront pas encore rencontrés.
Faudrait trouver un autre moyen à mon avis.