0

Androidで、ストレージ内の特定の拡張子をListに追加して、リストに表示するようにしたいです。
再帰の時、一つ上のディレクトリではなく、/sdcardを終了してしまいます。
何故でしょうか?

MainActivity.java

public class MainActivity extends AppCompatActivity {
    private String sdPath = getExternalStoragePath();

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        new MyAsyncTask(this).execute(new File(sdPath));
    }



    // 外部ストレージ(SDカード)のパスを取得する。
    public static String getExternalStoragePath() {
        String path;
        // MOTOROLA 対応
        path = System.getenv("EXTERNAL_ALT_STORAGE");
        if (path != null) {
            return path;
        }
        // Samsung 対応
        path = System.getenv("EXTERNAL_STORAGE2");
        if (path != null) {
            return path;
        }
        // 旧 Samsung + 標準 対応
        path = System.getenv("EXTERNAL_STORAGE");
        if (path == null) {
            path = Environment.getExternalStorageDirectory().getPath();
        }
        // HTC 対応
        File file = new File(path + "/ext_sd");
        if (file.exists()) {
            path = file.getPath();
        }
        // その他機種
        return path;
    }
}

MyAsyncTask.java

public class MyAsyncTask extends AsyncTask<File, Void, String> {
    Context context;
    private File[] files;
    private List<String> pickedFileList = new ArrayList<String>();
    private ListView lv;

    public MyAsyncTask(Context context) {
        this.context = context;
    }

    @Override
    protected String doInBackground(File... params) {
        pickOnDir(params[0]);
        return "hoge";
    }

    public void pickOnDir(File dir) {
    files = dir.listFiles();
    if (files != null) {
        Log.d("file Info", dir.getAbsolutePath() + "を検索中...");
        for (int i = 0; i < files.length; i++) {
            if (files[i].isDirectory()) {
                Log.d("file Info", files[i].getAbsolutePath() + "にフォルダを確認しました");
                pickOnDir(files[i]);
            }else if (files[i].isFile()) {
                if (files[i].getName().endsWith(".js")) {
                    Log.d("file Info", files[i].getAbsolutePath() + "にJSファイルを確認しました");
                    pickedFileList.add(files[i].getAbsolutePath());
                } else {
                    Log.d("file Info", files[i].getAbsolutePath() + "にファイルを確認しました");
                }
            } else {
                Log.d("file Info", files[i].getAbsolutePath() + "は不明なファイルです");
            }
        }
        Log.d("file Info", dir.getAbsolutePath() + "の検索を完了したので戻ります");
        return;
    }
}

    @Override
    protected void onPostExecute(String results) {
        lv = (ListView) ((Activity) this.context).findViewById(R.id.listView);
        ArrayAdapter<String> adapter = new ArrayAdapter<String>(this.context, android.R.layout.simple_expandable_list_item_1, pickedFileList);
        lv.setAdapter(adapter);
    }
}

ログ

/sdcardを検索中...
/sdcard/DCIMにフォルダを確認しました
/sdcard/DCIM/Cameraにフォルダを確認しました
/sdcard/DCIM/Cameraを検索中...
/sdcard/DCIM/Camera/XXX.jpgにファイルを確認しました
/sdcardの検索を完了したので戻ります

2
  • Toastではなく、Log出力しても結果は同じでしょうか? また、sdPathが期待したものかをLog出力してみるのもいいかもしれません。
    – webarata3
    Commented 2016年2月24日 19:44
  • 02-25 10:05:18.368 3112-3112/com.example.innsbluck.jspicker D/file Info: /sdcardを検索中... 02-25 10:05:18.370 3112-3112/com.example.innsbluck.jspicker D/file Info: /sdcard/DCIMにフォルダを確認しました 02-25 10:05:18.392 3112-3112/com.example.innsbluck.jspicker D/file Info: /sdcard/DCIMを検索中... 02-25 10:05:18.393 3112-3112/com.example.innsbluck.jspicker D/file Info: /sdcard/DCIM/Cameraにフォルダを確認しました 02-25 10:05:18.403 3112-3112/com.example.innsbluck.jspicker D/file Info: /sdcard/DCIM/Cameraを検索中... で止まりました。 Commented 2016年2月25日 1:07

1 件の回答 1

2

メインスレッドからこのコードを実行すると、大抵の端末では止まると思います。所謂ANR(Application Not Responding)が発生したのではないでしょうか?これは、メインスレッドで5秒以上かかる処理をやった場合に起こります。ファイルの検索は時間がかかるため、別スレッドで行うべきです。

5
  • 非同期処理を行う、ということでよろしいでしょうか? Commented 2016年2月25日 1:08
  • そうです。ストレージへのアクセスはそれなりに時間がかかりますので、ディレクトリの中身全てを再帰で検索したりする場合はANRが起きかねません。
    – M.M.
    Commented 2016年2月25日 1:20
  • /sdcardを検索中... /sdcard/DCIMにフォルダを確認しました /sdcard/DCIM/Cameraにフォルダを確認しました /sdcard/DCIM/Cameraを検索中.../sdcard/DCIM/Camera/XXX.jpgにファイルを確認しました /sdcardの検索を完了したので戻ります となるのは何故でしょうか...? 実際は/sdcard/DCIM/Cameraの検索を完了したので戻りますとなるべきなのですが... Commented 2016年2月25日 2:34
  • 多分フィールドにprivate File[] filesと定義して、再帰処理の内部でそこへアクセスしてしまっているからです。再帰処理の内部はfiles = dir.listFiles();ではなく、File[] files = dir.listFiles();としないと内容が再帰中に書き変わってしまいます。
    – M.M.
    Commented 2016年2月25日 4:01
  • 腑に落ちました うまくいきました ありがとうございます! Commented 2016年2月25日 4:58

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。