我正在使用PHP的SQL Server驱动程序连接到SQL Server 2008 Express。现在,我试图用存储过程替换所有的SELECT,UPDATE和INSERT语句。这对于只包含SELECT语句的SP来说工作正常。但是现在我试图用update来做一个,并且不断收到错误消息“直接执行SQL;没有游标”。我可以使用相同的参数值从Management Studio中调用SP。执行更新时存储过程失败
任何想法?
干杯 亚历
编辑:这里有一个更新程序。有趣的是,该过程实际上执行得很好,并更新了数据。但它仍然返回一个错误,导致一个异常。
首先,失败的PHP代码:
if (! $this->Result = sqlsrv_query($this->Conn, $strQuery, $arrParameters, array("Scrollable"=>SQLSRV_CURSOR_STATIC)))
{
$this->sendErrorMail($strQuery, $arrParameters);
throw new Exception(4001);
}
SQL
USE [testsite]
GO
/****** Object: StoredProcedure [dbo].[Items_countDownload] Script Date: 09/09/2010 18:03:28 ******/
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
-- =============================================
-- Author: Alexis Hildebrandt
-- Create date: 2010-09-09
-- Description: Increases the download count by 1
-- =============================================
ALTER PROCEDURE [dbo].[Items_countDownload]
@Id INT
AS
BEGIN
-- SET NOCOUNT ON added to prevent extra result sets from
-- interfering with SELECT statements.
SET NOCOUNT ON;
DECLARE @DownloadCount INT = 0, @MaxCount INT = 0, @Id2 INT = 0
DECLARE itemCursor CURSOR SCROLL
FOR
SELECT Id, Downloads
FROM Items
WHERE Id = @Id
OR SKU IN
(
SELECT SKU FROM Items WHERE Id = @Id
)
FOR UPDATE OF Downloads
OPEN itemCursor
FETCH NEXT FROM itemCursor
INTO @Id, @DownloadCount;
-- Find the largest Download count across all versions of the item
WHILE @@FETCH_STATUS = 0
BEGIN
IF @MaxCount < @DownloadCount
SET @MaxCount = @DownloadCount;
FETCH NEXT FROM itemCursor
INTO @Id, @DownloadCount;
END
-- Increase the download count by one for all versions
FETCH FIRST FROM itemCursor
INTO @Id, @DownloadCount;
WHILE @@FETCH_STATUS = 0
BEGIN
UPDATE Items
SET Downloads = @MaxCount + 1
WHERE CURRENT OF itemCursor
FETCH NEXT FROM itemCursor
INTO @Id, @DownloadCount;
END
CLOSE itemCursor;
DEALLOCATE itemCursor;
END
如果您可以显示您的更新存储过程代码,那么它可能会帮助我们确定问题 – 2010-09-09 15:35:08
您能用文字描述这段代码的目的是什么?它看起来像是通过直接的'update'语句而不是使用游标可以更高效地实现的。 – 2010-09-10 12:01:16
嗨马丁,我知道它被炸毁了。但是它需要处理两件事情:计算项目下载量并确保计数器对同一下载的不同版本同步。 – Alexis 2010-09-10 13:24:41