Discussion :

[Mandrake31]

Bonjour,
je bataille depuis un certain temps avec une procédure stockée avec sql server 2005. Je ne comprends pas pourquoi j' ai juste les noms des colonnes sans données en sortie et donc ce que je devrais faire pour afficher le résultat complet.
Merci à vous.

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
 
USE Magasin
IF EXISTS 
(
	SELECT name 
	FROM sysobjects 
	WHERE name ='ClientsParCategorie' 
	AND type = 'P'
) 
DROP PROCEDURE ClientsParCategorie
GO
 
CREATE PROCEDURE ClientsParCategorie @Nom_cat VARCHAR(30)
AS
BEGIN
	SELECT cli.Nom_client, com.Date_com,  prod.Titre_prod
	FROM client cli, categorie cat, commande com,
	 produit prod, detail_commande det
	WHERE cli.Id_client = com.Id_client
	AND com.Id_com = det.Id_com
	AND det.Id_prod = prod.Id_prod
	AND  prod.Id_prod = cat.Id_cat
	AND cat.id_cat = 
	( 	
		SELECT id_cat 
		FROM categorie
		WHERE UPPER(cat.Nom_cat) 
		LIKE UPPER('%'+@Nom_cat+'%')
	)
END
GO
 
 
EXECUTE ClientsParCategorie 'imprimantes'
GO

[seminoque]

Difficile de répondre car il faudrait connaître le contenu de chaque table Voici quelques suggestions.

Il me semble tout d'abord que la requête peut être simplifiée

Code :

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

1
2
3
4
5
6
7
8
 
SELECT cli.Nom_client, com.Date_com,  prod.Titre_prod
FROM client cli, categorie cat, commande com, produit prod, detail_commande det
WHERE cli.Id_client = com.Id_client
AND com.Id_com = det.Id_com
AND det.Id_prod = prod.Id_prod
AND  prod.Id_prod = cat.Id_cat
AND UPPER(cat.Nom_cat) LIKE UPPER('%'+@Nom_cat+'%')

Ensuite est-ce que la requête suivante renvoie des résultats (en dehors de la procédure bien sûr) ?

Code :

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

1
2
3
4
5
6
7
8
 
SELECT cli.Nom_client, com.Date_com,  prod.Titre_prod
FROM client cli, categorie cat, commande com, produit prod, detail_commande det
WHERE cli.Id_client = com.Id_client
AND com.Id_com = det.Id_com
AND det.Id_prod = prod.Id_prod
AND  prod.Id_prod = cat.Id_cat
AND UPPER(cat.Nom_cat) LIKE '%IMPRIMANTES%'

Si non, il faut commencer par une requête simple et la compliquer progressivement pour savoir à partir de quel moment il n'y a plus de résultat (TOP 1 permet de remonter juste une ligne et pas tout le contenu de la table ) :

Code :

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

1
2
3
4
 
SELECT TOP 1 *
FROM client cli, commande com
WHERE cli.Id_client = com.Id_client

Code :

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

1
2
3
4
5
 
SELECT TOP 1 *
FROM client cli, commande com, detail_commande det
WHERE cli.Id_client = com.Id_client
AND com.Id_com = det.Id_com

etc...
________________________________
Seminoque, créateur de
http://www.bingokaz.com

[elsuket]

Bonjour,

Vous devriez spécifier vos jointures par le mot-clé [INNER] JOIN :

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
-- Toutes versions 
SELECT CLI.Nom_client,
		COM.Date_com,
		PROD.Titre_prod
FROM dbo.client CLI,
JOIN commande COM ON CLI.Id_client = COM.Id_client
JOIN detail_commande DET ON COM.Id_com = DET.Id_com
JOIN produit PROD ON DET.Id_prod = PROD.Id_prod
JOIN categorie CAT ON PROD.Id_prod = CAT.Id_cat
JOIN
(
	SELECT id_cat 
	FROM categorie
	WHERE UPPER(cat.Nom_cat) 
	LIKE UPPER('%'+@Nom_cat+'%')
) TMP ON TMP.id_cat = CAT.id_cat
 
-- Depuis SQL Server 2005
WITH
	CTE AS
	(
		SELECT id_cat 
		FROM categorie
		WHERE UPPER(cat.Nom_cat) 
		LIKE UPPER('%'+@Nom_cat+'%')
	)
SELECT CLI.Nom_client,
		COM.Date_com,
		PROD.Titre_prod
FROM dbo.client CLI
JOIN dbo.commande COM ON CLI.Id_client = COM.Id_client
JOIN dbo.detail_commande DET ON COM.Id_com = DET.Id_com
JOIN dbo.produit PROD ON DET.Id_prod = PROD.Id_prod
JOIN dbo.categorie CAT ON PROD.Id_prod = CAT.Id_cat
JOIN CTE ON CTE.id_cat = CAT.id_cat

Ce qui vous permet de séparer vos jointures de vos filtres (WHERE) et de faciliter le travail du compilateur.
Dès lors pour tester où la jointure ne se fait pas, vous devrez écrire des demi-jointures (LEFT [OUTER] JOIN dans le cas présent).

@++

[Mandrake31]

Merci beaucoup pour tous ces bons conseils,
je suis débutant avec sql server mais j' ai fait un peu d' Oracle ; je pensais en fait que c' était dû à un paramétrage particulier qu' il fallait faire pour avoir ces données en sortie.
Je vais donc employer votre méthodologie et je vous tiens au courant

[SQLpro]

Code :

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

1
2
WHERE UPPER(cat.Nom_cat) 
LIKE UPPER('%'+@Nom_cat+'%')

Ceci est idiot en terme sémantique et plus encore en terme de perforances. Utilisez une collation pour résoudre ce problème. Venant d'Oracle il est normal que vous ne connaissiez pas le principe des collations qui fait partie de la norme SQL et qu'Oracle n'a jamais implémenté.

Code :

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

WHERE cat.Nom_cat LIKE '%'+@Nom_cat+'%' COLLATE French_CI_AI

voir l'article que j'ai écrit à ce sujet : http://sqlpro.developpez.com/cours/s...er/collations/

A +

[Mandrake31]

J' ai réussi à résoudre mon problème ainsi finalement. Merci à vous

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
 
USE Magasin
 
IF EXISTS 
	(
		SELECT name 
		FROM sysobjects 
		WHERE name ='ClientsParCategorie' 
		AND type = 'P'
	) 
	DROP PROCEDURE ClientsParCategorie
	GO
 
CREATE PROCEDURE ClientsParCategorie @Nom_cat VARCHAR(30)
	AS
BEGIN
	SELECT CLI.Nom_client, COM.Date_com, PROD.Titre_prod
          FROM client CLI 
           INNER JOIN commande COM ON COM.Id_client = CLI.Id_client
           INNER JOIN detail_commande DET ON DET.Id_com = COM.Id_com
           INNER JOIN produit PROD ON PROD.Id_prod = DET.Id_prod
           INNER JOIN categorie CAT ON CAT.Id_cat = PROD.Id_cat
           WHERE cat.Nom_cat LIKE '%'+@Nom_cat+'%' COLLATE French_CI_AI
END