[Curseur] Problème de boucle [Résolu]

[afrodje]

Bonjour,

J'ai suivi ce tuto pour établir le traitement ci-dessous :

Code :

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
-- Déclaration des variables
DECLARE @numero_facture int
DECLARE @statut VARCHAR(10)
DECLARE @id int
 
-- Déclaration du curseur
DECLARE curseur_facture CURSOR FOR 
SELECT id, cast(numero_facture AS INT) AS num_fact, RTRIM(statut) AS statut FROM Facture_statut_facture WHERE traiter = 0 ORDER BY numero_facture
 
-- Ouverture curseur
OPEN curseur_facture
 
FETCH NEXT FROM curseur_facture INTO @id, @numero_facture , @statut
 
-- Association curseur/variable
--FETCH curseur_facture INTO @numero_facture , @statut
 
-- Boucle
WHILE @@FETCH_STATUS = 0
BEGIN	
	-- erreur = 1
	IF @statut = 'ERR'
	BEGIN
		UPDATE dbo.Facture_entete
		SET erreur = 1
		WHERE numero_facture = @numero_facture
 
		UPDATE dbo.Facture_statut_facture
		SET traiter = 1
		WHERE id = @id
	END
 
	-- recu  = 1 + envoyé vers client = 1
	IF @statut = 'ENV'  
	BEGIN		
		UPDATE dbo.Facture_entete
		SET recu = 1, envoye_vers_client = 1
		WHERE numero_facture = @numero_facture
 
		UPDATE dbo.Facture_statut_facture
		SET traiter = 1
		WHERE id = @id
	END
 
	-- erreur_chez_client = 1
	IF @statut = 'RH2'  
	BEGIN		
		UPDATE dbo.Facture_entete
		SET erreur_chez_client = 1
		WHERE numero_facture = @numero_facture
 
		UPDATE dbo.Facture_statut_facture
		SET traiter = 1
		WHERE id = @id
	END
 
	-- recuperer par le client = 1
	IF @statut = 'REC'  
	BEGIN		
		UPDATE dbo.Facture_entete
		SET recuperer_par_client = 1
		WHERE numero_facture = @numero_facture
 
		UPDATE dbo.Facture_statut_facture
		SET traiter = 1
		WHERE id = @id
	END
 
	-- Ligne suivante
	FETCH NEXT FROM curseur_facture
 
END
 
CLOSE curseur_facture
DEALLOCATE curseur_facture

Mon problème est que, dans mon curseur curseur_facture j'ai 10 lignes par exemple, il faut que le traitement s’exécute 10 fois avant d'être tout traiter.
Alors qu'avec le WHILE @@FETCH_STATUS = 0, une seule exécution suffit normalement...

Merci.

[StringBuilder]

Normal, tu traîtes 10 fois de suite la première facrture.

Dans ton "fetch next" qui se trouve à la fin de ton code, met bien le "into".
=> Là, il ne fait que consommer une ligne dans le curseur, sans rien faire des données : donc tu refais le même traitement.

En remettant le INTO variables, tu recopies les données du curseur dans les variables, pour traiter ces nouvelles variables dans l'ittération suivante.

[afrodje]

Envoyé par StringBuilder

Normal, tu traîtes 10 fois de suite la première facrture.

Dans ton "fetch next" qui se trouve à la fin de ton code, met bien le "into".
=> Là, il ne fait que consommer une ligne dans le curseur, sans rien faire des données : donc tu refais le même traitement.

En remettant le INTO variables, tu recopies les données du curseur dans les variables, pour traiter ces nouvelles variables dans l'ittération suivante.

Merci, j'ignorai ce paramètre

[elsuket]

Bonjour,

Cela étant, il n'est ici nul besoin de curseur, comme bien souvent :

Code :

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
43
44
45
46
47
48
49
50
51
DECLARE @err_msg nvarchar(2048)
 
BEGIN TRY
	BEGIN TRANSACTION
 
	;WITH
		CTE AS
		(
			SELECT	CAST(numero_facture AS int) AS num_fact
				, RTRIM(statut) AS statut
			FROM	dbo.Facture_statut_facture
			WHERE	traiter = 0
		)
	UPDATE		dbo.Facture_entete
	SET		erreur = CASE C.statut WHEN 'ERR' THEN 1 ELSE erreur END
			, recu = CASE C.statut WHEN 'ENV' THEN 1 ELSE recu END
			, envoye_vers_client = CASE C.statut WHEN 'ENV' THEN 1 ELSE envoye_vers_client END
			, erreur_chez_client = CASE C.statut WHEN 'RH2' THEN 1 ELSE erreur_chez_client END
			, recuperer_par_client = CASE C.statut WHEN 'REC' THEN 1 ELSE recuperer_par_client END
	FROM		dbo.Facture_entete AS FE
	INNER JOIN	CTE AS C
				ON FE.numero_facture = C.num_fact
	;WITH
		CTE AS
		(
			SELECT	id
			FROM	dbo.Facture_statut_facture
			WHERE	traiter = 0
		)
	UPDATE		dbo.Facture_statut_facture
	SET		traiter = 1
	FROM		dbo.Facture_statut_facture AS FSF
	INNER JOIN	CTE AS C
				ON C.id = FSF.id
 
	COMMIT TRANSACTION
END TRY
BEGIN CATCH
	SET @err_msg = ERROR_MESSAGE()
 
	IF XACT_STATE() = 0
	BEGIN
		COMMIT TRANSACTION
	END
	ELSE
	BEGIN
		ROLLBACK TRANSACTION
 
		RAISERROR('%s', 16, 1, @err_msg)
	END	
END CATCH

Ce qui s'exécutera bien plus vite qu'avec le curseur.
N'oubliez pas que SQL est un langage ensembliste, et que par conséquent il est conçu pour traiter les données dans leur ensemble, et non dans leur unité comme vous le faites.

Enfin SQL est aussi un langage déclaratif : vous décrivez le résultat que vous souhaitez obtenir, mais pas la façon dont vous souhaitez l'obtenir, comme on le fait dans un langage fonctionnel.

@++

[afrodje]

Envoyé par elsuket

Bonjour,

Cela étant, il n'est ici nul besoin de curseur, comme bien souvent :

Code :

Ce qui s'exécutera bien plus vite qu'avec le curseur.
N'oubliez pas que SQL est un langage ensembliste, et que par conséquent il est conçu pour traiter les données dans leur ensemble, et non dans leur unité comme vous le faites.

Enfin SQL est aussi un langage déclaratif : vous décrivez le résultat que vous souhaitez obtenir, mais pas la façon dont vous souhaitez l'obtenir, comme on le fait dans un langage fonctionnel.

@++

Certes, plus complexe aussi que les curseurs...
Je me pencherai dessus.

Merci

[elsuket]

Certes, plus complexe aussi que les curseurs...

Non, vous avez 6 UPDATE avec un test à chaque fois, et pour chaque ligne.
J'ai deux instructions sans test et une meilleure gestion de la transaction.
A vous de voir, mais pour moi y'a pas photo

@++