スレッドセーフな再帰呼び出し（getTotalFiles）の作成方法

Question

私は再帰的なファイルチェックをしていますが、問題は私がそのメソッド自体の中にカウンターを持っていないので、私はそれを外に宣言しました。しかし、問題は、これはスレッドセーフではないということです。

private int countFiles = 0; 
private int getTotalFiles(String path) { 

    File file = new File(path); 
    File listFile[] = file.listFiles(); 

    for (File f : listFile) { 
     if (f.isFile()) { 
      countFiles++; 
     } 
     if (f.isDirectory()) { 
      getTotalFiles(f.getAbsolutePath()); 
     } 
    } 

    return countFiles; 
} 

クラス変数countFilesはスレッドセーフではありません。どのようにこのスレッドを安全にするには？

Answer 1

フィールドは必要ありません。再帰呼び出しの結果をcountFilesに追加するだけです。

private int getTotalFiles(String path) { 
    //... 
    if (f.isFile()) { 
     countFiles++; 
    } 
    if (f.isDirectory()) { 
     countFiles += getTotalFiles(path); 
    } 
} 

Answer 2

countFilesがここではクラスレベル変数である必要がある明確な理由はありません。 countFilesをローカル変数にすることで、コードをスレッドセーフにすることができます。この変数は、メソッドによって返されます。