Javaに関する様々な情報をご紹介します。

Javaに関する様々な情報をご紹介します。
評価

0

BufferedReaderを途中でClose

windows環境で以下のような
現在のメモリの空き容量を取得するメソッドを作成
しました。

//------------ここから
int memory = 0;

String[] command = {"cmd", "/c", "typeperf", "-sc", "1", "\"\\Memory\\Available MBytes\""};
Process pro = null;
try
{
    pro = Runtime.getRuntime().exec(command);
}
catch (Exception e)
{
    return 0;
}

InputStream in = pro.getInputStream();
String line;
BufferedReader br = null;
try
{
    br = new BufferedReader(new InputStreamReader(in,"SJIS"));
    int count = 1;
    while ((line = br.readLine()) != null)
    {
        if(count == 3)
        {
            String[] str = line.split(",");
            String edit = str[1].replace("\"", "");
            String[] edit2 = edit.split("[.]");
            memory = Integer.parseInt(edit2[0]);
            break;
        }
        count++;
    }
}
catch (IOException e)
{
    e.printStackTrace();
}
catch (Exception e)
{
}
finally
{
    try{
        if(br != null)
        {
            br.close();
        }
        if(in != null)
        {
            in.close();
        }
    }catch(Exception e1)
    {}
}

return memory;
//------------ここまで

while ((line = br.readLine()) != null)
{
  処理1
}

処理1の部分で文字列を読み取り編集し
メモリの値を取っているのですが
必要な部分を取得して編集が終わったら
すぐさまループを抜け終了しています。

この部分が正しいかどうかよくわかりません。
文字列を取得し終わってもwhileのループ処理が
終わるまでは抜け出さない方がよいのでしょうか。
(途中で抜けた場合、バッファが残るか心配)

3

回答

92605

閲覧

3件の回答

評価

10

>バッファが残るか心
残るが、どうせプロセス終了時には破棄される。

評価

30

>バッファが残るか心配
コードを拝見するとfinally節でちゃんとcloseしているの
で少なくともmainメソッドからreturnする前にバッファは
必ず開放されると思います。そうなることを意図して
finallyをかいたのでは?

余談ですが、Java7からだとtry-with-resource構文を使っ
たほうがさらにすっきりかけると思います。catchや
finallyのところでcloseしなくてよくなるので楽です。

try (BufferedReader br = new ...) {
 ...
} catch (IOException e) {
  ログを出すだけ
}
//この時点では自動的にcloseされている


評価

0

質問者です。

>KSwordOfHasteさん

丁寧な回答ありがとうございます。
勝手な思い込みで出力を全部読み取ってから
クローズ処理をしなきゃいけないと思っていたので
途中でも大丈夫かなと質問させていただきました。

try-with-resource構文について
途中までjava6の環境で作っていてそういう構文があるのを
知りませんでした。
現在はjava7環境で作っているので利用してみようと思います。

ありがとうございました。

質問から6ヶ月以上経過しているので、回答を書き込むことはできません。