Discussion :

[shimomura22]

Bonjour à tous
Voici mon problème ...
J'ai 2 tables


Et voici mon problème
Lors de la création d'un parking, j'aimerai, grâce au champ NombrePlace de ma table Parking , créer des emplacements pour ce parking dont le nombre serait égale au NombrePlace.
Je m'explique si je crée un parking et que je lui attribue 50 dans le champ NombrePlace automatiquement dans la table Emplacement je crée 50 entrées avec l'IdParking du parking insérer dans le champs IdParking de la table Emplacement.
J’espère que mes explications sont assez clair parce là je suis perdu.
Je me doute que j'aurai besoin d'une boucle et/ou d'un trigger mais je ne suis pas sûr de la syntaxe
Merci

[shimomura22]

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
USE [ParkAir]
GO
/****** Object:  Trigger [dbo].[trgParkingEmplacement]    Script Date: 22-06-16 20:41:01 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
ALTER trigger [dbo].[trgParkingEmplacement] on [dbo].[Parking]
after insert
as 
 
declare @NbrePlace int, @NbreEmplacement int, @IdParking int, @Label varchar(50)
 
set @NbrePlace = (select [NombrePlace] from inserted )
set @IdParking = ( select [IdParking] from inserted)
set @Label = (select [Nom] from inserted)
set @NbreEmplacement = (select count(*) from Emplacement e, inserted p where e.IdParking = p.IdParking )
 
while @NbreEmplacement < @NbrePlace
 
Begin
	insert into Emplacement ([IdParking],Numero)
	values
	(@IdParking,@Label + cast (@NbreEmplacement as varchar(10)))
	set @NbreEmplacement = @NbreEmplacement + 1
end

Voici mon code, je voudrais avoir un avis ou si possible une amélioration.
Le code en soi fonctionne mais je veux être sur qu'il soit le plus efficace possible

Merci

[aieeeuuuuu]

Bonjour,

Plusieurs choses ne vont pas :

1/Sur le modèle d'abord : la colonne NombreDePlace n'est pas sa place dans la table Parking. En effet, c'est une donnée qui se calcule.
Que se passera-t-il si cette valeur est modifiée? quelles lignes irez vous supprimer dans la table Emplacement pour garder la cohérence des données.
Par ailleurs, il faudra modifier la valeur de cette colonne si vous ajoutez ou supprimez des lignes de la table Emplacement.
Il serait préférable de retirer cette colonne de la table Parking
Faites plutôt une vue, qui contiendra cette colonne calculée par jointure entre les deux tables. et placez votre trigger sur cette vue.

2/Le trigger : son principal défaut est qu'il ne prends pas en compte la possibilité d’insérer plusieurs lignes dans la table Parking.
Par ailleurs, d'un point de vue performances, ajouter des lignes une par une dans une boucle n'est pas idéal.
Il faudrait donc ajouter toutes les lignes d'un coup.

[StringBuilder]

Comme aïeu² je verrais plutôt un truc du genre :

Code sql :

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
40
41
42
43
44
45
46
47
 
create table parking
(
	id int not null identity(1,1) primary key,
	nom varchar(30) not null unique
);
 
create table emplacement
(
	id int not null identity(1,1) primary key,
	parking_id int not null references parking(id),
	num int not null,
	unique (parking_id, num)
);
go
 
create view v_parking (id, nom, nb_emplacements)
as
select p.id, p.nom, count(*)
from parking p
inner join emplacement e on e.parking_id = p.id
group by p.id, p.nom;
go
 
create trigger trg_v_parking_i
on v_parking
instead of insert
as
begin
	insert into parking (nom)
	select nom from inserted;
 
	with cpt (num)
	as
	(
		select 1
		union all
		select cpt.num + 1 from cpt where num < 10000
	)
	insert into emplacement (parking_id, num)
	select p.id, c.num
	from inserted i
	inner join parking p on p.nom = i.nom
	inner join cpt c on c.num <= i.nb_emplacements
	OPTION (MAXRECURSION 9999);
end;
go

Qu'on peut tester de la sorte :

Code sql :

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

1
2
3
 
insert into v_parking (nom, nb_emplacements) values ('Dépose minute', 3), ('Sous-terrain', 400), ('Cours', 50);
select id, nom, nb_emplacements from v_parking;

[shimomura22]

Salut StringBuilder ..
Pourrais-tu m'expliquer d'où vient le "cpt" ????
Parce que je ne vois pas où il a été déclaré ???
Merci et désolé de répondre si tard

[shimomura22]

Merci StringBuilder et aieeeuuuuu
discussion résolue