我在我的工作线程中有以下代码（下面的ImageListView是派生自Control）：
if (mImageListView != null &&
mImageListView.IsHandleCreated &&
!mImageListView.IsDisposed)
{
if (mImageListView.InvokeRequired)
mImageListView.Invoke(
new RefreshDelegateInternal(mImageListView.RefreshInternal));
else
mImageListView.RefreshInternal();
}
然而，有时我会在上面的Invoke方法中遇到ObjectDisposedException。似乎在我检查IsDisposed和调用Invoke之间，控件可能已被处理。我该如何避免这种情况？
你这里遇到了一个竞争条件。最好直接捕获ObjectDisposed异常并结束它。事实上，在这种情况下，我认为这是唯一有效的解决方案。
try
{
if (mImageListView.InvokeRequired)
mImageListView.Invoke(new YourDelegate(thisMethod));
else
mImageListView.RefreshInternal();
}
catch (ObjectDisposedException ex)
{
// Do something clever
}
// the event handler. in this case preped for cross thread calls
void OnEventMyUpdate(object sender, MyUpdateEventArgs e)
{
if (!this.IsHandleCreated) return; // ignore events if we arn't ready, and for
// invoke if cant listen to msg queue anyway
if (InvokeRequired)
Invoke(new MyUpdateCallback(this.MyUpdate), e.MyData);
else
this.MyUpdate(e.MyData);
}
// the update function
void MyUpdate(Object myData)
{
...
}
基本问题：
使用Invoke设施时，将使用Windows消息队列，该队列会将消息放入队列中，以等待或立即触发跨线程调用，就像Post或Send消息一样。如果在Invoke消息之前有一个消息会使组件及其窗口句柄无效，或者该消息刚好放置在您尝试执行任何检查之后，那么您将会遇到麻烦。
x thread -> PostMessage(WM_CLOSE); // put 'WM_CLOSE' in queue
y thread -> this.IsHandleCreated // yes we have a valid handle
y thread -> this.Invoke(); // put 'Invoke' in queue
ui thread -> this.Destroy(); // Close processed, handle gone
y thread -> throw Invalid....() // 'Send' comes back, thrown on calling thread y
没有真正的方式可以知道控件将要从队列中移除自己，也没有任何合理的方法可以“撤销”调用。无论您进行多少检查或额外的锁定，都无法阻止其他人发出类似关闭或停用的操作。这种情况有很多。
一种解决方案：
首先要意识到的是，调用将失败，就像(IsHandleCreated)检查会忽略事件一样。如果目标是保护非UI线程上的调用方，则需要处理异常，并将其视为未成功的任何其他调用（以避免应用程序崩溃或执行其他操作）。除非你打算重写/重新生成调用设施，否则catch是你唯一的方法来知道。
// the event handler. in this case preped for cross thread calls
void OnEventMyWhatever(object sender, MyUpdateEventArgs e)
{
if (!this.IsHandleCreated) return;
if (InvokeRequired)
{
try
{
Invoke(new MyUpdateCallback(this.MyUpdate), e.MyData);
}
catch (InvalidOperationException ex) // pump died before we were processed
{
if (this.IsHandleCreated) throw; // not the droids we are looking for
}
}
else
{
this.MyUpdate(e.MyData);
}
}
// the update function
void MyUpdate(Object myData)
{
...
}
if(!myControl.Disposing)
; // invoke here
private bool IsControlValid(Control myControl)
{
if (myControl == null) return false;
if (myControl.IsDisposed) return false;
if (myControl.Disposing) return false;
if (!myControl.IsHandleCreated) return false;
if (AbortThread) return false; // the signal to the thread to stop processing
return true;
}
Isak Savo 提出的解决方案
try
{
myForm.Invoke(myForm.myDelegate, new Object[] { message });
}
catch (ObjectDisposedException)
{ //catch exception if the owner window is already closed
}
在C# 4.0中可以正常工作，但由于某些原因，在C# 3.0中失败了（无论如何都会引发异常）
因此，我使用了另一种解决方案，基于一个标志来指示窗体是否正在关闭，并且如果设置了该标志，则防止使用invoke。
public partial class Form1 : Form
{
bool _closing;
public bool closing { get { return _closing; } }
private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
_closing = true;
}
...
// part executing in another thread:
if (_owner.closing == false)
{ // the invoke is skipped if the form is closing
myForm.Invoke(myForm.myDelegate, new Object[] { message });
}
这样做的好处是完全避免了使用try/catch。
你可以使用互斥锁。
在线程的开始处：
Mutex m=new Mutex();
然后：
if (mImageListView != null &&
mImageListView.IsHandleCreated &&
!mImageListView.IsDisposed)
{
m.WaitOne();
if (mImageListView.InvokeRequired)
mImageListView.Invoke(
new RefreshDelegateInternal(mImageListView.RefreshInternal));
else
mImageListView.RefreshInternal();
m.ReleaseMutex();
}
无论您将mImageListView放置在何处：
m.WaitOne();
mImageListView.Dispose();
m.ReleaseMutex();
这将确保您无法同时处置和调用。
public partial class MyForm : Form
{
private void InvokeViaBgw(Action action)
{
BGW.ReportProgress(0, action);
}
private void BGW_ProgressChanged(object sender, ProgressChangedEventArgs e)
{
if (this.IsDisposed) return; //You are on the UI thread now, so no race condition
var action = (Action)e.UserState;
action();
}
private private void BGW_DoWork(object sender, DoWorkEventArgs e)
{
//Sample usage:
this.InvokeViaBgw(() => MyTextBox.Text = "Foo");
}
}
private void Form_FormClosing(object sender, FormClosingEventArgs e)
{
if (service.IsRunning)
{
service.Exit();
e.Cancel = true;
this.BeginInvoke(new Action(() => { this.Close(); }));
}
}
可能需要使用 lock(mImageListView){...} 吗？
请参考以下问题：
避免跨线程WinForm事件处理中的Invoke/BeginInvoke困境？
产生EventHandlerForControl实用程序类可以解决事件方法签名的问题。您可以调整该类或者审查其中的逻辑以解决该问题。
这里真正的问题是nobugz正确地指出，winforms中提供的用于跨线程调用的API本质上并不是线程安全的。即使在对InvokeRequired和Invoke/BeginInvoke进行调用时，也存在多个竞争条件可能导致意外行为。