优化/简化游标sql

Question

我有下面的代码，它运行得很好，只需要几秒钟来计算答案 - 我想知道是否有更快/更简洁的方法来编写这段代码 - 如果是这样，我在做什么错误的？

谢谢

select case when
    (select LSCCert from matterdatadef where ptmatter=$Matter$) is not null then
    (
        (select case when
            (SELECT top 1 dbo.matterdatadef.ptmatter 
             From dbo.workinprogress, dbo.MatterDataDef 
             where  ptclient=(
                 select top 1 dbo.workinprogress.ptclient 
                 from dbo.workinprogress 
                 where dbo.workinprogress.ptmatter = $matter$)
               and dbo.matterdatadef.LSCCert=(
                 select top 1 dbo.matterdatadef.LSCCert 
                 from dbo.matterdatadef 
                 where dbo.matterdatadef.ptmatter = $matter$)
             )=ptMatter then (
                 SELECT isnull((DateAdd(mm, 6, (
                         select top 1 Date 
                         from OfficeClientLedger 
                         where (pttrans=3) 
                             and ptmatter=$matter$ 
                         order by date desc))), 
                     (DateAdd(mm, 3, (
                         SELECT DateAdd 
                         FROM LAMatter 
                         WHERE  ptMatter = $Matter$)))
             )
        )
        end 
        from lamatter 
        where ptmatter=$matter$)
    )
    end

Answer 1

看起来您的 sql 是从报告工具生成的。问题是您正在对表lamatter的每一行执行SELECT top 1 dbo.matterdatadef.ptmatter...查询。进一步减慢执行速度的是，在该查询中，您将重新计算 ptclient 和 LSCCert 的比较值 - 这些值在执行期间不会更改。

最好使用正确的联接并通过避免相关子查询（引用联接表中的值并且必须对该表的每一行执行的查询）来设计查询以仅执行每个部分一次。计算值是可以的，只要它们只计算一次 - 即从最终的 where 子句中计算。

下面是一个演示相关子查询的简单示例：

错误的 sql：

select a, b from table1 
where a = (select c from table2 where d = b)

这里对每一行运行子选择，这会很慢，尤其是在 table2(d) 上没有索引的情况下

更好的 sql：

select a, b from table1, table2
where a = c and d = a

这里数据库最多会扫描每个表一次，速度会很快