c – 使交换机重量级功能更具可重用性

假设我有这个代码：

void ClassA::SomeFunction()
{
    switch(something)
    {
    case case1:
        doSomething();
        break;
    case case2:
        doSomethingElse();
        break;
       .
       . 
       .
}
...
void ClassB::SomeFunction()
{
    switch(something) // this 'something' is the same 'something' as in
    {                 // the above class
    case case1:
        doSomethingCompletelyUnrelatedToClassAFunction();
        break;
    case case2:
        doSomethingCompletelyUnrelatedToClassAOtherFunction();
        break;
       .
       . 
       .
}  

在完全相同的情况下,两个类中的两个函数做了相当不同的事情(两个开关中的所有情况完全相同).
基本上我正在编写一个小型的Chip-8模拟器以获得乐趣,这两个类代表我的“CPU”和反汇编程序.

CPU应解码操作码并执行一些操作,而反汇编程序应根据操作码简单地格式化字符串.

我试图想办法避免复制/粘贴整个开关,只是改变每种情况下调用的函数.

我提出的一个简单的解决方案是创建一个处理操作码的抽象类,并为交换机中的每个案例提供不同的方法. CPU和反汇编程序类都将扩展此类,并为每个方法实现自己的行为.但是,我希望有一个更优雅的解决方案.

谢谢.

编辑：

根据@Dogbert的评论,我添加了一大堆实际代码：

wxString* DebugWindow::DisassembleOpCode(uint16_t opCode)
{
wxString disLine;

uint8_t nibble1 = (opCode & 0xF000) >> 12;
uint8_t nibble2 = (opCode & 0x0F00) >> 8;
uint8_t nibble3 = (opCode & 0x00F0) >> 4;
uint8_t nibble4 = (opCode & 0x000F);

if (opCode == 0x00E0)
    disLine = "CLS";
else if (opCode == 0x00EE)
    disLine = "RET";
else
    switch (nibble1)
{
    case 0x1:
    {
                // JMP nnn
                disLine = wxString::Format("JMP %04x", nibble2 | nibble3 | nibble4);
                break;
    }
    case 0x2:
    {
                // CALL nnn
                disLine = wxString::Format("CALL %04x", nibble2 | nibble3 | nibble4);
                break;
    }
    case 0x3:
    {
                // SE V[x], nn  -- skip next instruction if V[x] == nn
                disLine = wxString::Format("SE V[%x], %02x", nibble2, nibble3 | nibble4);
                break;
    }

    ...
    ...
    ...

    case 0x8: // arithmetic operations between registers, see below
        {
                      switch (nibble4)
                      {
                      case 0x0:
                      {
                                  // LD V[x], V[y] -- sets value of register V[x] to value of register V[y]
                                  disLine = wxString::Format("ADD V[%x], V[%x]", nibble2, nibble3);
                                  break;
                      }
                      case 0x1:
                      {
                                  // OR V[x], V[y] -- performs bitwise OR of values of registers V[x] and V[y], the result is stored in V[x]
                                  disLine = wxString::Format("OR V[%x], V[%x]", nibble2, nibble3);
                                  break;
                      }

     ...
     ...

这是DisassembleOpcode函数的一部分. CPU类中的Step函数实际上应该执行填充而不是格式化字符串,但它由相同的开关/案例组成.

解决方法:

与dasblinkenlight建议的相同,我甚至会说“制表”指令集会大大提高设计的可读性.有点像：

struct opcode instructionset[] = {
{ 0x1000, "JMP %04x", &do_jump},
...

然后,您的代码将访问每个指令的表,可能表格将按操作码排序以便快速查找：

struct opcode op* = find(opCode, instructionset);
// for printing the instruction
disLine = wxString::Format(op->formatstring, opCode & op->format_mask);

你明白了……但关键是：你将拥有一个格式良好的已实现指令集表,我认为这将极大地简化维护/将来添加的指令.

标签：c-3,c,switch-statement,reusability
来源： https://codeday.me/bug/20190728/1562906.html