使用哪种设计模式来改进这个Java程序

Question

我编写了一个程序来解析XML文件，以获取一个名为SerialNum的特定标记值，该值包含在Header标记中。该文件的构造如下：

< li >它包含一个标题和一个正文 < li >标题可能包含许多SerialNum标签。我们需要提取最后一个标签的值。

我使用Stax解析器获取SerialNum值，并编写了以下代码：

public String findIdValue(HttpServletRequest request) {

    String serialNumberValue = null;

    if(request != null){
        ServletInputStream servletInstream;
        try {
            servletInstream = request.getInputStream();
            XMLInputFactory factory = XMLInputFactory.newInstance();
            XMLStreamReader xmlStreamReader = factory.createXMLStreamReader(servletInstream);
            //begin parsing if we get <Header>
            //end parsing if we get <Header/> or </Header>

            int event = xmlStreamReader.getEventType();
            boolean enableToParse = false;
            boolean gotSerialNumber = false;
            boolean parseComplete = false;

            while( (xmlStreamReader.hasNext()) && (!parseComplete) ){
                switch(event) {
                case XMLStreamConstants.START_ELEMENT:
                    if("Header".equals(xmlStreamReader.getLocalName())){
                        //tag is header, so begin parse
                        enableToParse = true;
                    }else if(("SerialNum".equals(xmlStreamReader.getLocalName())) && (enableToParse) ){
                        //tag is serialNum so enable to save the value of serial number
                        gotSerialNumber = true;
                    }
                    break;
                case XMLStreamConstants.CHARACTERS:
                    //get serial number and end the parsing
                    if(gotSerialNumber){
                        //get wsa and end the parsing
                        serialNumberValue = xmlStreamReader.getText();
                        gotSerialNumber = false;                            
                    }

                    break;
                case XMLStreamConstants.END_ELEMENT:
                    //when we get </Header> end the parse
                    //when we get </SerialNum> reinit flags
                    //when we get </Header> end the parse even we don't get a serial number
                    if("Header".equals(xmlStreamReader.getLocalName())){
                        parseComplete= true;
                    }else if("SerialNum".equals(xmlStreamReader.getLocalName())){
                        //reinit flag when we get </SerialNum> tag
                        gotSerialNumber = false;
                    }
                    break;
                default:
                    break;
                }
                event = xmlStreamReader.next();
            }


        } catch (final XMLStreamException e) {
            //catch block
            LOG.info("Got an XMLStreamException exception. " + e.getMessage());
        }
        catch (final IOException e1) {
            //catch block
            LOG.info("Got an IOException exception. " + e1.getMessage());
        }

    }


    return serialNumberValue;
}

此代码提取所需的值，但代码质量不是很好：它不容易阅读和维护。它包含一个开关案例，如果其他块嵌套在 while 循环中。使用哪种设计模式来提高代码质量？

Answer 1

我不认为你的代码需要一个设计模式。但是一个更干净的代码会非常好。

我同意Louis F.在评论中的建议：“作为第一步，您可以尝试在不同的方法中提取不同开关案例的代码，并给它们有意义的名称。”。

我认为你的代码也有太多的注释。这是一种< code >代码味道。仅举一个例子:

if("Header".equals(xmlStreamReader.getLocalName())){
    //tag is header, so begin parse
    enableToParse = true;
}

删除注释并用代码解释它怎么样？

if(isTagHeader(xmlStreamReader)) {
    enableToParse = true;
}

只是一些想法…你的代码看起来并不可怕。但我认为这里的重点不是设计模式。

如果您有兴趣深入了解它，我强烈推荐Rober C.Martin（Bob叔叔）的《清洁代码》一书。