Déclencheur je présume

**Just-Soft** · 19/08/2023, 11h22

Bonjour
j'ai une table mouvement avec les champs suivants :
id_mouv, date_mouv, recette, depense et solde.

je voudrais qu'à l'ouverture d'une nouvelle ligne, donc à l'ajout d'un enregistrement, récupérer du solde et l'ajouter à la nouvelle recette.
J'ai essayé de faire ça en utilisant un déclencheur mais je n'ai pas eu le résultat escompté.

voici mon code

Code :

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

1
2
3
4
5
6
7
8
9
10
CREATE OR ALTER trigger tb_mouvements_bi0 for tb_mouvements
active before insert position 0
AS
DECLARE VARIABLE s DECIMAL(18,0);
begin
  /* Trigger text */
  SELECT solde_journee FROM tb_mouvements ORDER BY 1 DESC into :s;
  if (inserting) then
    new.solde_journee =  new.solde_journee + s;
end

des idées ?

**Just-Soft** · 20/08/2023, 13h44

Bonjour

j'ai essayé de faire ça mais j'ai eu une erreur.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
CREATE OR ALTER trigger tb_mouvements_bi0 for tb_mouvements
active after update position 0
AS
declare variable id_m int;
declare variable s float;
begin
  /* Trigger text */
    SELECT MAX(id_mouv), sum(solde_journee) from tb_mouvements
    group by solde_journee
    INTO id_m,  :s;
 
    UPDATE tb_mouvements
    SET SOLDE_CUMULE = NEW.SOLDE_CUMULE + s;
end

je commece par récupérer le dernier enregistrement en utilsant max, puis le montant de la journée en utilisant sum. Après récupération des données, j'essai de faire une mise à jour sur la table en utilisant update mais là je reçois une erreur sur lavariable s :

Column does not belong to referenced table.
Dynamic SQL Error.
SQL error code = -206.
Column unknown.
S.
At line 13, column 43.

**licardentaistor** · 20/08/2023, 17h29

salut,

regarde ici : https://www.mysqltutorial.org/mysql-...nsert-trigger/

tu as normalement tous ce qu'il te faut

**SergioMaster** · 20/08/2023, 19h04

Si tu commençais par donner la structure de la table ?

**Just-Soft** · 21/08/2023, 09h45

Envoyé par SergioMaster

Si tu commençais par donner la structure de la table ?

je l'avais donné dans le 1er post

en fait, j'ai remodelé un peu la chose, j'ai ajouté un champ solde_cumulé afin d'alleger le traitement.

voici la structure

Code :

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

1
2
3
4
5
6
7
8
CREATE TABLE TB_MOUVEMENTS (
    ID_MOUV        INTEGER NOT NULL,
    DATE_MOUV      DATE,
    RECETTE        DOUBLE PRECISION,
    DEPENSE        DOUBLE PRECISION,
    SOLDE_JOURNEE  COMPUTED BY (recette - depense),
    SOLDE_CUMULE   DOUBLE PRECISION
);

et le déclencheur

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
CREATE OR ALTER trigger tb_mouvements_bi0 for tb_mouvements
active after update position 0
AS
declare variable id_m int;
declare variable s float;
begin
  /* Trigger text */
    SELECT MAX(id_mouv), sum(solde_journee) from tb_mouvements
    group by solde_journee
    INTO id_m, :s;
 
    UPDATE tb_mouvements
    SET SOLDE_CUMULE = NEW.SOLDE_CUMULE + s;
end

à l'éxécution il reconnait pas la variable s dans la dernière instruction :

Column does not belong to referenced table.
Dynamic SQL Error.
SQL error code = -206.
Column unknown.
S.
At line 13, column 43.

**SergioMaster** · 21/08/2023, 10h18

Quelques questions maintenant que je vois la structure :
Tout d'abord, je ne pense pas qu'il n'y ait qu'un seul mouvement par journée ?
Ensuite, si le solde cumulé est ainsi

Code :

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

1
2
    UPDATE tb_mouvements
    SET SOLDE_CUMULE = NEW.SOLDE_CUMULE + :s;

(manque un : )
toutes les lignes seront modifiées, cela me semble donc une colonne inutile, voire dangereux dans un contexte multi-user.

J'imagine que SOLDE_CUMULE est le SOLDE_CUMULE d'une journée et non de l'ensemble des jours ?
Donc, SOLDE_CUMULE étant un champ calculé , AMHA n'est pas à mettre dans la table, il serait préférable, si c'est ce qui est souhaité, utiliser une CTE pour obtenir le solde.

ie.

Code :

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

1
2
3
WITH SC AS (SELECT DATE_MOUV,SUM(recette - depense) SOLDEJOUR FROM TB_MOUVEMENTS GROUP BY DATE_MOUV)
SELECT  M.DATE_MOUV,COALESCE(M.RECETTE,0) RECETTE,COALESCE(M.DEPENSE,0) DEPENSE,(COALESCE(M.RECETTE,0)-COALESCE(M.DEPENSE,0),SC.SOLDEJOUR
FROM TB_MOUVEMENTS M JOIN SC ON SC.DATE_MOUV=M.DATE_MOUV

Un autre truc m'inquiète

Code :

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

1
2
3
  SELECT MAX(id_mouv), sum(solde_journee) from tb_mouvements
    group by solde_journee
    INTO id_m, :s;

ce groupement n'est certainement pas bon, GROUP BY DATE_MOUV me semble plus juste et le MAX(id_mouv) ne semble ne pas servir
et le UPDATE manque d'une clause WHERE

**Just-Soft** · 21/08/2023, 10h36

Quelques eclaircissements

Ce que je voudrais c'est avoir un solde après chaque transaction faite, c'est pourquoi j'avais pensé à mettre deux champs solde_journée qu'on pourrait changer par solde_transaction (plus explicite) et qui devrait à mon sens être un champs calculé (recette - dépense).
mon problème est le suivant, à la demande de l'utilisateur je voudrais imprimé un hostorique où l'on peut voir le mouvement, c'est pourquoi j'avais songé à mettre un champs sode_cumulé, je pensais que cela allait alleger le serveur puique le calcul se faisait à chaque transaction.

j'espère que j'ai été clair.

**escartefigue** · 21/08/2023, 10h39

Une colonne calculée telle que le solde cumulé n'a rien à faire dans une table.
Imaginez que le montant journalier des ventes d'une ligne remontant à il y a 3 ans soit erroné et qu'il faille donc le corriger, il faudra en ce cas modifier toutes les lignes postérieures à cette date pour en recalculer le solde ! C'est aberrant.
Toute valeur calculée doit l'être au moment où on en a besoin, il ne faut jamais la stocker.
Si le calcul est redondant, on peut le faire dans une vue ou une fonction pour se faciliter la tâche.

**Just-Soft** · 21/08/2023, 10h42

Envoyé par escartefigue

Une colonne calculée telle que le solde cumulé n'a rien à faire dans une table.
Imaginez que le montant journalier des ventes d'une ligne remontant à il y a 3 ans soit erroné et qu'il faille donc le corriger, il faudra en ce cas modifier toutes les lignes postérieures à cette date pour en recalculer le solde ! C'est aberrant.
.

j'imagine que ce que vous dites est juste mais je suis parti d'une supposition que les jounées une fois cloturées devraient être scellées.
La demande d'un historique de quelques annnées va prendre beacoup de temps pour le calcul n'est ce pas ?
il faut un compromis que je n'arrive pas à trouvé pour le moment.

**SergioMaster** · 21/08/2023, 11h21

Au point 1 : oui, Firebird permet des champs calculés (COMPUTED) mais uniquement sur la ligne et pas toujours nécessaire puisque
SOLDE_JOURNEE COMPUTED BY (recette - depense) = RECETTE-DEPENSE AS SOLDE_JOURNEE et non pas ce qui semble être voulu

ceci serait plus juste

Code :

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

1
2
3
4
5
 
CREATE VIEW MOUVEMENTS (ID_MOUV,DATE_MOUV,RECETTE,DEPENSE,SOLDE_JOURNEE)
AS
SELECT M.ID_MOUV,M.DATE_MOUV,M.RECETTE,M.DEPENSE,(SELECT SUM(RECETTE-DEPENSE) FROM TB_MOUVEMENTS T WHERE T.DATE_MOUV=M.DATE_MOUV AND T.ID_MOUV< M.ID_MOUV) SOLDE_JOURNEE  FROM TB_MOUVEMENTS M 
ORDER BY M.ID_MOUV;

(bien que je n'aime pas trop les SELECTs au sein d'un SELECT) un SELECT sur MOUVEMENTS sera alors facile

Au point 2 : imprimer un historique n'implique pas forcément des ajouts de champ ! le plus souvent le générateur d'état (Rave,Quickreport, FastReport) permet le plus souvent ce genre de chose. Et

sinon une simple UNION comme celle-ci

Code :

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

1
2
3
SELECT DATE_MOUV,0 AS ID_MOUV,0 AS IDREC,SUM(RECETTE),SUM(DEPENSE),SUM(RECETTE-DEPENSE) FROM TB_MOUVEMENTS GROUP BY DATE_MOUV -- Total journee
UNION 
SELECT DATE_MOUV,ID_MOUV,1,RECETTE,DEPENSE,RECETTE-DEPENSE FROM TB_MOUVEMENTS  -- mouvement du jour en ordre

permets d'avoir les totaux de la journée avant le détail

Une CTE Récursive me semble aussi être une autre solution

NB. Je suis étonné de voir RECETTE et DEPENSE dans une même ligne

**Just-Soft** · 21/08/2023, 12h28

Envoyé par SergioMaster

NB. Je suis étonné de voir RECETTE et DEPENSE dans une même ligne

Quelle est ta suggestion si possible ?

**SergioMaster** · 21/08/2023, 17h05

Généralement un mouvement est en entrée ou en sortie (credit ou débit, + ou - etc..) une colonne qui indique le type et un montant suffisent
donc une structure

Code :

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

1
2
3
4
5
6
7
CREATE TABLE TB_MOUVEMENTS (
    ID_MOUV        INTEGER NOT NULL,
    DATE_MOUV    DATE,
    SENS             CHAR(1) NOT NULL, 
    MONTANT       DOUBLE PRECISION,
 
   CONSTRAINT PK_MOUVEMENT PRIMARY KEY ID_MOUV

Voir ce dépot Gestion des comptes Personnels
où il y a un bdd contenant une table de ce genre

**Just-Soft** · 22/08/2023, 10h15

Envoyé par SergioMaster

Généralement un mouvement est en entrée ou en sortie (credit ou débit, + ou - etc..) une colonne qui indique le type et un montant suffisent

Exactement, suite à tes orientations j'ai déjà fait les changements et voilà ma nouvelle structure :

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
CREATE GENERATOR GEN_TB_MOUVEMENTS_ID;
 
CREATE TABLE TB_MOUVEMENTS (
    ID_MOUV    INTEGER NOT NULL,
    DATE_MOUV  TIMESTAMP NOT NULL,
    NATURE_OP  VARCHAR(1),
    MONTANT    DOUBLE PRECISION
);
 
 
ALTER TABLE TB_MOUVEMENTS ADD CHECK (nature_op in ('D', 'R'));
ALTER TABLE TB_MOUVEMENTS ADD CONSTRAINT PK_TB_MOUVEMENTS PRIMARY KEY (ID_MOUV);
 
SET TERM ^ ;
 
/* Trigger: TB_MOUVEMENTS_BI */
CREATE OR ALTER TRIGGER TB_MOUVEMENTS_BI FOR TB_MOUVEMENTS
ACTIVE BEFORE INSERT POSITION 0
as
begin
  if (new.id_mouv is null) then
    new.id_mouv = gen_id(gen_tb_mouvements_id,1);
end
^
SET TERM ; ^

là je reviens à ma 1ère question, comment puis-je avoir le solde des opérations entre deux dates p.e ?

**SergioMaster** · 22/08/2023, 10h31

De la même façon qu'indiqué #10 en adaptant bien sûr le calcul en fonction de la colonne NATURE_OP

**Just-Soft** · 22/08/2023, 11h01

Envoyé par SergioMaster

De la même façon qu'indiqué #10 en adaptant bien sûr le calcul en fonction de la colonne NATURE_OP

c'est ce que j'essai de faire mais l'adaptation m'est difficile.

Voici un jeu de données :

ID_MOUV DATE_MOUV NATURE_OP MONTANT
13 21/08/2023 14:25:13 R 400
14 21/08/2023 14:26:17 D 150
15 21/08/2023 14:26:35 D 800
16 21/08/2023 14:26:45 R 550
17 21/08/2023 15:09:38 D 180
18 21/08/2023 15:09:55 R 360

A partir de ce jeu, je voudrais avoir un historique tel que celui-ci :

ID_MOUV DATE_MOUV NATURE_OP MONTANT SOLDE
13 21/08/2023 14:25:13 R 400 +400
14 21/08/2023 14:26:17 D -150 +250
15 21/08/2023 14:26:35 D -800 -550
16 21/08/2023 14:26:45 R 550 0
17 21/08/2023 15:09:38 D -180 -180
18 21/08/2023 15:09:55 R 360 +180

des idées ?

**SergioMaster** · 22/08/2023, 11h32

Déjà, les colonnes sont changées puisque maintenant DATE_MOUV est devenu un TimeStamp => Plus difficile de faire un groupement par journée il va falloir utiliser un CAST

Pour le signe et la tranche de date

Code :

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

1
2
SELECT ID_MOUV, DATE_MOUV, NATURE_OP,IIF(NATURE_OP='R',MONTANT,MONTANT*-1) MONTANT FROM TB_MOUVEMENTS
WHERE CAST(DATE_MOUV AS DATE) BETWEEN :DEBUT AND :FIN

Mais je le répète, c'est au générateur d'état de faire le calcul du solde.

comment puis-je avoir le solde des opérations entre deux dates p.e ?

Quel est le solde initial ? ce n'est pas pris en compte dans la demande

c'est ce que j'essai de faire mais l'adaptation m'est difficile.

mais je ne vois aucun essai, est-ce à dire qu'il faut que je fasse tout le boulot ? Déjà que je change votre structure gratuitement

Enfin, si c'est bien de fournir un jeu d'essai et le résultat attendu, vous voudriez que je fasse la saisie alors que vous auriez pu me fournir un script (je bosse moi) !

**Just-Soft** · 22/08/2023, 11h37

Envoyé par SergioMaster

Déjà que je change votre structure gratuitement

Je ne vais répondre qu'à ceci, oui effectivement j'ai vu que vous avez changer la structure mais s'était une pure coincidence car je l'avais déjà fait hier.
Par contre, je croyais que s'était un site d'entraide !
Pour le mot "j'avais fait des essais voulait tout simplement dire que j'ai fais des requêtes sans arriver au résultat escompté donc pas la peine de noyer la discussion avec des codes inutiles.

Je finirais par vous dire grand merci et un autre merci pour ne pas répondre à mes posts puisque votre esprit est allé loin très loin dans la réflexion de la gratuité.

**SergioMaster** · 22/08/2023, 11h46

Envoyé par Just-Soft

s'était une pure coincidence car je l'avais déjà fait hier.

À d'autres, c'est moi qui vous l'ai suggéré à 12:21

Je finirais par vous dire grand merci et un autre merci pour ne pas répondre à mes posts puisque votre esprit est allé loin très loin dans la réflexion de la gratuité.

C'est noté, si vous n'êtes pas capable de comprendre qu'une phrase comme

c'est ce que j'essai de faire mais l'adaptation m'est difficile.

sans indiquer où vous en êtes dans vos essais fait ressembler à une impression de "faites le job à ma place" qui pousse à voir rouge

**Just-Soft** · 22/08/2023, 11h54

Envoyé par SergioMaster

"faites le job à ma place" qui pousse à voir rouge

J'ai dis merci mais je crois que vous poussez le bouchon un peu même trop loin Monsieur.
Poser vos armes on est des adultes et ce n'est pas parceque vous êtes modérateur que vous méprisez les gens.
Alors prière d'être aimable et gentil car vous ne faites pas le job pour moi, une requête n'est pas un job.
Je vous prie d'arrêter ça et merci encore pour votre aide.

**SergioMaster** · 22/08/2023, 12h33

Je suis têtu.
voilà ce que j'aurais voulu comme jeu d'essai

Code :

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

1
2
3
4
5
6
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('13', '20.08.2023, 14:25:13.000', 'R', '400', '400');
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('14', '21.08.2023, 14:26:17.000', 'D', '-150', '250');
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('15', '21.08.2023, 14:26:35.000', 'D', '-800', '-550');
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('16', '21.08.2023, 14:26:45.000', 'R', '550', '0');
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('17', '21.08.2023, 15:09:38.000', 'D', '-180', '-180');
INSERT INTO TB_MOUVEMENTS (ID_MOUV, DATE_MOUV, NATURE_OP, MONTANT, SOLDE) VALUES ('18', '21.08.2023, 15:09:55.000', 'R', '360', '180');