Problème de condition d'un update !

**Gwendolyne** · 22/11/2007, 14h03

Bonjour,

Je suis actuellement en train d'écrire une requete MSSQL pour mettre a jour une colonne d'une table répondant à une condition qui fonctionne très bien séparément sous forme d'un SELECT mais qui une fois mis en semble ne fonctionne pas correctement.

Je m'explique:
Ma requete classique affiche toutes les lignes d'une table commencent par HLS par exemple. Elle fonctionne bien avec la syntaxe suivante:
SELECT ...
FROM ...
WHERE ...
Tous s'affiche correctement.

Maintenant je veux faire mon UPDATE:

Je fais donc :
UPDATE ...
SET ...
FROM ... le meme que celui de la requete précédente
WHERE ... le meme que celui de la requete porécédente

Conclusion:
Ma requete fais bien le UPDATE, mais au lieu de mettre seulement les lignes concerné par le WHERE à jour elle mais toute la colonne à jour.

Je ne comprend pas j'ai l'impression qu'elle ne prend pas en compte la condition puisqu'en résultat j'ai "1 row affected"!

Pouvez-vous m'éclairer sur ce sujet. Je n'arrive pas a comprendre ce qui neconvient pas syntaxiquement à ma requete.

Je peux également la copier si vous voulez mieux comprendre mon problème ou si je n'ai pas été suffisament clair.

Merci pour vos réponses!

**Keihilin** · 22/11/2007, 14h07

Je ne suis pas certain de la tolérance de MSSQL aux syntaxes exotiques, mais il faudrait plutôt faire :

Code :

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

1
2
3
4
5

UPDATE table
SET column1 = value 1, column2 = value2, ...
WHERE condition

UPDATE SET FROM WHERE n'est pas correct à priori.

**Gwendolyne** · 22/11/2007, 14h23

OK mais dans ce cas la ou est ce que je peux définir les tables du FROM?

Il faut bien que les conditions de ma requete puisse aller chercher les tables de chaque attribut. Si j'enlève l'étape du FROM dans ma requete. Comment ma requete va-t-elle faire pour faire le lien entre les attributs des tables et les tables elle même qu'elle utilise dans la condition WHERE?

**Keihilin** · 22/11/2007, 14h31

Envoyé par Keihilin

Code :

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

1
2
3
4
5

UPDATE table
SET column1 = value 1, column2 = value2, ...
WHERE condition

ça aiderait peut être de voir ta requête complète...

**Gwendolyne** · 22/11/2007, 14h37

Voila la requete qui liste ce dont j'ai besoins.

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
SELECT	
ItemType_name,	
AMOModel_name,
AMOModelVariant_name,
Item_version,
D.Delivery_name
FROM
AMOModelVariant AMV,
AMOModel AMM,	
ItemParents IP,
Item I,
ItemType IT,
ItemStatus IST,
ItemDelivery ID,
Delivery D
WHERE
AMM.Program_id = 2
AND AMV.AMOModel_id=AMM.AMOModel_id
AND AMV.AMOModelVariant_id = IP.Parent_id
AND D.Delivery_id = ID.Delivery_id
AND ID.Item_id = I.Item_id
AND IP.ParentType_id = 11
AND IP.Item_id = I.Item_id 
AND I.ItemType_id = IT.ItemType_id
AND I.ItemStatus_id = IST.ItemStatus_id
AND AMM.AMOModel_name = 'HLS'
AND D.Delivery_name LIKE 'HLS_AC%'

VOila mon UPDATE qui bug:

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
UPDATE
	AMOModelVariant 
SET
	AMOModelVariant_name = 'AC'
FROM
AMOModelVariant AMV,
AMOModel AMM,	
ItemParents IP,
Item I,
ItemType IT,
ItemStatus IST,
ItemDelivery ID,
Delivery D
WHERE
AMM.Program_id = 2
AND AMV.AMOModel_id=AMM.AMOModel_id
AND AMV.AMOModelVariant_id = IP.Parent_id
AND D.Delivery_id = ID.Delivery_id
AND ID.Item_id = I.Item_id
AND IP.ParentType_id = 11
AND IP.Item_id = I.Item_id 
AND I.ItemType_id = IT.ItemType_id
AND I.ItemStatus_id = IST.ItemStatus_id
AND AMM.AMOModel_name = 'HLS'
AND D.Delivery_name LIKE 'HLS_AC%'

Et voila ce que je viens de faire mais qui marche toujours pas:

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
UPDATE
	AMOModelVariant
SET
	AMOModelVariant.AMOModelVariant_name = 'AC'
WHERE
AMOModel.Program_id = 2
AND AMOModelVariant.AMOModel_id=AMOModel.AMOModel_id
AND AMOModelVariant.AMOModelVariant_id = ItemParents.Parent_id
AND Delivery.Delivery_id = ItemDelivery.Delivery_id
AND ItemDelivery.Item_id = Item.Item_id
AND ItemParents.ParentType_id = 11
AND ItemParents.Item_id = Item.Item_id 
AND Item.ItemType_id = ItemType.ItemType_id
AND Item.ItemStatus_id = IST.ItemStatus_id
AND AMM.AMOModel_name = 'HLS'
AND Delivery.Delivery_name LIKE 'HLS_AC%'

J'ai aussi essayé de mettre toute les tables après le UPDATE mais ca n'a pas marché!

**Keihilin** · 22/11/2007, 14h47

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
 
UPDATE
	AMOModelVariant 
SET
	AMOModelVariant_name = 'AC'
WHERE
    AMOModelVariant_id IN (   
    SELECT	
    AMOModelVariant_id
    FROM
    AMOModelVariant AMV,
    AMOModel AMM,	
    ItemParents IP,
    Item I,
    ItemType IT,
    ItemStatus IST,
    ItemDelivery ID,
    Delivery D
    WHERE
    AMM.Program_id = 2
    AND AMV.AMOModel_id=AMM.AMOModel_id
    AND AMV.AMOModelVariant_id = IP.Parent_id
    AND D.Delivery_id = ID.Delivery_id
    AND ID.Item_id = I.Item_id
    AND IP.ParentType_id = 11
    AND IP.Item_id = I.Item_id 
    AND I.ItemType_id = IT.ItemType_id
    AND I.ItemStatus_id = IST.ItemStatus_id
    AND AMM.AMOModel_name = 'HLS'
    AND D.Delivery_name LIKE 'HLS_AC%' )

pas garantie à 100%, j'ai la flemme de décortiquer quand les jointures sont faites comme ça, mais à priori ça devrait fonctionner.

**Gwendolyne** · 22/11/2007, 15h01

La requete fonctionne mais encore une fois elle ne tient pas compte de la condition elle change toute la colonne d'un coup.

Je comprend pas, pourtant syntaxiquement ca a l'air correct cette fois!

**ppmax** · 22/11/2007, 15h11

et ça donne quoi ça ?

SELECT
AMOModelVariant_id
FROM
AMOModelVariant AMV,
AMOModel AMM,
ItemParents IP,
Item I,
ItemType IT,
ItemStatus IST,
ItemDelivery ID,
Delivery D
WHERE
AMM.Program_id = 2
AND AMV.AMOModel_id=AMM.AMOModel_id
AND AMV.AMOModelVariant_id = IP.Parent_id
AND D.Delivery_id = ID.Delivery_id
AND ID.Item_id = I.Item_id
AND IP.ParentType_id = 11
AND IP.Item_id = I.Item_id
AND I.ItemType_id = IT.ItemType_id
AND I.ItemStatus_id = IST.ItemStatus_id
AND AMM.AMOModel_name = 'HLS'
AND D.Delivery_name LIKE 'HLS_AC%' )

La bonne chose?

**Keihilin** · 22/11/2007, 15h13

et sous cette forme ?

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
	AMOModelVariant 
SET
	AMOModelVariant_name = 'AC'
WHERE
    AMOModelVariant_id IN (  
    SELECT  AMV.AMOModelVariant_id,	
    FROM    AMOModelVariant AMV, 
            JOIN AMOModel AMM ON AMM.AMOModel_id = AMV.AMOModel_id
            JOIN ItemParents IP ON IP.Parent_id = AMV.AMOModelVariant_id
            JOIN Item I ON I.Item_id = IP.Item_id
            JOIN ItemDelivery ID ON ID.Item_id = I.Item_id
            JOIN Delivery D ON D.Delivery_id = ID.Delivery_id
    WHERE   AMM.Program_id = 2
            AND IP.ParentType_id = 11
            AND AMM.AMOModel_name = 'HLS'
            AND D.Delivery_name LIKE 'HLS_AC%'
   )

au passage, j'ai enlever 2 tables non-utilisées...

**Gwendolyne** · 22/11/2007, 15h42

Non désolé ca marche toujours pas. Ca change la colonne entièrement!

Pour ppmax la première requete qui fonctionne bien: retourne tous les lignes que je veux UPDATER.
Mais quand j'essai de les updater au lieu de changer les attributs de la colonne(que j'ai sélectionné avec la requête qui fonctionne bien) elle change tous les attributs de la colonne. En deux mots elle ne respecte pas du tout les contraintes imposées par le WHERE.

**Keihilin** · 22/11/2007, 15h53

est-ce que tu peux lancer

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
SELECT  AMV.AMOModelVariant_id
    FROM    AMOModelVariant AMV
            JOIN AMOModel AMM ON AMM.AMOModel_id = AMV.AMOModel_id
            JOIN ItemParents IP ON IP.Parent_id = AMV.AMOModelVariant_id
            JOIN Item I ON I.Item_id = IP.Item_id
            JOIN ItemDelivery ID ON ID.Item_id = I.Item_id
            JOIN Delivery D ON D.Delivery_id = ID.Delivery_id
    WHERE   AMM.Program_id = 2
            AND IP.ParentType_id = 11
            AND AMM.AMOModel_name = 'HLS'
            AND D.Delivery_name LIKE 'HLS_AC%'

et t'assurer que cela te retourne ce que tu veux...

Au passage, vérifie quand même qu'il y a d'autres lignes en comparant avec un

Code :

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

1
2
3
 
SELECT AMV.AMOModelVariant_id
FROM AMOModelVariant

**Gwendolyne** · 22/11/2007, 16h16

La première requete que tu as écrit fonctionne bien. Elle me donne la meme chose que la requete que j'ai écrit au début mais plus rapidement que la mienne:

Résultat: 420 lignes

Si on prend la meme requete et qu'on enlève la condition AND D.Delivery_name LIKE 'HLS_AC%'

Résultat: 900 lignes (420 lignes HLS_AC% + autre ligne)

Ce que je veux c'est que mon UPDATE change juste le ModelVariant_name de ces 420 lignes et pas des 480 autres.

Peut etre devrais-je redémarrer mon SQL Query Analyzer? bien qu'il semble bien fonctionner...

**Keihilin** · 22/11/2007, 16h33

Envoyé par Gwendolyne

Peut etre devrais-je redémarrer mon SQL Query Analyzer? bien qu'il semble bien fonctionner...

ben ça tiendrait du miracle que le problème vienne de là, mais vu que je sèche, pourquoi ne pas essayer ?

**Gwendolyne** · 22/11/2007, 17h03

Bon ben j'ai redémarré tout, j'ai reéssayé et rien a faire ca marche toujours pas!

Snirf ...

**Keihilin** · 22/11/2007, 17h05

tu peux essayer de nous monter le plan d'exécution ?

**Gwendolyne** · 22/11/2007, 17h25

C'est à dire? Qu'est ce que tu entends par plan d'execution?

**Sacha999** · 24/11/2007, 16h16

Je pense savoir pourquoi ta requete UPDATE ne fonctionne pas comme tu le voudrais. A cause de l'alias que tu donne a la table, tu la reutilise pas dans le UPDATE

1er cas qui marche (pas d'ambiguité de table):
UPDATE TABLE_CLIENT
SET Sexe = 'Male'
WHERE Prenom = 'Jean'

2eme cas qui marche (meme que cas 1, mais j'ai rajouté le FROM)
UPDATE TABLE_CLIENT
SET Sexe = 'Male'
FROM TABLE_CLIENT <--- meme chose que 1er cas sauf cette ligne rajouté
WHERE Prenom = 'Jean'

par contre 3eme cas qui ne marchera pas (ta facon de faire)
UPDATE TABLE_CLIENT
SET Sexe = 'Male'
FROM TABLE_CLIENT tc<--- meme chose que 2eme cas sauf que la il y a un Alias pour le nom de la table
WHERE Prenom = 'Jean'

Pour que le 3eme cas fonctionne il faudrait plutot ecrire
UPDATE tc <--- ici on met l'alias de la table et pas son vrai nom
SET Sexe = 'Male'
FROM TABLE_CLIENT tc
WHERE Prenom = 'Jean'

**Gwendolyne** · 26/11/2007, 10h27

Bonjour Sacha,

J'ai essayé en enlevant tous les alias!

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
UPDATE
	AMOModelVariant
SET
	AMOModelVariant_name = 'AC'
WHERE
    AMOModelVariant_id IN (  
    SELECT  AMOModelVariant.AMOModelVariant_id
    FROM    AMOModelVariant
            JOIN AMOModel ON AMOModel.AMOModel_id = AMOModelVariant.AMOModel_id
            JOIN ItemParents ItemParents ON ItemParents.Parent_id = AMOModelVariant.AMOModelVariant_id
            JOIN Item ON Item.Item_id = ItemParents.Item_id
            JOIN ItemDelivery ON ItemDelivery.Item_id = Item.Item_id
            JOIN Delivery ON Delivery.Delivery_id = ItemDelivery.Delivery_id
    WHERE   AMOModel.Program_id = 2
            AND ItemParents.ParentType_id = 11
            AND AMOModel.AMOModel_name = 'HLS'
            AND Delivery.Delivery_name LIKE 'HLS_AC%'
   )

Mais ca marche toujours pas!
Ca marche pas non plus en mettant l'alias comme tu le montre dans ton dernier cas.

Si quelqu'un a une autre idée, n'hésitez pas!

**maverickbj** · 26/11/2007, 12h12

Salut Gwendolyne,

J'ai essayé cette syntaxe chez moi ça semble faire ce dont tu as besoin.
Attention si tu fais des join inutiles c'est normal que ça te change toute la colonne. Il faut que tu limites tes join aux tables sur lesquelles tu vas filtrer. Ta requête devrait du coup donner ça.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
 
UPDATE AMOModelVariant
SET AMOModelVariant.AMOModelVariant_name = 'AC'
FROM AMOModelVariant AS MV 
JOIN AMOModel AS M ON M.AMOModel_id=MV.AMOModel_id
JOIN ItemParents AS IP ON IP.Parent_id=MV.AMOModelVariant_id
JOIN ItemDelivery AS ID ON ID.Item_id=IP.Item_id
JOIN Delivery AS DEL ON DEL.Delivery_id=ID.Delivery_id
WHERE MV.AMOModel_name='HLS'
AND DEL.Delivery_name LIKE 'HLS_AC%'
AND IP.ParentType_id=11

Je pense que ça devrait fonctionner n'hésite pas aussi à revoir tes conditions peut être que tu zappes quelque chose ou que tes filtres ne sont pas bons.

Bon Courage

**Keihilin** · 26/11/2007, 12h18

Envoyé par maverickbj

Code :

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

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

UPDATE AMOModelVariant
SET AMOModelVariant.AMOModelVariant_name = 'AC'
FROM AMOModelVariant AS MV 
JOIN AMOModel AS M ON M.AMOModel_id=MV.AMOModel_id
JOIN ItemParents AS IP ON IP.Parent_id=MV.AMOModelVariant_id
JOIN ItemDelivery AS ID ON ID.Item_id=IP.Item_id
JOIN Delivery AS DEL ON DEL.Delivery_id=ID.Delivery_id
WHERE MV.AMOModel_name='HLS'
AND DEL.Delivery_name LIKE 'HLS_AC%'
AND IP.ParentType_id=11

Vous pourriez arrêter avec vos update..from ? Ca me pique les yeux !

Problème de condition d'un update !

MS SQL Server

Discussions similaires

Partager

Partager