Convert.ChangeType () falha em tipos anuláveis

301

Desejo converter uma string em um valor de propriedade do objeto, cujo nome eu tenho como string. Estou tentando fazer o seguinte:

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

O problema é que isso está falhando e gerando uma exceção de conversão inválida quando o tipo de propriedade é um tipo anulável. Este não é o caso dos valores que não podem ser convertidos - eles funcionarão se eu fizer isso manualmente (por exemplo DateTime? d = Convert.ToDateTime(value);) eu já vi algumas perguntas semelhantes, mas ainda não consigo fazê-lo funcionar.

iboeno
fonte
1
Estou usando ExecuteScalar <int?> Com PetaPoco 4.0.3 e falha pelo mesmo motivo: return (T) Convert.ChangeType (val, typeof (T)) na linha 554
Larry

Respostas:

409

Não testado, mas talvez algo assim funcione:

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);
}
LukeH
fonte
12
Só precisava daquele pedaço de código. Obrigado por Nullable.GetUnderlyingType! Me ajudou bastante quando construí o ModelBinder de um homem pobre para um projeto que precisava dele. Eu lhe devo uma cerveja!
Maxime Rouiller
3
Talvez em vez de (value == null) ? nullusar (value == null) ? default(t)?
threadster
Parece não funcionar para uniqueidentifier para string.
Anders Lindén
Existe algum motivo específico para criar a safeValuevariável em vez de apenas reatribuir a value?
Coloradocolby #
@ threadster Você não pode usar o operador padrão em uma variável do tipo 'Tipo'. Veja stackoverflow.com/questions/325426/…
andy250
75

Você precisa obter o tipo subjacente para fazer isso ...

Tente isso, eu usei com sucesso com genéricos:

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

Eu o uso em vários locais do meu código; um exemplo é um método auxiliar que eu uso para converter valores de banco de dados de maneira segura:

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

Chamado usando:

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

Escrevi uma série de postagens no blog, incluindo esta em http://www.endswithsaurus.com/2010_07_01_archive.html (Role para baixo até o Adendo, @JohnMacintyre realmente identificou o bug no meu código original que me levou pelo mesmo caminho que você ligado agora). Eu tenho algumas pequenas modificações desde essa postagem que incluem a conversão de tipos de enum também, portanto, se sua propriedade for um Enum, você ainda poderá usar a mesma chamada de método. Basta adicionar uma linha para verificar os tipos de enumeração e você estará pronto para as corridas usando algo como:

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

Normalmente, você tem alguma verificação de erro ou usa o TryParse em vez do Parse, mas você obtém a imagem.

BenAlabaster
fonte
Obrigado - ainda estou perdendo um passo ou não estou entendendo algo. Estou tentando definir um valor de propriedade, por que estou recebendo o objeto em que está no tipo subjacente? Também não sei como obter do meu código um método de extensão como o seu. Não saberei qual será o tipo para fazer algo como value.Helper <Int32?> ().
iboeno 20/08/10
@iboeno - Desculpe, estava fora de uma reunião, então não pude ajudá-lo a conectar os pontos. Ainda bem que você tem uma solução.
precisa saber é o seguinte
9

Isso é um pouco longo, por exemplo, mas é uma abordagem relativamente robusta e separa a tarefa de transmitir de valor desconhecido para tipo desconhecido.

Eu tenho um método TryCast que faz algo semelhante e leva em consideração tipos anuláveis.

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

É claro que o TryCast é um método com um parâmetro de tipo, portanto, para chamá-lo dinamicamente, você deve construir o MethodInfo:

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

Em seguida, para definir o valor real da propriedade:

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

E os métodos de extensão para lidar com 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;
}
bopapa_1979
fonte
6

Eu tinha uma necessidade semelhante, e a resposta de LukeH me apontou na direção. Eu vim com essa função genérica para facilitar.

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

O uso é assim:

        NotNullableDateProperty = CopyValue(NullableDateProperty, NotNullableDateProperty);

Observe que o segundo parâmetro é usado apenas como um protótipo para mostrar à função como converter o valor de retorno, portanto, na verdade, não precisa ser a propriedade de destino. Ou seja, você também pode fazer algo assim:

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

Eu fiz dessa maneira, em vez de usar uma saída porque você não pode usar com propriedades. Como é, ele pode trabalhar com propriedades e variáveis. Você também pode criar uma sobrecarga para passar o tipo, se quiser.

Steve In CO
fonte
0

Obrigado @LukeH
eu mudei um pouco:

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;
}
hs586sd46s
fonte
0

Eu fiz assim

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