Question

In classes that implement INotifyPropertyChanged I often see this pattern :

    public string FirstName
    {
        get { return _customer.FirstName; }
        set
        {
            if (value == _customer.FirstName)
                return;

            _customer.FirstName = value;

            base.OnPropertyChanged("FirstName");
        }
    }

Precisely the lines

            if (value == _customer.FirstName)
                return;

are bothering me. I’ve often did this but I am not that sure it’s needed nor good. After all if a caller assigns the very same value I don’t want to reassign the field and, especially, notify my subscribers that the property has changed when, semantically it didn’t.

Except saving some CPU/RAM/etc by freeing the UI from updating something that will probably look the same on the screen/whatever_medium what do we obtain?

Could some people force a refresh by reassigning the same value on a property (NOT THAT THIS WOULD BE A GOOD PRACTICE HOWEVER)?

1. Should we do it or shouldn’t we?

2. Why?

Answer 1

Yes, you should return immediately when the consumer is setting a property value that is equal to the value which is already being persisted.

First of all, there is no reason to waste any time or resources in the setter of the property – the value is already set so no further action is needed. Also you should never call OnPropertyChanged if the value stored in the property’s backing field hasn’t changed – the method is intended to be raised when the value has changed not when the property’s setter has been called.

All that being said, however – if the setter didn’t have a call to OnPropertyChanged I wouldn’t have bothered to check the value first. In the case of a simple setter that only sets the backing field’s value and nothing else it is actually going to be faster to always the set the value rather than checking first then setting the value. Only use this pattern when the property’s setter has additional logic that either shouldn’t fire or may incur an unnecessary performance penalty.