我已经阅读了之前关于用状态/策略模式替换 Java 中的条件逻辑(例如 IF/ELSE)的堆栈交换，但我不确定我的案例是否适合替换。 这里有耦合我看了 - Long list of if statements in Java和 Converting many 'if else' statements to a cleaner approach

我实际上是在编写一个文件下载管理器，这些是我的 IF/ELSE 结构:

1. 如果文件和它的 zip 文件存在，则将 zip 文件移动到 zip 文件目录并读入文件
2. 如果zip文件存在则解压文件并将zip文件移动到zip文件目录并读入文件
3. 如果 zip 文件不存在则从指定的 URL 下载它 然后解压缩并读入文件并将zip文件移动到指定目录。
4. 如果 URL 中不存在 zip 文件，则创建空白文件并将其写入磁盘。

基本上，据我了解，您将这四个条件作为键放在散列映射中，然后值是需要发出的“命令”。 但是我相信您仍然需要一个 If/Else 来决定对作为输入给出的键调用什么操作。所以我看不到好处。有人可以解释一下吗？

最佳答案

我认为你应该使用 GoF 模式 Chain of responsibility .您应该引入两个接口(interface):1) Condition 您将在其中检查适当的条件，例如“如果 zip 文件不存在”并返回 boolean 结果 - 如果满足条件则为“true”，否则为“else”，2) Execution strategy，将运行指定条件的操作，例如“从指定的 URL 下载它然后解压缩并读入文件并将 zip 文件移动到指定的目录。”因此，第一个界面将回答“何时”问题，第二个界面将回答“然后”问题。 “条件”实现和“执行策略”实现应该组合成“元组”(或对、条目等)。这个“元组”应该按您所描述的顺序移动到集合中。然后，当您需要处理 zip 文件时，您将遍历集合、调用条件并检查结果，如果结果为“真”，则调用适当的“执行策略”。此外，条件可以与执行策略相结合，并使用两种方法移动到单一接口(interface)/实现中。描述 zip 文件当前状态的上下文可以在条件/执行策略之间传递。 希望这会有所帮助。

更新。 代码示例(Java 语言)。

/**
 * All implementations should check proper condition
 */
interface Condition { 

  /**
   * Check if condition is satisfied
   *
   * @param pathToFile path to target file
   * @return 'true' if condition is satisfied, otherwise 'false'
   */
  boolean isSatisfied(String pathToFile); //i've made an assumption that you'll manipulate file path for checking file
}
...
/**
 * Childs will wrap some portion of code (if you'll use language, that supports lambdas/functors, this interface/implementation can be replaced with lambda/functor)
 */
interface Action {

  /**
   * Execute some portion of code
   *
   * @param pathToFile path to target file
   */ 
  void execute(String pathToFile);
}
...
class ZipFileExistsCondition implements Condition {

  @Override
  public boolean isSatisfied(String pathToFile) {
   ... //check if zip file exists
  }
}
...
class ZipFileDoesNotExists implements Condition {
  @Override
  public boolean isSatisfied(String pathToFile) {
   ... //download zip file and move it to some temp directory
   //if file downloaded ok, than return 'true' otherwise 'false'
  }
}
...
class AlwaysSatisfiedCondition implements Condition {
  @Override
  public boolean isSatisfied(String pathToFile) {
   ... //always returns 'true', to run action assigned with this condition
  }
}
...
Collection<Map.Entry<Condition, Action>> steps = Arrays.asList(
 new AbstractMap.ImmutableEntry<Condition, Action>(new ZipFileExistsCondition(), 
 new Action() { /*move zip file to zip file directory and read in file*/ }),
 new ZipFileDoesNotExists(), new Action() { /*download it from specified URL and then unzip it and read in file and move zip file to specified directory*/ },
 new AlwaysSatisfiedCondition(), new Action() { /*create blank file and write it out to disk*/  }
);
...
String pathToFile = ...
...
for(Map.Entry<Condition, Action> step: steps) {
 if(!step.getKey().isSatisfied(pathToFile))
   continue;

 step.getValue().execute(pathToFile); 
}  

备注: 1)您可以将“条件”实现为匿名类， 2) 'AlwaysSatisfiedCondition' 可以是单例， 3) 如果您使用的是 Java/Groovy/Scala，则可以使用 Guava/Apache Commons“Predicate”代替“Condition”、“Function”或“Closure”代替“Action”。

如果您需要在第一个“满足”条件和适当的操作执行后退出，那么只需在操作执行后放置“break”/“return”。

关于java - 用状态/策略模式替换 if/else 逻辑，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/26571351/