Discussion :

[benjamin002]

Bonjour,

J'ai un petit problème de requête dans mon site qui gère les prêts de matériels.
Dans ma page d'accueil, je récupère dans un tableau les prêts dont l'utilisateur est responsable et un autre tableau qui récupère les prêts dont il est titulaire.

Mes tables sont comme ceci:
personnes(id_personne, Nom, Prenom, login, pass, email..)
prets(id_pret, date_depart, date_retour, id_personne, id_personne_D, prenom_D, nom_D, BL, id_materiel)

Dans ma table pret, "id_personne" correspond à l'id de la personne responsable du prêt, "id_personne_D" à l'id de la personne destinataire du prêt et nom_D/prenom_D sert à enregistrer les nom et prenom des personnes qui ne sont pas enregistrées.

voici le code de mon accueil:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
<?php
include('connexion.php');
 
//Requête qui récupère les prêts dont l'utilisateur connecté ($cookieLOG est récupérer grace aux cookies) est responsable						
$req="SELECT DISTINCT P.id_personne, PE.id_personne, Nom_D, Prenom_D, id_personne_D, PE.id_materiel, date_retour, login, id_pret
FROM personnes P, prets PE, materiel M
WHERE login = '".$cookieLOG."'
AND PE.id_personne = P.id_personne
ORDER BY date_retour ASC";
 
$resultat = mysql_query($req)or die(mysql_error());
 
$id="";
while($row = mysql_fetch_row($resultat)){
	$id_pers = $row[4];
	$datejour = date('Y-m-d');
	$datefin = $row[6];
	$datejour = new DateTime($datejour);
	$datejour = $datejour->format('Ymd');
	$datefin = new DateTime($datefin);
	$datefin = $datefin->format('Ymd');
 
	//vérification si l'on récupère le nom d'une personne enregistrée dans la bdd ou non, si elle n'est pas enregistrée on récupère le nom_D, prenom_D
	if($id_pers == 0){
                //change la couleur de la ligne en vert si la date de retour n'est pas dépassée
		if($datejour <= $datefin){
			$id = $row[7]; //récupère l'id du prêt
			echo "<tr>";
			echo "<td class='couleur'>$row[2]</td>";
			echo "<td class='couleur'>$row[3]</td>";
			echo "<td class='couleur'>$row[5]</td>";
			echo "<td class='couleur'>$row[6]</td>";
			echo "<td class='couleur'><a href='Retour_Pret.php?id=".$id."'>--></font></a></td>";
			echo "</tr>";
                 //Sinon on change la couleur de la ligne en rouge
		} else {
			$id = $row[7]; //récupère l'id du prêt
			echo "<tr>";
			echo "<td class='couleur2'>$row[3]</td>";
			echo "<td class='couleur2'>$row[4]</td>";
			echo "<td class='couleur2'>$row[5]</td>";
			echo "<td class='couleur2'>$row[6]</td>";
			echo "<td class='couleur2'><a href='Retour_Pret.php?id=".$id."'>--></font></a></td>";
			echo "</tr>";
		}
        si celle-ci est enregistrée on traduit l'id_personne en récupérant son Nom et Prenom dans la table personnes
	}else{
	      $requete="SELECT DISTINCT P.id_personne, id_personne_D, P.Nom, P.Prenom
			       FROM personnes P, prets PE
			       WHERE P.id_personne = '".$id_pers."'
			       AND id_personne_D = P.id_personne";
 
				$result = mysql_query($requete)or die(mysql_error());
 
		while($row1 = mysql_fetch_row($result)){
			if($datejour <= $datefin){
				$id = $row[7]; //récupère l'id du prêt
				echo "<tr>";
				echo "<td class='couleur'>$row1[2]</td>";
				echo "<td class='couleur'>$row1[3]</td>";
				echo "<td class='couleur'>$row[5]</td>";
				echo "<td class='couleur'>$row[6]</td>";
				echo "<td class='couleur'><a href='Retour_Pret.php?id=".$id."'>--></font></a></td>";
				echo "</tr>";
			} else {
				$id = $row[7]; //récupère l'id du prêt
				echo "<tr>";
				echo "<td class='couleur2'>$row1[3]</td>";
				echo "<td class='couleur2'>$row1[4]</td>";
				echo "<td class='couleur2'>$row[5]</td>";
				echo "<td class='couleur2'>$row[6]</td>";
				echo "<td class='couleur2'><a href='Retour_Pret.php?id=".$id."'>--></font></a></td>";
				echo "</tr>";
			}
		}
	}
}
?>

mes requêtes ne doivent pas être bonnes, j'ai des doublons (alors que j'ai rajouter DISTINCT). Ma logique est-elle bonne ou est-ce qu'il existe une meilleure solution?

Merci,

Benjamin

[CinePhil]

1) Les jointures s'écrivent depuis 1992 avec l'opérateur JOIN. Il serait temps de s'y mettre !

2) En écrivant les jointures correctement, tu te serais aperçu que tu n'as qu'une condition de jointure pour deux jointures !
Du coup, tu as un produit cartésien entre la table materiel et le reste !

3) DISTINCT opère sur la totalité des colonnes du SELECT.
N'est-il pas probable que tu aies, pour une personne, plusieurs matériels et plusieurs prêts ?

4) Pour qu'on puisse t'aider efficacement, donne la structure de tes tables et seulement le texte de la requête, indentée correctement, plutôt qu'inclue dans du code PHP.