Comments on: Quick and dirty “dirty checking” for Windows form, C#

By: Guillaume JAY

Guillaume JAY — Tue, 30 Mar 2010 11:22:11 +0000

This is a very good post. As you said, a straight forward time saver. This should go to the new blog.

Anthony, I’d like to see your whole class.

email guillaume(dot)jay(at)gmail(dot)com

Thanks !

By: Guest

Guest — Wed, 17 Mar 2010 08:59:52 +0000

All this is handled on text change event of a textbox or selected index changed of drop downs. as i am assigning values to controls on grid cell double click, that time also the change event get fired, i only want to track that is user made some changes in existing values or not. then only i want to prompt to save changes.
can anyone guide me?

By: Anthony

Anthony — Thu, 11 Mar 2010 13:07:46 +0000

I implemented a similary is dirty logic but it is dynamic and based on the event. It uses some reflection so it may be a bit slower then your implementation, but it allows you to add new events without code changes. Let me know what you think:

private List _dirtyEvents = new List();
_dirtyEvents.Add(new DirtyEvent(“TextChanged”));
_dirtyEvents.Add(new DirtyEvent(“SelectedIndexChanged”));

private void recurseControls(Control.ControlCollection Controls)
{
foreach (Control c in Controls)
{
if (c is INotifyPropertyChanged)
{
((INotifyPropertyChanged)c).PropertyChanged += new PropertyChangedEventHandler(DirtyFormBase_PropertyChanged);

}
for (int i = 0; i < _dirtyEvents.Count; i++)
{
EventInfo ei = c.GetType().GetEvent(_dirtyEvents[i].EventName);
if (ei != null)
{
Delegate textChangedDelegate = Delegate.CreateDelegate(ei.EventHandlerType, typeof(DirtyFormBase).GetMethod(“GenericHandler”, BindingFlags.Static | BindingFlags.NonPublic | BindingFlags.IgnoreCase));
ei.AddEventHandler(c, textChangedDelegate);
}
}

if (c.HasChildren)
recurseControls(c.Controls);
}
}

private static void GenericHandler(object sender, EventArgs args)
{
DirtyFormBase form = (DirtyFormBase)((Control)sender).FindForm();
if (!form.SuspendIsDirtyCheck)
{
form.IsDirty = true;
}
}

This is a bit more code as it is a base class for all forms that need dirty logic. If you are interested let me know and I will e-mail the class.

By: Wayne

Wayne — Mon, 13 Apr 2009 23:32:28 +0000

Seems to work like a charm! Thanks! You saved me a few hours (possible much more) with this snippet.

By: Wayne

Wayne — Mon, 13 Apr 2009 22:24:04 +0000

Figured it out. Instead of the handler you have above, using:

void InputControls_OnChange(object sender, EventArgs e)

does the trick. I’ll update you with anything else I find to round out your post. Cheers!

By: Wayne

Wayne — Mon, 13 Apr 2009 22:19:54 +0000

Hmm, trying to work out an issue with your code. I get a “No overload for ‘InputControls_OnChange’ matches delegate ‘System.EventHandler” error. Still debugging it….using .net 2.0. Any clue?

By: Wayne

Wayne — Mon, 13 Apr 2009 21:49:41 +0000

Thanks, I think I’ll steal this for my own nefarious purposes.

Actually, I’m going to use this to detect the ‘isDirty’ on each tab change (36 tabs, 300+ controls, yes, I know…)

Thanks for providing the code!