BTW det finns så många tekniska brister i proc och datatyp som du använder. Det verkliga problemet är här.
- I din proc kan du sätta var datumvillkor i första CTE själv.
- Om möjligt kan du lagra extrahours i andra i tabellen. Det betyder att Extrahours är int eller bigint.like -1000 eller 1000.varchar kommer aldrig att lösa ditt problem. Det kommer att rädda dig från många konverteringar och därför snabbt.
- Att använda så många kolumner i grupp i sig är i sig fel tillvägagångssätt. Speciellt att använda varchar-kolumn i grupp efter. Du bör använda nyckelkolumn i grupp genom att sedan gå ihop igen med tabellen för att få andra kolumner i resultatuppsättningen.
med dina exempeldata får jag -29:-51:-30.0 istället för -31:50:46 .gör det så här,
DECLARE @t TABLE (ExtraHrs VARCHAR(20))
INSERT INTO @t
VALUES ('00:59:38')
,('-03:59:37')
,('-08:59:39')
,('-08:52:36')
,('-08:59:16');
WITH cte
AS (
SELECT ExtraHrs
,CASE
WHEN left(ExtraHrs, 1) = '-'
THEN - 1
ELSE 1
END AS multiply
,right(ExtraHrs, 8) AS timestring
,
--get hours in seconds:
DATEPART(HOUR, right(ExtraHrs, 8)) * 3600 AS h_in_s
,
--get minutes in seconds:
DATEPART(MINUTE, right(ExtraHrs, 8)) * 60 AS m_in_s
,
--get seconds:
DATEPART(SECOND, right(ExtraHrs, 8)) AS s
FROM @t
)
,CTE3
AS (
SELECT *
,c.h_in_s + c.m_in_s + c.s AddExtra
FROM cte c
)
,cte4
AS (
SELECT sum(AddExtra * multiply) mn
FROM cte3
)
,cte5
AS (
SELECT mn / 3600 hh
,(mn % 3600) / 60 mi
,(mn % 3600.0) % 60 ss
FROM cte4
)
SELECT CASE
WHEN hh < 0
THEN '-'
ELSE ''
END
,cast(hh AS VARCHAR) + ':' + cast(mi AS VARCHAR) + ':' + cast(ss AS VARCHAR)
FROM cte5