Comments on: Annoyed with INotifyPropertyChanged?

By: Russell Shilling

Russell Shilling — Thu, 12 Feb 2015 17:46:30 +0000

Wow! Two thumbs up. This will become a part of every program I write from now on!

Great work, and thank you.

By: timoch

timoch — Fri, 25 Apr 2014 06:38:51 +0000

Applying memento here is a great idea. One could also easily implement undo/redo using snapshots or generate diffs for auditing/logging purposes. I think I’ll write a couple of posts on that. It’s been a while …

By: peterthorpe81

peterthorpe81 — Thu, 24 Apr 2014 11:42:25 +0000

This is really cool I extended it to support IEditableObject using a second dictionary as a memento. That way I can cancel/save changes in winforms grids and similar.

Its a shame there isn’t a [CallerMemberType] attribute we could use that could get us to just Get(); rather than Get();

By: timoch

timoch — Wed, 04 Sep 2013 08:43:57 +0000

Krumelur, you are right. Apparently, I used a version of the code before I refactored it. I hit the same issue myself.
Btw, sorry for the late reply, I was enjoying some hollidays

By: Krumelur

Krumelur — Sun, 25 Aug 2013 19:45:50 +0000

Oh, just noticed: you’re missing code. That’s causing the build error. Set should probably be:

protected void Set(T value, [CallerMemberName] string name = null) {
Debug.Assert(name != null, “name != null”);
if (Equals (value, Get (name)))
{
return;
}
_properties[name] = value;
OnPropertyChanged(name);
}

By: Krumelur

Krumelur — Sun, 25 Aug 2013 19:40:55 +0000

B-R-I-L-L-I-A-N-T idea! However: I cannot get it to build I tried VS2012 and Xamarin Studio.

In the Set I’m getting:

Error 1 The best overloaded method match for ‘Set(string, string)’ has some invalid arguments
Error 2 Argument 2: cannot convert from ‘T’ to ‘string’