2013-05-10 97 views
-2

我在这里有一个查询来更新每个代表最近的库存清单类型的表。但是,我得到它的唯一方法是使用游标,这真的会影响性能。以下是我正在使用的表格声明和查询。我还能做些什么来更快地达到预期效果?如何提高查询性能?

表:

create table #inv (
Rep_LName nvarchar (50), 
Rep_FName nvarchar (50), 
Rep_ID nvarchar (50), 
Rep_Email nvarchar (100), 
Rep_Status nvarchar (50), 
Rep_BU nvarchar (50), 
Sales_Force nvarchar (50), 
Territory nvarchar (50), 
Sample_Eligibility nvarchar (50), 
DM_Name nvarchar (100), 
Phys_Inv_Date datetime, 
Last_Reconciled datetime, 
Inv_Type nvarchar(50), 
Days_Since_Last_inv int) 

查询:

declare Inventory_type cursor 
for select rep_ID, inventory_type 
from inv_header 
where rep_id in (select rep_id from #inv) 
order by call_date desc 
declare @rep_ID nvarchar(50) 
declare @inventory_type nvarchar(50) 
declare @ls_Sql as nvarchar(max) 
declare @param as nvarchar(max) 
select @ls_Sql='' 
select @param='' 
-- open cursor 
open Inventory_type 

fetch next from Inventory_type 
into @rep_ID, @inventory_type 
while (@@fetch_status = 0) 
begin 
    --use parameterized dynamic sql 
    SET @param='@rep_ID nvarchar(50),@inventory_type nvarchar(50)' 
    SET @ls_Sql='update #inv set Inv_Type = @inventory_type 
WHERE rep_id = @rep_id AND Inv_Type IS NULL' 

    --pass parameter to dynamic query 
    exec sp_executesql @ls_Sql,@param,@rep_ID,@Inventory_type 

    fetch next from Inventory_type 
    into @rep_ID, @Inventory_type 
end 

close Inventory_type 
deallocate Inventory_type 
+2

当然你可以做到这一点作为UPDATE语句? – Simon 2013-05-10 13:19:26

+2

RBAR警报! (通过激动的行排列)......... – granadaCoder 2013-05-10 13:21:25

+1

这里是一个学习的例子.....它展示了一个RBAR解决方案,然后将它重构为一个基于集合的解决方案。 http://granadacoder.wordpress.com/2008/07/24/cursors-setbased-and-scalar-udf/ – granadaCoder 2013-05-10 13:23:00

回答

4

不要使用游标;他们破坏查询性能。是的,他们出于某种原因出现在产品中,但他们是最后一招的工具,几乎总是可以通过基于集合的方法完成工作。一个简单的更新语句在这里可以很好地工作。

update i 
set i.Inv_Type = h.inventory_type 
from #inv i 
inner join inv_header h on i.rep_id = h.rep_id 
    and h.Call_date = 
    (select Max(Call_Date) 
    from inv_header i2 
    where i2.rep_id = i.rep_id) 
where i.inv_type is null 
+0

我的第一个想法。 – 2013-05-10 13:27:10

+3

这将不会得到最新呼叫日期inventory_type – 2013-05-10 13:32:53

+0

@CharlesBretana优秀点;编辑。 – 2013-05-10 13:33:59

0

了我的头顶部,不知道你的数据库中,这样的事情应该工作:

UPDATE #inv set Inv_Type = (SELECT inventory_type FROM inv_header 
    WHERE #inv.Rep_ID = inv_header.rep_id) 
    WHERE Inv_Type IS NULL 

你可以也可以将它作为连接来使用,可能会更有效,具体取决于您拥有的行数。

应该让你至少开始....

0

我认为他想拥有最新的赎回日的inv_type ..

update inv 
set inv.Inv_Type = ih.inventory_type 
from (
    select rep_ID, max(call_date) max_call_date 
    from inv_header 
    where rep_id in (select rep_id from #inv) 
    group by rep_ID) a 
inner join inv_header ih 
    on ih.rep_ID = a.rep_ID 
    and ih.call_date = a.max_call_date 
inner join #inv inv 
    on inv.rep_ID = ih.rep_id 
    and inv.Inv_Type is null 
1

试试这个:

update i set 
    Inv_Type = h.inventory_type 
    From #inv i join inv_header h 
    on h.rep_id = i.Rep_ID 
     And h.Call_date = 
      (Select Max(Call_Date) 
      From inv_header 
      Where rep_id = i.rep_id) 
    where i.Inv_Type Is Null