Pb de syntaxe CASE WHEN

Bonjour,

Je souhaite créer une colonne calculée.
J'ai des factures et des encaissements qui sont lettrés.
J'ai fait une clé "Compte_Client + Lettre"
Si la clé est la même pour une pièce de type facture et une autre pièce qui n'est PAS une facture, je veux ramener la date la plus lointaine de cette autre pièce, dans mon exemple ci-dessous "27/12/19".
Ex :

Clé	Type de pce	Compte_Client	Num_Pce	Lettrage	Date
C0040 - K	ENC	C0040	A1903	K	15/12/19
C0040 - K	FAC	C0040	A1901	K	27/12/19

Je bloque totalement sur le code.
J'ai fait cela

Code:

---

1 2 3 4 5 6 7 8 9 10 11

CASE WHEN ( (Cle) AND Type_Pce = 'FAC' ) = ( (Cle) AND Type_Pce <> 'FAC' ) THEN '1' ELSE '0' END

---

Ce qui me donne cette erreur :
Msg*156, Niveau*15, État*1, Ligne*1
Syntaxe incorrecte vers le mot clé 'CASE'.

Ce qui me semble normal puisque la condition liée à ma clé est bancale.

J'apprends seule SQL server, je vous remercie par avance pour votre aide.

Quelque chose comme :

Code:

---

1 2 3 4 5

SELECT MIN(Date) FROM MaTable AS T1 LEFT OUTER JOIN MaTable AS T2 ON T1.Cle = T2.Cle AND T2.Type_de_pce <> 'FAC' WHERE T1.Type_de_pce = 'FAC'

---

A +

J'ai édité mon code pour plus de clarté :)

Je te remercie, on va en fait créer deux tables temporaires et les lier ?

Ma table s'appelle GACCENTRYD.
Mais quand je passe le code (avec les vrais noms), j'ai cette erreur.

Code:

---

1 2 3 4 5 6

SELECT MAX(Date) FROM GACCENTRYD AS T1 LEFT OUTER JOIN GACCENTRYD AS T2 ON (T1.BPR_0 + '-' + T1.MTC_0) = (T2.BPR_0 + '-' + T2.MTC_0) AND T2.TYP <> 'FAC' WHERE T1.TYP = 'FAC'

---

Erreur :
Msg*208, Niveau*16, État*1, Ligne*15
Nom d'objet 'GACCENTRYD' non valide.

Saurais-tu pourquoi ?

Je te remercie par avance,

bonjour,

êtes vous sûre d'être connectée à la bonne base ? (et non sur la base master par défaut) ?
sinon la table est peut-être sur un autre schéma que le schéma par défaut.
ou vous l'avez mal écrite !

au passage, remplacez

Code:

---

1 2	ON (T1.BPR_0 + '-' + T1.MTC_0) = (T2.BPR_0 + '-' + T2.MTC_0) AND T2.TYP <> 'FAC'

---

Par

Code:

---

1 2 3 4

ON T1.BPR_0 = T2.BPR_0 AND T1.MTC_0 = T2.MTC_0 AND T2.TYP <> 'FAC'

---

Vous ne pourrez qu'améliorer les performances.

Bonjour,

Je vous remercie, j'ai bien regardé et compris comment je devais nommer mes tables pour les alias.

J'ai un nouveau problème avec un message, je ne sais pas l'interpréter :

Code:

---

1 2 3 4 5 6 7 8

SELECT MAX(GACCENTRYD.ACCDAT_0) FROM C.GACCENTRYD AS T1 LEFT OUTER JOIN C.GACCENTRYD AS T2 ON T1.BPR_0 = T2.BPR_0 AND T1.MTC_0 = T2.MTC_0 AND T2.TYP_0 <> 'FAC' WHERE T1.TYP_0 = 'FAC'

---

Erreur :
L'identificateur en plusieurs parties "GACCENTRYD.ACCDAT" ne peut pas être lié.

Si je lui demande plus simple (la date comptable d'une pièce en particulier), ça fonctionne :

Code:

---

1 2	SELECT ACCDAT_0 from C.GACCENTRYD WHERE GACCENTRYD.NUM_0 = 'A1905ODG00126'

---

Il y a qqch qui ne va pas dans le FROM, mais quoi ?

Je vous remercie par avance pour votre aide :)

Bonjour,

Puisque vous avez affecté un alias T1 pour la table, essayez en remplaçant SELECT MAX(GACCENTRYD.ACCDAT_0) par SELECT MAX(T1.ACCDAT_0)

Bonjour,
Même si mon alias vient après cette commande ?

J'ai essayé, j'ai cette erreur :

Syntaxe incorrecte vers le mot clé 'SELECT'.

Postez à chaque fois la requête exacte effectuée !!!! Nos boules de cristal sont en grève...
A +

Désolée, vous avez raison.

En plus, la copier m'a montrée que j'avais écrit 2 fois SELECT... bravo...

Elle fonctionne !
Comme ceci :

Code:

---

1 2 3 4 5

SELECT MAX(T2.ACCDAT_0) FROM C.GACCENTRYD AS T1 LEFT OUTER JOIN C.GACCENTRYD AS T2 ON (T1.BPR_0 + '-' + T1.MTC_0) = (T2.BPR_0 + '-' + T2.MTC_0) AND T2.TYP_0 <> 'FAC' WHERE T1.TYP_0 = 'FAC'

---

Merci encore !

Au passage vos parenthèses ne servent à rien….

A +

Je vous remercie, j'ai vu cela :)
Pour l'instant, elles me servent à me repérer dans ce que je fais.
Bonne soirée.

bonjour,

Citation:

---

Envoyé par Ahinha

Elle fonctionne !
Comme ceci :

Code:

---

1 2 3 4 5

SELECT MAX(T2.ACCDAT_0) FROM C.GACCENTRYD AS T1 LEFT OUTER JOIN C.GACCENTRYD AS T2 ON (T1.BPR_0 + '-' + T1.MTC_0) = (T2.BPR_0 + '-' + T2.MTC_0) AND T2.TYP_0 <> 'FAC' WHERE T1.TYP_0 = 'FAC'

---

---

Elle fonctionne peut être mais attention les perfs ! votre prédicat de jointure n'est pas indexable (non "Sargable"), si vos tables contiennent un peu de volume, le temps de réponse risque d'être rédhibitoire
Appliquez la préconisation faite par aieeeuuuuu :

Citation:

---

Envoyé par aieeeuuuuu

au passage, remplacez

Code:

---

1 2	ON (T1.BPR_0 + '-' + T1.MTC_0) = (T2.BPR_0 + '-' + T2.MTC_0) AND T2.TYP <> 'FAC'

---

Par

Code:

---

1 2 3 4

ON T1.BPR_0 = T2.BPR_0 AND T1.MTC_0 = T2.MTC_0 AND T2.TYP <> 'FAC'

---

Vous ne pourrez qu'améliorer les performances.

---

Citation:

---

Envoyé par Ahinha

Je vous remercie, j'ai vu cela :)
Pour l'instant, elles me servent à me repérer dans ce que je fais.
Bonne soirée.

---

Utilisez plutôt l'indentation, c'est beaucoup plus lisible…. :

Code:

---

1 2 3 4 5 6 7 8

SELECT MAX(T2.ACCDAT_0) FROM C.GACCENTRYD AS T1 LEFT OUTER JOIN C.GACCENTRYD AS T2 ON T1.BPR_0 + '-' + T1.MTC_0 = T2.BPR_0 + '-' + T2.MTC_0 AND T2.TYP_0 <> 'FAC' WHERE T1.TYP_0 = 'FAC';

---

Certains SGBDR incorporent une fonction pour formater le code (par exemple SQL Server Management Studio avec Apex Refactor).
Sinon il existe de nombreux outils en ligne, plus ou moins bien conçus pour ce faire :
http://www.dpriver.com/pp/sqlformat.htm
https://sql-format.com/
https://sql-formatter.online/options/formatting
https://sqlformat.org/

…

A +

Bonjour,
La vue fonctionne parfaitement avec un test unitaire, mais me fait planter quand je dois la lancer dans une requête.
Par exemple, si je veux voir tous les lettrages fait la semaine dernière.
Voici la vue [DATE_LATEST_LETTRAGE]:

Code:

---

1 2 3 4 5 6 7 8 9

SELECT max(T1.ACCDAT_0) AS DATE_LETTRAGE,T1.TYP_0,T2.NUM_0 FROM C.C.GACCENTRYD AS T1 INNER JOIN C.C.GACCENTRYD AS T2 ON T1.BPR_0 = T2.BPR_0 AND T1.MTC_0 = T2.MTC_0 AND T1.TYP_0 <> 'FAC' WHERE LEN(T2.MTC_0)>0 AND LEN(T2.BPR_0)>0 AND LEN(T1.MTC_0)>0 AND LEN(T1.BPR_0)>0 AND T2.TYP_0 = 'FAC' GROUP BY T1.TYP_0,T2.NUM_0

---

J'ai essayé de créer des index pour l'améliorer, mais cela ne semble pas suffisant.

Y a-il qque chose que je puisse faire pour améliorer les perfs ?

Dans mon processus global (SEI), voici à quoi ressemble la requête complète, pour une facture (ici NUM_FAC).

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

SELECT [GACCDUDATE].[NUM_0] ,[SINVOICEV].[INVDAT_0] ,[DATE_LATEST_LETTRAGE].[DATE_LETTRAGE] FROM [C].[C].[GACCDUDATE] [GACCDUDATE] LEFT JOIN [C].[C].[SINVOICEV] [SINVOICEV] ON [GACCDUDATE].[NUM_0] = [SINVOICEV].[NUM_0] LEFT JOIN [C].[C].[GACCENTRYD] [GACCENTRYD] ON [GACCDUDATE].[LIG_0] = [GACCENTRYD].[LIN_0] AND [GACCDUDATE].[NUM_0] = [GACCENTRYD].[NUM_0] AND [GACCDUDATE].[TYP_0] = [GACCENTRYD].[TYP_0] LEFT JOIN [C].[SEI_C].[DATE_LATEST_LETTRAGE] [DATE_LATEST_LETTRAGE] ON [GACCENTRYD].[NUM_0] = [DATE_LATEST_LETTRAGE].[NUM_0] LEFT JOIN [C].[C].[GACCENTRY] [GACCENTRY] ON [GACCDUDATE].[TYP_0] = [GACCENTRY].[TYP_0] AND [GACCDUDATE].[NUM_0] = [GACCENTRY].[NUM_0] LEFT JOIN [C].[C].[COMPANY] [COMPANY] ON [GACCDUDATE].[CPY_0] = [COMPANY].[CPY_0] LEFT JOIN [C].[C].[GTYPACCENT] [GTYPACCENT] ON [COMPANY].[LEG_0] = [GTYPACCENT].[LEG_0] AND [GTYPACCENT].[TYP_0] = GACCDUDATE.TYP_0 LEFT JOIN [C].[C].[GJOURNAL] [GJOURNAL] ON [GTYPACCENT].[DEFJOU_0] = [GJOURNAL].[JOU_0] WHERE ([GACCDUDATE].[NUM_0] = 'NUM_FAC')

---

Je vous remercie par avance pour votre aide.

Bonjour,

Si la requête est très lente, il faut vérifier que

• toutes les colonnes de jointure sont bien de même type et de même longueur
• il n'y a pas de fonctions (conversion de type, substring...) appliquées sur les critères de jointure et de filtrage
• si les colonnes de jointure ou de filtrage possèdent un nombre significatif de valeurs distinctes, il existe des index pour ces colonnes
• les statistiques sont à jour et les espaces ont été réorganisés

Par ailleurs

• pourquoi ajoutez-vous des crochets autour des noms, c'est inutile tant que ces noms ne sont pas des mots réservés SQL
• utiliser des alias, c'est pratique, à condition d'en profiter pour simplifier les noms. Par exemple DATE_LATEST_LETTRAGE, y a sans doute moyen de trouver plus court ;)

Communiquez la description des index de chacune des tables utilisées

Bonjour,
Je vais passer la demande à mon intégrateur, pq ça dépasse de loin mes compétences.
Merci pour votre réponse :)