Discussion :

[feldi]

Bon, j'en suis arrivé la avec le merge :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
merge into i
using (Select max(f.date_facture)
        FROM f
        WHERE i.invoice_date < f.date_facture
        AND (i.csc_euro IS NULL 
        OR i.prg_euro IS NULL
        OR i.prs_ssa_euro IS NULL)) f
ON (i.customer_code = f.code_client
    i.invoice_code = f.code_facture
    I.PRODUCT_CODE = f.code_produit
    i.deliveried_quantity = f.quantite)
When matched then update set i.csc_euro = f.csc / f.quantite * I.DELIVERIED_QUANTITY;

Juste pour la première partie, le reste ne changera pas tellement par rapport à ça.
Par contre j'ai une erreur parenthèse de droite absente qu'il faut que je règle encore et je ne comprends rien à la fonction KEEP (DENSE_RANK last order by).
J'ai googlé la fonction mais il n'y a pas vraiment de vrais tutos explicatifs (à la rigueur le blog de laurent schneider qui est aussi sur dvp).
Qu'est ce qu'elle est censée effectué cette fonction dans cet exemple :

Code :

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

1
2
 
SELECT max(factures.csc) KEEP(DENSE_RANK LAST ORDER BY invoice_date)

Merci

[Rei Ichido]

Code :

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

1
2
 
SELECT max(factures.csc) KEEP(DENSE_RANK LAST ORDER BY invoice_date)

On demande de prendre une seule valeur de factures.csc, en ne gardant (KEEP) que le dernier (LAST) des enregistrements rangés (ORDER BY) sans trou lié aux ex-eaquo (DENSE_RANK) par invoice_date.

EDIT : précision sur le dense_rank.

[pacmann]

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
 
MERGE INTO invoiceD t
USING (                            
    SELECT i.La_PK, max(f.prs_ssa) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prs_ssa, 
           max(f.csc) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) csc, 
           max(f.prg) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prg,        
    FROM invoice i 
      JOIN factures f ON i.customer_code = factures.code_client 
                  AND i.invoice_code = f.numero_facture 
                  AND i.product_code = f.code_article
                  AND i.deliveried_quantity = f.quantite
                  AND i.invoice_date < f.date_facture
    GROUP BY i.La_PK
       ) s
ON (i.La_PK = s.La_PK)
WHERE t.csc_euro IS NULL 
    OR t.prg_euro IS NULL
    OR t.prs_ssa_euro IS NULL  
WHEN MATCHED THEN UPDATE t.prs_ssa = s.prs_ssa, t.csc = s.csc, t.prg = s.prg

Ce que tu mets dans le USING, c'est le résultset que tu veux utiliser pour mettre à jour InvoiceD :
-Tu fais la jointure entre InvoiceD et Factures parce que tu a besoin de la date dans InvoiceD pour choisir tes factures
- Tu fais un GROUP BY parce que tu veux une seule ligne par invoice à mettre à jour. La_PK, c'est la primary key de InvoiceD qui te permet d'identifier de manière sûre.
- KEEP .. machin : ça veut dire : garde la valeur qui correspond à la plus grande invoice_date

[feldi]

Comme ils sont marrants la ou je travaille, il n'y a pas de clé primaire dans la table !
Donc j'ai essayé d'adapter par rapport aux conditions ton merge :

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
 
MERGE INTO invoice inD
USING (                            
    SELECT i.product_code, i.invoice_code, i.customer_code, i.deliveried_quantity, 
    max(f.prs_ssa) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prs_ssa, 
    max(f.csc) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) csc, 
    max(f.prg) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prg,        
    FROM invoice i 
      JOIN factures f 
            ON (i.customer_code = f.code_client 
            AND i.invoice_code = f.numero_facture 
            AND i.product_code = f.code_article
            AND i.deliveried_quantity = f.quantite
            AND i.invoice_date > f.date_facture)
    GROUP BY i.product_code, i.invoice_code, i.deliveried_quantity, i.customer_code
       ) s
ON (i.product_code = s.product_code, 
    i.invoice_code = s.invoice_code, 
    i.customer_code = s.customer_code, 
    i.deliveried_quantity = s.deliveried_quantity)
WHERE inD.csc_euro IS NULL 
    OR inD.prg_euro IS NULL
    OR inD.prs_ssa_euro IS NULL  
WHEN MATCHED THEN UPDATE inD.prs_ssa_euro = f.prs_ssa, inD.csc_euro = f.csc, inD.prg_euro = f.prg;

et j'ai un message d'erreur :

ORA-00936: expression absente

ça vient du group by qui ramène plusieurs colonnes ?

Merci pour votre aide !

[pacmann]

Ca vient de la virgule en trop dans le SELECT du USING

[feldi]

/facepalm

J'ai un autre message d'erreur

ORA-00907: parenthèse de droite absente

Question subsidiaire : pourquoi le débuggeur // anti-erreur des logiciels oracle n'est-il pas plus précis ? Dans le genre, au moins indiquer la ligne et pas juste
"Error at line 1"...

[lola06]

Code :

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

1
2
3
4
ON (i.product_code = s.product_code, 
    i.invoice_code = s.invoice_code, 
    i.customer_code = s.customer_code, 
    i.deliveried_quantity = s.deliveried_quantity)

Ce ne sont pas des virgules mais des AND qu'il faut mettre :

Code :

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

1
2
3
4
ON (i.product_code = s.product_code
   AND i.invoice_code = s.invoice_code
   AND i.customer_code = s.customer_code
   AND i.deliveried_quantity = s.deliveried_quantity)

[feldi]

Merci à toi !

Mais

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
 
MERGE INTO invoice inD
USING (                            
    SELECT i.product_code, i.invoice_code, i.customer_code, i.deliveried_quantity, 
    --prendre une seule valeur de f.prs_ssa, en ne gardant (KEEP) que le dernier (LAST) des enregistrements rangés (ORDER BY) sans trou lié aux ex-eaquo (DENSE_RANK) par invoice_date.
    max(f.prs_ssa) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prs_ssa, 
    max(f.csc) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) csc, 
    max(f.prg) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prg       
    FROM invoice i 
      JOIN factures f 
            ON (i.customer_code = f.code_client 
            AND i.invoice_code = f.numero_facture 
            AND i.product_code = f.code_article
            AND i.deliveried_quantity = f.quantite
            AND i.invoice_date > f.date_facture)
    GROUP BY i.product_code, i.invoice_code, i.deliveried_quantity, i.customer_code
       ) s
ON (i.product_code = s.product_code 
    AND i.invoice_code = s.invoice_code 
    AND i.customer_code = s.customer_code 
    AND i.deliveried_quantity = s.deliveried_quantity)
WHERE (inD.csc_euro IS NULL 
    OR inD.prg_euro IS NULL
    OR inD.prs_ssa_euro IS NULL)
WHEN MATCHED THEN UPDATE inD.prs_ssa_euro = f.prs_ssa, inD.csc_euro = f.csc, inD.prg_euro = f.prg;

j'ai encore un message d'erreur (j'espère que ça va bientot s'arrêter)

ORA-00905: mot-clé absent

J'essaye de faire le comparatif avec la doc oracle pour voir ce qu'il pourrait manquer mais je vois pas...


Edit : ce serait le Where dans le Using mais est-ce que le join peut le remplacer ?
Edit 2 : en rajoutant

Code :

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

1
2
3
4
 
WHERE (i.csc_euro IS NULL 
        OR i.prg_euro IS NULL
        OR i.prs_ssa_euro IS NULL)

avant le join, j'ai de nouveau le message concernant la parenthèse de droite...

Edit 3 : merci, mais même en rajoutant le SET j'obtiens le même message d'erreur.

ORA-00905: mot-clé absent

[lola06]

Tu as oublié le SET après UPDATE

Code :

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

UPDATE SET

[pacmann]

Tu peux mettre le WHERE à l'intérieur du USING je pense

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
 
MERGE INTO invoice inD
USING (                            
    SELECT i.product_code, i.invoice_code, i.customer_code, i.deliveried_quantity, 
    --prendre une seule valeur de f.prs_ssa, en ne gardant (KEEP) que le dernier (LAST) des enregistrements rangés (ORDER BY) sans trou lié aux ex-eaquo (DENSE_RANK) par invoice_date.
    max(f.prs_ssa) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prs_ssa, 
    max(f.csc) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) csc, 
    max(f.prg) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prg       
    FROM invoice i 
      JOIN factures f 
            ON (i.customer_code = f.code_client 
            AND i.invoice_code = f.numero_facture 
            AND i.product_code = f.code_article
            AND i.deliveried_quantity = f.quantite
            AND i.invoice_date > f.date_facture)
    WHERE (i.csc_euro IS NULL 
        OR i.prg_euro IS NULL
        OR i.prs_ssa_euro IS NULL)            
    GROUP BY i.product_code, i.invoice_code, i.deliveried_quantity, i.customer_code    
       ) s
ON (i.product_code = s.product_code 
    AND i.invoice_code = s.invoice_code 
    AND i.customer_code = s.customer_code 
    AND i.deliveried_quantity = s.deliveried_quantity)
WHEN MATCHED THEN UPDATE SET inD.prs_ssa_euro = f.prs_ssa, inD.csc_euro = f.csc, inD.prg_euro = f.prg;

[feldi]

Ouaip, c'est exactement ce que j'avais fait.
Par contre il me dit que certains champs n'existent pas... Faut que je vérifie bien avec les alias et tout.

Je vous tiens au courant. Merci !


Edit :
Le script qui fonctionne :

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
 
MERGE INTO scn_sales_scp_dev.invoice inD
USING (                            
    SELECT i.product_code, i.invoice_code, i.customer_code, i.DELIVERIED_QUANTITY, 
    --prendre une seule valeur de f.prs_ssa, en ne gardant (KEEP) que le dernier (LAST) des enregistrements rangés (ORDER BY) sans trou lié aux ex-eaquo (DENSE_RANK) par invoice_date.
    max(f.prs_ssa) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prs_ssa, 
    max(f.csc) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) csc, 
    max(f.prg) KEEP(DENSE_RANK LAST ORDER BY i.invoice_date) prg       
    FROM scn_sales_scp_dev.invoice i 
      JOIN infoc.scn_factures f 
            ON (i.customer_code = f.code_client 
            AND i.invoice_code = f.numero_facture 
            AND i.product_code = f.code_article
            AND i.DELIVERIED_QUANTITY = f.quantite
            AND i.invoice_date > f.date_facture)
      WHERE (i.csc_euro IS NULL 
            OR i.prg_euro IS NULL
            OR i.prs_ssa_euro IS NULL)
    GROUP BY i.product_code, i.invoice_code, i.DELIVERIED_QUANTITY, i.customer_code
       ) s
ON (inD.product_code = s.product_code 
    AND inD.invoice_code = s.invoice_code 
    AND inD.customer_code = s.customer_code 
    AND inD.DELIVERIED_QUANTITY = s.deliveried_quantity)
WHEN MATCHED THEN UPDATE SET inD.prs_ssa_euro = s.prs_ssa, inD.csc_euro = s.csc , inD.prg_euro = s.prg;

ou pas, puisqu'il ne met rien à jour

[pacmann]

Mais t'exécutes avec quel outil ?

Pour ce genre d'erreur, quand je lance avec Toad par exemple, il me sélectionne le mot clef où il vautre.
Avec SQL*Plus, il t'affiche l'endroit avec une étoile.
...

[feldi]

J'utilise également Toad mais il ne m'indique rien !
Simplement un
"Error at line 1
ORA-XXXX ..."

Il m'indique le noms des colonnes fausses mais pour les problèmes de mots-clés ou de parenthèses, je dois me débrouiller.

[feldi]

Vous n'allez peut être pas me croire mais j'ai fait cette requête (par désespoir) :

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
 
UPDATE invoice invoiceD
    SET invoiceD.csc_euro = (SELECT DISTINCT factures.csc / FACTURES.QUANTITE * INVOICED.DELIVERIED_QUANTITY
    FROM factures
    WHERE invoiceD.customer_code = factures.code_client 
        AND invoiceD.invoice_code = factures.numero_facture 
        AND invoiceD.product_code = factures.code_article
        AND invoiceD.deliveried_quantity = factures.quantite
        AND (invoiceD.csc_euro IS NULL 
        OR invoiceD.prg_euro IS NULL
        OR invoiceD.prs_ssa_euro IS NULL)),
    invoiceD.prg_euro = (SELECT DISTINCT factures.prg / FACTURES.QUANTITE * INVOICED.DELIVERIED_QUANTITY
    FROM factures
    WHERE invoiceD.customer_code = factures.code_client 
        AND invoiceD.invoice_code = factures.numero_facture 
        AND invoiceD.product_code = factures.code_article
        AND invoiceD.deliveried_quantity = factures.quantite
        AND (invoiceD.csc_euro IS NULL 
        OR invoiceD.prg_euro IS NULL
        OR invoiceD.prs_ssa_euro IS NULL)),
    invoiceD.prs_ssa_euro = (SELECT DISTINCT factures.prs_ssa / FACTURES.QUANTITE * INVOICED.DELIVERIED_QUANTITY
    FROM factures
    WHERE invoiceD.customer_code = factures.code_client 
        AND invoiceD.invoice_code = factures.numero_facture 
        AND invoiceD.product_code = factures.code_article
        AND invoiceD.deliveried_quantity = factures.quantite
        AND (invoiceD.csc_euro IS NULL 
        OR invoiceD.prg_euro IS NULL
        OR invoiceD.prs_ssa_euro IS NULL))
    WHERE invoiceD.INVOICE_DATE < trunc(to_date(add_months(sysdate, -6)), 'MONTH')
        AND (invoiceD.csc_euro IS NULL 
        OR invoiceD.prg_euro IS NULL
        OR invoiceD.prs_ssa_euro IS NULL);

et ça semble avoir fonctionné (il me semble que lola06 en avait parlé mais comme je pensais l'avoir déjà fait, j'ai refusé son idée, désolé).

Merci à tous pour m'avoir guidé dans cette difficile expérience.

[lola06]

Il ne faudrait pas mettre la table que tu update par hasard ?

Code :

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

UPDATE <MaTable> SET

Ou c'est le MERGE qui la définie ?

[feldi]

Je suppose que c'est la table définie dans la clause merge into.