急：大家帮我看下这段存储过程应该怎么写才正确？

Question

ALTER PROCEDURE [dbo].[GetSortData]
@condtions varchar(200),
@tbName varchar(50),
@starttime varchar(20) 
AS
declare @str varchar(5000)
BEGIN
    SET NOCOUNT ON;
    SET ARITHABORT off;
    SET ANSI_WARNINGS off;    if object_id('dbo.sorttmp') is not null 
    drop table sorttmp 
    
    declare @p int
    declare @sql nvarchar(4000) 
 
    set @p = CharIndex(',',@condtions) 
    if @p=0
    begin
      set @condtions=@condtions
    end
    else
    begin
      set @condtions = substring (@condtions, 1,@p-1 )
    end    set @sql='select * from '+@tbName+' where '+@condtions+' and (CONVERT(varchar(100), coltime, 20) like ''%'+@starttime+'%'')'
    exec(@sql)
    if @@rowCount>0
    begin
      set @sql='select * into sorttmp from '+@tbName+' where '+@condtions+' and (CONVERT(varchar(100), coltime, 20) like ''%'+@starttime+'%'')'
      exec(@sql)
      alter table sorttmp add sortcon varchar(200)
      update sorttmp set sortcon='+@condtions+'
    end
    
    while @p>0
    begin
      set @condtions= substring (@condtions, @p+1,len(@condtions) )
      set @p = CharIndex(',',@condtions)
      if @p=0
      begin
        set @condtions=@condtions
      end
      else
      begin
        set @condtions = substring (@condtions, 1,@p-1 )
      end
      set @sql='select * from '+@tbName+' where '+@condtions+' and (CONVERT(varchar(100), coltime, 20) like ''%'+@starttime+'%'')'
      exec(@sql)
      if @@rowCount>0
      begin
        if object_id('dbo.tmp') is not null 
        drop table tmp
        set @sql='select * into tmp from '+@tbName+' where '+@condtions+' and (CONVERT(varchar(100), coltime, 20) like ''%'+@starttime+'%'')'
        exec(@sql)
        alter table tmp add sortcon varchar(200)
        update tmp set sortcon='+@condtions+'
        insert into sorttmp select * from tmp
      end
    end
    select * from sorttmp group by sortcon
END

Answer 1

为了让大家能更好的看清代码，我补充说明一下。
    @condtions 是条件字符串，就是WHERE后面跟的条件语句，而且我程序中传递过来的参数是个用’，‘分割的字符串，分割开的N个不同的条件。
    @tbName是数据表名的字符串。
    @starttime是时间字符串，因为名为@tbName的数据表里有不同年度月份的数据。
    存储过程大致流程是：
    用@p来表示分隔号’，‘在@condtions字符串中的位置，然后@condtions = substring (@condtions, 1,@p-1 )截取第一个逗号之前的condtion，然后对名为@tbName的数据表做查询，查询结果集保存到表sorttmp，再在sorttmp加一列sortcon，将condtion写入到sortcon，表示刚才这些查询结果是根据sortcon列中保存的字符串来做的。
    然后再用WHILE循环不断的截取字符串@condtions，每次将一个condtion截取出来做查询，然后再做上面的操作，只是将结果保存到表tmp中，然后再将TMP表中的数据INSERT到表sorttmp中。这样循环直到没有逗号为止，然后再做select * from sorttmp group by sortcon,得到想要的结果集。

Answer 2

set @p = CharIndex(',',@condtions) 
可以这样用吗？
不太懂耶！

Answer 3

    呵呵，问题已解决，原来是ALTER PROCEDURE，应该是CREATE PROCEDURE,因为我原来是用SQL2000，2000里新建PROCEDURE，系统自动出来语句是CREATE PROCEDURE [OWNER].[PROCEDURE NAME] AS，我习惯了，现在改到SQL2005，2005系统新建PROCEDURE开始是CREATE PROCEDURE，建好以后自动改为ALTER PROCEDURE，我不喜欢用2005系统自动给出的新建语句（因为一大堆啰嗦的语句），我就把自己其他的已经建立好的PROCEDURE开头复制过来了，忘记把ALTER改CREATE，一开始还以为是我登陆用户权限问题呢，弄了半天，原来是这个。