PrintWritersと.get（）メソッドのArrayList

Question

私は現在ArrayListとPrintWriterを使用することを学んでいます。私の教授は、複数のファイルを使用してファイルを分割する必要がある場所を割り当てました。PrintWriterArrayList。だから私はこのような何かに見えるためのループでPrintwriter秒を初期化：writersはArrayList<PrintWriter>とfilePatternある

for (int i=1; i<=n; i++) { 
    try { 
    this.writers.add(new PrintWriter(new File(filePattern + i))); 
    } 
    catch (java.io.IOException e) { 
    System.out.println("File " + i + " not found."); 
    } 
} 

がソースです。 しかし私のwriteメソッドでは、私は実際に物事を書くためにこれらのPrintWriterを呼び出す必要があります。 int currentFileを使用してファイルを分割して回転しますが、名前を付けていないときにはにあるPrintWriterの使い方が正確ではありません。次のように "get"メソッドを使用できますか？ ：

public void write (String s) throws IOException { 
    if (this.writers.size() > 0) { 
    PrintWriter p = this.writers.get(currentFile); 
    p.println(s); 

    if (this.currentFile == (this.writers.size()-1)) { 
     this.currentFile = 0; 
    } 
    else { 
     this.currentFile++; 
    } 
    } 
    else { 
    System.out.println("No Writers currently exist"); 
    } 
} 

これは機能しませんか？もしそうでなければ、別の方法がありますか？

Answer 1

あなたはthis.currentFileが0とthis.writers.size()-1の間にあるかどうかを確認していない場合は、まだIndexOutOfBoundsExceptionにもたらすことができるthis.writers.get(this.currentFile)かthis.writers[this.currentFile]を行うことができます。これは、シリアル（すなわち非並列）実行には有効です。

さらに、getterを使用してデータ（つまり、currentFile）にアクセスし、セッターがメンバーメソッドでもデータをクリーンに変更するよう助けてくれます。