请看一下这个简单的SQL 语句..

[panxiaosen]

在一个商业游戏服务器端里有下面这样一个存储过程,
CREATE PROCEDURE LOAD_USER_DATA
@id char(21)
AS

DECLARE @nRet smallint
SET @nRet = 0

SELECT @nRet = COUNT(strUserId) FROM USERDATA WHERE strUserId = @id
IF @nRet = 0
RETURN 0

SELECT Nation, Race, Class, HairColor, Rank, Title, [Level], [Exp], Loyalty, Face, City FROM USERDATA WHERE strUserId = @id

RETURN 1

红色部分可不可以去掉呢?改成下面这样??

CREATE PROCEDURE LOAD_USER_DATA
@id char(21)
AS

SELECT Nation, Race, Class, HairColor, Rank, Title, [Level], [Exp], Loyalty, Face, City FROM USERDATA WHERE strUserId = @id

if @@ROWCOUNT=0
RETURN 0
ELSE
RETURN 1

[青锋-SS]

应该可以

[-狙击手-]

SELECT @nRet = COUNT(strUserId) FROM USERDATA WHERE strUserId = @id 
IF @nRet = 0 
RETURN 0 

-------------

可以写成：
IF NOT EXISTS (SELECT 1 FROM USERDATA WHERE strUserId = @id )
   return 0

[-狙击手-]

商业游戏服务器端考虑效率

[-狙击手-]

效率低，而且浪费服务器资源

[dobear_0922]

可以，，，

[panxiaosen]

[Quote=引用 4 楼 happyflystone 的回复:]
效率低，而且浪费服务器资源
[/Quote]

是说原来的代码比较浪费资源吗?

[panxiaosen]

如果按我那样的改的,两句SELECT变成一句,是不是会省点资源? 但是商业的游戏那样写应该也有他的道理吧?

[ma_yan_jun_78]

子木 :按你改的就没有判断了.
无枪狙击手 方法及分析都很正确

[pt1314917]

IF NOT EXISTS (SELECT 1 FROM USERDATA WHERE strUserId = @id )
   return 0

[zzr82528]

[Quote=引用 7 楼 panxiaosen 的回复:]
如果按我那样的改的,两句SELECT变成一句,是不是会省点资源? 但是商业的游戏那样写应该也有他的道理吧?
[/Quote]
在查不到的时候，改之后更浪费资源。
可以测试一下。
本来为空，查不到，却要查一遍。
从功能上说没什么问题，如果要效率，最好实际测试一下。

[ssc16864]

可以的。

[zheninchangjiang]

you should not use procedure,your server will do at least one select sql.
direct use select sql to fetch rows,and the client does the job.

[dawugui]

你那句是干嘛的没看明白.

不过之前的
SET @nRet = 0
这句到是多余的.