这个thread.abort（）是否正常且安全？

Z时代
2024-01-10
分类：问答

我创建了一个自定义自动完成控件，当用户按下一个键时，它将在另一个线程上查询数据库服务器（使用远程处理）。当用户快速键入时，程序必须取消先前执行的请求/线程。

我以前首先将其实现为AsyncCallback，但我发现它很麻烦，要遵循的内部规则过多（例如AsyncResult，AsyncState，EndInvoke），另外您还必须检测BeginInvoke’d对象的线程，以便可以终止先前执行的线程。此外，如果我继续执行AsyncCallback，则那些AsyncCallbacks上没有任何方法可以正确终止先前执行的线程。

EndInvoke无法终止线程，它仍将完成待终止线程的操作。我仍然会最终在线程上使用Abort（）。

因此，我决定仅使用纯线程方法来实现它，而无需使用AsyncCallback。这是thread.abort（）正常且对您安全吗？

public delegate DataSet LookupValuesDelegate(LookupTextEventArgs e);
internal delegate void PassDataSet(DataSet ds);
public class AutoCompleteBox : UserControl
{
   Thread _yarn = null;
   [System.ComponentModel.Category("Data")]
   public LookupValuesDelegate LookupValuesDelegate { set; get; }
   void DataSetCallback(DataSet ds)
   {
      if (this.InvokeRequired)
         this.Invoke(new PassDataSet(DataSetCallback), ds);
      else
      {
         // implements the appending of text on textbox here
      }
   }
   private void txt_TextChanged(object sender, EventArgs e)
   {
      if (_yarn != null) _yarn.Abort();
      _yarn = new Thread(
         new Mate
         {
            LookupValuesDelegate = this.LookupValuesDelegate,
            LookupTextEventArgs =
            new LookupTextEventArgs
            {
               RowOffset = offset,
               Filter = txt.Text
            },
            PassDataSet = this.DataSetCallback
         }.DoWork);
      _yarn.Start();
   }
}
internal class Mate
{
   internal LookupTextEventArgs LookupTextEventArgs = null;
   internal LookupValuesDelegate LookupValuesDelegate = null;
   internal PassDataSet PassDataSet = null;
   object o = new object();
   internal void DoWork()
   {
      lock (o)
      {
         // the actual code that queries the database
         var ds = LookupValuesDelegate(LookupTextEventArgs);
         PassDataSet(ds);
      }
   }
}

笔记

在用户连续键入键时取消上一个线程的原因，不仅是为了防止文本的添加发生，而且还取消了上一个网络往返，因此该程序不会因为连续执行而消耗过多的内存。网络操作。

我担心是否完全避免使用thread.Abort（），该程序可能会占用太多内存。

这是不带thread.Abort（）的代码，使用一个计数器：

internal delegate void PassDataSet(DataSet ds, int keyIndex);
public class AutoCompleteBox : UserControl
{
   [System.ComponentModel.Category("Data")]
   public LookupValuesDelegate LookupValuesDelegate { set; get; }
   static int _currentKeyIndex = 0;
   void DataSetCallback(DataSet ds, int keyIndex)
   {
      if (this.InvokeRequired)
         this.Invoke(new PassDataSet(DataSetCallback), ds, keyIndex);
      else
      {
         // ignore the returned DataSet
         if (keyIndex < _currentKeyIndex) return;
         // implements the appending of text on textbox here...
      }
   }
   private void txt_TextChanged(object sender, EventArgs e)
   {
      Interlocked.Increment(ref _currentKeyIndex);
      var yarn = new Thread(
         new Mate
         {
            KeyIndex = _currentKeyIndex,
            LookupValuesDelegate = this.LookupValuesDelegate,
            LookupTextEventArgs =
            new LookupTextEventArgs
            {
               RowOffset = offset,
               Filter = txt.Text
            },
            PassDataSet = this.DataSetCallback
         }.DoWork);
      yarn.Start();
   }
}
internal class Mate
{
   internal int KeyIndex;
   internal LookupTextEventArgs LookupTextEventArgs = null;
   internal LookupValuesDelegate LookupValuesDelegate = null;
   internal PassDataSet PassDataSet = null;
   object o = new object();
   internal void DoWork()
   {
      lock (o)
      {
         // the actual code that queries the database
         var ds = LookupValuesDelegate(LookupTextEventArgs);
         PassDataSet(ds, KeyIndex);
      }
   }
}