Discussion :

[kate59]

Bonjour,
je dois afficher une liste deroulante et des infos derrière sur une page au choix de la liste.
Ce qui m'embete et sur la liste dérounante j'ai plusieurs fois la même valeur du titre de voyaye qui est le pays repeter plusieurs fois a part quand je fais group by mais la j'ai des voyages qui disparait
si je fais distinct ayant besoin de plusieurs valeur pour affiche les resultats ca ne marche pas

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
	public function getListGroupByDestinationByType($type_voyage)
	{
 
		$listeVoyage = array();
 
		$sql = 'SELECT *
		FROM voyage where page_voyage="'.$type_voyage.'" and active_voyage="1"
		group by titre_voyage';
 
 
		$requete = $this->db->query($sql);
 
		while ($Voyage = $requete->fetch(PDO::FETCH_ASSOC))
		$listeVoyage[] = new Voyage ($Voyage);
 
		$requete->closeCursor();
 
 
		return $listeVoyage;
 
 
	}

une idée svp ? MERI

[tatayo]

Bonjour,
Je suis étonné que SqlServer accepte ta requête: un SELECT * avec un GROUP BY sur une seule colonne.
Je suppose donc que ta table ne contient que 2 colonnes.
Le jour où tu ajoutes une colonne dans ta table, ta requête ne fonctionnera plus...
C'est (entre autre) pour ça qu'il faut bannir les SELECT *.

Cela étant dit, si tu veux des valeurs distinctes, il faut utiliser DISTINCT.
Si tu veux utiliser des fonctions d'agrégation (SUM, COUNT, MAX...), il faut utiliser GROUP BY.

Là tu nous dis que "ça ne fonctionne pas".
Mais encore ?
Est-ce que tu peux nous donner la structure de ta table, un jeu de test et le résultat attendu ?
Sans ces éléments (ni la requête et l'éventuel message d'erreur), on ne pourra pas t'aider.

Tatayo.

[escartefigue]

Si le but est de masquer les valeurs répétées dans la liste déroulante, il faut le faire coté application et non pas dans la requête *.
Si le but est de rechercher des valeurs distinctes, alors le mot clef DISTINCT est de rigueur.
Si enfin, le but est de calculer des agrégats, alors c'est GROUY BY qui est requis, mais en effet, comme le signale justement Tatayo, il faut que les colonnes soient cohérentes avec la clause SELECT.

Dans tous les cas, SELECT * est à proscrire dans les traitements : résultat instable, transport de colonnes inutiles et index couvrants inéligibles.
SELECT * ne doit être utilisé que pour des requêtes à la volée.

* EDIT une solution toutefois par requête : utiliser la fonction LAG combinée avec un CASE
Par exemple :

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
with tx(xid, xval) as
    (select 001, 'abcdef'  union all
     select 002, 'abcdef'  union all
     select 003, 'toto'    union all
     select 004, 'xxxxx'   union all
     select 005, 'abcdef'  union all
     select 006, 'toto'    union all
     select 007, 'azerty'  union all
     select 008, 'azerty'  union all
     select 009, 'toto'
    )
select xid
     , case when xval=lag(xval) over(order by xval, xid) then '' else xval end as xval
from tx

Donne :

xid	xval
1	abcdef
2
5
7	azerty
8
3	toto
6
9
4	xxxxx

[Dendrite]

Je ne comprends pas bien ta question, mais au cas où, teste déjà ça :

Code SQL :

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

1
2
3
SELECT DISTINCT titre_voyage
FROM voyage
where page_voyage="'.$type_voyage.'" and active_voyage="1"