在不同位置关闭 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，因为在将其作为参数传递给 storeToXML 后，您似乎不需要 FileOutputStream。

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

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

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

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

Best coding practice for closing of FileOutputStream at various location

java

fileoutputstream