Update un champ à partir d'un count effectué sur une autre table selon certain conditions

Salut tout le monde,

J'essaie de mettre à jour un champ dans le tableau A avec un certain nombre d'occurrences de certaines données dans une autre table B. La condition est que les champs de B doivent avoir les mêmes clés étrangères que dans le tableau A.
merci à adavance

Pièce jointe 152740

j'aimerais faire ce COUNT :

Code:

---

1 2 3 4

select COUNT(1) as NbrFacture,DateFacturation_FK, Site_FK, Client_FK from DistrisysDW.dbo.FactFacture group by DateFacturation_FK, Site_FK, Client_FK order by 2

---

Et mettre à jour "NbFacture" dans l'autre table dans un état que les lignes correspondent à ces trois champs DateFacturation_FK, Site_FK, Client_FK.

J'ai essayé ça mais ça ne fonctionne pas :

Code:

---

1 2 3 4 5 6 7 8 9 10

Update DistrisysDW.dbo.FactFactureEntete SET NumFacture=T.NbrFacture from (select COUNT(1) as NbrFacture,DateFacturation_FK, Site_FK, Client_FK from DistrisysDW.dbo.FactFacture group by DateFacturation_FK, Site_FK, Client_FK order by 2 ;) T INNER JOIN DistrisysDW.dbo.FactFactureEntete ENT ON T.DateFacturation_FK=ENT.DateFacturation_FK AND T.Site_FK=ENT.Site_FK AND T.Client_FK=ENT.Client_FK

---

Bonjour,

2 petites remarques.

Il est impossible, et surtout particulièrement inutile de placer un ORDER BY dans une sous requête. Quoi qu'il se passe, l'ensemble des données sera traité, qu'importe l'ordre, c'est de la cosmétique qui ne devrais pas se gérer en SQL.

Lorsqu'on fait un UPDATE, la fonction SET se comporte ainsi.

Code:

---

1 2 3

UPDATE maTable SET ChampDeMaTableAUpdate = ChampDeLaSource

---

Dans votre cas, vous avez fait quelques mélanges, il me semble, entre votre modèle et votre besoin.

Vous dites vouloir mettre à jour la colonne NbFacture de la table Entete, cependant, votre code ne correspond pas

Code:

---

1 2	SET NumFacture = T.NbrFacture

---

Probablement une confusion de champ.

Pour votre solution, je vous propose quelque chose comme ça

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

WITH COUNT_Facture AS ( SELECT DateFacturation_FK , Site_FK , Client_FK , COUNT(*) AS NbrFacture FROM DistrisysDW.dbo.FactFacture GROUP BY DateFacturation_FK, Site_FK, Client_FK )   UPDATE F SET NbFacture = NbrFacture FROM COUNT_Facture CF INNER JOIN DistrisysDW.dbo.FactFactureEntete F ON CF.DateFacturation_FK = F.DateFacturation_FK AND CF.Site_FK = F.Site_FK AND CF.Client_FK = F.Client_FK;

---

Avec une CTE qui va agréger vos données et que l'on va exploiter dans le FROM.

Attention cependant, vous êtes dans le langage SQL, un FROM n'est pas une norme SQL mais une largesse de SQL Server.

Cordialement,
Lyche

Juste une petite remarque concernant ce que vous cherchez à faire.

Vous cherchez à stocker une valeur calculée.
C'est tout à votre honneur, j'imagine dans une optique d'optimisation ou de simplification de certaines requêtes.
Cependant, ce n'est pas une bonne pratique, car il y a peu de chances pour que vous arriviez à maintenir l'intégrité à tout instant des données.

Et parfois, il faut mieux attendre 2 secondes de plus et avoir un résultat juste plutôt que de se précipiter et avoir un résultat faux.

Donc, en premier lieux, évitez autant que possible de stocker des valeurs calculées.
Si vraiment vous ne pouvez pas faire autrement, optez en priorité, si votre SGBD le permet, pour des colonnes calculées : elles ont l'avantage de pouvoir être persistantes (si leur type et le calcul le permet) et surtout, garantissent une intégrité à tout instant.
Si votre SGBD ne supporte pas les colonnes calculées, alors optez en priorité pour une vue : le principal inconvénient par rapport à une colonne calculée, c'est que le calcul est fait à chaque accès à la vue, même si on ne lit pas la colonne.
Et si vraiment c'est les performances qui sont vitales, optez pour une mise à jour de cette colonne par trigger.

Ne remplissez jamais une colonne calculée par requête de type "one shot". Car les données sont alors potentiellement incohérentes (et donc fausses) instantanément après la lecture.

Sinon, pour répondre à la requête oneshot, en plus simple :

Code:

---

1 2 3 4

update FactFactureEntete f1 set nbfacture = (select COUNT(*) from FactFacture f2 where f2.DateFacturation_FK = f1.DateFacturation_FK AND f2.Site_FK = f1.Site_FK AND f2.Client_FK = f1.Client_FK), nbarticle = (select sum(quantite) from FactFacture f2 where f2.DateFacturation_FK = f1.DateFacturation_FK AND f2.Site_FK = f1.Site_FK AND f2.Client_FK = f1.Client_FK)

---

Plus simple à mon avis.

heu, pourquoi la somme?

Et oui, j'ai hésité à proposer la solution avec la requête dans le SET. J'ai simplement voulu conserver en partie sa syntaxe.

Cependant, je ne vois pas où se trouve la somme :koi:

Bah le count(*) c'est pour compter les factures.
Et le sum(quantite) c'est pour remplir la colonne "nbrarticlefacture".

Justement, je ne vois pas ce besoin dans sa demande, mais je suis peut-être juste fou :aie:

Ou est-ce simplement une anticipation, qui serait bien vue dans ce cas ;)

C'est une anticipation ;)