如何避免很多其他条件

Question

我已经阅读了很多关于代码重构和避免if else语句的主题。 其实，我有一个课，我使用了很多if - else条件。

更多细节：我使用的是拉解析器和我SOAP响应的每一行，我会检查是否有标签我感兴趣的，如果没有，检查另一个标记等：

if(eventType == XmlPullParser.START_TAG) { 
      soapResponse= xpp.getName().toString(); 

      if (soapResponse.equals("EditorialOffice")){ 
       eventType = xpp.next(); 
       if (xpp.getText()!=null){ 
       editorialOffice += xpp.getText(); 
       } 
      } 
      else if (soapResponse.equals("EditorialBoard")){ 
       eventType = xpp.next(); 
       if (xpp.getText()!=null){ 
       editorialBoard += xpp.getText(); 
       } 
      } 
      else if (soapResponse.equals("AdvisoryBoard")){ 
       eventType = xpp.next(); 
       if (xpp.getText()!=null){ 
       advisoryBoard += xpp.getText(); 
       } 
      } 
     } 
     eventType = xpp.next(); 
    } 

现在，我想用其他的，而不是那些如果其他条件，但我不知道是什么。

你可以给我一个例子或一个很好的教程页吗？

谢谢。

Answer 1

在这个特定的情况下，由于码是所有3的情况下，除了字符串基本上相同被附加到，我将不得不为每个字符串的映射条目正在构建：

Map<String,String> map = new HashMap<String,String>(); 
map.put("EditorialOffice",""); 
map.put("EditorialBoard",""); 
map.put("AdvisoryBoard",""); 
// could make constants for above Strings, or even an enum 

，然后将您的代码更改为以下内容

if(eventType == XmlPullParser.START_TAG) { 
    soapResponse= xpp.getName().toString(); 
    String current = map.get(soapResponse); 
    if (current != null) { 
     eventType = xpp.next(); 
     if (xpp.getText()!=null){ 
      map.put(soapResponse, current += xpp.getText()); 
     } 
    } 
    eventType = xpp.next(); 
} 

否“if ... then ... else”。甚至没有为战略模式增加多个类的复杂性等。地图是你的朋友。在某些情况下策略很棒，但是这个策略非常简单，无需解决。

Answer 2

除了zzzzzzz（etc。）的评论......请记住，您正在使用XmlPullParser，它使您可以编写您喜欢的代码。你可以注册一些回调来分割你的代码并使其更好，但如果可能的话，只需使用SimpleXML库或类似的代码。

此外，你可以重构你的代码，使其更具可读性和更少的冗长。例如，为什么你在每个if语句中调用xpp.next()？为什么不只在外面调用它：

if(eventType == XmlPullParser.START_TAG) { 
    soapResponse= xpp.getName().toString(); 
    if (soapResponse.equals("EditorialOffice") && xpp.getText()!=null){ 
     editorialOffice += xpp.getText(); 
    } 
    else if (soapResponse.equals("EditorialBoard") && xpp.getText()!=null){ 
     editorialBoard += xpp.getText(); 
    } 
    else if (soapResponse.equals("AdvisoryBoard") && xpp.getText()!=null){ 
     advisoryBoard += xpp.getText(); 
    } 
} 
eventType = xpp.next(); 

Answer 3

在Java 7中，您可以在字符串上进行切换。你可以使用它，如果你可以使用它;-)

Answer 4

试着看看战略模式。

• 做一个接口类用于处理响应（IMyResponse）
  • 使用此IMyResponse创建AdvisoryBoardResponse，EditorialBoardResponse类
• 创建与soapresponse值为键，你的战略，作为一个词典值
• 然后，您可以使用IMyResponse类的方法从字典中获取它

小例子：

// Interface 
public interface IResponseHandler { 
    public void handleResponse(XmlPullParser xxp); 

} 

// Concrete class for EditorialOffice response 
private class EditorialOfficeHandler implements IResponseHandler { 
    public void handleResponse(XmlPullParser xxp) { 
     // Do something to handle Editorial Office response 
    } 
} 

// Concrete class for EditorialBoard response 
private class EditorialBoardHandler implements IResponseHandler { 
    public void handleResponse(XmlPullParser xxp) { 
     // Do something to handle Editorial Board response 
    } 
} 

在您需要创建的处理程序的地方：

Map<String, IResponseHandler> strategyHandlers = new HashMap<String,IResponseHandler>(); 
strategyHandlers.put("EditorialOffice", new EditorialOfficeHandler()); 
strategyHandlers.put("EditorialBoard", new EditorialBoardHandler()); 

如果您收到响应：

IResponseHandler responseHandler = strategyHandlers.get(soapResponse); 
responseHandler.handleResponse(xxp); 

Answer 5

你可以创建一个对ResponseHandler接口三个实现，一个用于if/else构造的每个分支。

然后有一个映射将不同的soapResponses映射到一个处理程序，或者是一个包含所有处理程序的列表（如果它可以处理该soapResponse）。

您还应该能够将一些样板代码移动到响应处理程序类的常见可能抽象实现。

由于经常有这样的许多变化。通过利用代码重复一个实际上只需要一个实现：

class ResponseHandler{ 
    String stringToBuild = "" // or what ever you need 
    private final String matchString 

    ResponseHandler(String aMatchString){ 
     matchString = aMatchString 
    } 
    void handle(XppsType xpp){ 
     if (xpp.getName().toString().equals(matchString){ 
      eventType = xpp.next(); 
      if (xpp.getText()!=null){ 
       editorialOffice += xpp.getText(); 
      } 
     } 
    } 
} 

你的代码变得

List<ResponseHandler> handlers = Arrays.asList(
    new ResponseHandler("EditorialOffice"), 
    new ResponseHandler("EditorialBoard"), 
    new ResponseHandler("AdvisoryBoard")); 
if(eventType == XmlPullParser.START_TAG) { 
    for(ResponseHandler h : handlers) 
     h.handle(xpp); 
} 

Answer 6

广大疑问，这是一个并没有真正的答案。 （我不使用肥皂经常）

这里只是一些想法基于代码：

首先，你可以狮重复的代码

if (soapResponse.equals("EditorialOffice") 
||soapResponse.equals("EditorialBoard") 
||soapResponse.equals("AdvisoryBoard")){ 

你可以做的另一个好处是发挥周围的开关staments像：

switch(soapResponse){ 
case "EditorialOffice": 
case "EditorialBoard": 
case "AdvisoryBoard": 
eventType = xpp.next(); 
       if (xpp.getText()!=null){ 
       advisoryBoard += xpp.getText(); 
       } 
break; 

你也应该考虑打破你考到小功能：

public bool interestingTag(string s){ 
return (soapResponse.equals("EditorialOffice") 
    ||soapResponse.equals("EditorialBoard") 
    ||soapResponse.equals("AdvisoryBoard")); 
} 

    public processData(xpp){ 
    eventType = xpp.next(); 
        if (xpp.getText()!=null){ 
        editorialBoard += xpp.getText(); 
        } 
    ....} 

这样你可以处理所有的答案在一个while循环，你超长的if else变成5〜10线功能

但正如我说，有做同样的事情这么多的好方法

Answer 7

你还没有提到，如果你可以或使用Java 7.从那个Java版本，你可以使用Strings in switch statements。

除此之外，封装逻辑每种情况下是一个好主意，例如：

Map<String, Department> strategyMap = new HashMap<String, Department>(); 
strategyMap.put("EditorialOffice", new EditorialOfficeDepartment()); 
strategyMap.put("EditorialBoard", new EditorialBoardDepartment()); 
strategyMap.put("AdvisoryBoard", new AdvisoryBoardDepartment()); 

然后，你可以简单地从地图上选择了正确的策略，并使用它：

String soapResponse = xpp.getName(); 
Department department = strategyMap.get(soapResponse); 
department.addText(xpp.getText()); 

Department当然是在接口...

Answer 8

你可以定义一个枚举如下：

public enum SoapResponseType { 
    EditorialOffice(1, "description here") { 
     public void handle(XmlPullParser xpp) { 
      //do something you want here 
      return null; 
     } 
    }, 
    EditorialBoard(2, "description here") { 
     public void handle(XmlPullParser xpp) { 
      //do something you want here 
      return null; 
     } 
    }, 
    AdvisoryBoard(3, "description here") { 
     public void handle(XmlPullParser xpp) { 
      //do something you want here 
      return null; 
     } 
    }; 

    public static SoapResponseType nameOf(String name) { 
     for (SoapResponseType type : values()) { 
      if (type.getName().equalsIgnoreCase(name)) { 
       return type; 
      } 
     } 
     return null; 
    } 

    public void handle(XmlPullParser xpp) { 
     return null; 
    } 
} 

使用上述枚举这样的：

SoapResponseType type = SoapResponseType.nameOf("input string"); 
if (type != null) { 
    type.handle(xpp); 
} 

它是干净的代码，是不是！