Discussion :

[Ahinha]

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 :

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

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.

[SQLpro]

Quelque chose comme :

Code :

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

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 +

[Ahinha]

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 :

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

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,

[aieeeuuuuu]

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 :

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

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

Par

Code :

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

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.

[Ahinha]

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 :

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

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 :

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

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

[escartefigue]

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)

[Ahinha]

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'.

[SQLpro]

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

[Ahinha]

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 :

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

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 !

[SQLpro]

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

A +

[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.

[escartefigue]

bonjour,

Elle fonctionne !
Comme ceci :

Code :

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

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 :

au passage, remplacez

Code :

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

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

Par

Code :

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

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.

[SQLpro]

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 :

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

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 +

[Ahinha]

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 :

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

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 :

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
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.

[escartefigue]

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

[Ahinha]

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