Jag har en SP som blev rätt lång och som jag tror går att korta ner en hel del. Det jag vill göra med den är att sätta in en annons, sen kan man ha max fem bilder till annonsen som ska in i en annan tabell. Detta körs då givetvis i en transaktion. SP fungerar som den är men jag tycker att det borde gå att lösa insättningen av bilderna på ett annat sätt än jag har gjort. Är ju alltid bra om man får lära sej nåt nytt också:) jag har funderat på om det inte skulle gå att köra någon while eller nåt? Så här ser den ut idag:

<code>
alter proc addAdvertise
@typeOfCustomerID int,
@lanID int,
@categoryID int,
@typeOfAdvertiseID int,
@name varchar(50),
@mail varchar(150),
@phone varchar(20),
@cellPhone varchar(20),
@zipCode varchar(6),
@header varchar(100),
@text text,
@price money,
@password varchar(20),
@pic1 varchar(150) = null,
@pic2 varchar(150) = null,
@pic3 varchar(150) = null,
@pic4 varchar(150) = null,
@pic5 varchar(150) = null
as

set nocount on
declare @errMess varchar(100)
declare @returnID int
begin try
begin tran
insert into Advertises(typeOfCustomerID, lanID, categoryID, typeOfAdvertiseID, [name], mail, phone, cellPhone, zipCode, header, [text], price, isApproved, password, inDate)
values(@typeOfCustomerID, @lanID, @categoryID, @typeOfAdvertiseID, @name, @mail, @phone, @cellPhone, @zipCode, @header, @text, @price,
'False', @password, getdate())
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
select @returnID = @@identity
if(@pic1 is null)
begin
commit tran
return @returnID
end
else
begin
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@returnID, @pic1)
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
if(@pic2 is null)
begin
commit tran
return @returnID
end
else
begin
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@returnID, @pic2)
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
if(@pic3 is null)
begin
commit tran
return @returnID
end
else
begin
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@returnID, @pic3)
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
if(@pic4 is null)
begin
commit tran
return @returnID
end
else
begin
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@returnID, @pic4)
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
if(@pic5 is null)
begin
commit tran
return @returnID
end
else
begin
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@returnID, @pic5)
if(@@rowcount <> 1)
begin
set @errMess = 'Det gick inte att lägga in annonsen, var god försök igen.'
raiserror(@errMess, 16, 1)
end
else
begin
commit tran
return @returnID
end
end
end
end
end
end
end
end
end
end
end
end try

begin catch
raiserror(@errMess, 16, 1)
rollback tran
return
end catch
</code>

En sak som inte fungerar är "return @returnID" varför gör det inte det? Jag måste verkligen få med id tillbaka.

--------------------------EDIT-------------------------
Det där med return löste jag. hade ju missat att ha med "select @returnID", när jag ändrade till detta så fungerade det.
<code>
set @returnID = @@identity
select @returnID
</code>
Resten av koden ser likadan ut. Kvarstår gör dock om det går att lösa insättningen av bilderna på nåt enklare sätt?

Du kan ju bryta ut den här koden

if(@pic1 is null)
begin

-----

end

till en egen procedure eller function

så att du helt enkelt kör fem st anrop

execute MyInsert @Id1, @pic1
execute MyInsert @Id1, @pic2
execute MyInsert @Id1, @pic3
....

Ja det var smart, det ska jag prova. jag återkommer om jag får problem.

Funkade klockrent!