Gespeicherte Prozedur Abfrage löschen

Question

Ich habe gespeicherte Prozedur:

ALTER PROCEDURE [dbo].[k_ShoppingCart_DELETE] 

@cartGUID nvarchar 

AS 

DELETE FROM 
    [dbo].[k_ShoppingCart] 
WHERE 
    CartGUID = @cartGUID 

Als ich dies ausführen,

exec dbo.k_ShoppingCart_DELETE '32390b5b-a35a-4e32-8393-67d5629192f0' 

Ergebnis: 0 Zeile (n) betroffen.

Aber, wenn ich versuche, diese Abfrage:

Delete FROM k_ShoppingCart Where CartGUID = '32390b5b-a35a-4e32-8393-67d5629192f0' 

Ergebnis: 2 Zeilen betroffen.

Was ist daran falsch?

Answer 1

Wenn Sie darauf bestehen, mit NVARCHAR statt UNIQUE, müssen Sie die Größe angeben:

@cartGUID nvarchar(36) 

Ohne sie Ihre guids abgeschnitten werden (to 30 characters).

Sie können dieses Verhalten bestätigen diese modifizierte Version Ihrer Arbeits Abfrage ausgeführt wird:

DECLARE @cart nvarchar, @sizedcart nvarchar(36) 
SET @cart  = '32390b5b-a35a-4e32-8393-67d5629192f0' 
SET @sizedcart = '32390b5b-a35a-4e32-8393-67d5629192f0' 

-- works 
Delete FROM k_ShoppingCart Where CartGUID = '32390b5b-a35a-4e32-8393-67d5629192f0' 

-- will not work 
Delete FROM k_ShoppingCart Where CartGUID = @cart 

-- should work 
Delete FROM k_ShoppingCart Where CartGUID = @sizedcart 

ich mit @Marc Gravell zustimmen, obwohl, unique der Weg hier zu gehen.

Answer 2

Ihr gespeichertes SQL scheint ein bisschen seltsam, haben Sie es richtig kopiert?

Dieses sieht nicht wie es kompiliert ehrlich zu sein:

WHERE 
    CartGUID =@cartGUID 

in (Select 

Answer 3

Was ist mit, dass IN-Anweisung am Ende Ihrer gespeicherten proc? Es macht keinen Sinn. Einfach loswerden. Außerdem müssen Sie allen Tabellen in der Abfrage und im Sproc "dbo" voranstellen (wer weiß, vielleicht haben Sie in Ihrem Standardschema eine andere Version derselben Tabelle). Diese zwei Dinge zu tun sollte den Trick tun.

Answer 4

Sollte @cartGUID eine uniqueidentifier anstatt eine sein? Es ist effizienter, einen einzelnen varchar in eine GUID zu konvertieren und die GUIDs zu vergleichen, als alle GUIDs in varchar zu konvertieren, und hoffen, dass es das gleiche Format verwendet (andernfalls würde die Gleichheit trotzdem fehlschlagen).

Wie andere darauf hingewiesen haben, sieht die WHERE Klausel funky aus, aber mein Geld ist auf der Varchar-Konvertierung der Schuldige.

Answer 5

ich Ihnen die gleiche Tabelle ... unter mehreren Schemas nicht annehmen, haben

dbo.k_ShoppingCart 

vs. 

user1.k_ShoppingCart 

Die gespeicherte Prozedur wird mit „dbo“, während der Ad-hoc-Abfrage den aktuellen Benutzer Referenz verwendet.

Answer 6

Noch eine Sache, die mir seltsam erscheint: Wenn cartID eine GUID (und eine eindeutige Kennung) ist, warum wirkt sich Ihre Löschanweisung auf zwei Zeilen aus?

Versuchen Sie auch, eine Länge zur Definition von cartID hinzuzufügen, vielleicht führt eine funkige Standardlänge dazu, dass der Eingabeparameter gekürzt oder mit Leerzeichen oder etwas gefüllt wird.

Answer 7

Ich löste es.

. Lösung: ALTER PROCEDURE [dbo] [k_ShoppingCart_DELETE]

@cartGUID nvarchar (50)