1

我正在为我的 android 设备构建一个 HTTP 服务器。

我正在使用很多 IF-ELSE 语句来处理不同的请求。

由于我将与其他人共享我的代码以供以后使用，因此我必须使其尽可能清晰易读。现在，我什至无法轻松阅读我的代码。

我认为问题出在一个类中使用了很多 IF-ELSE 语句。例如。

if(purpose.equals("readProfile"){
     .....
}
else if(purpose.equals("writeProfile"){
     .....
}
    ....

我尝试将它们分类，并根据它们的类别对条件进行排序。但并没有改善很多易读性。然后我尝试在每个条件前写简短的评论。但这让事情变得更加混乱。

可以做些什么来增加条件语句的易读性？

4

5 回答 5

4

正如Luiggi Mendoza所说，这是对上一个问题的跟进......

如果您使用的是Java 7，则可以对字符串使用switch-case 语句

    //month is a String
    switch (month.toLowerCase()) {
        case "january":
            monthNumber = 1;
            break;
          //partsleft out for sake of brevity ..
        default: 
            monthNumber = 0;
            break;
    }

（摘自上面引用的 Oracle Java 教程。）

重构

然而，这个巨大的 if-else 只是问题的一部分。由于这似乎是一个随着时间的推移而增长的结构，我建议进行彻底的重构，并使用在我看来是一种策略模式。你应该：

制定一个涵盖所有用例边界的接口：

interface MyStrategy {
  void execute(MyInputContext input, MyOutputContext output);
}

（使用带有 MyInputContext 和 MyOutputContext 的 void 方法只是一种方法，这只是一个示例，但是要处理具有响应的请求，这是有道理的，就像 Servlet 的工作方式一样）

将大 IF-ELSE 语句的内容重构为该接口的实例（这些将是策略）：

//VERY simplified...
class ReadProfileStrategy implements MyStrategy {
  void execute(MyInputContext input, MyOutputContext output) {
    //do the stuff that was in the if-else block in the "readProfile" part
  }
}

//... at the branching part:
MyInputContext input; //build this here
MyOutputContext output; //build this here

switch (purpose) {
    case "readProfile":
         // no need to always instantiate this, it should be stateless...
         new ReadProfileStrategy().execute();
         break;
    //... left out for sake of brevity
}

重构步骤 2

如果这样做了，您可以将字符串 ID 添加到接口和实例本身，并完全摆脱 if-else 或 switch 语句，您可以创建一个即使通过 IOC 容器（如）填充的 Map，最新，并且完全灵活。

class ReadProfileStrategy implements MyStrategy {
  String getID() {
      return "readProfile";
  }

  void execute(MyInputContext input, MyOutputContext output) {
    //do the stuff that was in the if-else block in the "readProfile" part
  }
}

在处理请求时的类中

private final Map<String, MyStrategy> strategyMap; //fill the map using your favorite approach, like using Spring application context, using the getCode() to provide the key of the map

在处理逻辑中：

MyStrategy strategy = strategyMap.get(purpose);
if(strategy!=null) {
    strategy.execute();
}
else {
    //handle error here
}
于 2013-09-12T08:26:09.207 回答
2

这可能超出范围，但只是一个观察

尝试使用

if("readProfile".equals(purpose){}代替

if(purpose.equals("readProfile"){}.

这将有助于避免 null pinter 异常

于 2013-09-12T08:24:22.953 回答
2

枚举可以提供帮助 - 您还可以向它们添加功能。

public void test(String purpose) {
  if (purpose.equals("readProfile")) {
    // Read.
  } else if (purpose.equals("writeProfile")) {
    // Write.
  }
}

enum Purpose {
  readProfile {
    @Override
    void doIt() {
      // Read.
    }
  },
  writeProfile {
    @Override
    void doIt() {
      // Write.
    }
  };

  abstract void doIt();

}
public void test2(String purpose) {
  Purpose.valueOf(purpose).doIt();
}
于 2013-09-12T08:30:03.987 回答
1

您可以尝试为每个块使用某种带有实现的动作接口，并使用该动作的具体实现预加载地图。

interface Action {
    void execute();
}

Map<String, Action> actions = new HashMap<>();
actions.put("readProfile", new Action() { ... });
actions.put("writeProfile", new Action() { ... });

actionMap.get(purpose).execute();

这也会降低你的圈复杂度。当然，您应该只预加载一次地图。

于 2013-09-12T08:27:59.167 回答
1

好吧，如果将 if-else 条件中的代码分离到另一个类是有意义的，那么也许使用工厂模式。还要使所有分离的类都MyActivity.class使用诸如execute().

工厂根据您传递的字符串决定必须创建什么对象（等），然后调用ReadProfile.class方法。WriteProfile.classexecute()

MyActivity obj = MyFactory.createMyActivity(String)
obj.execute(...);
于 2013-09-12T08:39:08.297 回答