通过反射设置值的奇怪问题

时间:2016-06-21 14:39:48

标签: c# .net reflection

我有这个辅助方法 SetProperty ,它通过反射设置对象的属性。下面是我使用该方法的两个场景。第一种方法 CreateInstance 工作正常,但第二种方法插入不起作用。

在第二种方法中,只要方法 SetProperty 返回,对象上设置的属性就会丢失。我通过visual studio调试了它。该对象的属性设置为最后的花括号。然后,当控件返回给调用者插入时,属性值将丢失。

设置属性的方法

[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static void SetProperty(object destination, string propertyName, object value)
{
    var type = destination.GetType();
    var property = type.GetProperty(propertyName);
    var convertedVal = Convert(value, property.PropertyType);

    property.SetValue(destination, convertedVal);
}

SetProperty方法在此方法中正常工作

public static T CreateInstance<T>(SqlDataReader row, IEnumerable<CLASS> columns)
{
    var type = typeof(T);
    var obj = Activator.CreateInstance(type, true);

    foreach (var column in columns)
    {
        SetProperty(obj, column.BackingPropertyName, column.Name);
    }

    return (T)obj;
}

SetProperty方法在此方法中不起作用

public T Insert<T>(T obj, string table = null)
{
    // CODE CHUNK
    using (var conn = new SqlConnection(this.ConnectionString))
    {
        conn.Open();

        using (var cmd = new SqlCommand(query.ToString(), conn))
        {
            // CODE CHUNK

            var autoGeneratedValue = cmd.ExecuteScalar();

            if (temp.AutoGeneratedColumn != null)
            {
                ReflectionHelper.SetProperty(
                    obj,
                    temp.AutoGeneratedColumn.BackingPropertyName,
                    autoGeneratedValue
                );
            }
        }
    }

    return obj;
}

编辑 - 添加控制台应用以启用重复

要复制创建新的控制台应用,请将此代码粘贴到Program.cs(或等效的)

using System;
using System.Runtime.CompilerServices;

namespace ConsoleApplication1
{
    public struct Person
    {
        public int ID { get; set; }
        public string FirstName { get; set; }
        public string LastName { get; set; }
        public string Gender { get; set; }
        public int Age { get; set; }
    }

    internal class Program
    {
        private static void Main(string[] args)
        {
            var p = new Person
            {
                ID = 93
            };

            var res = SetProperty<Person>(ref p, "Age", 34);
            Console.WriteLine(p.Age);
            Console.WriteLine(res.Age);
            Console.Read();
        }

        [MethodImpl(MethodImplOptions.AggressiveInlining)]
        public static T SetProperty<T>(ref T destination, string propertyName, object value)
        {
            var type = destination.GetType();
            var property = type.GetProperty(propertyName);
            var convertedVal = Convert(value, property.PropertyType);

            property.SetValue(destination, convertedVal);

            return (T)destination;
        }

        private static object Convert(object source, Type destinationType)
        {
            if (destinationType == null)
            {
                throw new ArgumentNullException("destinationType");
            }

            if (destinationType.IsGenericType &&
                destinationType.GetGenericTypeDefinition().Equals(typeof(Nullable<>)))
            {
                if (source == null)
                {
                    return null;
                }
                destinationType = Nullable.GetUnderlyingType(destinationType);
            }

            return System.Convert.ChangeType(source, destinationType);
        }
    }
}

1 个答案:

答案 0 :(得分:2)

从你的repro,我可以看出问题是什么。

这是由您的Person结构框组成的。

考虑SetProperty<T>()方法中的这行代码:

property.SetValue(destination, convertedVal);

SetValue()的第一个参数是object类型。这意味着当结构destination传递给它时,它被装箱并且在盒装副本上设置属性 - 保持原始结构不变。 (请注意,装箱只影响值类型,例如结构 - 如果Person是一个类,它就不会被装箱并且它会按预期工作。)

要解决此问题,您必须通过将Person结构分配给object来“手动”框,然后在该对象上设置该属性,最后将其分配回原始结构参考,像这样:

[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static T SetProperty<T>(ref T destination, string propertyName, object value)
{
    var type = destination.GetType();
    var property = type.GetProperty(propertyName);
    var convertedVal = Convert(value, property.PropertyType);

    object boxed = destination;

    property.SetValue(boxed, convertedVal);

    destination = (T) boxed;

    return (T)boxed;
}