优化重复代码，提升复用性方法分享

在软件开发中，重复的条件判断和操作会导致代码冗余，降低可读性和维护性。本文针对这一问题，提出一种高效的重构策略：将条件判断与后续操作封装成单一职责的函数，实现“检查并执行”的完整逻辑。以奥赛罗棋盘游戏为例，通过重构棋盘逻辑中重复的`if`语句块，展示了如何利用函数封装来简化代码、提高可维护性，并遵循DRY原则。优化后的代码不仅减少了重复，还提升了可测试性，使得业务逻辑更加清晰。掌握此技巧，有助于开发者编写更健壮、更优雅的代码，提升软件系统的整体质量。

冗余条件判断的挑战

在软件开发中，我们经常会遇到需要对相似条件进行多次判断，并执行相应操作的场景。一个典型的例子是，当某个操作（method2）依赖于某个条件（method1）时，如果这个模式需要应用于不同的参数集，就可能出现以下形式的重复代码：

if (method1(parametersA)) {
  method2(parametersA);
}
if (method1(parametersB)) {
  method2(parametersB);
}
// ...以此类推，针对不同的参数集重复多次

这种模式在奥赛罗（Othello）这样的棋盘游戏中尤为常见，例如，在下棋时需要检查所有八个方向是否存在可翻转的对手棋子，并执行翻转操作。如果对每个方向都编写一个独立的if语句，代码将变得冗长、难以阅读，且不易维护。每次需要修改检查逻辑或翻转逻辑时，都必须在多个地方进行重复修改，增加了出错的风险。

封装条件与操作：提升代码优雅性

为了解决上述问题，一种高效的重构策略是将条件判断 (method1) 和后续操作 (method2) 封装到一个单一的、高内聚的函数中。这个新函数将负责完成“检查并执行”的完整逻辑。

核心思想

将if (condition) { action(); }这一组合逻辑抽象为一个新的函数。这个新函数接收所有必要的参数，并在内部完成条件判断和操作执行。

示例：奥赛罗棋盘逻辑重构

假设在奥赛罗游戏中，canFlip(board, row, col, direction) 方法用于检查给定方向上是否有可翻转的棋子，而 flipPieces(board, row, col, direction) 方法用于实际翻转这些棋子。

重构前的代码（冗余示例）：

// 检查并翻转第一个方向
if (canFlip(board, currentRow, currentCol, Direction.NORTH)) {
    flipPieces(board, currentRow, currentCol, Direction.NORTH);
}

// 检查并翻转第二个方向
if (canFlip(board, currentRow, currentCol, Direction.NORTHEAST)) {
    flipPieces(board, currentRow, currentCol, Direction.NORTHEAST);
}

// ...以此类推，重复八次，每次针对一个方向
// if (canFlip(..., Direction.EAST)) { flipPieces(..., Direction.EAST); }
// if (canFlip(..., Direction.SOUTHEAST)) { flipPieces(..., Direction.SOUTHEAST); }
// ...

重构后的代码（封装示例）：

首先，定义一个封装了检查与翻转逻辑的新方法：

/**
 * 检查指定方向是否有可翻转的棋子，如果有则执行翻转操作。
 * @param board 当前棋盘状态
 * @param row 当前落子行
 * @param col 当前落子列
 * @param direction 检查的方向
 */
public void checkAndFlip(Board board, int row, int col, Direction direction) {
    if (canFlip(board, row, col, direction)) {
        flipPieces(board, row, col, direction);
    }
}

然后，在主逻辑中，我们可以通过循环遍历所有方向来调用这个封装好的方法，大大简化了代码：

// 定义所有可能的方向
Direction[] allDirections = {
    Direction.NORTH, Direction.NORTHEAST, Direction.EAST, Direction.SOUTHEAST,
    Direction.SOUTH, Direction.SOUTHWEST, Direction.WEST, Direction.NORTHWEST
};

// 遍历所有方向，执行检查和翻转
for (Direction dir : allDirections) {
    checkAndFlip(board, currentRow, currentCol, dir);
}

通过这种方式，原本重复的八个if语句被一个简洁的循环所取代，代码量减少，逻辑更加清晰。

封装的优势

1. 提高可读性： 业务逻辑被抽象到具有明确名称的函数中，使得代码意图一目了然。例如，checkAndFlip 明确表达了其功能。
2. 减少代码重复（DRY原则）： 避免了相同的条件判断和操作逻辑在多处出现，降低了维护成本。
3. 增强可维护性： 如果canFlip或flipPieces的内部逻辑发生变化，只需要修改checkAndFlip函数内部即可，无需在多处进行修改。
4. 提升可测试性： checkAndFlip作为一个独立的单元，更容易编写单元测试来验证其行为。
5. 更好的抽象： 将底层细节隐藏在新函数内部，使得上层调用者无需关心具体的检查和翻转过程，只需关注“检查并翻转”这个高层概念。

注意事项与扩展

• 函数命名： 封装后的函数应采用清晰、描述性的名称，准确反映其功能，如checkAndFlip、validateAndProcess等。
• 参数设计： 确保封装函数接收所有必需的参数，以便在内部完成其职责。
• 单一职责原则： 封装函数应尽量遵循单一职责原则，即一个函数只做一件事情。在本例中，checkAndFlip 的职责是“检查是否满足条件并在满足时执行操作”，这仍然是一个单一的、高层次的职责。
• 何时不封装： 如果每个if语句中的条件或操作逻辑差异巨大，以至于封装后并不能带来明显的简化，反而可能增加抽象的复杂性，那么此时不进行封装可能是更好的选择。封装的目的是简化，而不是为了封装而封装。
• 更复杂的场景： 对于更复杂的条件组合或操作序列，可以考虑使用策略模式、命令模式或责任链模式等设计模式来进一步解耦和组织代码。

总结

通过将重复的条件判断与后续操作封装到一个独立的函数中，我们可以有效地消除代码冗余，提升代码的可读性、可维护性和复用性。这种重构技巧简单而强大，是编写高质量、专业级代码的重要实践之一。在面对大量相似的if语句时，优先考虑这种封装模式，将有助于构建更加健壮和优雅的软件系统。

终于介绍完啦！小伙伴们，这篇关于《优化重复代码，提升复用性方法分享》的介绍应该让你收获多多了吧！欢迎大家收藏或分享给更多需要学习的朋友吧~golang学习网公众号也会发布文章相关知识，快来关注吧！