简化查询以用作 winform rdlc 报告中的数据集

Posted 2023-05-08

【中文标题】简化查询以用作 winform rdlc 报告中的数据集

【英文标题】：Simplify Query to be used as dataset in winform rdlc report

【发布时间】：2018-10-30 05:34:47

【问题描述】：

我正在尝试让此查询在我正在构建的 winform 应用程序中填充报告查看器（并将在填充要查看的报告之前将查询中的参数更改为在表单上选择的值）。但是over功能无法正常工作。用户将从单选框、下拉列表等中输入参数，单击搜索，reportviewer 将打开以便他们可以打印。查询非常丑陋，我已经研究了一段时间。这是我获得所需格式的结果集的唯一方法。

SQL QUERY 寻求帮助以缩短相同的结果

DECLARE @EndReport DATETIME, @StartReport DATETIME, @Location INT, @Department varchar(50) 
SET @EndReport = '10-15-2018' SET @StartReport = '10-15-2018' SET @Department = 'fb' SET @Location = 10

SELECT row_number() over (order by (ai.FirstName + ' ' + ai.LastName)) RowNum
    ,AssociateName = isnull(upper(ai.FirstName + ' ' + ai.LastName),'**' + cast(t.ID as varchar(30)) + '**')
    ,ID = t.ID
    ,Codes = (t.DeptCode + '-' +  t.OpCode)
    ,TimeSUM = cast(SUM(datediff(second,StartTime, FinishTime) /3600.0) as decimal (6,2))
    ,Units = SUM(Units)
    ,UPH = cast(isnull(sum(Units) / nullif(sum(datediff(minute,StartTime,FinishTime))*1.0,0),0.0)*60  as decimal(10,0))
    into temptable10
FROM TimeLogNEW t LEFT JOIN AssociateInfo ai
ON t.ID = ai.ID
JOIN GoalSetUp g
ON (t.DeptCode + '-' + t.OpCode) = (g.DeptCode + '-' + g.OpCode)
WHERE EventDate between @StartReport and @EndReport 
and t.Location = @Location and g.location= @Location and  ((t.DeptCode + t.OpCode) in (g.DeptCode + g.OpCode)) and t.DeptCode = @Department
GROUP BY t.DeptCode,t.OpCode,ai.FirstName,ai.LastName, t.ID

SELECT 
 [Associate Name] = AssociateName
,[Codes] = Codes 
,[TimeSUM] = TimeSUM
,[Units] = Units
,[UPH] = UPH
,[UPH Target] = Goal
,[Actual %] = CASE WHEN goal = 0 then '0%' 
        else convert(varchar,cast(100* (isnull(UPH,0)/nullif(Goal,0)) as decimal(10,0))) + '%' END   
FROM goalsetup g join temptable10 on g.DeptCode = left(codes,2)and g.opcode = RIGHT(codes,2) 
WHERE g.Location = @Location    
ORDER BY Codes, UPH Desc
drop table temptable10

SQL 结果集

Visual Studio 错误

---

添加 Visual Studio 屏幕截图。在下面回答后更新

【问题讨论】：

row_number() 列生成后好像没有使用，从查询中删除即可。

【参考方案1】：

我不知道这是否有效，因为我没有要测试的表。 （即，如果出现错误，您将不得不决定要做什么。）

row_number() 列没有明显的原因，所以我删除了它 Codes = (t.DeptCode + '-' + t.OpCode) 制作表格时，后跟： join temptable10 on g.DeptCode = left(codes,2)and g.opcode = RIGHT(codes,2) 这是低效且不必要的，只需保留两列

事实上，很多部门代码和操作代码串联在一起，这似乎是不必要的，只需使用 2 列而不需要额外的努力。

between 在用于日期范围时是一条狗，我强烈建议使用下面查询中的内容。这个由>= 和< (+1 day) 组成的日期范围结构适用于所有日期/时间数据类型

请在引用任何列时使用表别名，像我这样的读者无法知道其中一些列来自哪个表，这使得调试/维护变得更加困难。

建议查询：

DECLARE @EndReport datetime
      , @StartReport datetime
      , @Location int
      , @Department varchar(50)
SET @EndReport = '10-15-2018'
SET @StartReport = '10-15-2018'
SET @Department = 'fb'
SET @Location = 10

SELECT
    AssociateName = ISNULL(UPPER(ai.FirstName + ' ' + ai.LastName), '**' + CAST(t.ID AS varchar(30)) + '**')
  , ID =            t.ID
  , Codes =         (t.DeptCode + '-' + t.OpCode)
  , t.DeptCode
  , t.OpCode
  , TimeSUM =       CAST(SUM(DATEDIFF(SECOND, StartTime, FinishTime) / 3600.0) AS decimal(6, 2))
  , Units =         SUM(Units)
  , UPH =           CAST(ISNULL(SUM(Units) / NULLIF(SUM(DATEDIFF(MINUTE, StartTime, FinishTime)) * 1.0, 0), 0.0) * 60 AS decimal(10, 0)) 
INTO temptable10
FROM TimeLogNEW t
LEFT JOIN AssociateInfo ai
    ON t.ID = ai.ID
JOIN GoalSetUp g
    ON t.DeptCode = g.DeptCode  AND t.OpCode = g.OpCode
WHERE EventDate >= @StartReport AND EventDate < dateadd(day,1,@EndReport)
AND t.Location = @Location
AND g.location = @Location
AND t.DeptCode = @Department
GROUP BY
    t.DeptCode
  , t.OpCode
  , ai.FirstName
  , ai.LastName
  , t.ID

SELECT
    [Associate Name] =  t10.AssociateName
  , [Codes] =           t10.Codes
  , [TimeSUM] =         t10.TimeSUM
  , [Units] =           t10.Units
  , [UPH] =             t10.UPH
  , [UPH Target] =      g.Goal
  , [Actual %] =       
                CASE
                    WHEN g.goal = 0 THEN '0%'
                    ELSE CONVERT(varchar, CAST(100 * (ISNULL(t10.UPH, 0) / NULLIF(g.Goal, 0)) AS decimal(10, 0))) + '%'
                END
FROM goalsetup g
JOIN temptable10 t10
    ON g.DeptCode = t10.DeptCode
    AND g.opcode = t10.opcode
WHERE g.Location = @Location
ORDER BY
     t10.Codes
  ,  t10.UPH DESC

DROP TABLE temptable10

【讨论】：

你是对的，over 完全没有被使用，而且你的代码在 sql 中运行良好。现在，当我尝试将此 SP 设置为 Visual Studio 中的数据集时，我可以将 WinForm 参数传递给它并使用其数据集打开 reportviewer，但由于临时表，我无法设置它。有什么方法可以解决这个问题或在不需要临时表的情况下获得相同的结果集？

只需删除“进入 temptable10”（一旦你不创建它就不要丢弃它:-)

第二个查询正在获取我需要的数据并加入临时表。

对不起。将新的第一个查询放入“cte”with cte as ( ... ) 然后第二个查询可以使用该名称而不是临时表，例如with cte as ( ... ) select * from cte join x on 1=1 您不必使用名称“cte”，您可以随意命名它甚至是“temptable10”，尽管将来可能会令人困惑