NullableタイプでConvert.ChangeType()が失敗する


301

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

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);)同様の質問がいくつかありますが、機能しません。


1
ペタポコ4.0.3でExecuteScalar <int?>を使用していますが、同じ理由で失敗します:554行で(T)Convert.ChangeType(val、typeof(T))を返します
Larry

回答:


409

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

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);
}

12
自分でそのコードを必要としていただけです。Nullable.GetUnderlyingTypeをありがとう!それを必要とするプロジェクトのために貧乏人のModelBinderを構築したとき、私はたくさん助けてくれました。私はあなたにビールを借りています!
Maxime Rouiller

3
たぶん代わりに(value == null) ? null使用(value == null) ? default(t)
スレッドスター2017年

uniqueidentifierが文字列に対応していないようです。
AndersLindén2018年

safeValueへの再割り当てだけではなく、変数を作成する特別な理由はありますvalueか?
コロラド

@threadster 'Type'型の変数ではデフォルトの演算子を使用できません。stackoverflow.com/questions/325426/…を
andy250

75

それを行うには、基になる型を取得する必要があります...

これを試してください、私はジェネリックでうまく使用しました:

//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が実際に私の元のコードにあるバグを見つけました。今)。列挙型の変換も含まれているので、いくつかの小さな変更があります。そのため、プロパティが列挙型の場合でも、同じメソッド呼び出しを使用できます。列挙型をチェックする行を追加するだけで、次のようなものを使用してレースに出かけます。

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

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


ありがとうございます。まだ足りない点や理解できない点があります。プロパティ値を設定しようとしていますが、それがその基になるタイプのオブジェクトを取得するのはなぜですか?私のコードからあなたのような拡張メソッドにどのように移行するかもわかりません。value.Helper <Int32?>()のようなことをするために型がどうなるかわかりません。
iboeno 2010

@iboeno-申し訳ありませんが、会議に参加できなかったので、ドットをつなぐ手助けができませんでした。あなたが解決策を手に入れてよかったです。
BenAlabaster、2010

9

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

同様の処理を行い、null許容型を考慮に入れるTryCastメソッドがあります。

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;
}

6

私にも同様のニーズがあり、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を使用する代わりに、この方法を使用しました。そのまま、プロパティと変数を使用できます。必要に応じて、代わりに型を渡すオーバーロードを作成することもできます。


0

ありがとう@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

こうやってやった

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;
    }
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.