哪位大侠批评一下这个存储过程，有哪些缺点！是高手写的还是三流水平写的?

Question

其他的看不出来，至少SQL语句是高手！！为什么
set nocount off

Answer 1

高手！！1,判断过程名,好风格
2,ISNULL替换函数的使用
3,set nocount off 过程不显示
4,程序风格不象初手写的

Answer 2

SQL语句非常流畅简洁，不过我觉得有两句话修改一下写法看起来会更清晰更规范：,rating = (select isNull(sum(rating),0) from critiques where workId=b.WorkId) 
改成：
,(select isNull(sum(rating),0) from critiques where workId=b.WorkId)AS rating 同理,workNum = (select isNull(count(workId),0) from works where userId=a.UserId)
改成
(select isNull(count(workId),0) from works where userId=a.UserId)
AS workNum

Answer 3

这个sp写得比较清晰，比较规范。
肯定不是新手所为，作者应该至少有１年的功底。
如果非要找毛病的话，那幺要把：
if exists (select * from sysobjects where id = object_id('upWorksGetList')) 
    drop procedure upWorksGetList
这句改成
if exists (select * from sysobjects where id = object_id('upWorksGetList') and xtype = 'P') 
    drop procedure upWorksGetList但是因为这个sp的商务逻辑实在太简单，因此也无法看出作者是否高手。

Answer 4

程序写的很规范！
不过平时语句都是一气下来，也没有机会整的这么工整哦！除非是应聘的时候！
set nocount off
是显示结果
嘿嘿！是有一定熟练程度的！where a.UserId = isNull(@UserId,a.UserId) and  
      b.albumId = isNull(@AlbumId,b.albumId)                       
这个地方处理不错！不过COUNT（），SUM（）还会返回NULL？

Answer 5

这个存储过程用了交互式的字查询，这样在纪录多的情况下会带来速度上的瓶颈，最好是把汇总查询的语句放到FROM后面，用关联得到相应的结果。

Answer 6

查询语句可以这样改
Select  
(b.workId)
,(a.userId )
,(a.lastName)
,(a.firstName)
,C.workNum
,(b.albumId) 
,(b.[fileName])
,(b.title) 
,ISNULL(B.rating,0)  
,(b.res)from  Users a  INNER JOIN 
         (select userId, count(workId) AS workNum  from works GROUP BY userId) C 
      on a.UserId = C.UserId
      Left OUTER Join       
         (select A.UserId,A.albumId,A.[fileName],A.b.title,A.res,A.workId,sum(B.rating) AS rating 
          from Works A INNER JOIN critiques B ON A.WorkID=B.WorkID 
          GROUP BY A.UserId,A.albumId,A.[fileName],A.b.title,A.res,A.workId) B 
      on a.UserId = b.UserId AND b.albumId = isNull(@AlbumId,b.albumId)
where a.UserId = isNull(@UserId,a.UserId)

Answer 7

你可以把查询的语句这样改一下
Select b.workId,a.userId ,a.lastName,a.firstName,C.workNum,b.albumId 
,b.[fileName],b.title,ISNULL(B.rating,0) AS rating 
,b.resfrom  Users a  INNER JOIN 
         (select userId, count(workId) AS workNum  from works GROUP BY userId) C 
      on a.UserId = C.UserId
      Left OUTER Join       
         (select A.UserId,A.albumId,A.[fileName],A.b.title,A.res,A.workId,sum(B.rating) AS rating 
          from Works A INNER JOIN critiques B ON A.WorkID=B.WorkID 
          GROUP BY A.UserId,A.albumId,A.[fileName],A.b.title,A.res,A.workId) B 
      on a.UserId = b.UserId AND b.albumId = isNull(@AlbumId,b.albumId)where a.UserId = isNull(@UserId,a.UserId)

Answer 8

他在前面求和的语句中加入的查询的条件，而他的条件是关系到下面的表的条件，这样就会造成SQL Server在做查询计划的时候出现循环的情况，就是说每当下面的语句查询出一条记录时，就要相应的对works 和 critiques 表进行一次统计，想一想，这样做的开销有多大？上面我写的语句还是罗索 现在更正Select b.workId,a.userId ,a.lastName,a.firstName,
       ISNULL(B.workNum,0) AS workNum,
       b.albumId ,b.[fileName],b.title,
       ISNULL(B.rating,0) AS rating ,b.resfrom  Users a  Left OUTER Join       
     (select A.UserId,A.albumId,A.[fileName],
          A.b.title,A.res,A.workId,
          sum(ISNULL(B.rating,0)) AS rating ,
          count(workId) AS workNum 
          from Works A LEFT OUTER JOIN critiques B ON A.WorkID=B.WorkID 
          GROUP BY A.UserId,A.albumId,A.[fileName],
                   A.b.title,A.res,A.workId) B 
      on a.UserId = b.UserId AND b.albumId = isNull(@AlbumId,b.albumId)

Answer 9

不过这位写存储过程的先生对SQL的语句运用的很灵活，只是对SQL内部的机制不太了解，应该算是写SQL语句的高手，但不是高明的DBA。嘻，我这样的评论，千万不要生气，我是就事论事，请原谅。

Answer 10

另外，如果你在Works 表的USERID 和 critiques 表的workId 有索引
这样写实最快的
Select b.workId,a.userId ,a.lastName,a.firstName,
       ISNULL(C.workNum,0) AS workNum,
       b.albumId ,b.[fileName],b.title,
       ISNULL(C.rating,0) AS rating ,b.resfrom  Users a  Left OUTER Join  
      Works B ON A.USERID=B.USERID  AND b.albumId = isNull(@AlbumId,b.albumId)  
      LEFT OUTER JOIN 
     (select A.UserId, 
          sum(ISNULL(B.rating,0)) AS rating ,
          count(workId) AS workNum 
          from Works A LEFT OUTER JOIN critiques B ON A.WorkID=B.WorkID 
          GROUP BY A.UserId,B.workId) C 
      on a.UserId = C.UserId where a.UserId = isNull(@UserId,a.UserId)

Answer 11

gzhughie(hughie)写得不错，但是业务逻辑不对!1) workNum 要求是用户作品总数；
2）Rating  要求是作品的评分总数；