Discussion :

[jeanmarc1234]

Bonjour,

J'ai 2 Tables :

[Categories]
Cat_Id
Cat_Nom


[Annonces]
Ann_Id
.....
Ann_Categories
Ann_Categories1
Ann_Categories2

Grâce à énormément d'aide de pas mal de personnes de ce forum, j'ai réussi à obtenir une requête qui me donne le total d'annonces par catégories pour le champs Ann_Categories.

Et je voudrais en fait obtenir le total d'annonces par categories pour les 3 champs confondu(Ann_Categories, Ann_Categories1, Ann_Categories2)

Voici le code pour 1 seul Champ:

Code :

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

1
2
3
4
5
6
 
SELECT C.Cat_Id, C.Cat_Nom, COUNT(*) AS Total_annonce 
FROM Categories C
LEFT JOIN Annonces A ON A.Ann_Categories = C.Cat_Id 
GROUP BY C.Cat_Id, C.Cat_nom
ORDER BY Total_annonce DESC

J'ai testé cela mais bien évidement cela m'affiche un total érroné:

Code :

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

1
2
3
4
5
6
7
8
 
SELECT C.Cat_Id, C.Cat_Nom, COUNT(*) AS Total_annonce 
FROM Categories C
LEFT JOIN Annonces A ON A.Ann_Categories = C.Cat_Id
LEFT JOIN Annonces A1 ON A1.Ann_Categories1 = C.Cat_Id 
LEFT JOIN Annonces A2 ON A2.Ann_Categories2 = C.Cat_Id  
GROUP BY C.Cat_Id, C.Cat_nom
ORDER BY Total_annonce DESC

Une petit idée ? car je ne suis vraiment pas doué

[jnore]

Bonsoir,

Essaie ceci:

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
 
SELECT Cat_Id,Cat_Nom,Count(Ann_Categories) AS Total_annonce
FROM Categories JOIN
(
	SELECT Ann_Categories
	FROM Annonces
	WHERE Ann_Categories IS NOT NULL
 
	UNION ALL
 
	SELECT Ann_Categories1
	FROM Annonces
	WHERE Ann_Categories1 IS NOT NULL
 
	UNION ALL
 
	SELECT Ann_Categories2
	FROM Annonces
	WHERE Ann_Categories2 IS NOT NULL
) sous_requete
ON Categories.Cat_Id=sous_requete.Ann_Categories
GROUP BY Cat_Id,Cat_Nom
ORDER BY Count(Ann_Categories) DESC

Je n'ai pas de quoi vérifier si c'est ok.
Dis nous si c'est bon

[SQLpro]

Votre table ne repsctant pas les formes normales et notamment la première (repétiton des catégories) vos requêtes seront toujours :
1) difficile à écrire
2) totalement contre performantes

Commencez par rectifier votre modèle afin de normaliser vos catégories...

Petite question : que se passera t-il s'il faut une 4e catégorie ?

A +

[jeanmarc1234]

Bonjour,
jnore ta requete me renvoi des résultats érronés (mais qui ce rapproche du nombre d'annonces réels)

SQLPRO Vous parlez de normaliser mes categories. Auriez vous un modèle de tables afin de régulariser tout cela ?
Si il faut une 4e Categories, je serais obligé d'ajouter un champs Ann_categories3 dans ma table Annonces.

[jnore]

Pour la requete que je t'ai proposée, dis nous en un peu plus, qu'est-ce qui ne va pas? Donne-nous quelques données, ce sera plus aisé!

Concernant les propos de SQLPro, l'idéal aurait été d'avoir un seul champ "Catégories" dans la table "Annonce" . Ce champ reprenant l'identité (id ou clé unique )"Cat_id" de ta table "Catégories".

L'intéret d'une base de données est de mettre des champs en relation de façon définitive.
Une requete dépend de bcp des liaisons. Si, sous pretexte que tu veux rajouter une catégorie, t'es obligé de modifier ta requete, cela veut dire que tes tables sont mal conçues.
L'ajout de données se faisant en ligne et non pas en colonnes.

[jeanmarc1234]

Bonjour,

En fait on sais mal compris, mes tables sont comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
 
[Categories]
Cat_Id
Cat_Nom
 
[Annonces]
Ann_Id
....
Ann_Categories
Ann_Categories1
Ann_Categories2
Ann_Categories3

Les champs
Ann_Categories, Ann_Categories1, Ann_Categories2, Ann_Categories3 reprennent bien l'Id de Cat_Id.
Il y a quatre champs dans ma table [Annonces] pour qu'une annonce déposée puisse avoir plusieurs catégories.

Exemple d'une ligne annonce :

Code :

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

1
2
3
4
5
 
Ann_Id	Ann_Nom	Ann_Categories	Ann_Categories1	Ann_Categories2	
 
 
1	    DURAND		1	           5	                     34

Ensuite pour ta requête :

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
 
SELECT Cat_Id,Cat_Nom,Count(Ann_Categories) AS Total_annonce
FROM Categories JOIN
(
	SELECT Ann_Categories
	FROM Annonces
	WHERE Ann_Categories IS NOT NULL AND Ann_Valide='true'
 
	UNION ALL
 
	SELECT Ann_Categories1
	FROM Annonces
	WHERE Ann_Categories1 IS NOT NULL AND Ann_Valide='true'
 
	UNION ALL
 
	SELECT Ann_Categories2
	FROM Annonces
	WHERE Ann_Categories2 IS NOT NULL AND Ann_Valide='true'
 
	UNION ALL
 
	SELECT Ann_Categories3
	FROM Annonces
	WHERE Ann_Categories3 IS NOT NULL AND Ann_Valide='true'
) sous_requete
ON Categories.Cat_Id=sous_requete.Ann_Categories
GROUP BY Cat_Id,Cat_Nom
ORDER BY Count(Ann_Categories) DESC

par exemple pour la categorie 1, la requête me renvoi le résultat :
255 annonces

Alors quand utilisant cette requête :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT * FROM [Annonces] 
LEFT JOIN Categories C ON Annonces.Ann_Categories = C.Cat_Id 
LEFT JOIN Categories C1 ON Annonces.Ann_Categories1 = C1.Cat_Id 
LEFT JOIN Categories C2 ON Annonces.Ann_Categories2 = C2.Cat_Id
 LEFT JOIN Categories C3 ON Annonces.Ann_Categories3 = C3.Cat_Id 
WHERE ([Ann_Valide] = 'true' 
AND ([Ann_Categories] = 1 
OR [Ann_Categories1] = 1 
OR [Ann_Categories2] = 1  
OR [Ann_Categories3] = 1))

cela me renvoi un tableau contenant 250 annonces.

PS : dans ma dernière requete la jointure me sert simplement à afficher le champ Cat_Nom plutôt que l'id de la catégorie.


Merci de votre aide !

[Alain Defrance]

Pourquoi ne pas adopter un modèle logique comme celui ci :


Categories(idCat, nomCat);
Annonces(idAnn, ...);
Appartenir(#idCat+#idAnn);
Dans ce cas la requete deviens beaucoup plus simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT
     nomCat AS NOM,
     COUNT(*) AS NOMBRE
FROM
     Categories AS Cat
INNER JOIN
     Appartenir AS App ON
          Cat.idCat=App.idCat
GROUP BY
     Cat.idCat

[jeanmarc1234]

Bonjour,

Je pense en effet que je vais adopter ton modèle, cela devient tout dessuite plus simple !


Merci à vous tous !!!