T-SQL problem

Hvordan ser din kaldende kode ud?

PS: Jeg synes at du skullle bruge et ikke-keyword som alias!

Sikkert et dumt spørgsmål, men har du sørget for, at din primærnøgle bliver sat når du indsætter?

Det er mare mig der har siddet og sovet for at sikre at den samme bruger ikke blev oprettet to gange havde jeg til føjet en if sætning og det var den der fejlet !! men smid et svar så i kan få nogle points for jeres ulejlighed!!

Så koden der virker ser sådan her ud til andre der skulle have et ligende problem:

CREATE PROCEDURE dbo.[AddUser]

@Name varchar(max),
@UserName varchar(max),
@Email varchar(max),
@Password varchar(max)


AS

IF NOT EXISTS (Select Id From Users where UserName= @UserName)
BEGIN

INSERT INTO Users (Name,UserName,Email,Password,CreateDate)
VALUES (@Name,@UserName,@Email,@Password,GETDATE())

SELECT SCOPE_IDENTITY() AS [Id]
END
ELSE
BEGIN

Select Id From Users where UserName = @UserName

END

Jeg vil mene, at det vil være mere korrekt, at lave et uniqueindex på UserName.

Så skal du ikke selv teste det, og det er sikkert også hurtigere.

Kan så ikke lige huske, hvordan du så fanger fejlen efterfølgende, men der kan arne_v sikkert hjælpe lidt :)

Jeg er helt enig om at et unikt index paa username er bedre.

Koden i #4 kan have samtidigheds problemer medmindre der bruges transaktioner og et hoejt transaction isolation level.

Fange fejlen ved insert af duplikat kan haandteres paa forskellig vis:
- i applikationen som faar en fejl ved kald af SP
- i SP (fra SQLServer 2005 kan man nemlig bruge TRY CATCH i en SP!)

og et svar fra mig

Jeg mente bare at der fandtes en anden løsning end Try-Catch, der jo koster lidt på performance. Men jeg husker måske forkert?

Maaske er der en anden loesning, men jeg kender den ikke.

Jeg ville ikke bekymre mig over over performance af try catch for bruger oprettelse.

Lad os sige at det koster 1 ms mere CPU tid end alternativet. Og lad os sige at sitet faar 100000 nye bruger om aaret. Og lad os sige at alle fejler en gang. Det vil braende ca. 0.25 CPU sekund mere af i doegnet.

Der er uden tvivl andre ting som vil give mere at optimer.