[Script SQL] Faire des conditions

**Gregory.M** · 16/02/2009, 15h05

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 :

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

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 :

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

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

**cmako** · 16/02/2009, 15h09

Envoyé par Gregory.M

Code :

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

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 :

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

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

**elsuket** · 16/02/2009, 15h12

Bonjour,

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

@++

**Gregory.M** · 16/02/2009, 15h14

j'utilise sql server 2005

**cavo789** · 17/02/2009, 12h20

As-tu essayé la proposition de cmako ?

**Gregory.M** · 17/02/2009, 12h25

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

**Gregory.M** · 18/02/2009, 13h03

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 :

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
 
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 :

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

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...

**cmako** · 18/02/2009, 13h11

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...

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

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

Les commandes CREATE PROCEDURE et ALTER PROCEDURE doivent doivent être toujours au début ou après la commande GO

**Gregory.M** · 18/02/2009, 13h16

Envoyé par cmako

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

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 :

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

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.

**cmako** · 18/02/2009, 13h18

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 :

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

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

**Gregory.M** · 18/02/2009, 13h21

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

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
 
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 :

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

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'.

**cmako** · 18/02/2009, 13h22

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

Code :

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

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

**cmako** · 18/02/2009, 13h23

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 :

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
 
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 :

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

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

**Gregory.M** · 18/02/2009, 13h45

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 :

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

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 :

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

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

**cmako** · 18/02/2009, 13h54

T'as pas mis le GO devant le CREATE PROCEDURE

**elsuket** · 18/02/2009, 14h05

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.

@++

**Gregory.M** · 18/02/2009, 14h09

J'ai toujour la meme erreur!

Code :

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

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

**Gregory.M** · 18/02/2009, 14h18

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

Code :

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

drop proc Portal_UpdateLanguageVisibility

**cmako** · 18/02/2009, 15h11

Envoyé par Gregory.M

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

Code :

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

drop proc Portal_UpdateLanguageVisibility

Enlève le GO après AS

[Script SQL] Faire des conditions

Développement SQL Server

Discussions similaires

Partager

Partager