[英]SQL very ugly GROUP BY
Daylight savings calculation. 夏令时计算。 We do have a very ugly working model below.
我们在下面确实有一个非常丑陋的工作模型。
Have seen this - SQL - Ugly combination of GROUP BY and COALESCE 看到了这一点-SQL-GROUP BY和COALESCE的丑陋组合
If we can set variables.. This does not work - also, have removed the count(*) to illustrate the issue we are seeing.. 如果我们可以设置变量,则此方法不起作用-同样,已删除count(*)以说明我们所遇到的问题。
Declare @subtractor int;
select top 10 case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then
Set @subtractor = 5;
else
Set @subtractor = 6;
end
CAST(DATEADD(HH, -@subtractor,LOGGED_TIME) AS date) AS ForDate
, txtW=DATENAME(WEEKDAY, DATEADD(HH, -@subtractor,LOGGED_TIME))
, intW=DATEPART(WEEKDAY, DATEADD(HH, -@subtractor,LOGGED_TIME))
, intH=DATEPART(HOUR, DATEADD(HH, -@subtractor,LOGGED_TIME))
from SUPPORT_DATABASE.dbo.LOG (nolock)
This does work.. However, it is a very, very ugly GROUP BY.. Is there a way to clean this up? 确实可以。.但是,它是一个非常非常丑陋的GROUP BY ..有没有办法清理它?
select top 10
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then CAST(dateadd(hh,-5,LOGGED_TIME) AS date) -- DST
else CAST(dateadd(hh,-6,LOGGED_TIME) AS date) -- DST
end as ForDate,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATENAME(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATENAME(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as txtW,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATENAME(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATENAME(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as intW,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATEPART(HOUR,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATEPART(HOUR,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as intH
, COUNT(*) AS Totals
FROM SUPPORT_DATABASE.dbo.LOG WITH (nolock)
GROUP BY case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then CAST(dateadd(hh,-5,LOGGED_TIME) AS date) -- DST
else CAST(dateadd(hh,-6,LOGGED_TIME) AS date) -- DST
end,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATENAME(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATENAME(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end
,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATEPART(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATEPART(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATEPART(HOUR,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATEPART(HOUR,dateadd(hh,-6,LOGGED_TIME)) -- DST
end
You could wrap it in a cte, so you can just GROUP BY
the aliases: 您可以将其包装在cte中,这样就可以使用别名
GROUP BY
了:
;WITH cte AS ( select
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then CAST(dateadd(hh,-5,LOGGED_TIME) AS date) -- DST
else CAST(dateadd(hh,-6,LOGGED_TIME) AS date) -- DST
end as ForDate,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATENAME(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATENAME(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as txtW,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATENAME(WEEKDAY,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATENAME(WEEKDAY,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as intW,
case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then DATEPART(HOUR,dateadd(hh,-5,LOGGED_TIME)) -- DST
else DATEPART(HOUR,dateadd(hh,-6,LOGGED_TIME)) -- DST
end as intH
FROM SUPPORT_DATABASE.dbo.LOG WITH (nolock))
SELECT *,COUNT(*) CT
FROM cte
GROUP BY ForDate, txtW, intW, intH
That's a simple way to clean things up a little bit, sure it needs refactoring as well. 这是一种简单的清理方式,确保它也需要重构。
You could just create a UDF: 您可以创建一个UDF:
CREATE FUNCTION dbo.YourFunctionName (@DateTime DATETIME)
RETURNS DATE
AS
BEGIN
RETURN
(
SELECT case
when @DateTime between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(@DateTime) as varchar(4)) as datetime)))
end
then CAST(dateadd(hh,-5,@DateTime) AS date) -- DST
else CAST(dateadd(hh,-6,@DateTime) AS date) -- DST
end
);
END
Then your query is simply: 那么您的查询就是:
SELECT dbo.YourFunctionName(LOGGED_TIME) AS ForDate,
DATENAME(WEEKDAY, dbo.YourFunctionName(LOGGED_TIME)) AS txtW
....
GROUP BY dbo.YourFunctionName(LOGGED_TIME);
Or, if you will never re-use this function and don't wnat to create it just for this, you can move your logic to an APPLY, then reference your new column: 或者,如果您永远不会重复使用此功能并且不为此而创建,可以将逻辑移至APPLY,然后引用新列:
SELECT fd.ForDate,
DATENAME(WEEKDAY, fd.ForDate) AS txtW
FROM ...
CROSS APPLY
(
SELECT case
when LOGGED_TIME between
case DATEPART(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-4,CAST('03/08/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))--2am minus 6 hours for conversion from UST going to DST
else dateadd(hh,-4,DATEADD(d,15-datepart(dw,CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('03/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
and
case DATEPART(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime))
when 1 then dateadd(hh,-3,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)) --2am minus 5 hours for conversion from UST going from DST
else dateadd(hh,-3,DATEADD(d,8-datepart(dw,CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)),CAST('11/01/'+CAST (YEAR(LOGGED_TIME) as varchar(4)) as datetime)))
end
then CAST(dateadd(hh,-5,LOGGED_TIME) AS date) -- DST
else CAST(dateadd(hh,-6,LOGGED_TIME) AS date) -- DST
end as ForDate
) AS fd
GROUP BY fd.ForDate;
Or finally you could use a subquery and only group in the outer query. 或最后,您可以使用子查询,而仅在外部查询中使用组。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.