Discussion :

[defmsy]

Bonjour,

Je débute en T-SQL et je suis bloqué sur une requête.

J'ai la requête suivante qui me retourne la concaténation de plusieurs champs des résultats grâce à un curseur :

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
SET @no_facture = '66151'
 
DECLARE C_LINES CURSOR FOR
      SELECT T1.NO_FACT, T1.INCIDENT , T1.CODE_LIGNE, T1.NO_SEQUENC, T1.REFERENCE, T1.LIBELLE
      FROM AVENTEPI as T1
      WHERE T1.NO_FACT = @no_facture
 
      union
 
      SELECT T1.NO_FACT, T1.INCIDENT , T1.CODE_LIGNE, T1.NO_SEQUENC, T1.CODE_DIVER, T1.LIBELLE
      FROM AVENTEMO as T1
      WHERE T1.NO_FACT = @no_facture
 
      union
 
      SELECT T1.NO_FACT, T1.INCIDENT , T1.CODE_LIGNE, T1.NO_SEQUENC, T1.CODE_DIVER, T1.LIBELLE
      FROM AVENTEDI as T1
      WHERE T1.NO_FACT = @no_facture
 
      ORDER BY 2, 3, 4
 
OPEN C_LINES 
DECLARE @no_fact INT
DECLARE @incident CHAR(1)
DECLARE @code_ligne INT
DECLARE @no_sequence INT
DECLARE @ref VARCHAR(17)
DECLARE @libelle VARCHAR(35)
DECLARE @concat VARCHAR(8000)
SET @concat = ''
FETCH C_LINES INTO @no_fact, @incident, @code_ligne, @no_sequence, @ref, @libelle
WHILE @@FETCH_STATUS = 0
BEGIN
      IF @no_fact IS NOT NULL
            SET @concat = @concat + @ref + ' - ' + @libelle + ' / '
      FETCH C_LINES INTO @no_fact, @incident, @code_ligne, @no_sequence, @ref, @libelle
END
CLOSE C_LINES
DEALLOCATE C_LINES
IF @concat IS NOT NULL
      SET @concat = SUBSTRING(@concat, 1, LEN(@concat) - 1)
SELECT @concat

Je dois maintenant récupérer les factures sur une période donnée et pour chacune d'entre elle retourner le résultat de la requête ci-dessus correspondant.

[elsuket]

Bonjour,

N'utilisez pas les curseurs : c'est lourd, lent, et contre-performant par définition. En effet SQL est par essence ensembliste : c'est à dire qu'il est conçu pour traiter les données dans leur ensemble, et pas dans leur unité.

On peut donc réécrire votre requête comme suit :

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
 
SET @no_facture = '66151';
 
WITH
	CTE AS
	(
		      SELECT	NO_FACT
				, INCIDENT
				, CODE_LIGNE
				, NO_SEQUENC
				, REFERENCE
				, LIBELLE
		      FROM	dbo.AVENTEPI AS T1
		      WHERE	NO_FACT = @no_facture 
		UNION 
		      SELECT	NO_FACT
				, INCIDENT
				, CODE_LIGNE
				, NO_SEQUENC
				, REFERENCE
				, LIBELLE
		      FROM	dbo.AVENTEMO AS T1
		      WHERE	NO_FACT = @no_facture 
		UNION 
		      SELECT	NO_FACT
				, INCIDENT
				, CODE_LIGNE
				, NO_SEQUENC
				, REFERENCE
				, LIBELLE
		      FROM	dbo.AVENTEDI AS T1
		      WHERE	NO_FACT = @no_facture 
		ORDER BY INCIDENT, CODE_LIGNE, NO_SEQUENC
	)
SELECT	@concat = CASE WHEN @concat IS NULL THEN '' ELSE @concat END + @ref +  + ' - ' + REFERENCE + ' / '
FROM	CTE

Vous pouvez peut-être écrire UNION ALL à la place de UNION si vous êtes absolument certain que tous les lignes de chaque table sont mutuellement exclusives : cela évite le tri en vue de l'élimination automatique des lignes en doublon. Par ailleurs il est important de qualifier le nom des tables par le nom du schéma auquel elles appartiennent : par défaut c'est dbo.

En ce qui concerne la recherche par dates, il est évident que si vous ne donnez ni le nom de la colonne date, ni la structure des tables, il nous est impossible de vous aider plus avant

@++