Question

I want to create a generic extension method that will set a value to an Object or a Struct if their value equals to their default value.

so i have the following code:

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

and here is a call example:

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

I don’t know if it have anything to do with it but i use entity framework and MVC.

What actually happens in a debugger I see that the line in the extension method i_ObjectToUpdate = i_DefaultValue; is working and the values are changes but when the debugger gets out of the extension method I see that the value of i_UnitToCreate.EntityCreationDate remains unchaged.

Any ideas what went wrong ?

Answer 1

You have two references in your code. One is i_UnitToCreate.EntityCreationDate which points to some address in memory. And other is i_ObjectToUpdate in your extension method, which initially also points to that address (you are creating copy of address when passing i_UnitToCreate.EntityCreationDate reference to your method). Later you change second reference to point on other object in memory, but that does not change first reference, because they are independent.

Workaround

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

You can use expression to pass property selector (not property value) to extension method. Retrieve value from compiled expression. If it is default value, then with reflection (you can easily get PropertyInfo by casting member expression) set new value. Usage:

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

---

PS There is one remark about my first answer – I thought about generic T type as a reference type, when talked about copying reference values. With value types (as DateTime) whole object is copied. It does not change result (you can’t assign new value), but needs to be mentioned.