使用泛型回调是不好的形式?

时间:2012-08-30 14:32:40

标签: c# multithreading generics callback

我有这个代码(好吧,类似的东西)。

private delegate void GenericCallback<T>(T Info);

private void DoWork()
{
    System.Threading.Thread Worker = new System.Threading.Thread(
            delegate() 
            {
                TestMethod(TestMethodCallback<string>);
            }
    );
    Worker.Start();
}

private void TestMethod(GenericCallback<string> Callback)
{
    System.Threading.Thread.Sleep(1000);
    if(Callback != null)
    {
        Callback("Complete");
    }
}

private void TestMethod(GenericCallback<int> Callback)
{
    System.Threading.Thread.Sleep(1000);
    if(Callback != null)
    {
        Callback(25);
    }
}

private void TestMethodCallback<T>(T Info)
{
    MessageBox.Show(Info.ToString());
}

这允许我根据参数的类型调用TestMethod的不同版本,同时还允许我使用单个回调方法。

这是一种不好的形式,还是一种公认​​的做法?

4 个答案:

答案 0 :(得分:7)

看起来您可能正在寻找Action delegate type。它基本上就是你在这里拥有的:一个通用的返回委托类型。

答案 1 :(得分:5)

这种既定的做法,已经完成了一些工作。在这种情况下Action,如果你返回一个值,Func是通用委托,就像这样。如果我看到你的签名,那就是优势:

private void TestMethod(GenericCallback<string> Callback)

我必须查看GenericCallback<string>是什么。如果我看到:

private void TestMethod(Action<string> callback)

我已经知道了。

答案 2 :(得分:1)

完全没问题。在这个非常简单的情况下,最好指定一个字符串(因为你的泛型类型,没有约束,只能作为一个Object处理,所以ToString()是你可以调用的极少数有用的东西之一,无论什么你通过了),但显然这是示例代码,所以有很多可接受的方法来做到这一点。 Action<>Func<>确实是很好的内置函数,但是它们有一些缺点,例如减少了自我文档(很难说Func应该做什么,除了显而易见的“取一个整数并返回一个整数”。

答案 3 :(得分:0)

假设这将用于异步返回数据(否则,为什么不等待返回值?),听起来你没有发现Task Parallel Library。特别是,Task<T>是一种更为普遍的方式,可以实现类似回调的行为。

例如:

private Task<int> TestMethod()
{
    TaskCompletionSource<int> tcs=new TaskCompletionSource<int>();
    //do something asynchronous
    //when asynchronous job is complete
    //tcs.SetResult(25);
    return tcs.Task;

}