Question

我认为由于抽象设计不佳，我使用了太多方法。这就是我提出这个问题的原因，与this one和类似的问题不同，后者只关注方法本身的数量。

长话短说，我正在做一个Hexapod。我之所以这么说是因为有一些不同寻常的设计考虑因素：你不能立刻从前进到倒退;所以有一些过渡时间。此外，还有空闲时间，即没有行动时。

我正在重构我的代码，但为了达到这个目的，我发现我创建了许多不应该在顶级抽象中的方法，但是在下面：

void loop()
  {
  // Potentiometer
  String action = Controller.action();
  Serial.println(action);

  // Set the spider to the forward start position
  if (action == "toForward")
    {
    Spidey.toForward();
    }

  // Move the spider forward
  else if (action == "forward")
    {
    Spidey.forward();
    }

  // Get it to standby position
  else if (action == "fromForward")
    {
    Spidey.fromForward();
    }

  // MANY MORE METHODS HERE

  // Default action (stand by)
  else
    {
    Spidey.idle();
    }

  delay(10);
  }

在此级别中，只有Spidey.forward();方法应该可见，而不是过渡方法。 如何设置转换操作的抽象级别？

我的选择：

让Controller返回另一个参数，＆＃34; transition＆＃34;，它将被传递给被调用的方法。问题：这使代码过于复杂。
不是让Controller返回toForward，forward和fromForward，而是让它只返回forward，然后让Spidey处理内部事物（将变量lastAction存储在Spidey内，而不是像我现在所做的那样存储Controller。问题：这会创建更多私有方法，但根据the question linked，这是可以的。
还有其他想法吗？

Answer 1

这取决于Controller现在发生了什么。某处需要有一个类保持当前状态的记录并处理转换序列，并将“转发”请求转换为转换中间状态的序列。也许这是控制器，或者它可能是蜘蛛侠。

对我来说，根据很多不同的魔术字符串调用不同的函数，看起来很糟糕。这看起来更像是一个不处理中间状态的顶级表示。使用枚举，内部状态转换逻辑看起来会更舒服。然后可能有一个成员函数toState（），它将对状态进行排序以达到该状态。

请记住，当中间状态完成时，需要轮询或通知此类，以便可以启动下一个类。如果只有Controller知道这一点，那么在那里拥有那个逻辑可能是有意义的。不要试图在两个类之间复制或拆分该逻辑。

伺服转换应该具有哪些抽象级别？

1 个答案: