假设我有这个代码:
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和反汇编程序类都将扩展此类,并为每个方法实现自己的行为.但是,我希望有一个更优雅的解决方案.
谢谢.
编辑:
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]",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],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);