2017-06-21 8 views
-3

私はあなたがこれにあなたのコードをリファクタリングする必要がこのコードを簡単にするには?

private void outputPatternList(String message) throws IOException { 

    try (BufferedWriter out = new BufferedWriter(new FileWriter(patternList, true))) { 
     out.write(message); 
     out.write("\n"); 
    } 
} 
+2

あなたはここでしか2件の書き込みを行っています。これが最後の書き込みかどうかは分かりません。発信者は知っていますか?おそらくファイルがすでに存在するかどうかをチェックし、接尾辞としてではなく、*接頭辞*(既に存在する場合)として '\ n'を書いてください。 –

+0

"ファイルがすでに存在するかどうかをチェックし、接尾辞としてではなく接頭辞として\ nを書いてください(既に存在する場合)。 私はこれが良いと思います。ありがとう! –

答えて

1

simplierこのコードを作成する方法についていくつかのヒントをお願いしたいと思います:

private void outputPatternList(List<String> lines) throws IOException { 
    StringBuilder builder = new StringBuilder(); 
    for (String line : lines) { 
     builder.append(line).append("/n"); 
    } 
    String message = builder.toString().trim(); 
    try (BufferedWriter out = new BufferedWriter(new FileWriter(patternList, true))) { 
     out.write(message); 
    } 
} 
関連する問題