Discussion :

[shimomura22]

Bonjour à tous et à toutes

Voici mon problème
Je souhaite effacer des données reliées entre elles par une contrainte type FK le plus efficacement possible...
J'ai 2 tables

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CREATE TABLE [dbo].[Emplacement](
	[IdEmplacement] [int] IDENTITY(1,1) NOT NULL,
	[IdReservation] [int] NULL,
	[IdParking] [int] NOT NULL,
	[Numero] [varchar](50) NOT NULL,
	[Occupied] [bit] NULL,
	[Prix] [float] NULL,
 CONSTRAINT [PK_Emplacement] PRIMARY KEY CLUSTERED 
(
	[IdEmplacement] ASC
)

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
 
CREATE TABLE [dbo].[Parking](
	[IdParking] [int] IDENTITY(1,1) NOT NULL,
	[Etage] [int] NOT NULL,
	[ZoneTarifaire] [int] NOT NULL,
	[DescriptionParking] [varchar](250) NOT NULL,
	[Nom] [varchar](50) NOT NULL,
	[route] [varchar](50) NOT NULL,
	[postal_code] [varchar](50) NOT NULL,
	[locality] [varchar](50) NOT NULL,
	[administrative_area_level_1] [varchar](50) NOT NULL,
	[country] [varchar](50) NOT NULL,
	[street_number] [varchar](50) NOT NULL,
 CONSTRAINT [PK_Parking] PRIMARY KEY CLUSTERED 
(
	[IdParking] ASC
)

L'id de ma table parking est référencé dans la table emplacement.
J'aimerai lors de la suppression d'un parking supprimer ses références dans la table emplacement
Pour ce faire, j'utilise une SP

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
 
alter PROCEDURE sp_Delete_Parking
	-- Add the parameters for the stored procedure here
	@id int
AS
BEGIN
	-- SET NOCOUNT ON added to prevent extra result sets from
	-- interfering with SELECT statements.
	SET NOCOUNT ON;
	alter table [dbo].[Emplacement]
	NoCheck constraint [FK_Emplacement_Parking]
 
 
 
 
	delete from [dbo].[Parking]
		where [IdParking] = @id
    -- Insert statements for procedure here
 
	delete from [dbo].[Emplacement]
		where [IdParking] = @id 
 
	alter table [dbo].[Emplacement]
	check constraint [FK_Emplacement_Parking]
END
GO

Je voudrais savoir si il y a un moyen disons plus professionnel d'y arriver.
Je voudrais aussi une gestion d'erreur juste au cas où ????
Je pensais utiliser une transaction mais je ne suis pas sur de bien la programmer

Merci et bonne journée

[al1_24]

Regarde du côté des contraintes d'intégrité référentielle (clés étrangères) et de la clause ON DELETE CASCADE...

[shimomura22]

Bonjour
Je voulais utiliser ON DELETE CASCADE... mais on m'a dit que c’était assez dangereux et pas "propre" du tout dans le code.
Je voulais utiliser une transaction pour vérifier que la première étape se soit bien déroulé ensuite passer à la suivante. Si erreur il y a je voudrais faire un rollback et informer l'utilisateur que la suppression a échouée
Merci

[SQLpro]

ON DELETE CASCADE fait cela automatiquement....
1) dans une seule transaction avec annulation si problème
2) en minimisant les opérations car c'est du code interne plus rapide que du transact SQL

On vous a dit que c'était pas bien ? Pourquoi ???? Citez vos sources !

Oui l'emploi SYSTÉMATIQUE du DELETE CASCADE est catastrophique. Mais pour des tables conjointes comme Facture/Ligne de facture, c'est plutôt pas mal !

A +

[StringBuilder]

En alternative au CASCADE, on peut faire un TRIGGER "INSTEAD OF", faire le DELETE des lignes liées dedans, puis le delete des lignes à supprimer à propremement parler.

Mais comme le dit SQL Pro, aucune raison valable de faire ça alors que le SGBD le gère nativement.

Le trigger ne sera utile que dans le cas où on doit faire des vérifications avant.
Par exemple, sur une livraison, interdire sa suppression si des lignes ont été facturées.

[aieeeuuuuu]

Bonjour,

J'ajouterais quelques remarques concernant votre procédure stockée en l'état :

1/ la FK est mal gérée : vous la désactivez, puis la réactivez sans vérification. En faisant cela, vous pouvez vous retrouver avec des lignes orphelines (emplacement pointant vers un parking qui n'existe plus). De plus, l'optimiseur n'utilisera plus la contrainte pour améliorer les requêtes.
Il faudrait donc ajouter la clause de vérification WITH CHECK pour revalider la contrainte :

Code :

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

1
2
3
 
alter table [dbo].[Emplacement] WITH CHECK
	check constraint [FK_Emplacement_Parking]

2/ il est d'ailleurs inutile en l’occurrence de désactiver la contrainte (cela risque en plus de produire une recompilation des plan d’exécution) : il suffit de commencer par supprimer les emplacements, avant de supprimer le parking.