Discussion :

[Negaton]

Bonjour,
Je souhaiterais optimiser un update de ce type :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
update TablePrincipale
set somme1=(select sum(montant1*produit)
		FROM TableSecondaire
		WHERE TablePrincipale.ID=TableSecondaire.ID
	   )
,somme2=(select sum(montant2/produit)
		FROM TableSecondaire
		WHERE TablePrincipale.ID=TableSecondaire.ID
	   )
,somme3=(select sum(montant1*produit)
		FROM TableSecondaire
		WHERE TablePrincipale.ID=TableSecondaire.ID
	   )

Comme vous le voyez, mes champs "somme1" et "somme3" sont calculés selon la même manière : je cherche donc à ne faire la sous requête qu'une seule fois. Je trouve comment faire sur oracle sur le net (syntaxe du genre update table set (somme1,somme3)=(select...) ) mais cette syntaxe n'est pas valable pour sql server.

A noter : ma sous requête fait une jointure sur l'id de la table à mettre à jour : je ne pense donc pas pouvoir pré calculer la valeur de somme1 et somme 3 par avance

Merci pour votre aide

[ATMorphine]

Code :

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

1
2
3
4
5
6
UPDATE TablePrincipale
Set somme1 = sum(ts.montant1*produit)
, somme2 = sum(ts.montant2/produit)
, somme3 = sum(ts.montant1*produit)
FROM TablePrincipale  tp
INNER JOIN TableSecondaire ts on ts.Id = tp.id

[Negaton]

Merci pour ton aide.
Je teste ça dès que possible

[Negaton]

Sauf erreur de ma part (j'avais d'ailleurs déjà testé) celà ne marche pas à cause du "sum". Sql server me renvoit l'erreur "An aggregate may not appear in the set list ok un update statement"

D'autres idées ?

[ATMorphine]

Code :

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

1
2
3
4
5
6
7
UPDATE TablePrincipale
SET somme1 = sum(ts.montant1*produit)
, somme2 = sum(ts.montant2/produit)
, somme3 = sum(ts.montant1*produit)
FROM TablePrincipale  tp
INNER JOIN TableSecondaire ts ON ts.Id = tp.id
GROUP BY tp.id

J'avais oublié que tu voulais un somme, donc il faut un group by

[Negaton]

Tu es sur qu'on peut faire un group by dans un update ?
Je viens d'essayer par acquis de consience et j'arrive pas à le plugger sans erreur

[ATMorphine]

Dans ce cas ce que tu peux faire :

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
 
select ID
, Sum(montant1*produit) as somme1
, Sum(montant2/produit) as somme2
, Sum(montant1*produit) as somme3
Into #temp
FROM TableSecondaire
group by ID
 
UPDATE TablePrincipale
SET somme1 = sum(ts.montant1*produit)
, somme2 = sum(ts.montant2/produit)
, somme3 = sum(ts.montant1*produit) 
FROM #temp tp
INNER JOIN TablePrincipale ts ON ts.Id = tp.id

Mais je pense pas que ça soit la meilleur méthode.

[Invité]

Quelque chose dans ce goût :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
UPDATE TablePrincipale TP
SET somme1 = temp.somme1
, somme2 = temp.somme2
, somme3 = temp.somme3
FROM TP
 JOIN (
	SELECT ID
	, Sum(montant1*produit) AS somme1
	, Sum(montant2/produit) AS somme2
	, Sum(montant1*produit) AS somme3
	FROM TableSecondaire
	GROUP BY ID
) Temp
	ON temp.ID = TP.ID

Je n'ai pas testé.

[Negaton]

Merci encore pour ton aide ATMorphine

J'ai effectivement cette solution pour mon problème qui ne me convient qu'à moitié pour l'avoir testé au préalable (problème de place disque qui est limité et du nombre d'occurrence assez élevé)

C'est pour celà que je cherchais à connaitre un syntaxe qui m'aurait échapé.

Si personne d'autre n'a d'idées, je vais effectivement devoir m'y résoudre. dommage que ms sql serveur n'ai pas mis à dispo la même syntaxe qu'oracle

@Jerome MTL : je teste celà illico. Merci

[ATMorphine]

Je suis trop bête, c'est bien la bonne solution, je n'avais pas pensé a la sous requete pour calculer les sommes

[Negaton]

Je vais avoir du mal à tester ca avant lundi : on me tombe dessus!
Je vous tien au courant dès que j'ai testé mais ca semble bon effectivement (je n'y ai pas pensé non plus !)

[elsuket]

SVP, arrêtez d'utiliser des tables temporaires à torts et à travers !

@++

[Negaton]

Merci atmorphine et Jerome MTL : la solution proposée fonctionne correctement et améliore sensiblement les choses.
@elsuket : super interressent ton article