如何避免许多if-else条件

Question

我读过很多关于代码重构和避免if-else语句的主题。实际上，我有一个类，我使用了很多if-else条件。

更多细节：我正在使用pull解析器，在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();
     }

现在，我想用别的东西，而不是那些if else条件，但我不知道是什么。

你能给我举个例子吗？

Answer 1

在Java7中，你可以在字符串上切换。如果你可以使用它，你可以使用它；-）

Answer 2

在这种特殊情况下，由于除了附加的字符串之外，所有 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 && xpp.getText()!=null) {
        map.put( soapResponse, current += xpp.getText());
    }
    eventType = xpp.next();
}

没有“如果”...然后...否则”。甚至没有为策略模式等增加多个类的复杂性。地图是你的朋友。在某些情况下，策略是伟大的，但是这个简单到不需要策略也能解决。

Answer 3

试着看看策略模式。

< li >创建一个接口类来处理响应(IMyResponse) < ul > < li >使用此IMyResponse创建AdvisoryBoardResponse，EditorialBoardResponse类

小例子：

// 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);