8

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ます。

これは偽陽性ですか？かなりバニラの実行パス分析のようです。

4

3 に答える 3

11

私の意見では、これは偽陽性です。リソースは「最終的に」ブロックで閉じられているため、ここで何が問題になるかわかりません。

補足として、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);
    }
}
于 2012-08-02T22:22:40.527 に答える
0

open と close の両方を最初の try 句に移動するとどうなりますか? それらは同じタイプの例外をスローします。if os != null を削除します。

于 2012-08-02T22:50:40.357 に答える
-2

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) {
        }
    }
}
于 2012-08-02T22:35:55.590 に答える