Tabのフラグメントクラスは共通化できますか？

Question

Tabのフラグメントクラスを共通化できないか？と考えています。

		@Override
public Fragment getItem(int position){
			switch (position){
				case 0:
					return AllFragment.newInstance(tabList.get(position));
				case 1:
					return CountryFragment.newInstance();
				case 2:
					return RegionFragment.newInstance();
				case 3:
					return PrefecturesFragment.newInstance();
				case 4:
					return EtcFragment.newInstance();
				default:
					return null;
           }
}

という書き方がネットで調べるとベターな感じなのですが、ほとんどレイアウトがそれぞれ違っているので、こういう書き方でもいいんじゃないか？と思うのですが、今回作っているのは全部のタブページが表示しているデータをフィルターかけているだけなので、レイアウトも全く同じでいいので、フラグメントクラスは１つでいけるんじゃないか？と考えました。

ただ、それが実現できなかったので質問させてください…。

		@Override
		public Fragment getItem(int position) {

			AllFragment fragment = new AllFragment();
			fragment.setTab(tabList.get());
			return fragment;
}

こういう感じで毎回newしてやれば、毎回生成されてできるんじゃないか？と思ったのですが、そう簡単ではなかったようで…。

今後は表示ごとにnewされてしまって、挙動がおかしくなってしまいました…。

そこで、質問なのですが、この場合の打開策としてはどういった手段が効果的なんでしょうか？

すでに実体化されているかどうかを見に行って、もし「null」だったらnewしない。みたいな書き方にしたほうがいいのでしょうか？

それとも、ここは共通化を諦めて個別にシングルトンクラスを作ったほうがいいんでしょうか？

Answer 1

fragment.setTabメソッドはAllFragmentで実装したsetterでしょうか？

Fragmentの生成時はsetArgumentsを使って値を渡すようにしましょう。

Comments

[SmartBuzz]

自前クラスで値を渡したいので今回使えないかなと…。

[meilcli]

まずFragmentにsetArgumentsで値を渡したほうがいい理由についてはこちらの記事を参照ください(http://qiita.com/HideMatsu/items/ddf640899cbe1b2027ed)

そしてsetArgumentsで渡すBundleへの自前クラス保存はParcelableを使えばいいです(参考 http://y-anz-m.blogspot.jp/2010/03/androidparcelable.html)

Answer 2

試したやり方で問題ないように思います。
挙動がおかしいとは具体的にどのような挙動になっていますか？

Comments

[SmartBuzz]

５つタブを作っているのですが、１つ目２つ目が空で３つ目、４つ目、５つ目は想定通りの物が入っています。そこから１つ目を開くと今度は５つ目に入っていたデータが入っていて、２つ目をまた開くと５つ目のデータ、あとは全部５つ目のデータが入っている。

という様な挙動です。

[yona]

共通化する前のようにファクトリメソッドを追加して、引数を渡すようにしましょう。

[SmartBuzz]

共通化の前は、同じフラグメントクラスをいっぱい作っていただけなのでファクトリクラスとかではないです…。

ファクトリクラス（使った事も無く30分ぐらいhttp://qiita.com/torinist/items/9dd095d9dcb4b2621948読んでましたが理解できず…。）を使って、getItemメソットでAllFragmentクラスを１つnewInstanceにして、引数をpositionとタブ情報を送ってやるという事ですか？

[yona]

newInstanceのことです。