Solved

Conversion failed when converting from a character string to uniqueidentifier.

Posted on 2009-05-07

ASP.NET

1 solution

1,555 Views

Last Modified: 2012-05-06

I am trying to pass a parameter from ASP.NET to a SQL Server Stored procedure.

string strUserID = currentUser.ProviderUserKey.ToString();
Returns "ba275a68-9e77-4301-9d97-7cc7dfecda0d"

SqlCommand cmd = new SqlCommand("usp_addWatchList", conStockSelect);

cmd.Parameters.Add("@UserID", SqlDbType.VarChar).Value = strUserID;
cmd.ExecuteNonQuery();
Conversion failed when converting from a character string to uniqueidentifier.

---------
Stored procedure executes successfully from SQL Server Manager
ALTER PROCEDURE [dbo].[usp_addWatchList]
@SymbolID int,
@UserID varchar(50)

/*
exec usp_addWatchList
@SymbolID =329,
@UserID = 'BA275A68-9E77-4301-9D97-7CC7DFECDA0D'
*/

What am I missing?

Thanks,

Comment

Question by:Dovberman

[X]

Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

Help others & share knowledge
Earn cash & points
Learn & ask questions

Join The Community

5 Comments

LVL 12

Expert Comment

by:GuitarRich

ID: 243310932009-05-07

when adding the parameter it should be passed as SqlDbType.UniqueIdentifier
Like this:
cmd.Parameters.Add("@UserID", SqlDbType.UniqueIdentifier).Value = strUserID;

LVL 12

Expert Comment

by:GuitarRich

ID: 243311092009-05-07

sorry - don't think I read it properly - the other problem you could be having is it not putting the value into the parameter right. I always add parameters like this:
cmd.Parameters.AddWithValue("@UserID", strUserID);
I've had problems in the past the other way where the value was not actually stored in the parameter object properly.

LVL 12

Accepted Solution

by:GuitarRich

GuitarRich earned 250 total points

ID: 243311242009-05-07

one final thing - it could be in the stored proc when you compare the table field to @UserID - if the table field is a UniqueIdentifier that will fail. Change the parameter in the stored proc to a UniqueIdentifier and add the parameter as System.UniqueIdentifier
Theres a few things to look at anyway :)

Author Comment

by:Dovberman

ID: 243315022009-05-07

I set everything to uniqueidentifier.
Error: Failed to convert parameter value from a String to a Guid.

Also tried cmd.Parameters.AddWithValue("@UserID", strUserID);

Error:Conversion failed when converting from a character string to uniqueidentifier.

Author Comment

by:Dovberman

ID: 243316712009-05-07

I set everything to uniqueidentifier.

AND
Also tried cmd.Parameters.AddWithValue("@UserID", strUserID);
AND changed the sp from this:
INSERT INTO WatchList
(UserID,SymbolID,PickDatePrice,PickDate,MarketID)
VALUES('@UserID',@SymbolID,23.45,'2009-04-23',3)

To this:
INSERT INTO WatchList
(UserID,SymbolID,PickDatePrice,PickDate,MarketID)
VALUES(@UserID,@SymbolID,23.45,'2009-04-23',3)

Now it works.
Thanks

Featured Post

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Some ASP.NET Practical tips

Article by: Easwaran

In an ASP.NET application, I faced some technical problems. In this article, I list them out and show the solutions that I found. I hope it will be useful. Problem: After closing a pop-up window, the parent page should be refreshed automaticall…

ASP.NET

Minify and Concatenate Your Scripts and Stylesheets

Article by: kevp75

Today is the age of broadband. More and more people are going this route determined to experience the web and it’s multitude of services as quickly and painlessly as possible. Coupled with the move to broadband, people are experiencing the web via …

ASP.NET

Network monitoring: why having a policy is the best policy?

Video by: AdRem

Monitoring a network: why having a policy is the best policy? Michael Kulchisky, MCSE, MCSA, MCP, VTSP, VSP, CCSP outlines the enormous benefits of having a policy-based approach when monitoring medium and large networks. Software utilized in this v…