Discussion :

[ImuraDarling]

Bonjour, je me remet doucement dans le SQL et je ne comprend pas pourquoi cette requête ne fonctionne pas.

Requête qui fonctionne :

Code :

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

1
2
3
4
select (case when origine_typeopcom='PCN' or origine_typeopcom='MAJ' then ORIGINE_LIBOPCOM else 'MANUEL' end) as Motif,
DATE_MAJ_CAISSE
from CAISSE.CHGT_PRIX
where date_MAJ_CAISSE >='2018-09-19' and date_MAJ_CAISSE <='2018-11-15'

Mais j'ai plusieurs fois le même Motif, à la même Date_Maj_caisse, je souhaite donc utiliser count(*) pour les regrouper et avoir un tableau plus lisible. C'est là que je bloque :

Code :

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

1
2
3
4
5
select (case when origine_typeopcom='PCN' or origine_typeopcom='MAJ' then ORIGINE_LIBOPCOM else 'MANUEL' end) as Motif,
DATE_MAJ_CAISSE, count(*)
from CAISSE.CHGT_PRIX
where date_MAJ_CAISSE >='2018-09-19' and date_MAJ_CAISSE <='2018-11-15'
group by DATE_MAJ_CAISSE,(case when origine_typeopcom='PCN' or origine_typeopcom='MAJ' then ORIGINE_LIBOPCOM else 'MANUEL' end) as Motif

ça me renvoi :
Error: ILLEGAL SYMBOL as; VALID SYMBOLS ARE M else 'MANUEL' end). .. (State:37000, Native Code: FFFFFF98)
Error: An error occurred during implicit system action type '2'. Information returned for the error includes SQLCODE '-104', SQLSTATE '42601' and message tokens 'as|M else 'MANUEL' end)|..'. (State:56098, Native Code: FFFFFD29)

Merci d'avance.

ImuraDarling

[CinePhil]

Commençons par simplifier et écrire plus lisiblement votre première requête en l'incrémentant :

Code :

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

1
2
3
4
5
6
7
8
SELECT 
	CASE 
		WHEN origine_typeopcom IN('PCN', 'MAJ') THEN origine_typeopcom 
		ELSE 'MANUEL' 
	END as Motif,
	DATE_MAJ_CAISSE
FROM CAISSE.CHGT_PRIX
WHERE date_MAJ_CAISSE BETWEEN '2018-09-19' and '2018-11-15'

=>
1) IN à la place du OR
2) Parenthèses inutiles
3) BETWEEN à la place de >= AND <=

Mais j'ai plusieurs fois le même Motif, à la même Date_Maj_caisse, je souhaite donc utiliser count(*) pour les regrouper et avoir un tableau plus lisible.

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT 
	CASE 
		WHEN origine_typeopcom IN('PCN', 'MAJ') THEN origine_typeopcom 
		ELSE 'MANUEL' 
	END AS Motif,
	date_maj_caisse,
	COUNT(*) AS Nombre
FROM caisse.chgt_prix
WHERE date_maj_caisse BETWEEN '2018-09-19' and '2018-11-15'
GROUP BY date_maj_caisse, Motif

=>
4) Réutilisation de l'alias Motif dans le GROUP BY à la place du CASE.
Si vous réécrivez le CASE dans le GROUP BY, ne mettez pas l'alias ! L'alias ne sert qu'à nommer des colonnes dans le SELECT pour faciliter l'affichage ou pour réutiliser plusieurs fois dans une requête les mêmes éléments (notamment les tables dans les requêtes avec jointures).

[ImuraDarling]

Effectivement, l'utilisation du "IN" et du "BETWEEN" rend la requête plus lisible. Je vais penser à les utiliser plus souvent dans mes requêtes.

Au début, j'avais bien réutilisé l'alias dans le groupe by, mais ça ne fonctionnait pas, c'est pour ça que j'ai réutilisé le case. Là, je viens de réessayer, ça ne fonctionne toujours pas avec l'alias, mais en réutilisant le case oui. (sans l'alias, du coup).

Merci beaucoup !