Requête - Fusion de résultats

**QuestVba** · 05/06/2020, 07h18

Hello

Forum,

Le contexte : mes agents peuvent télécharger directement des documents électroniques dans leurs dossiers et/ou dans les dossiers des autres gestionnaires.

Mon but : avoir une liste mensuelle par agent qui me permet de savoir dans combien de dossiers :

au moins un document a été téléchargé par lui-même
au moins un document a été téléchargé par un autre
au moins un document a été téléchargé pour un autre

J'exécute cette requête (c'est une adaptation d'un post https://www.developpez.net/forums/d2.../#post11555647 - Encore merci à vanagreg).

Ma première question (je sais, c'est pas de l'aide habituelle) : est-ce que ce que j'ai réalisé - et qui me donne les bons résultats - est optimal ? En effet, je suis tout nouveau dans le domaine et j'ai certainement plein de choses à apprendre. Un grand merci,

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
-- BY_HIMSELF / BY_OTHER / FOR_OTHER
with uploaded_docs 
as (select to_char(ad.process_date,'mm/yyyy') as Month_Select
           , ad.file_number
           , ad.lot_number
           , ad.agent_number as ID_SCANNER
           , ag2.first_name || ' ' || ag2.last_name as SCANNER
            , ag.agent_number AS ID_FILEMANAGER
            , ag.first_name || ' ' || ag.last_name as FILE_MANAGER
            , CASE
                WHEN ad.agent_number = ag.agent_number THEN 1
                ELSE 0
              END AS BY_HIMSELF
            , CASE
                WHEN ad.agent_number <> ag.agent_number  THEN 1
                ELSE 0
             END AS BY_OTHER
            , CASE
                WHEN ad.agent_number <> ag.agent_number  THEN 1
                ELSE 0
             END AS FOR_OTHER
    from archived_documents ad
    inner join file_managements fm on fm.file_number = ad.file_number
    inner join agents ag on ag.agent_id = fm.file_manager_id 
        and to_char(start_date,'yyyymm') < '202012' and end_date is null 
        and fm.filemanagement_type = 'PERMANENT'
    left join agents ag2 on ag2.agent_number = ad.agent_number 
    where to_char(ad.process_date,'yyyymm') = '202005'
        and ad.archived_type='DIRECT'
   ), S_BY_HIMSELF_AND_OTHER as (
select  Month_Select
      , FILE_MANAGER AS FILE_MANAGER
      , count(distinct case when BY_HIMSELF = 1 then file_number end) AS INJECTED_DOCUMENT_BY_HIMSELF
      , count(distinct case when BY_OTHER = 1 then file_number end) AS INJECTED_DOCUMENT_BY_other
from uploaded_docs
group by Month_Select, FILE_MANAGER
), S_FOR_OTHER as (
select  Month_Select 
      , SCANNER AS SCANNER
      , count(distinct case when FOR_OTHER = 1 then file_number end) AS INJECTED_DOCUMENT_FOR_other
from uploaded_docs
group by Month_Select, SCANNER
)
select 
  nvl(B.Month_Select, FR.Month_Select) as Month_Select
  , nvl(B.FILE_MANAGER, FR.SCANNER) as FILE_MANAGER
  , B.INJECTED_DOCUMENT_BY_HIMSELF
  , B.INJECTED_DOCUMENT_BY_OTHER
  , FR.INJECTED_DOCUMENT_FOR_other
from 
  S_BY_HIMSELF_AND_OTHER B
  full outer join S_FOR_OTHER FR on B.FILE_MANAGER = FR.SCANNER
order by 2
;

Ma seconde question : est-il possible de remplacer les données Null en 0 ?

**JeitEmgie** · 05/06/2020, 08h47

Les formules BY_OTHER et FOR_OTHER sont identiques, prob. un copy/paste non corrigé

**vanagreg** · 05/06/2020, 09h57

Bonjour,

Si tu n'as pas de problème de performance avec la requête alors inutile de se poser trop de questions. Juste une remarque sur la sélection des colonnes date. Pour qu'un index puisse éventuellement être utilisé, il ne faut pas appliquer de fonction sur les colonnes dates, mais plutôt sur les valeurs. Donc tes conditions:

Code :

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

1
2
to_char(start_date,'yyyymm') < '202012'
to_char(ad.process_date,'yyyymm') = '202005'

peuvent être réécrites ainsi:

Code :

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

1
2
start_date < to_date('20201201', 'yyyymmdd')
ad.process_date >= to_date('20200501', 'yyyymmdd') and ad.process_date < to_date('20200601', 'yyyymmdd')

**QuestVba** · 05/06/2020, 15h33

JeitEmgie

Effectivement,il y a une redondance. Mais elle est utile car elle permet de voir ce qui est fait par un autre. Je vais donc améliorer le sql en gardant une occurrence OTHER que j'utiliserai en fonction du FILE_MANAGER ou du SCANNER. Cela permet donc de voir ce qui a été fait pour un autre ou ce qu'un autre a fait pour l'agent.

J'arrive à ceci :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
-- BY_HIMSELF / BY_OTHER / FOR_OTHER
with uploaded_docs 
as (select to_char(ad.process_date,'mm/yyyy') as Month_Select
           , ad.file_number
           , ad.lot_number
           , ad.agent_number as ID_SCANNER
           , ag2.first_name || ' ' || ag2.last_name as SCANNER
            , ag.agent_number AS ID_FILEMANAGER
            , ag.first_name || ' ' || ag.last_name as FILE_MANAGER
            , CASE
                WHEN ad.agent_number = ag.agent_number THEN 1
                ELSE 0
              END AS BY_HIMSELF
            , CASE
                WHEN ad.agent_number <> ag.agent_number  THEN 1
                ELSE 0
             END AS OTHER
    from archived_documents ad
    inner join file_managements fm on fm.file_number = ad.file_number
    inner join agents ag on ag.agent_id = fm.file_manager_id 
        and to_char(start_date,'yyyymm') < '202012' and end_date is null 
        and fm.filemanagement_type = 'PERMANENT'
    left join agents ag2 on ag2.agent_number = ad.agent_number 
    where to_char(ad.process_date,'yyyymm') = '202001'
        and ad.archived_type='DIRECT'
   ), S_BY_HIMSELF_AND_OTHER as (
select  Month_Select
      , FILE_MANAGER AS FILE_MANAGER
      , count(distinct case when BY_HIMSELF = 1 then file_number end) AS INJECTED_DOCUMENT_BY_HIMSELF
      , count(distinct case when OTHER = 1 then file_number end) AS INJECTED_DOCUMENT_BY_other
from uploaded_docs
group by Month_Select, FILE_MANAGER
), S_FOR_OTHER as (
select  Month_Select 
      , SCANNER AS SCANNER
      , count(distinct case when OTHER = 1 then file_number end) AS INJECTED_DOCUMENT_FOR_other
from uploaded_docs
group by Month_Select, SCANNER
)
select 
  nvl(B.Month_Select, FR.Month_Select) as Month_Select
  , nvl(B.FILE_MANAGER, FR.SCANNER) as FILE_MANAGER
  , B.INJECTED_DOCUMENT_BY_HIMSELF
  , B.INJECTED_DOCUMENT_BY_OTHER
  , FR.INJECTED_DOCUMENT_FOR_other
from 
  S_BY_HIMSELF_AND_OTHER B
  full outer join S_FOR_OTHER FR on B.FILE_MANAGER = FR.SCANNER
order by 2
;

**QuestVba** · 05/06/2020, 15h40

vanagreg

Merci pour le retour.
La façon que j'utilisais permettait de ne changer qu'une date à un seul endroit. C'était une astuce ...

**vanagreg** · 05/06/2020, 15h44

Dans la clause SELECT pourquoi pas, mais dans les clauses ON ou WHERE il faut éviter, et puisque ta question était de savoir si la requête était optimale.

Requête - Fusion de résultats

SQL Oracle

Vue hybride

Discussions similaires

Partager

Partager