13

SQL Server を使用しています。@variable が null の場合、何らかの値に設定されます。

set nocount on

IF object_id('tempdb..##tmp') IS NOT NULL 
BEGIN
     DROP TABLE ##tmp
END
CREATE TABLE ##tmp (week varchar (25), users int, stamps int)

  declare @inter varchar(100)
  declare @qt_users int
  declare @qt_stamps int
  declare @comando varchar(5000)
  declare @start date = null
  declare @end date = null
  declare @week datetime = DATEADD(DAY, 6, @start)

  --if @start is null, set it to a value:

  if (null!=@start)
  begin
    set @start = '20130101'
  end
  if (null!=@end)
  begin
    set @end = GETDATE()
  end

  while @start < @end
  begin
    select @qt_users = COUNT(distinct id_user) 
      from stamps 
      where dt_synchronization between @start and @week

    select @qt_stamps = COUNT(id_stamp) 
    from stamps 
    where dt_synchronization between @start and @week

    set @inter =  convert(varchar(10),@start,105) + ' até ' + 
    convert(varchar(10),@week,105)

    set @comando = 'insert into ##tmp(week, users, stamps) values (''' + 
      @inter + ''','''+ 
      cast(@qt_users as varchar) + 
      ''',''' + cast(@qt_stamps as varchar) + ''')'
    exec (@comando)
    set @start = @week + 1
    set @week = dateadd(day, 6, @start)

  end

select week, users, stamps from ##tmp
4

1 に答える 1

25

代わりに IS NULL を使用してチェックします。

IF (@start IS NULL)
   SET @start = '20130101'

または、1 行で:

SET @start = ISNULL(@start, '20130101')

更新: また、 @week の設定が早すぎます:

declare @week datetime = DATEADD(DAY, 6, @start) -- @start is NULL

への変更：

declare @week datetime
-- IF checks here to set @start/@end if null...
SET @week = DATEADD(DAY, 6, @start)

余談ですが、パフォーマンスのためにループをセットベースのアプローチにリファクタリングする価値もあります。集計/数値表タイプのアプローチは、調査のオプションです。

于 2013-03-20T17:13:09.347 に答える