2

我想创建一个通用扩展方法，如果它们的值等于它们的默认值，它将为 Object 或 Struct 设置一个值。

所以我有以下代码：

public static void setIfNull<T>(this T i_ObjectToUpdate, T i_DefaultValue)
{
    if (EqualityComparer<T>.Default.Equals(i_ObjectToUpdate, default(T)))
    {
        i_ObjectToUpdate = i_DefaultValue;
    }
}

这是一个调用示例：

public OrganizationalUnit CreateOrganizationalUnit(OrganizationalUnit i_UnitToCreate)
{
    i_UnitToCreate.EntityCreationDate.setIfNull(DateTime.Now); //Here is a call
    i_UnitToCreate.EntityLastUpdateDate.setIfNull(DateTime.Now); //And another one
    m_Context.DomainEntities.Add(i_UnitToCreate);
    return i_UnitToCreate;
}

我不知道它是否与它有关，但我使用实体框架和 MVC。

在调试器中实际发生的事情我看到扩展方法中的行i_ObjectToUpdate = i_DefaultValue;正在工作并且值发生了变化，但是当调试器退出扩展方法时，我看到 的值i_UnitToCreate.EntityCreationDate保持不变。

任何想法出了什么问题？

4

3 回答 3

2

您的代码中有两个引用。一种是i_UnitToCreate.EntityCreationDate指向内存中的某个地址。其他i_ObjectToUpdate在您的扩展方法中，它最初也指向该地址（在传递对您的方法的引用时，您正在创建地址的副本i_UnitToCreate.EntityCreationDate）。稍后您将第二个引用更改为指向内存中其他对象的点，但这不会更改第一个引用，因为它们是独立的。

解决方法

public static void SetIfDefault<T, TProperty>(this T arg,
    Expression<Func<T, TProperty>> propertySelector, TProperty value)
{
    TProperty currentValue = propertySelector.Compile()(arg);
    EqualityComparer<TProperty> comparer = EqualityComparer<TProperty>.Default;
    if (!comparer.Equals(currentValue, default(TProperty)))
        return;
    PropertyInfo property = (PropertyInfo)((MemberExpression)propertySelector.Body).Member;
    property.SetValue(arg, value);
}

您可以使用表达式将属性选择器（而不是属性值）传递给扩展方法。从编译的表达式中检索值。如果它是默认值，则通过反射（您可以PropertyInfo通过转换成员表达式轻松获取）设置新值。用法：

i_UnitToCreate.SetIfDefault(x => x.EntityCreationDate, DateTime.Now);
i_UnitToCreate.SetIfDefault(x => x.EntityLastUpdateDate, DateTime.Now);

PS关于我的第一个答案有一个评论 -T当谈到复制引用值时，我认为泛型类型是一种引用类型。使用值类型（如 DateTime）复制整个对象。它不会改变结果（您不能分配新值），但需要提及。

于 2012-11-24T09:43:57.807 回答
0

问题是，在您的扩展方法中，i_ObjectToUpdate是对位于堆中的某个对象的本地引用（指针）。当你设置：

i_ObjectToUpdate = i_DefaultValue

您实际上并没有更改堆中的对象，而是将本地引用指向指向i_ObjectToUpdate的对象i_DefaultValue，但仅在扩展方法的范围内。调用上下文中的原始引用：

i_UnitToCreate.EntityCreationDate
i_UnitToCreate.EntityLastUpdateDate

维持不变。

要解决此问题，您必须实际将指针传递给指向扩展方法的指针。这是通过添加ref关键字来完成的：

    public static void setIfNull<T>(this ref T i_ObjectToUpdate, T i_DefaultValue)
    {
        if (EqualityComparer<T>.Default.Equals(i_ObjectToUpdate, default(T)))
        {
            i_ObjectToUpdate = i_DefaultValue;
        }
    }

不幸的是，扩展方法不支持 ref 关键字。您的替代方法是将该方法用作标准静态方法：

    public static void setIfNull<T>(ref T i_ObjectToUpdate, T i_DefaultValue)
    {
        if (EqualityComparer<T>.Default.Equals(i_ObjectToUpdate, default(T)))
        {
            i_ObjectToUpdate = i_DefaultValue;
        }
    }

并这样称呼它：

Program.setIfNull(i_UnitToCreate.EntityCreationDate, DateTime.Now);

理想情况下，您可以创建一个仅进行空值检查的不同扩展方法，然后自己设置变量：

    public static T IfNull<T>(this T i_ObjectToUpdate, T i_DefaultValue)
    {
        if (EqualityComparer<T>.Default.Equals(i_ObjectToUpdate, default(T)))
        {
            return i_DefaultValue;
        }
        return i_ObjectToUpdate;
    }

并通过以下方式调用它：

i_UnitToCreate.EntityLastUpdateDate = _UnitToCreate.EntityLastUpdateDate.IfNull(DateTime.Now);

这相当于（使用引用类型时）：

i_UnitToCreate.EntityLastUpdateDate = i_UnitToCreate.EntityLastUpdateDate ?? DateTime.Now;
于 2012-11-24T09:51:03.447 回答
0

我相信更好的方法是使用 C# 4.0??运算符：

   var nullable;

   . . . 

   nullable = nullable ?? "defaultValue";

你想达到什么目的？在我自己的个人工作习惯中，当我编写了一个扩展方法IDictionary的引用键不存在时，C# 对象抛出异常（dict['not here']抛出异常）让我感到非常沮丧：

public static TValue Safeget<TValue, TKey>(this IDictionary<TKey, TValue> arg, TKey key, TValue defaultValue = default(TValue))
        {
            if (arg.ContainsKey(key))
            {
                return arg[key];
            }

            return defaultValue;
        }

这是你所追求的吗？

于 2012-11-24T10:29:18.063 回答