Stored procedure - review

Hvad laver RETURN(0) ?

Er det ikke den som sender Username retur?

Forslag til rettelser modtages meget gerne. Er voldsomt nærtagene, men skal nok give points ;-)

normal vil det være
CREATE PROCEDURE sp_do_userinfo_by_email
@BO_Email1  varchar(65)
AS
BEGIN
    SELECT Username FROM TUser WHERE BO_Email1 = @BO_Email1 order by ID desc
END

GO


intet return på

Husk en e-mail kan være på optil 255 chars

Takker for svar.
Er der mere man kan forbedre. Lig gerne svar ved forslag. Så kan jeg lige give points om 5 min.

Ikke umiddelbart hvad jeg kan se.
Måske ldanielsen har nogle foreslag.

Hvis du kun skal have et username ud, så ville jeg lave den med en outputparameter:
CREATE PROCEDURE sp_do_userinfo_by_email
@BO_Email1  varchar(255),
@UserName varchar(100) output
AS
BEGIN
    SELECT @UserName=Username FROM TUser WHERE BO_Email1 = @BO_Email1
END

GO

dr_chaos har jo vist de to måder at returnere fra en sp, så jeg har ikke mere at tilføje.

Men det kunne måske være en ide at lave det i en User defined function i stedet. Der sker intet med data, den laver blot et opslag og returnerer en værdi. Præcis det en function kan.

Men det kommer jo an på hvor og hvordan du bruger den.

Hvis du vi have begge dele kan du lade sp'en kal de din function.

Tak for ekstra hjælp til ldanielsen

Lige et par kommentarer.

Sp'ere bør ikke prefix'es med sp_
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/createdb/cm_8_des_01_116b.asp

Vil forøvrigt mene, at BEGIN END kan slettes.

Er vistnok best practice at bruge "fully qualified name" i stil med
SELECT @UserName = dbo.TUser.Username
FROM  dbo.TUser
WHERE  dbo.TUser.BO_Email1 = @BO_Email1

Tak for forklaring og link

Du han hente et "Best practice" analyse-værktøj her:
http://www.microsoft.com/downloads/details.aspx?displayla%20ng=en&familyid=B352EB1F-D3CA-44EE-893E-9E07339C1F22&displaylang=en