pb de requete

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

$critere = $_GET['critere']; $annee = $_GET['annee']; $table = "COTISATIONS_GEST";   if ($critere == "cotisant") {   $requete = "SELECT * FROM g WHERE appel_coti LIKE 'O' ";   $result = mysql_query($requete); while ($data = mysql_fetch_array($result)) { $requete2 = "SELECT * FROM $table WHERE code_gest=".$data['code_gest']." AND annee="".$annee."; $result2 = mysql_query($requete2);

---

j'ai une sélection d'une année que je retrouve dans critère
est ce que ma requete 2 est correcte au niveau ecriture
Je souhaite obtenir les noms dont le code est identique et dont l'année correspond à mon critere annee.

merci
vivi

1- Ne jamais au grand jamais entrer dans une requête des données utilisateurs sans qu'elles aient été filtrée (dans ton cas $annee)

2- Pourquoi faire un select * si tu n'as pas besoin de tous les champs ? Soit selectif sur les données à extraire de la base.

3- Exécuter une requête dans l'itération des résultats de la précédente c'est à mon avis une mauvaise idée.

Exécutes ta première requête , extrait les données nécessaire puis finalement exécute une autre (ou plusieurs si c'est pas possible en 1 seule) requête avec les données précédemment récupéré.

4- Si ce n'est pas trop tard et que tu ta vesrion de mysql est >= 4.1.3 tu devrais envisager l'utilisation de mysqli ou de pdo à la place des mysql_*

j'ai dejà pal mal de souci a construire car débutante.

Peux tu m'aiguiller sur mon code présent ou pas

Bonjour,

Est-ce que tu as testé ce script ? Si tu obtient un message d'erreur, copie le nous. Autrement mis à part les remarques de grunk qui ne sont pas à prendre à la légère, à priori la requête devrait marcher :roll:

j'ai une " en trop c'est bon cela fonctionne.

Merci quand meme
vivi