4

我有一个通用接口ICommandHandler<>,它将有许多实现,每个实现都用于处理 的特定实现ICommand,例如:

public class CreateUserCommand : ICommand { ... }
public class CreateUserCommandHandler : ICommandHandler<CreateUserCommand> { ... }

当我得到一个ICommand对象时,我试图将它动态分派到正确的ICommandHandler. 目前,我Invoke在调度程序类中使用了一种非常简单的反射方法:

public void Dispatch<T>(T command) where T : ICommand
{
    Type commandType = command.GetType();
    Type handlerType = typeof(ICommandHandler<>).MakeGenericType(commandType);
    object handler = IoC.Get(handlerType);
    MethodInfo method = handlerType.GetMethod("Handle");

    method.Invoke(handler, new object[] { command });
}

这种方法有两个问题。首先它使用慢反射。其次,如果该方法抛出任何类型的异常,那么它将被包裹在 a 中TargetInvocationException,如果我重新抛出它,我将丢失堆栈跟踪。

我想出了一种通过创建委托并使用来进行调用的方法,DynamicInvoke但这并不能解决异常问题(而且我不确定DynamicInvoke是否真的比 更好Invoke):

public void Dispatch<T>(T command) where T : ICommand
{
    Type commandType = command.GetType();
    Type handlerType = typeof(ICommandHandler<>).MakeGenericType(commandType);
    object handler = IoC.Get(handlerType);
    MethodInfo method = handlerType.GetMethod("Handle");

    Type actionType = typeof(Action<>).MakeGenericType(commandType);
    Delegate action = Delegate.CreateDelegate(actionType, handler, method);
    action.DynamicInvoke(command);
}

我的问题是,有没有更好的方法来实现我想要做的事情?最好我可以进行强类型调用,而不是获取object并查找MethodInfo. 我认为这是不可能的,因为在编译时不知道类型。

如果这是不可能的,那么一个更“本机”抛出异常的有效解决方案将是下一个最佳选择。

编辑:更新代码示例以阐明我使用 IoC (Ninject)ICommandHandler在运行时创建,而不是Activator.CreateInstance()像我最初提出的那样。包括一个如何按要求使用的示例:

var command = new CreateUserCommand() { Name = "Adam Rodger" };
var dispatcher = new CommandDispatcher();
dispatcher.Dispatch(command);
// this would send the message to CreateUserCommandHandler.Handle(command) 
// dynamically and any exceptions would come back 'natively'

编辑 2:如下所示,我无法将结果转换为IoC.Get(handlerType)ICommandHandler<T>因为我InvalidCastException在运行时得到一个。这是因为在运行时T实际上是ICommand,我假设是因为命令类是通过 WCF 到达的,并且以某种方式设法失去了它们的强类型。调用调度程序的代码类似于:

[ServiceContract]
public class CommandService
{
    [OperationContract]
    public void Execute(ICommand command) // no type information
    {
        var dispatcher = new CommandDispatcher(); // injected by IoC in real version
        dispatcher.Dispatch(command);
    }
}
4

2 回答 2

7

大多数 DI 容器(包括 Ninject)允许您执行以下操作:

public void Dispatch<T>(T command) where T : ICommand
{
    ICommandHandler<T> handler = IoC.Get<ICommandHandler<T>>();
    handler.Handle(command);
}

如果您不知道命令的类型(换句话说, if typeof(T) != command.GetType()),使用双分派是最简单的方法:

class SomeCommand : ICommand
{
    // ...

    public void Dispatch(IoC ioc)
    {
        var handler = ioc.Get<IHandle<SomeCommand>>();
        handler.Handle(this);
    }
}

但是,如果您发现将此代码添加到所有命令中令人反感,则可以进行反思。

编辑这里是一个基于反射的版本。您可以(并且应该)缓存已编译的委托。

interface ICommand { }
interface IHandle<TCommand> where TCommand : ICommand
{
    void Handle(TCommand command);
}

class CreateUserCommand : ICommand { }
class CreateUserHandler : IHandle<CreateUserCommand>
{
    public void Handle(CreateUserCommand command)
    {
        Console.Write("hello");
    }
}

[TestMethod]
public void build_expression()
{
    object command = new CreateUserCommand();
    object handler = new CreateUserHandler();

    Action<object, object> dispatcher = BuildDispatcher(command.GetType());
    dispatcher(handler, command);
}

private static Action<object, object> BuildDispatcher(Type commandType)
{
    var handlerType = typeof(IHandle<>).MakeGenericType(commandType);
    var handleMethod = handlerType.GetMethod("Handle");

    var param1 = Expression.Parameter(typeof(object));
    var param2 = Expression.Parameter(typeof(object));

    var handler = Expression.ConvertChecked(param1, handlerType);
    var command = Expression.ConvertChecked(param2, commandType);
    var call = Expression.Call(handler, handleMethod, command);

    var lambda = Expression.Lambda<Action<object, object>>(call, param1, param2);
    return lambda.Compile();
}
于 2013-04-10T16:21:45.393 回答
1

尝试这个

dynamic handler=Activator.CreateInstance(handlerType);
try
  {
         handler.Handle((dynamic)command);
   }

   catch
   {
   // do whatever you want 
   }
于 2013-04-10T15:58:40.837 回答