java - 在不同位置关闭 FileOutputStream 的最佳编码实践

Question

properties.storeToXML(new FileOutputStream(new File(m_wcontext.getProject().getBaseDirectory() +
         "/" + m_wcontext.getServiceName() + ".properties")),null);

我在 10 个不同的地方调用了类似的方法。在代码审查期间，建议我需要关闭该资源。推荐的方法是使用如下修改代码:但这会使代码变得笨拙，因为我需要重复代码 10 次。

try {
    fios = new FileOutputStream(new File(m_wcontext.getProject().getBaseDirectory() +
                "/" + m_wcontext.getServiceName() + ".properties"));
    properties.storeToXML(fios, null);
} finally {
    if(fios!=null) {
         fios.close();
    }
}

请问下面的方法没问题。还有更好的吗？

    FileOutputStream fios = getFileOutPutStream(m_wcontext.getProject().getBaseDirectory()  +  "/" + m_wcontext.getServiceName() + ".properties");        
    properties.storeToXML(fios, null);

    // ...      

private FileOutputStream getFileOutPutStream(String path) throws IOException {
    FileOutputStream fios=null;
    try {
        fios = new FileOutputStream(new File(m_wcontext.getProject().getBaseDirectory() +
                "/" + m_wcontext.getServiceName() + ".properties"));
        properties.storeToXML(fios, null);
    } finally {
        if(fios!=null) {
            fios.close();
        }
    }
    return fios;
}

Answer 1

假设您使用 Java 7 或更高版本进行编译，您可以使用 try-with-resources 语句来自动关闭资源:

private void writeXML(String path) throws IOException {
    try (FileOutputStream fios = new FileOutputStream(new File(path))) {
        properties.storeToXML(fios, null);
    }
}

我已将方法设为 void，因为在将 FileOutputStream 作为参数传递给 storeToXML 之后，您似乎不需要它>.

要调用此方法，您可以使用以下方法:

String path = m_wcontext.getProject().getBaseDirectory() +
            File.separator + m_wcontext.getServiceName() + ".properties";

try {
    writeXML(path);
} catch (IOException e) {
    e.printStackTrace();
}