3

我正在学习创建事件和创建多线程应用程序。

该方法Thread由另一个类调用，该类使用搜索条件填充参数。ABackgroundWorker被创建，执行搜索并将结果返回到worker_RunWorkerCompleted。

在 内worker_RunWorkerCompleted，我想将结果发送回订阅该Fireendofsearch事件的 UI。

我无法理解为什么下面的代码会引发以下错误

你调用的对象是空的。

当我触发事件时Fireendofsearch

public class BackgroundSearch
{
    public event SearchResultCompleteThreaded Fireendofsearch;
    public EventArgs a = null;
    public delegate void SearchResultCompleteThreaded(object seachresults, EventArgs a);
    internal void Thread(string folder, string parms)
    {
        var Argument = new List<object> { folder, parms };
        var worker = new BackgroundWorker {WorkerReportsProgress = false, WorkerSupportsCancellation = false};
        worker.DoWork += worker_DoWork;          
        worker.RunWorkerCompleted += worker_RunWorkerCompleted;
        worker.RunWorkerAsync(Argument);
    }
    void worker_DoWork(object sender, DoWorkEventArgs e)
    {
        var passedAugue = e.Argument as List<object>;
        var returnResult = new List<string[]>();
        if (passedAugue != null)
        {
            var result = Directory.GetFiles(passedAugue[0].ToString(), passedAugue[1].ToString(), SearchOption.AllDirectories);
            foreach (string s in result)
            {
                var t = new string[4];
                t[0] = s;
                t[1] = File.GetCreationTime(s).ToString();
                t[2] = File.GetLastAccessTime(s).ToString();
                t[3] = File.GetLastWriteTime(s).ToString();
                returnResult.Add(t);
            }
        }
        if (returnResult.Count != 0)
        {
            e.Result = returnResult;
        }            
    }
    void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
    {
        if (e.Result != null)
        {
            Fireendofsearch(e.Result, a);
        }
    }
}
4

5 回答 5

2

Firendofsearch 将是空的，直到有人订阅它，将您的工作完成事件处理程序更改为此来修复它。

void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (e.Result != null)
    {
        var friendOfSearch = Fireendofsearch;
        if(friendOfSearch != null)
           friendOfSearch (e.Result, a);
    }
}

我将它复制到变量的原因是，如果另一个线程中的某个人是最后一个在 null 检查和引发事件之间取消订阅的人，您仍然会得到 null 引用异常，通过首先处理另一个变量可以解决该问题。

但是，如果我在哪里编写它，我会进行一些其他更改，您EventArgs出于某种原因重新调整 null 并将结果作为传统事件模式中的“发送者”传递回。我会将您的代码更改为此

public event EventHandler<FriendOfSearchArgs> FirendsOfSearch;

void worker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (e.Result != null)
    {
        RaiseFriendOfSearch(e.Result);
    }
}

protected virtual void RaiseFriendOfSearch(object result)
{
    var friendOfSearch = FirendsOfSearch;
    if(friendOfSearch != null)
        friendOfSearch(this, new FriendOfSearchArgs(result));
}


public class FriendOfSearchArgs : EventArgs
{
   public FriendOfSearchArgs(object result)
   {
       Result = result;
   }

   public object Result {get; private set;}
}

这一切都写在 SO 文本框中，因此可能存在一两个错误。

于 2012-11-04T17:54:33.613 回答
0

出于某种原因（很可能是优化），一个事件仅在第一个处理程序方法订阅它时才被实例化。

您必须在代码中检查这一点。

以下是我通常声明事件的方式：

public event SearchResultCompleteThreaded Fireendofsearch;

private void RaiseFireEndOfSearchEvent(EventArgs e)
{
    if (Fireendofsearch != null)
    {
       Fireendofsearch(this, e);
    }
}

每当我需要引发事件时，我都会调用辅助方法。

于 2012-11-04T17:54:45.153 回答
0

在尝试调用委托之前，您应该检查 null。而且您必须首先将其拉入一个单独的变量中以避免线程问题。

var ev = Fireendofsearch;
if ( ev != null ) ev( ... );

我还发现在这种情况下使用扩展方法很有用：

public static void Raise ( this EventHandler h, object sender )
{
    if ( h != null) h( sender, EventArgs.Empty );
}

接着：

MyEvent.Raise ( this );
于 2012-11-04T18:01:03.603 回答
0

在您的公共事件“背后”，有一个类型为 的隐式私有变量SearchResultCompleteThreaded。该类型SearchResultCompleteThreaded是委托类型。

在 .NET 中，所有代表都是“多播”代表。这意味着他们有一个调用列表（您的GetInvocationList()方法 onSearchResultCompleteThreaded派生自System.Delegate.GetInvocationList().

现在，在 .NET 中，调用列表保证包含一个或多个项目（不是零个或多个）。任何委托类型都是不可变类型。但是，如果您尝试通过“减去”现有实例的调用列表中的所有成员来创建新实例，例如var newDel = oldDel - oldDel;or reuseDel -= reuseDel;，那么您将获得一个空引用，而不是获得一个具有零长度调用列表的新实例！

这样做的好处是您不必担心“空”委托实例（否则可能会被允许）和空引用之间的细微差别。关于它的坏处是你上面的问题。

于 2012-11-04T18:29:46.047 回答
-1

放：

public event SearchResultCompleteThreaded Fireendofsearch =  delegate { };

可能需要初始化吗？

于 2012-11-04T17:47:09.383 回答