Question

Eclipse（Juno）发出以下警告：

潜在的资源泄漏：'os'可能无法关闭

在此代码中try正文的第一行：

static void saveDetails(byte[] detailsData) {
    OutputStream os = null;
    try {
        os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE);
        os.write(detailsData);
    } catch (IOException e) {
        Log.w(LOG_TAG, "Unable to save details", e);
    } finally {
        if (os != null) {
            try {
                os.close();
            } catch (IOException ignored) {
            }
        }
    }
}

声明方法openFileOutput会抛出FileNotFoundException。

这是假阳性吗？这似乎是一个相当普遍的执行路径分析。

Answer 1

在我看来，这是误报。您的资源在“finally”块中关闭，因此我无法看到此处可能出现的问题。

作为旁注，如果您使用的是Java 7，我建议使用“try-with-resources”习惯用法。

static void saveDetails(byte[] detailsData) {    
    try (OutputStream os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE);) {
        os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE);
        os.write(detailsData);
    } catch (IOException e) {
        Log.w(LOG_TAG, "Unable to save details", e);
    }
}

Answer 2

如果你同时打开和关闭第一个试试条款怎么办？他们抛出相同类型的异常。并删除if os！= null。

Answer 3

我的猜测是因为你在结束前有if (os != null)。由于它是有条件的，因此OutputStream可能没有关闭。

如果你尝试会发生什么：

static void saveDetails(byte[] detailsData) {
    OutputStream os = null;
    try {
        os = sContext.openFileOutput(DETAILS_FILE_NAME, Context.MODE_PRIVATE);
        os.write(detailsData);
    } catch (IOException e) {
        Log.w(LOG_TAG, "Unable to save details", e);
    } finally {
        try {
            os.close();
        } catch (IOException ignored) {
        }
    }
}

为什么此代码会生成“潜在资源泄漏”警告？

3 个答案: