Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

INotifyPropertyChanged property name - hardcode vs reflection?

What is the best way to specify a property name when using INotifyPropertyChanged?

Most examples hardcode the property name as an argument on the PropertyChanged Event. I was thinking about using MethodBase.GetCurrentMethod.Name.Substring(4) but am a little uneasy about the reflection overhead.

like image 216
Kerry Jenkins Avatar asked Sep 26 '08 19:09

Kerry Jenkins


2 Answers

Don't forget one thing : PropertyChanged event is mainly consumed by components that will use reflection to get the value of the named property.

The most obvious example is databinding.

When you fire PropertyChanged event, passing the name of the property as a parameter, you should know that the subscriber of this event is likely to use reflection by calling, for instance, GetProperty (at least the first time if it uses a cache of PropertyInfo), then GetValue. This last call is a dynamic invocation (MethodInfo.Invoke) of the property getter method, which costs more than the GetProperty which only queries meta data. (Note that data binding relies on the whole TypeDescriptor thing -- but the default implementation uses reflection.)

So, of course using hard code property names when firing PropertyChanged is more efficient than using reflection for dynamically getting the name of the property, but IMHO, it is important to balance your thoughts. In some cases, the performance overhead is not that critical, and you could benefit from some kind on strongly typed event firing mechanism.

Here is what I use sometimes in C# 3.0, when performances would not be a concern :

public class Person : INotifyPropertyChanged {     private string name;      public string Name     {         get { return this.name; }         set          {              this.name = value;             FirePropertyChanged(p => p.Name);         }     }      private void FirePropertyChanged<TValue>(Expression<Func<Person, TValue>> propertySelector)     {         if (PropertyChanged == null)             return;          var memberExpression = propertySelector.Body as MemberExpression;         if (memberExpression == null)             return;          PropertyChanged(this, new PropertyChangedEventArgs(memberExpression.Member.Name));     }      public event PropertyChangedEventHandler PropertyChanged; } 

Notice the use of the expression tree to get the name of the property, and the use of the lambda expression as an Expression :

FirePropertyChanged(p => p.Name); 
like image 196
Romain Verdier Avatar answered Sep 28 '22 05:09

Romain Verdier


In .NET 4.5 (C# 5.0) there is a new attribute called - CallerMemberName it helps avoid hardcoded property names preventing the onset of bugs if developers decide to change a property name, here's an example:

public event PropertyChangedEventHandler PropertyChanged = delegate { };  public void OnPropertyChanged([CallerMemberName]string propertyName="") {     PropertyChanged(this, new PropertyChangedEventArgs(propertyName)); }  private string name; public string Name {     get { return name; }     set      {          name = value;         OnPropertyChanged();     } } 
like image 20
Denys Wessels Avatar answered Sep 28 '22 03:09

Denys Wessels