Discussion :

[Arnaud F.]

Bonjour,

j'ai le jeu de donnée suivant :

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
drop table #dco, #prise, #pick
go
CREATE TABLE #prise (interval   int     NOT NULL)
 
CREATE TABLE #dco (code       int         NOT NULL,
                   nb         int         NOT NULL)
 
 
CREATE TABLE #pick (code       int         NOT NULL,
                   res         int         NOT NULL)
 
insert #prise
select 1
union
select 6
 
insert #dco
select 1, 5
union
select 2, 7
 
insert #pick
select 1, -1
union
select 2, -1

Qui donne donc :

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
interval     
------------ 
           1 
           6 
 
2 Row(s) affected
 
code   nb     
------ ------ 
     1      5 
     2      7 
 
2 Row(s) affected
 
code   res    
------ ------ 
     1     -1 
     2     -1 
 
2 Row(s) affected

J'ai fais la requête suivante :

Code :

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

1
2
3
4
5
 
select #dco.code, #dco.nb, #prise.interval
FROM #dco, #prise, #pick
WHERE #pick.code = #dco.code
  AND #dco.nb    > #prise.interval

Qui me renvoie :

Code :

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

1
2
3
4
5
code         nb           interval     
------------ ------------ ------------ 
           1            5            1 
           2            7            1 
           2            7            6

En fait, j'aimerai obtenir que la valeur max de interval afin de l'utiliser dans l'update de #pick.res et donc faire un update par "intervalle"

L'update :

Code :

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

1
2
3
4
5
update #pick
   set #pick.res = interval
FROM #dco, #prise
WHERE #pick.code = #dco.code
  AND #dco.nb    > #prise.interval

Me renvoi :

Code :

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

1
2
3
4
code         res          
------------ ------------ 
           1            1 
           2            1

Au lieu de :

Code :

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

1
2
3
4
code         res          
------------ ------------ 
           1            1 
           2            6

C'est possible de faire ça d'un coup d'un seul?

[Waldar]

Vous n'avez pas fait d'erreur de copier coller ?

Car la condition suivante :

Code :

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

#dco.nb    > #prise.interval

devrait éliminer ce tuple :

Code :

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

1
2
3
code         nb           interval     
------------ ------------ ------------ 
           2            7            6

[Arnaud F.]

Non, pas d'erreur de copier / coller

#dco.nb = 7 dans le second cas, et 7 est bien strictement supérieur à 1 et 6...

[Waldar]

Houlala, c'est moi qui suis fatigué, j'ai lu à l'envers !

Revenons à votre problème.
Vous pouvez (devez) utiliser un agrégat pour ne sélectionner que le plus grand :

Code :

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

1
2
3
4
5
6
  SELECT #dco.code, #dco.nb,
         max(#prise.interval) as interval_max
    FROM #dco, #prise, #pick
   WHERE #pick.code = #dco.code
     AND #dco.nb    > #prise.interval
GROUP BY #dco.code, #dco.nb

Il faudrait aussi utiliser des jointures normalisées :

Code :

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

1
2
3
4
5
6
7
  SELECT #dco.code, #dco.nb, max(#prise.interval) as interval_max
    FROM #dco
         INNER JOIN #pick
           ON #pick.code = #dco.code
         INNER JOIN #prise
           ON #dco.nb > #prise.interval
GROUP BY #dco.code, #dco.nb

Je ne sais pas que SGBD vous utilisez, donc je ne sais pas si la requête suivante est acceptée :

Code :

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

1
2
3
4
5
6
UPDATE #pick
   SET #pick.res = max(#prise.interval)
  FROM #dco
       INNER JOIN #prise
         #dco.nb > #prise.interval
 WHERE #pick.code = #dco.code

[Arnaud F.]

Le SGBD utilisé est Sybase (v12.5).

J'avais déjà pensé au max, seulement on n'obtient pas le résultat escompté...


La requête :

Code :

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

1
2
3
4
5
UPDATE #pick
   SET #pick.res = max(#prise.interval)
  FROM #dco, #prise
 WHERE #pick.code = #dco.code
   AND #dco.nb    > #prise.interval

renvoie :

Code :

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

1
2
3
4
5
 
code   res    
------ ------ 
     1      6 
     2      6

Au lieu de :

Code :

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

1
2
3
4
code   res    
------ ------ 
     1      1 
     2      6

[Arnaud F.]

En fait, le select avec le group by fonctionne bien, mais l'update sans group by ne fonctionne pas ...


Pour les jointures normalisées, ce n'est pas le débat ici

[CinePhil]

Pour les jointures normalisées, ce n'est pas le débat ici

Ben si un petit peu quand même !
On est dans le forum "Langage SQL" qui est consacré au SQL standard, normatif.
Et la norme depuis 1992 pour écrire les jointures utilise le mot clé JOIN. Il serait peut-être temps de s'y mettre 17 ans après !

[Waldar]

L'update classique fonctionne (j'enlève les dièses, ça met tout en commentaire) :

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
UPDATE pick
   SET pick.res = (select max(prise.interval)
                     FROM dco
                          INNER JOIN prise 
                            ON dco.nb > prise.interval
                    Where pick.code = dco.code)
WHERE EXISTS (select null
                FROM dco
                     INNER JOIN prise 
                       ON dco.nb > prise.interval
               Where pick.code = dco.code);
 
 2 enregistrement(s) affecté(s) 
 
SELECT * FROM pick;
 
 code     res    
 -------  ------ 
 1        1      
 2        6

[Arnaud F.]

Ben si un petit peu quand même !
On est dans le forum "Langage SQL" qui est consacré au SQL standard, normatif.
Et la norme depuis 1992 pour écrire les jointures utilise le mot clé JOIN. Il serait peut-être temps de s'y mettre 17 ans après !

L'Homme n'aime pas le changement, il y est même réfractaire. Dans l'entreprise, les jointures sont écrites à l'ancienne manière depuis depuis plus de 17 ans justement. Rédiger les requêtes avec les jointures normalisées provoquent de l'urticaire chez certains...

Je m'adapte...

@Waldar : je teste

[CinePhil]

17 ans de résistance au changement, ça tient de l'antropologie là !

[Arnaud F.]

Waldar, ça fonctionne bien en effet, même si j'ai du mal à le comprendre

J'ai fais des recherches de mon côté...

J'en suis arrivé à trouver l'update :

Code :

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

1
2
3
4
5
6
7
8
update #pick
   set #pick.res = interval
FROM #dco, #prise
WHERE #pick.code = #dco.code
  and interval = (select max(interval)
                   from #dco d, #prise p
                   where d.code = #dco.code
                     and d.nb   > p.interval)

qui fonctionne également

donc

[Arnaud F.]

17 ans de résistance au changement, ça tient de l'antropologie là !

J'ai déjà tenté de faire une révolution SQL, c'était voué à l'échec

[elsuket]

J'ai tenté aussi, ça ne fonctionne pas, donc je quitte l'entreprise qui m'emploie encore

@++