Discussion :

[guigui69]

Bonjour à tous,

je voudrais créer une fonction sur une base bdd sqlserver, mais je débute.

ci-dessous la fonction que j'ai commencé à creer (non testé) je voudrais savoir si j'ai des erreurs dans la conception, des éléments que j'ai oublié

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
 
CREATE FUNCTION departement_concession(@Separateur char(1), @id_concession char(3))
BEGIN
 
 
DECLARE  departement CURSOR FOR
   SELECT code_departement FROM concession_to_departement WHERE id_concession= @id_concession
   OPEN departement 
   SET @Resultat = ''   
   FETCH NEXT FROM departement  INTO @code_departement
   WHILE @@FETCH_STATUS = 0    
     BEGIN
			SET @Resultat =  LTRIM( RTRIM(@code_departement))+ ' ' + rtrim(@Separateur )
 
     FETCH NEXT FROM departement   INTO @code_departement
     END
   CLOSE departement   
   DEALLOCATE departement   
   RETURN @Resultat
END

Merci d'avance pour vos conseils

guigui69

[SQLpro]

Erreurs :

• Il manque la déclaration de la variable @resultat en tant que variable de retour
• Il manque la déclaration de la variable @code_departement en tant que variable dans le traitement
• Vous utilisez le séparateur "caractères blanc," alors que vous aviez prévu un séparateur @Separateur en argument de votre fonction

Conseils :
mettez un "point-virgule" à la fin de chaque commande
Vous pouvez faire cette opération d'autres manières :
1) avec un SELECT de concaténation
2) avec la fonction TRING_AGG
L'ouverture d'un curseur comme vous l'avez fait, n'est pas la meilleure option. Il existe plusieurs paramètres pour minimiser les ressources :
https://docs.microsoft.com/fr-fr/sql...l-server-ver15
Notamment je vous conseillerais :

Code :

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

1
2
3
4
5
6
DECLARE departement CURSOR
   LOCAL FORWARD_ONLY STATIC READ_ONLY
   FOR
      SELECT code_departement 
      FROM concession_to_departement 
      WHERE id_concession= @id_concession;

Indentez votre code !

A +

[guigui69]

Bonjour SQLpro,

merci pour vos réponses je ne connaissais pas string_agg.

voici l'élément mise à jour, suite a vos retours:

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
 
CREATE FUNCTION departement_concession(@Separateur char(1), @id_concession char(3))
BEGIN
  DECLARE  @Resultat char(64)
 
		DECLARE  departement CURSOR
			LOCAL FORWARD_ONLY STATIC READ_ONLY
				FOR
 
				SELECT STRING_AGG (code_departement,';') 
				FROM concession_to_departement 
				WHERE id_concession= @id_concession;
		OPEN departement 
			SET @Resultat = ''   
			SET @code_departement = ''  
		FETCH NEXT FROM departement  INTO @code_departement
			WHILE @@FETCH_STATUS = 0    
				BEGIN
				SET @Resultat =  LTRIM( RTRIM(@code_departement)) + rtrim(@Separateur )
 
				FETCH NEXT FROM departement   INTO @code_departement
				END
		CLOSE departement   
		DEALLOCATE departement   
   RETURN @Resultat
END

Est-ce que cela vous parait bon, optimisé?

[SQLpro]

Non.. pas optimisé du tout....

Voici comment faire :

Code :

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

1
2
3
4
5
6
7
8
CREATE FUNCTION departement_concession(@Separateur char(1), @id_concession char(3))
RETURNS @Resultat varchar(max)
BEGIN
   SELECT @Resultat = STRING_AGG(code_departement, @Separateur) 
	FROM   concession_to_departement 
   WHERE  id_concession= @id_concession;
   RETURN @Resultat;
END

A +