Discussion :

[p_oum]

Bonjour,

Voici mon problème :
Je dispose de 3 tables, tableA, tableB, tableC qui se lient de la manière suivante

tableA | tableB | tableC
Code----Code | Prix
Prix | Id------Id

Je voudrait mettre a jour le prix de la tableC grâce au prix de la tableA

J'ai essayer la requête suivante :

Code :

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

1
2
3
4
5
6
7
8
9
UPDATE tableC SET Prix = (SELECT tableA.Prix
FROM tableC INNER JOIN
tableB ON tableC.Id = tableB.Id INNER JOIN
tableA ON tableB.Code = tableA.Code) 
 
WHERE tableC.Id = (SELECT tableB.Id
FROM tableC INNER JOIN
tableB ON tableC.Id = tableB.Id INNER JOIN
tableA ON tableB.Code = tableA.Code)

Mais j'obtiens l'erreur suivante :

Serveur*: Msg 512, Niveau 16, État 1, Ligne 1
La sous-requête a retourné plusieurs valeurs. Cela n'est pas autorisé quand la sous-requête suit =, !=, <, <= , >, >= ou quand elle est utilisée en tant qu'expression.
L'instruction a été arrêtée.

Est ce que le problème vient de ma requête ?

Merci

[elsuket]

Bonjour,

Faites simple !

Essayez :

Code :

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

1
2
3
4
5
6
7
UPDATE dbo.tableC
SET Prix = A.prix
FROM dbo.TableC AS C
JOIN dbo.tableB AS B
	ON C.Id = B.Id
JOIN dbo.TableA AS A
	 ON B.Code = A.Code

@++

[p_oum]

Bonjour,

Un grand merci sa fonctionne très bien

Cependant je voudrais ajouter une condition sur le champ 'Type' de la tableC

Ma requête deviens

Code :

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

1
2
3
4
5
6
7
8
UPDATE dbo.tableC
SET Prix = A.prix
WHERE tableC.Type='A'
FROM dbo.TableC AS C
JOIN dbo.tableB AS B
	ON C.Id = B.Id
JOIN dbo.TableA AS A
	 ON B.Code = A.Code

Seulement j'obtiens l'erreur :

Code :

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

Syntaxe incorrecte vers le mot clé 'FROM'.

Avez-vous solution ?

[narutobaka]

Bonjour,

Je crois qu'il faut mettre la condition FROM ainsi que les JOINTURES avant la condition WHERE mais je peux me tromper.

cordialement

[p_oum]

Oui désoler ....

La requête deviens donc :

Code :

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

1
2
3
4
5
6
7
8
UPDATE dbo.tableC
SET Prix = A.prix
FROM dbo.TableC AS C
JOIN dbo.tableB AS B
	ON C.Id = B.Id
JOIN dbo.TableA AS A
	 ON B.Code = A.Code
WHERE tableC.Type='A'

Mais j'obtiens l'erreur :

Code :

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

Le préfixe de colonne 'tableC' ne correspond ni au nom de table ni au nom d'alias utilisés dans la requête.

Alors que mon nom de champ est correct

[narutobaka]

bonjour,

ah je n'avait pas vu les "dbo.".

dbo pour database object.

retire les dbo. devant les noms de tes tables.

Un autre élément me parait bizarre dans la syntaxe de tes jointures.

Si je dit pas de betise la syntaxe est INNER JOIN.

Mais vu qu'il existe plusieurs type de jointure je ne sais pas laquelle tu souhaites utilisé.

Je pense que ca devrait resoudre ton soucis.

++

[Waldar]

retire les dbo. devant les noms de tes tables.

Très mauvaise idée, et vous allez en plus vous faire taper sur les doigts !
Il suffit simplement d'utiliser les alias des tables précédement déclarés :

Code :

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

1
2
3
4
5
6
7
8
UPDATE C
   SET C.Prix = A.prix
  FROM dbo.TableC AS C
       INNER JOIN dbo.tableB AS B
         ON B.Id = C.Id
       INNER JOIN dbo.TableA AS A
         ON A.Code = B.Code
 WHERE C.Type = 'A'

[elsuket]

Très mauvaise idée, et vous allez en plus vous faire taper sur les doigts !

Je confirme ce que dit Waldar, et la fessée étant en passe d'être interdite, je ne vois pas d'autre solution que de devoir effectivement vous taper sur les doigts !
Non, en fait il est important de mettre dbo devant le nom des tables car cela évite à SQL Server de rechercher le propriétaire de la table dans les tables de métadonnées et de le comparer à l'utilisateur qui exécute la procédure stockée.
C'est un moyen très simple d'obtenir un gain de performances important sur une charge de travail sous SQL Server 2000 (gain de 300-400 transactions par secondes)
Le gain est plus faible dès SQL Server 2005, mais il y a gain quand même : 100 transactions par seconde

J'ai "omis" le INNER dans la spécification de la jointure par habitude de codage.
En fait l'INNER spécifie une équi-jointure, mais à mon sens quand on effectue une jointure sans préciser le mot clé INNER, on spécifie une jointure naturelle ...

Dans vos requêtes, une fois que vous avez aliasé vos tables, toutes les références que vous y faites en dehors de la jointure (FROM et JOIN) doit utiliser l'alias

@++

[narutobaka]

Bonjour,

Le gain est plus faible dès SQL Server 2005, mais il y a gain quand même : 100 transactions par seconde

Autant pour moi je ne savais pas que laisser cette extension permettait un gain de performance. Merci pour l'astuce. Enregistré et je ne ferai plus l'erreur.

++

[p_oum]

Bonjour,

Merci pour votre aide et ces informations intéressantes

++