Activityの中で、DBからの値を取りたい

Question

スレッド間通信について教えて下さい。

ActivityのonCreateの中でMainFragmentPagerAdapterを呼び出す前に、DBからデータをリスト形式で取ってきて、Activityのグローバル変数に格納したいのですが、うまくいきません。

やりたい事は、タブの名前をDBに格納された「tab」テーブルを一覧取得して、「title」列を全件タブのタイトルに登録したいというものです。

	@Override
	protected void onCreate(Bundle saveInstanceState){

		super.onCreate(saveInstanceState);
		setContentView(R.layout.activity_collection_list);

		tabList = new ArrayList<>();

		Toolbar toolbar = (Toolbar)findViewById(R.id.toolbar);
		setSupportActionBar(toolbar);

		try{
		getTabInfo(this);
		Thread.sleep(1000);
		}catch (Exception e){

		}
		
		mPagerAdapter = new MainFragmentPagerAdapter(getSupportFragmentManager());

		mViewPager = (ViewPager)findViewById(R.id.container);
		mViewPager.setAdapter(mPagerAdapter);

		TabLayout tabLayout = (TabLayout)findViewById(R.id.tabs);
		tabLayout.setupWithViewPager(mViewPager);
	}

現状は、クリエイトの中で、MainFragmentPagerAdapterでタブを生成する前に、スリープをさせて、getTabInfo(this)

	private void getTabInfo(final Activity activity){
		new Thread(new Runnable() {
			@Override
			public void run() {
				final List<Tab> tabList = ((DataCollecotor) getApplication()).getDataCollecotorDatabase().getTabList();

				activity.runOnUiThread(new Runnable() {
					@Override
					public void run() {
						titleTest = tabList.get(0).getTitle();
					}
				});
			}
		}).start();
	}

final List<Tab> tabList = ((DataCollecotor) getApplication()).getDataCollecotorDatabase().getTabList();

この部分でtabListには入って欲しいと思って書いています。

titleTest = tabList.get(0).getTitle();

この部分は試験的に書きましたがうまくいきませんでした。

どうやればDBから値を取ってActivityの中で使えるようになるのか、ご教授下さい。
よろしくお願いします。

Answer 1

DataCollecotorがどんなクラスかはわかりませんが、非同期処理で行う理由はなんでしょうか。
そんなにたくさんのデータを処理するつもりですか？

Comments

[SmartBuzz]

DataCollectorの中で、DataCollectorDatabaseをnewして、ここでnewしたものをActivtyに渡せないので、非同期処理でやっています。

DB丸ごとというわけではないですが、タブの情報が入ったテーブル６カラムを取りに行ってタブを生成するサブクラスのgetPageTitleにその情報を渡したいと思ってこの処理を作っています。

[yona]

インスタンスを渡せないから非同期処理でやるの意味がわかりません。

それならgetDataCollecotorDatabaseはなにを返却しているんですか？

[SmartBuzz]

あああああーーーーー！
そうか…。そらですね…。

前任者のコードが、別スレッド作ってDBからリストを取ってくる処理になってたのでそれを真似して作っていたのですが…。
確かにスレッドを作らなくても
((DataCollector)getApplication()).getDataCollectorDatabase();
でDBの情報が取れますね！！

スレッドを使う意味ないですね…。

[yona]

ないことはないです。
UIスレッドで大量のデータにアクセスするときはUIスレッドの進行をブロックしないように非同期処理で書きます。今回は画面開始時にタブは必ず必要なのでUIスレッドでデータを取得した方がいいと思いますよ。

でも、理解せずに真似するのは危険ですね。

[SmartBuzz]

大量のデータを取得する時はスレッドを分ける。
少量のデータを取得する時はスレッドを分け無くていい。

スレッドを分ける意味は、処理を邪魔しないように…という事なんですね！
ありがとうございます！

コードもろくにかけず会社で突然Android開発を言われ、期限だけが迫ってきて誰も教えてくれない状況下で、リファレンスも読めないままでとにかく理解よりもなんとか作らないと…と付け焼刃でこの様なコピペしまくり状態でつまづいて、いろんな方のブログを読んでも理解できないといつも、teratailに書き込んでいます…。

いつも素早い回答本当に大変助かります。ありがとうございます！

[yona]

本当はできるだけ非同期にしたいけど、非同期処理の管理が複雑化するのでその辺はバランスをとらないといけないですね。

なるほど、困った状況ですね。
「こうゆう理解、認識をしている」といった事柄も質問に書いてもいいかもしれないですね。