Problème dans ma fonction

[bsangoku]

Bonjour à tous,

J'ai un problème dans mon for: pouvez vous le régler svp? (je suis débutant en sql: et c'est la première fois que je fais une boucle for en sql...

Le problème se situe au niveau de Update Historique SET Hit_or_miss="true";

Explication du trigger
Au début j'ai historique(FilmID,SouscriID,UtilisID)que j'ai remplis
Puis j'ajoute une colonne hit_or_miss qui est un boolean. Ce boolean renvoie true sosu certaine condition et false dans dautre. Et ce trigger va essayer de mettre à jour la colonne hit_or_miss

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
CREATE OR REPLACE FUNCTION functionHit_or_miss_version()
returns  TRIGGER AS
$$
DECLARE 
i record;
hit_or_miss BOOLEAN;
BEGIN
FOR i IN SELECT FilmID FROM historique 
LOOP
IF (SELECT GenreID FROM definieSur WHERE definieSur.GenreID IN (SELECT genreID FROM caracteriser))
THEN 
UPDATE Historique SET hit_or_miss="true";
ELSE 
UPDATE Historique SET hit_or_miss="false";
END IF;
RETURN Next i;
END LOOP;
RETURN;
END;
$$
LANGUAGE 'plpgsql';
 
 
CREATE TRIGGER INSERTIONHistorique
AFTER INSERT  ON Historique
FOR each statement 
execute procedure functionHit_or_miss_version();

Merci d'avance

[iberserk]

hit_or_miss est'il une colonne de votre table historique
car vous la définissez comme variable au début du script?

De même vous mettez à jour la table entière historique! sans filtrer sur FilmdID...


Détaillez votre erreur...

[elsuket]

Dans cette discussion, vous semblez travailler sous SQL Server.

Le trigger serait dans ce cas :

Code :

1
2
3
4
5
6
7
8
9
10
11
12
CREATE TRIGGER monTrigger
	ON maTable
	AFTER INSERT
AS
BEGIN
	UPDATE		historique
	SET		hit_or_miss = CASE WHEN DS.GenreId IS NULL THEN 'True' ELSE 'False' END
	FROM		historique AS H
	LEFT JOIN	definieSur AS DS ON ???
	LEFT JOIN	caracteriser AS C ON C.genreID = DS.genreID
	INNER JOIN	inserted AS I ON I.cle_primaire = H.cle_primaire
END

[estofilo]

Envoyé par elsuket

Dans cette discussion, vous semblez travailler sous SQL Server.

Non il s'agit de postgresql, c'est pourquoi il y a une fonction et un CREATE TRIGGER séparés.
Le RETURN NEXT i dans la fonction est une erreur car cette construction est réservée aux fonctions qui retournent un ensemble de tuples (SETOF quelquechose) ce qui n'est pas le cas des fonctions utilisables en TRIGGER.
Du coup la boucle FOR elle-même n'a pas de sens et pour ma part je ne saisis pas du tout la logique de ce bout de code.