web-dev-qa-db-ja.com

Nullable型でConvert.ChangeType()が失敗する

文字列をオブジェクトプロパティ値に変換したいのですが、その名前を文字列として持っています。私はこれを次のようにしようとしています:

string modelProperty = "Some Property Name";
string value = "SomeValue";
var property = entity.GetType().GetProperty(modelProperty);
if (property != null) {
    property.SetValue(entity, 
        Convert.ChangeType(value, property.PropertyType), null);
}

問題は、これが失敗し、プロパティタイプがnull入力可能なタイプである場合に無効なキャスト例外をスローすることです。これは、変換できない値の場合ではありません-手動でこれを実行すると動作します(例:DateTime? d = Convert.ToDateTime(value);)似たような質問をいくつか見ましたが、まだ動作しません。

273
iboeno

テストされていませんが、おそらく次のようなものが動作します:

string modelProperty = "Some Property Name";
string value = "Some Value";

var property = entity.GetType().GetProperty(modelProperty);
if (property != null)
{
    Type t = Nullable.GetUnderlyingType(property.PropertyType) ?? property.PropertyType;

    object safeValue = (value == null) ? null : Convert.ChangeType(value, t);

    property.SetValue(entity, safeValue, null);
}
377
LukeH

あなたはそれを行うために基礎となるタイプを取得する必要があります...

これを試して、ジェネリックで正常に使用しました:

//Coalesce to get actual property type...
Type t = property.PropertyType();
t = Nullable.GetUnderlyingType(t) ?? t;

//Coalesce to set the safe value using default(t) or the safe type.
safeValue = value == null ? default(t) : Convert.ChangeType(value, t);

私はコード内の多くの場所でそれを使用しています。1つの例は、タイプセーフな方法でデータベース値を変換するために使用するヘルパーメソッドです。

public static T GetValue<T>(this IDataReader dr, string fieldName)
{
    object value = dr[fieldName];

    Type t = typeof(T);
    t = Nullable.GetUnderlyingType(t) ?? t;

    return (value == null || DBNull.Value.Equals(value)) ? 
        default(T) : (T)Convert.ChangeType(value, t);
}

次を使用して呼び出されます:

string field1 = dr.GetValue<string>("field1");
int? field2 = dr.GetValue<int?>("field2");
DateTime field3 = dr.GetValue<DateTime>("field3");

これを含む一連のブログ投稿を http://www.endswithsaurus.com/2010_07_01_archive.html に書きました(下にスクロールして @ JohnMacintyre 実際にバグを発見元のコードでは、今と同じ道をたどりました)。列挙型の変換も含む投稿以降、いくつかの小さな変更があります。そのため、プロパティがEnumの場合でも、同じメソッド呼び出しを使用できます。列挙型を確認するために行を追加するだけで、次のようなものを使用してレースに参加できます。

if (t.IsEnum)
    return (T)Enum.Parse(t, value);

通常、エラーチェックを行うか、Parseの代わりにTryParseを使用しますが、状況はわかります。

70
BenAlabaster

これは一例として少し長いですが、これは比較的堅牢なアプローチであり、キャストのタスクを未知の値から未知のタイプに分離します

同様のことを行うTryCastメソッドがあり、null許容型を考慮に入れています。

public static bool TryCast<T>(this object value, out T result)
{
    var type = typeof (T);

    // If the type is nullable and the result should be null, set a null value.
    if (type.IsNullable() && (value == null || value == DBNull.Value))
    {
        result = default(T);
        return true;
    }

    // Convert.ChangeType fails on Nullable<T> types.  We want to try to cast to the underlying type anyway.
    var underlyingType = Nullable.GetUnderlyingType(type) ?? type;

    try
    {
        // Just one Edge case you might want to handle.
        if (underlyingType == typeof(Guid))
        {
            if (value is string)
            {
                value = new Guid(value as string);
            }
            if (value is byte[])
            {
                value = new Guid(value as byte[]);
            }

            result = (T)Convert.ChangeType(value, underlyingType);
            return true;
        }

        result = (T)Convert.ChangeType(value, underlyingType);
        return true;
    }
    catch (Exception ex)
    {
        result = default(T);
        return false;
    }
}

もちろん、TryCastは型パラメーターを持つメソッドなので、動的に呼び出すには、MethodInfoを自分で作成する必要があります。

var constructedMethod = typeof (ObjectExtensions)
    .GetMethod("TryCast")
    .MakeGenericMethod(property.PropertyType);

次に、実際のプロパティ値を設定します。

public static void SetCastedValue<T>(this PropertyInfo property, T instance, object value)
{
    if (property.DeclaringType != typeof(T))
    {
        throw new ArgumentException("property's declaring type must be equal to typeof(T).");
    }

    var constructedMethod = typeof (ObjectExtensions)
        .GetMethod("TryCast")
        .MakeGenericMethod(property.PropertyType);

    object valueToSet = null;
    var parameters = new[] {value, null};
    var tryCastSucceeded = Convert.ToBoolean(constructedMethod.Invoke(null, parameters));
    if (tryCastSucceeded)
    {
        valueToSet = parameters[1];
    }

    if (!property.CanAssignValue(valueToSet))
    {
        return;
    }
    property.SetValue(instance, valueToSet, null);
}

そして、property.CanAssignValueを扱う拡張メソッド...

public static bool CanAssignValue(this PropertyInfo p, object value)
{
    return value == null ? p.IsNullable() : p.PropertyType.IsInstanceOfType(value);
}

public static bool IsNullable(this PropertyInfo p)
{
    return p.PropertyType.IsNullable();
}

public static bool IsNullable(this Type t)
{
    return !t.IsValueType || Nullable.GetUnderlyingType(t) != null;
}
8
Eric Burcham

同様のニーズがありましたが、LukeHからの答えがその方向を示してくれました。簡単にするために、この汎用関数を思いつきました。

    public static Tout CopyValue<Tin, Tout>(Tin from, Tout toPrototype)
    {
        Type underlyingT = Nullable.GetUnderlyingType(typeof(Tout));
        if (underlyingT == null)
        { return (Tout)Convert.ChangeType(from, typeof(Tout)); }
        else
        { return (Tout)Convert.ChangeType(from, underlyingT); }
    }

使用方法は次のとおりです。

        NotNullableDateProperty = CopyValue(NullableDateProperty, NotNullableDateProperty);

2番目のパラメーターは、戻り値をキャストする方法を関数に示すためのプロトタイプとして使用されているため、実際には宛先プロパティである必要はありません。次のようなこともできるという意味です:

        DateTime? source = new DateTime(2015, 1, 1);
        var dest = CopyValue(source, (string)null);

プロパティではoutを使用できないため、outを使用する代わりにこの方法で実行しました。そのままで、プロパティと変数を使用できます。必要に応じて、代わりに型を渡すオーバーロードを作成することもできます。

6
Steve In CO

このようにしてやった

public static List<T> Convert<T>(this ExcelWorksheet worksheet) where T : new()
    {
        var result = new List<T>();
        int colCount = worksheet.Dimension.End.Column;  //get Column Count
        int rowCount = worksheet.Dimension.End.Row;

        for (int row = 2; row <= rowCount; row++)
        {
            var obj = new T();
            for (int col = 1; col <= colCount; col++)
            {

                var value = worksheet.Cells[row, col].Value?.ToString();
                PropertyInfo propertyInfo = obj.GetType().GetProperty(worksheet.Cells[1, col].Text);
                propertyInfo.SetValue(obj, Convert.ChangeType(value, Nullable.GetUnderlyingType(propertyInfo.PropertyType) ?? propertyInfo.PropertyType), null);

            }
            result.Add(obj);
        }

        return result;
    }
0
AnishJain87

ありがとう@LukeH
少し変更しました:

public static object convertToPropType(PropertyInfo property, object value)
{
    object cstVal = null;
    if (property != null)
    {
        Type propType = Nullable.GetUnderlyingType(property.PropertyType);
        bool isNullable = (propType != null);
        if (!isNullable) { propType = property.PropertyType; }
        bool canAttrib = (value != null || isNullable);
        if (!canAttrib) { throw new Exception("Cant attrib null on non nullable. "); }
        cstVal = (value == null || Convert.IsDBNull(value)) ? null : Convert.ChangeType(value, propType);
    }
    return cstVal;
}
0
hs586sd46s