views:

3467

answers:

6
+4  Q: 

Nullable GUID

In my database, in one of the table I have a GUID column with allow nulls. I have a method with a Guid? parameter that inserts a new data row in the table. However when I say myNewRow.myGuidColumn = myGuid I get the following error: "Cannot implicitly convert type 'System.Guid?' to 'System.Guid'."

+6  A: 

The ADO.NET API has some problems when it comes to handling nullable value types (i.e. it simply doesn't work correctly). We've had no end of issues with it, and so have arrived at the conclusion that it's best to manually set the value to null, e.g.

myNewRow.myGuidColumn = myGuid == null ? (object)DBNull.Value : myGuid.Value

It's painful extra work that ADO.NET should handle, but it doesn't seem to do so reliably (even in 3.5 SP1). This at least works correctly.

We've also seen issues with passing nullable value types to SqlParameters where the generated SQL includes the keyword DEFAULT instead of NULL for the value so I'd recommend the same approach when building parameters.

Greg Beech
Type of conditional expression cannot be determined because there is no implicit conversion between 'System.DBNull' and 'System.Guid'
iulianchira
Ah, apologies. Fixed.
Greg Beech
You might consider using the null coalescing operator when retrieving a fallback value for a possible null object: "(object)myGuid ?? DBNull.Value"
Emperor XLII
@Emperor XLII - you can't do that because you want DBNull.Value when the value you have *is* null, not when it isn't.
Greg Beech
This is telling me Error: Cannot implicitly convert type 'object' to 'System.Guid?'. An explicit conversion exists (are you missing a cast?) -- but explicitly casting to Guid? doesn't work either.
Daniel Coffman
+3  A: 

OK; how is myGuidColumn defined, and how is myGuid defined?

If myGuid is Guid? and myGuidColumn is Guid, then the error is correct: you will need to use myGuid.Value, or (Guid)myGuid to get the value (which will throw if it is null), or perhaps myGuid.GetValueOrDefault() to return the zero guid if null.

If myGuid is Guid and myGuidColumn is Guid?, then it should work.

If myGuidColumn is object, you probably need DBNull.Value instead of the regular null.

Of course, if the column is truly nullable, you might simply want to ensure that it is Guid? in the C# code ;-p

Marc Gravell
+1  A: 

You can use a helper method:

public static class Ado {
    public static void SetParameterValue<T>( IDataParameter parameter, T? value ) where T : struct {
        if ( null == value ) { parameter.Value = DBNull.Value; }
        else { parameter.Value = value.Value; }
    }
    public static void SetParameterValue( IDataParameter parameter, string value ) {
        if ( null == value ) { parameter.Value = DBNull.Value; }
        else { parameter.Value = value; }
    }
}
TcKs
+2  A: 

or:

    internal static T CastTo<T>(object value)
    {
        return value != DBNull.Value ? (T)value : default(T);
    }
A: 

If you want to avoid working with nullable GUIDs in your c# code (personally, I often find it cumbersome to work with nullable types) you could somewhere early assign Guid.Empty to the .NET data which is null in the db. That way, you don't have to bother with all the .HasValue stuff and just check if myGuid != Guid.Empty instead.

Isak Savo
I allready use an empty GUID with a different "meaning" .
iulianchira
+1  A: 

same as Greg Beech's answer

myNewRow.myGuidColumn = (object)myGuid ?? DBNull.Value
cheeves