Disable events temporarily

[Marc Solé]

Hello everybody,

I have a little "problem".

I have a DataGridView that is loaded automatically by code sometimes, and
manually by the user other times.

When the user introduces data in the DataGridView, there are some events
that control the introduced data for any errors the user can do. That's OK.
But when the data is loaded automatically, it is supposed that this data is
correct, so I don't need that these events are called.
I want to do that because with a lot of data, the load is very slowly, and
with the events disabled it takes only few seconds.

Anyone can help me, and tell me how can I disable these events?

I have done that, but it doesn't work.

Thanks in advance!

Marc

private void EventDisabler(b ool Enable)

{

if (Enable)

{

//Enable event handler

dgvItems.CellCl ick += new DataGridViewCel lEventHandler(d gvItems_CellCli ck);

dgvItems.CellVa lueChanged += new
DataGridViewCel lEventHandler(d gvItems_CellVal ueChanged);

dgvItems.CellEn dEdit += new
DataGridViewCel lEventHandler(d gvItems_CellEnd Edit);

dgvItems.CellVa lidated += new
DataGridViewCel lEventHandler(d gvItems_CellVal idated);

dgvItems.RowEnt er += new DataGridViewCel lEventHandler(d gvItems_RowEnte r);

dgvItems.RowLea ve += new DataGridViewCel lEventHandler(d gvItems_RowLeav e);

dgvItems.RowVal idated += new
DataGridViewCel lEventHandler(d gvItems_RowVali dated);

}

else

{

//Disable event handler

dgvItems.CellCl ick -= new DataGridViewCel lEventHandler(d gvItems_CellCli ck);

dgvItems.CellVa lueChanged -= new
DataGridViewCel lEventHandler(d gvItems_CellVal ueChanged);

dgvItems.CellEn dEdit -= new
DataGridViewCel lEventHandler(d gvItems_CellEnd Edit);

dgvItems.CellVa lidated -= new
DataGridViewCel lEventHandler(d gvItems_CellVal idated);

dgvItems.RowEnt er -= new DataGridViewCel lEventHandler(d gvItems_RowEnte r);

dgvItems.RowLea ve -= new DataGridViewCel lEventHandler(d gvItems_RowLeav e);

dgvItems.RowVal idated -= new
DataGridViewCel lEventHandler(d gvItems_RowVali dated);

}

}

[Marc Gravell]

I take it by "it doesn't work" you mean that they are still firing.
What you have should work, if used correctly. Is it possible that it has
been enabled twice (without an unsubscribe) through some route? As if so, it
will be subscribed twice, and fired twice. This will a: be slower, and b: if
you unsubscribe (-=) it will only remove the first found instance, perhaps
leaving one behind.

If this is the case, I would sugggest either keeping a bool field to track
if you think you are currently subscribed, or, *always* run the -= (which
unsubscribes if possible), then (if enabled) run the +=, so you can only
ever have 1 instance of each. The bool is cleaner.

Marc

[Marc Solé]

Thanks Marc,

I thought before to do that with the bool field, but it will always rise the
event and then compare the bool, isn't it?

Finally I choose the option to run always the -= .

Thanks a lot for your help.

Marc
"Marc Gravell" <ma**********@g mail.comescribi ó en el mensaje
news:e0******** ******@TK2MSFTN GP04.phx.gbl...

>I take it by "it doesn't work" you mean that they are still firing.
What you have should work, if used correctly. Is it possible that it has
been enabled twice (without an unsubscribe) through some route? As if so,
it will be subscribed twice, and fired twice. This will a: be slower, and
b: if you unsubscribe (-=) it will only remove the first found instance,
perhaps leaving one behind.

If this is the case, I would sugggest either keeping a bool field to track
if you think you are currently subscribed, or, *always* run the -= (which
unsubscribes if possible), then (if enabled) run the +=, so you can only
ever have 1 instance of each. The bool is cleaner.

Marc

[Marc Gravell]

No, I meant in your method... i.e.

bool eventsBound = false;
void BindEvents(bool enable) {
if(enable && eventsBound) return; // nothing to do; already bound...
if(!enable && !eventsBound) return; // nothing to do; not bound...

if(enable) {
someControl.Som eEvent += someHandler;
someControl.Som eOtherEvent += someOtherHandle r;
} else {
someControl.Som eEvent -= someHandler;
someControl.Som eOtherEvent -= someOtherHandle r;
}
eventsBound = enable;
}

Marc

[Marc Solé]

Thank you very much for your help.

It works fine now.

Marc
"Marc Gravell" <ma**********@g mail.comescribi ó en el mensaje
news:em******** ******@TK2MSFTN GP03.phx.gbl...

No, I meant in your method... i.e.

bool eventsBound = false;
void BindEvents(bool enable) {
if(enable && eventsBound) return; // nothing to do; already bound...
if(!enable && !eventsBound) return; // nothing to do; not bound...

if(enable) {
someControl.Som eEvent += someHandler;
someControl.Som eOtherEvent += someOtherHandle r;
} else {
someControl.Som eEvent -= someHandler;
someControl.Som eOtherEvent -= someOtherHandle r;
}
eventsBound = enable;
}

Marc