Violation de la contrainte PRIMARY KEY

**franculo_caoulene** · 12/12/2007, 16h26

Salut,

Je suis en train d'écrire une petite procédure stockée afin d'exporter des données d'une table à une autre.
Lorsqu'une ligne est déjà insérée je souhaite faire une mise à jour d'un champ.
Le problème est que la condition censée détecter le cas ne fonctionne pas.
La condition :

Code :

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

if (select count(COL_2) from TABLE_1 where COL_1 = @id and COL_2 = @2) = 0

me retourne toujours 0 dans la procédure. Si j'exécute la requête à part, pour tester, j'ai bien un résultat cohérent.

Je pense que la définition des tables n'est pas utile, mais je suis disposé à les donner si nécessaire.

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
31
32
33
34
35
36
37
38
39
open CURS_TMP
fetch next from CURS_TMP into @id, @p, @1, @2, @3, @4, @5, @6, @7, @8, @9
while @@fetch_status = 0
	begin
		if isnull(charindex(';', @1), 0) <> 0
		begin 
			set @ideb = 0
			set @ifin = 0
			while (@ifin < len(@1))
			begin
				set @ideb = @ifin
				set @ifin = charindex(';', @1, (@ideb+1))
				if @ifin<>1 and @ifin<>len(@1)
				begin
					insert into TABLE_1 (COL_1, COL_2, COL_3) values (@id, substring(@1,(@ideb+1),(@ifin-@ideb-1)), 23)
				end
			end
		end
		if isnull(charindex(';', @2), 0) <> 0
		begin 
			set @ideb = 0
			set @ifin = 0
			while (@ifin < len(@2))
			begin
				set @ideb = @ifin
				set @ifin = charindex(';', @2, (@ideb+1))
				if @ifin<>1 and @ifin<>len(@2)
				begin
					if (select count(COL_2) from TABLE_1 where COL_1 = @id and COL_2 = @2) = 0
						insert into TABLE_1 (COL_1, COL_2, COL_3) values (@id, substring(@2,(@ideb+1),(@ifin-@ideb-1)), 19)
					else
						update TABLE_1 set COL_3 = (COL_3*19) where COL_1 = @id and COL_2 = @2
				end
			end
		end
		fetch next from CURS_TMP into @id, @p, @1, @2, @3, @4, @5, @6, @7, @8, @9
	end
close CURS_TMP
deallocate CURS_TMP

Je pense que ça ne peut pas lire et compter les données déjà insérées dans ces conditions, mais je ne sais pas vraiment pourquoi (juste une idée floue), ni comment faire pour que je fasse le select sur les données déjà insérées.

Merci par avance.

**kagemaru** · 12/12/2007, 21h12

bonjour,

est-ce que le curseur est sur TABLE_1 ? for readonly ou for update ?

et quand vous lancez tout le code du curseur hors procédure, le résultat du select est bon ?

désolé poour l'éventuelle redite, mais c'est pour avoir des certitudes :-)

merci

**franculo_caoulene** · 12/12/2007, 21h32

Je n'ai pas le code sous les yeux, je ne peux donc que fournir des réponses approximatives.

Envoyé par kagemaru

est-ce que le curseur est sur TABLE_1 ?

Non le curseur est sur une autre table. Ce sont des données que "j'exporte" de TABLE_0 vers TABLE_1

Envoyé par kagemaru

for readonly ou for update ?

Bonne question, on va dire par défaut. C'est la première fois que j'utilise vraiment les curseurs, pour des raisons historiques on va dire.

Envoyé par kagemaru

et quand vous lancez tout le code du curseur hors procédure, le résultat du select est bon ?

C'est à dire? Ce n'est pas encore dans une procédure stockée mais un simple fichier SQL que j'exécute, je suis encore en phase de test. Quand j'exécute le SELECT, mais juste le select, j'ai bien le résultat attendu après l'exécution du code affiché, car TABLE_1 est vide à l'origine.

Envoyé par kagemaru

désolé poour l'éventuelle redite, mais c'est pour avoir des certitudes :-)

Mais c'est plus que bienvenu, merci de t'intéresser à mon problème.

**franculo_caoulene** · 13/12/2007, 09h08

En effet, j'ai oublié la déclaration du curseur :

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
declare CURS_TMP cursor for
select
	CH_ID,
	CH_P,
	CH_1, 
	CH_2, 
	CH_3, 
	CH_4, 
	CH_5, 
	CH_6, 
	CH_7, 
        CH_8, 
	CH_9,
	CH_A,
	CH_T,
	CH_F
from TABLE_1
where CH_P = 1
 
open CURS_TMP
-- code posté précédemment

Le curseur est donc pour un simple select.

**kagemaru** · 13/12/2007, 18h26

Non le curseur est sur une autre table. Ce sont des données que "j'exporte" de TABLE_0 vers TABLE_1

Pourtant la déclaration du curseur indique TABLE_1, sans vouloir abuser ....

Parfois il vaut mieux citer les noms de tables tels quels...

me retourne toujours 0 dans la procédure

je pensais que vous êtiez dans une procédure ...

Bonne question, on va dire par défaut

Le mode par défaut est readonly (jeu d'enregistrement en lecture seule)
Mais si de toute façon le curseur parcourt une table autre que celle mise à jour le problème ne vient pas du curseur.
Les curseurs ont parfois des comportements différents en fonction du mode d'ouverture, des index placés sur la table (PK, ...).
Mais dans votre cas, ce n'est pas lié.

Votre problème est étrange en effet, la condition étant ok. Si vous n'avez pas déja trouvé, peut-être pourriez-vous décrire les premiers tours de boucle avec un enregistrement d'exemple ....

**SQLpro** · 13/12/2007, 23h04

Cette conditon n'a aucun sens :

Code :

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

1
2
3
4
IF (SELECT count(COL_2) FROM TABLE_1 WHERE COL_1 = @id AND COL_2 = @2) = 0
						INSERT INTO TABLE_1 (COL_1, COL_2, COL_3) VALUES (@id, substring(@2,(@ideb+1),(@ifin-@ideb-1)), 19)
					else
						UPDATE TABLE_1 SET COL_3 = (COL_3*19) WHERE COL_1 = @id AND COL_2 = @2

En effet une requête renvoie une table non une valeur !

Il faut faire quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
IF EXISTS (SELECT count(COL_2) 
           FROM   TABLE_1 
           WHERE  COL_1 = @id 
             AND  COL_2 = @2
           HAVING count(COL_2) = 0)
   INSERT INTO TABLE_1 (COL_1, COL_2, COL_3) 
          VALUES (@id, substring(@2,(@ideb+1),(@ifin-@ideb-1)), 19)
ELSE
   UPDATE TABLE_1 
   SET    COL_3 = (COL_3*19) 
   WHERE  COL_1 = @id 
     AND  COL_2 = @2;

A +

**franculo_caoulene** · 14/12/2007, 10h43

Et bien l'erreur était bel et bien dans ma condition. Je l'avais testé mais mal testé! Au lieu de récupérer réellement la requête exécutée, je remplacais bêtement ma variable à la main par la valeur qu'elle devait avoir.

Merci à vous, merci à kagemaru qui en me demandant plus d'infos m'a forcé à plus me plonger dans le code et non plus d'être sûr de ce que j'avais.

SQLpro : c'est vrai qu'une requête retourne une table et non une valeur. Après quelques mois sans coder j'avais oublié "exists". Mais la condition comme je l'ai affiché ici fonctionne aussi. Je ne connais pas le coeur de MSSQL pour l'expliquer mais ça fonctionne.

Merci encore à vous deux pour vous être intéressé à mon problème.

Violation de la contrainte PRIMARY KEY

MS SQL Server

Discussions similaires

Partager

Partager