[Script SQL] Faire des conditions

Version imprimable

Bonjour,

J'ai fait une procédure stockée qui met à jour 2 tables, elle fonctionne tres bien.
J'ai par contre pas pensé à une chose.

Voila en gros ma requete toute simple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 ... UPDATE Portal_maTable SET Title=@Title, WHERE ContactId = @ContactId AND LangId = @LangId; ...
Mais en fait ce que je voudrais faire c'est de vérifier qu'il existe déja une ligne répondant au condition de mon WHERE et si ce n'est pas le cas créer la ligne donc faire un INSERT.

Quelqu'un pourrait il me dire comment créer une condition dans ma procédure.

je pensais faire quelque chose comment cela:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 select * from matable WHERE ContactId = @ContactId AND LangId = @LangId; SI (la requete retourne quelquechose) ALORS ----- je fais le update SINON ------ je fais le INSERT

Citation:

Envoyé par Gregory.M

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
select * from matable
WHERE
 ContactId = @ContactId
AND 
LangId = @LangId;
 
 
SI (la requete retourne quelquechose) ALORS
----- je fais le update
SINON
------ je fais le INSERT

Code:

1
2
3
4
5
6
IF EXISTS(SELECT * FROM matable WHERE condition)
BEGIN
-- alors UPDATE
END ELSE BEGIN
-- sinon INSERT
END

16/02/2009, 16h12
elsuket

Bonjour,

Si vous êtes sous SQL Server 2008, pensez à l'ordre MERGE.

@++ ;)
16/02/2009, 16h14
Gregory.M

j'utilise sql server 2005
17/02/2009, 13h20
cavo789

As-tu essayé la proposition de cmako ?
17/02/2009, 13h25
Gregory.M

Merde je ne l'avais meme pas vu!!! :oops:

Bonjour,

J'essaie d'appliquer ce que vous m'avez dit dans les messages précédents mais lorsque j'execute ce code j'ai une erreur...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
IF EXISTS(Select * from sysobjects where name LIKE 'Portal_UpdateLanguageVisibility' AND Xtype='P')
BEGIN
	set ANSI_NULLS ON
	set QUOTED_IDENTIFIER ON
	go
	ALTER Procedure [dbo].[Portal_UpdateLanguageVisibility]
	(
		@Visible  bit,
		@LangID int
	)
	AS
 
	UPDATE Portal_Languages
	SET Visible = @Visible
	WHERE Langid = @LangID;
 
END

Erreur:

Code:

1
2
3
4
5
 
Msg 102, Level 15, State 1, Line 4
Syntaxe incorrecte vers 'ON'.
Msg 102, Level 15, State 1, Procedure Portal_UpdateLanguageVisibility, Line 12
Syntaxe incorrecte vers 'END'.

Par contre cela marche quand j'enleve BEGIN et END, mais quand j'ajoute un else ca ne marche plus... :aie::aie:

18/02/2009, 14h11
cmako

Citation:

Envoyé par Gregory.M

Bonjour,

J'essaie d'appliquer ce que vous m'avez dit dans les messages précédents mais lorsque j'execute ce code j'ai une erreur...

Par contre cela marche quand j'enleve BEGIN et END, mais quand j'ajoute un else ca ne marche plus... :aie::aie:

IL ne faut jamais mettre GO à l'intérieur d'une procédure :nono:

Ensuite, on ne met JAMAIS un ALTER PROCEDURE après un BEGIN :kill:

Les commandes CREATE PROCEDURE et ALTER PROCEDURE doivent doivent être toujours au début ou après la commande GO
18/02/2009, 14h16
Gregory.M
Citation:

Envoyé par cmako

IL ne faut jamais mettre GO à l'intérieur d'une procédure :nono:

Ah oui? pourtant qu'on fait create procédure il y a déjà plusieurs GO. :?

En fait la je ne fais plus une procédure stockée, je fais un script qui regarde si la procédure existe et si elle existe je la met à jour sinon je la créé

Si j'enleve le go j'ai les erreurs suivantes:
Code:

1 2 3 4 5 Msg 156, Level 15, State 1, Line 6 Syntaxe incorrecte vers le mot clé 'Procedure'. Msg 137, Level 15, State 2, Line 14 La variable scalaire "@Visible" doit être déclarée.
18/02/2009, 14h18
cmako
Citation:

Envoyé par Gregory.M

Ah oui? pourtant qu'on fait create procédure il y a déjà plusieurs GO. :?

En fait la je ne fais plus une procédure stockée, je fais un script qui regarde si la procédure existe et si elle existe je la met à jour sinon je la créé

NON, on peut faire ça:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 CREATE PROCEDURE toto AS BEGIN ... END GO CREATE PROCEDURE titi AS BEGIN ... END GO

J'ai fait ca sur le modèle qui tu viens de faire. Mais ca ne marche pas non plus :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
IF EXISTS(Select * from sysobjects where name LIKE 'Portal_UpdateLanguageVisibility' AND Xtype='P')
BEGIN
	ALTER Procedure [dbo].[Portal_UpdateLanguageVisibility]
	(
		@Visible  bit,
		@LangID int
	)
	AS
	BEGIN
		UPDATE Portal_Languages
		SET Visible = @Visible
		WHERE Langid = @LangID;
	END
GO
END

Code:

1
2
3
4
5
6
7
8
 
 
Msg 156, Level 15, State 1, Line 3
Syntaxe incorrecte vers le mot clé 'Procedure'.
Msg 137, Level 15, State 2, Line 11
La variable scalaire "@Visible" doit être déclarée.
Msg 102, Level 15, State 1, Line 1
Syntaxe incorrecte vers 'END'.

Si tu veux modifier une procédure il faut faire ça:

Code:

1
2
3
4
5
6
7
8
IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = object_id(N'Portal_UpdateLanguageVisibility'))
	DROP Portal_UpdateLanguageVisibility
GO
 
CREATE PROCEDURE Portal_UpdateLanguageVisibility(@Visible  bit, @LangID int)
AS
...
GO

Citation:

Envoyé par Gregory.M

J'ai fait ca sur le modèle qui tu viens de faire. Mais ca ne marche pas non plus :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
IF EXISTS(Select * from sysobjects where name LIKE 'Portal_UpdateLanguageVisibility' AND Xtype='P')
BEGIN
	ALTER Procedure [dbo].[Portal_UpdateLanguageVisibility]
	(
		@Visible  bit,
		@LangID int
	)
	AS
	BEGIN
		UPDATE Portal_Languages
		SET Visible = @Visible
		WHERE Langid = @LangID;
	END
GO
END

Code:

1
2
3
4
5
6
7
8
 
 
Msg 156, Level 15, State 1, Line 3
Syntaxe incorrecte vers le mot clé 'Procedure'.
Msg 137, Level 15, State 2, Line 11
La variable scalaire "@Visible" doit être déclarée.
Msg 102, Level 15, State 1, Line 1
Syntaxe incorrecte vers 'END'.

Normal, CREATE PROCEDURE doit être au tout début
et GO doit être tout à la fin

J'y suis presque sauf qu'il y a un probleme sur le drop de la procedure stockée :(

j'ai essayé avec un ';' ou 'dbo' devant le nom de la procédure ... :(

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = object_id(N'Portal_UpdateLanguageVisibility'))
	drop Portal_UpdateLanguageVisibility
 
CREATE Procedure [dbo].[Portal_UpdateLanguageVisibility]
(
    @Visible  bit,
	@LangID int
)
AS
 
UPDATE Portal_Languages
SET Visible = @Visible
WHERE Langid = @LangID;

Code:

1
2
3
 
Msg 102, Level 15, State 1, Line 2
Syntaxe incorrecte vers 'Portal_UpdateLanguageVisibility'.

18/02/2009, 14h54
cmako

T'as pas mis le GO devant le CREATE PROCEDURE
18/02/2009, 15h05
elsuket

Bonjour,

GO n'est pas une instruction T-SQL, mais une commande qui était destinée au départ à l'utilitaire en ligne de commandes OSQL, puis à SQLCMD.
Elle est reconnue par SQL Server Management Studio.

Ces 3 outils interprètent GO comme le signal qu'ils doivent soumettre le lot T-SQL au moteur de base de données.

Le lot "en cours" se compose de toutes les instructions :

- comprises entre la dernière commande GO et la suivante ou entre le dernier GO et la fin d'un script
- depuis le début de la session

Dans le dernier bout de code de Gregory.M, il n'y a pas de GO entre l'instruction DROP PROCEDURE et CREATE PROCEDURE, donc le lot est transmis tel quel à l'instance, ce qui explique la remarque de cmako.

@++ ;)

J'ai toujour la meme erreur!

Code:

1
2
3
4
5
6
7
8
9
10
11
IF EXISTS (SELECT * FROM dbo.sysobjects WHERE id = object_id(N'Portal_UpdateLanguageVisibility'))
	drop Portal_UpdateLanguageVisibility

GO
CREATE Procedure [dbo].[Portal_UpdateLanguageVisibility] (@Visible  bit,@LangID int)
AS
GO
UPDATE Portal_Languages
SET Visible = @Visible
WHERE Langid = @LangID

18/02/2009, 15h18
Gregory.M

ah voila j'ai trouvé. il faut faire:

Code:

drop proc Portal_UpdateLanguageVisibility
18/02/2009, 16h11
cmako

Citation:

Envoyé par Gregory.M

ah voila j'ai trouvé. il faut faire:

Code:

drop proc Portal_UpdateLanguageVisibility

Enlève le GO après AS