Jag har gjort nåt tokigt i denna SP. det den gör är ratt lägga in massor med poster i AdvertisePics. det jag vill är att den ska lägga in de poster som finns i tempPic om det finns några. Nån som kan hjälpa mej rätt med detta?

<code>
ALTER proc [IsellItAdmin].[moveTempPic]
@userSession varchar(256),
@advertiseID int
as
set nocount on
declare @errMess varchar(100)
declare @picName varchar(150)
declare @numOfPics int
begin try
if(exists(select pic from tempPic where userSession = @userSession))
begin
select @numOfPics = count(*) from tempPic where userSession = @userSession
while(@numOfPics > 0)
begin
begin tran
select @picName = pic from tempPic where userSession = @userSession
insert into AdvertisePics(AdvertiseID, AdvertisePic)
values(@advertiseID, @picName)
delete from tempPic
where userSession = @userSession and pic = @picName
if(@@error <> 0)
begin
set @errMess = 'Det gick inte att lägga till bilderna.'
raiserror(@errMess, 16, 1)
end
else
commit tran
end
end
end try

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

När @numOfPics blir större än 0, och du går in i WHILE satsen blir den en evighetsloop.
Villkoret i WHILE satsen blir alltid sant, då du aldrig räknar ner @numOfPics.

/Håkan

Räknar ner gör jag ju med denna rad:
delete from tempPic
where userSession = @userSession and pic = @picName

Eller? Den ska ju radera posten, alltså måste ju @numOfPics bli en mindre varvet efter?

Nej.
För att @numOfPics ska bli mindre måste den tilldelas ett nytt värde.
Du tilldelar numOfPics värdet med SELECT satsen före WHILE-loopen, sedan rör du den inte mer.
Den blir inte tilldelat något bara för att du tar bort något i en tabell.
SELECT-satsen som tilldelar @numOfPics värdet ligger utan för WHILE-loopen.
Om du vill fånga upp ett nytt COUNT(*) värde, efter din DELETE får du köra SELECT-satsen igen
efter DELETE, och tilldela resultatet till @numOfPics.

/Håkan

Ja så dum man kan vara! Ska prova detta ikväll.

Varför inte något sådanthär:

ALTER proc [IsellItAdmin].[moveTempPic]
@userSession        varchar(256),
@advertiseID        int
as
set nocount on

DECLARE @errMess VARCHAR(100)
DECLARE @intErrorCode INT

BEGIN TRAN
	INSERT INTO AdvertisePics (AdvertiseID, AdvertisePic)
	SELECT @advertiseID, pic
	FROM tempPic
	WHERE userSession = @userSession

	IF (@intErrorCode <> 0) GOTO PROBLEM

	DELETE FROM tempPic
	WHERE userSession = @userSession

	IF (@intErrorCode <> 0) GOTO PROBLEM

COMMIT TRAN

PROBLEM:
IF (@intErrorCode <> 0) BEGIN
	ROLLBACK TRAN
	SET @errMess = 'Det gick inte att lägga till bilderna.'
	RAISERROR(@errMess, 16, 1)
END

ska testa ditt förslag Andreas. Jag tänkte ha bara en tabell från början, problemet är dock att jag inte vet advertiseID när bilderna läggs in i tempPic. Alltså måste jag lägga in annonsen och få det id:t innan jag kan lägga in bilderna.

Det verkar funka kanon:)


------EDIT----------

Glömde att jag skulle fråga ett par saker om din kod. Den verkar ju vara smidigare än min:) Vi fick lära oss på en kurs att det kunde bli problem med "GOTO" och att det var bättre att skriva koden så som jag ahr gjort, stämmer det? om något skulle gå fel, var ändrar du värdet på @intErrorCode?

Du kan ha rätt. Du kan ju använda:

ALTER proc [IsellItAdmin].[moveTempPic]
@userSession        varchar(256),
@advertiseID        int
as
set nocount on

SET XACT_ABORT ON;
GO

BEGIN TRAN
    INSERT INTO AdvertisePics (AdvertiseID, AdvertisePic)
    SELECT @advertiseID, pic
    FROM tempPic
    WHERE userSession = @userSession

    DELETE FROM tempPic
    WHERE userSession = @userSession

COMMIT TRAN

ALTER proc [IsellItAdmin].[moveTempPic]
@userSession        varchar(256),
@advertiseID        int
as
set nocount on

BEGIN TRAN
    BEGIN TRY

        INSERT INTO AdvertisePics (AdvertiseID, AdvertisePic)
        SELECT @advertiseID, pic
        FROM tempPic
        WHERE userSession = @userSession

        DELETE FROM tempPic
        WHERE userSession = @userSession
    END TRY
    BEGIN CATCH
        RAISERROR('Det gick inte att lägga till bilderna.', 16, 1)
        ROLLBACK TRAN
    END CATCH;
COMMIT TRAN