使用命令模式通过串行发送和接收命令

Question

我正在使用一个桌面应用程序，它应该通过串行接口发送和接收命令，固件由我的同事编程。

我们设计了一个协议，其中命令类型由ASCII字母表示，每个命令类型可能包含一个有效载荷(在命令类型之间是任意的，但是对于每种类型是固定的)，并且命令被包装在方括号中。例如，我们有以下命令:

• [S1234]-

所以我有一个接收和解析传入字节的类，每次成功解析命令时，都会引发一个事件，并带有以下暂定签名：

internal event EventHandler<SerialCommand> CommandReceived;

其中< code>SerialCommand有不同的子类型:BatteryCommand、AdcCommand、SerialCommand等。每种命令类型都与其各自的“字符代码”相关联。

我的问题是：客户端代码应该如何使用它？例如，当我收到命令时的当前实现有一个带有硬编码char文字的开关/案例，我发现它非常脆弱和丑陋：

    void CommandReceivedHandler(object sender, SerialCommand command)
    {
        switch (command.Code)
        {
            case 'A':
                int value= Convert.ToInt32(command.Value);
                _frameStreamer.AddFrame(new Frame<int>(new[] { value}));
                break;
            case 'B':
                BatteryLevel= (int)command.Value;
                break;
            case 'D':
                DoSomething((byte)command.Value);
                break;
            case 'S':
                SerialNumber = (int)command.Value;
                break;
        }           
    }

目前，这些“字符代码”分布在一堆类中，如果我需要更改给定的代码，我需要环顾四周以查找每个事件（霰弹枪手术反模式）。

我需要做的是两件事：

• 仅将字符代码封装在命令中，而不是客户端代码中;
• 在客户端（CommandReceived 事件使用者）上多态执行操作，最好不使用开关/case 语句。

Answer 1

你可以试试这样的方法：

public abstract class BaseCommand
{
    //Code not needed at all, because logic encapsulated into command
    //public char Code { get; set; }         
    public abstract void Action(IClient client);
}

public abstract class BaseCommand<T> : BaseCommand
{
    public T value { get; set; }
}

public class CommandA : BaseCommand<int>
{            
    public override void Action(IClient client)
    {        
        client.someInt = value * 2;
    }
}

public class CommandB : BaseCommand<string>
{
    public override void Action(IClient client)
    {
        client.someString = value.Trim();
    }
}

public interface IClient
{
    void CommandReceivedHandler(object sender, BaseCommand command);    
    int someInt { get; set; }
    string someString { get; set; }
}

public class Client : IClient
{
    public void CommandReceivedHandler(object sender, BaseCommand command)
    {
        command.Action(this);
    }

    public int someInt { get; set; }
    public string someString { get; set; }
}