java中的流关闭异常

Question

请看看我做了什么

private InputSource getContent(String fName) throws SAXException, IOException, ServiceException {
        // Some code here
        if(currentNodeRef != null)
        {
            ContentReader reader = contentService.getReader(currentNodeRef,ContentModel.PROP_CONTENT);

            InputStream inputStream = null;
             try 
             {  
                    inputStream = new BufferedInputStream(reader.getContentInputStream(),16384);

                    return new InputSource(inputStream);
             }
             finally
             {
                 if(inputStream!=null)
                    try 
                    {
                        inputStream.close();
                    } catch (IOException e) {
                        // TODO Auto-generated catch block
                        e.printStackTrace();
                    }
             }
        }
        return new InputSource();
}

在我的 parseDocument 方法中，我调用了上述方法。

parseDocRoot(getContent(fName),path);

在 parseDocRoot 中

public  void parseDocRoot (InputSource ins, String path) throws SAXException, IOException,
  ParserConfigurationException, ServiceException
  {
          DocumentBuilderFactory factory = DocumentBuilderFactory.newInstance();
          DocumentBuilder builder = factory.newDocumentBuilder();
            builder.setEntityResolver(new EntityResolver() {
                public InputSource resolveEntity(String publicId, String systemId)
                        throws SAXException, IOException {
                     return new InputSource(new ByteArrayInputStream(new byte[0])); 
                }
            });
        Document doc = builder.parse(ins);
        NodeList list = doc.getChildNodes();
        parseDocument(list,path);
  }

我得到一个错误说 Stream Not closed 并且在调试上面的代码时我发现错误就行了

Document doc = builder.parse(ins);

请帮我找到解决办法。

Answer 1

我认为错误是流 已关闭。原因是你有一个 finally 块：

         try 
         {  
                inputStream = new BufferedInputStream(reader.getContentInputStream(),16384);

                return new InputSource(inputStream);
         }
         finally
         {
             if(inputStream!=null)
                try 
                {
                    inputStream.close();
                } catch (IOException e) {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }
         }

您关闭 finally 块中的流。此代码在从方法返回之前立即执行。重要的是，直到方法返回并由parse 方法处理后，InputStream 才会使用。

您应该将 finally 块放在整个代码块周围——当您真正完成时关闭流。最简单的方法是内联您的getContent 方法并在调用parse 之后放置finally 块。完成此操作后，您可能会想出一种封装该逻辑的方法，但这会有点棘手，因为您肯定需要保留 InputStream 的句柄，直到完成解析才能关闭它.

另一个更简单的选择是让getContent 返回一个Document，而不是简单地在该方法中移动parseDocRoot(getContent(fName),path);。