Activityの責務について

Question

Android アプリケーションの作成について質問させて頂きます．

現在，私が作成しているアプリケーションはMainActivityをNavigationDrawerを用いています．メニューの数だけFragmentを作成しています．各Fragmentには，アクティビティへのコールバック用のインタフェースを設けており，MainActivityで実装しています．Activityは，UIにおける大きな土台であると認識しています．
また，作成中のアプリケーションではパーミッションの取得が必要になっています．そのため，パーミッションに関するコードもMainActivityに記述しています．

本来，Fragmentのコールバック用インタフェースの実装やパーミッションに関する記述はActivityの中で行うものだと思っていますが，Navigation Drawerにしているのと複数のパーミッションを扱う必要があるため，MainActivityだけでコードが300行くらいになっています．

私が今悩んでいるのは，MainActivityだけで300行になっている点で，このActivityに複数の責務を課しているのではないかという疑問があるからです．これは仕方がないで済ましていいのでしょうか？それとも，パーミッション関係は別でクラスを用意するべきでしょうか？(スプラッシュ画面を用意して，そのActivityに任せるというやり方もできるとは思いますが...)

Answer 1

ソースコードを書かれていない以上は適切なアドバイスはできませんが、クラス分けをするのは可読性に繋がりますから、分けれるところは分けるべきです。

300行程度ならまだ良いという意見や、30行以内に収めろという意見もあるのでその辺は貴方次第です。

Comments

[mN6gZkd8fr]

ご回答ありがとうございます．

私の判断で良いということで，今のままでも問題はないのですが，どうも複数のFragmentに対するコールバック用インタフェースの実装とパーミッションの記述がMainActivityに密集しすぎている感じがしました．

> クラス分けをするのは可読性に繋がりますから、分けれるところは分けるべきです。

コールバックの実装をFragmentを扱うMainActivity (Navigation Drawer) 以外で行うのは分かりづらくなるため，MainActivityから切り離すとすればパーミッションの記述だと思います．もし分けるとしたら，新たにActivityを設けるべきでしょうか？それともパーミッション用のFragmentを用意すればいいのでしょうか？
一般的にどのようなやり方で対応しているのかご存じであれば教えて頂けませんか？(Drawer Navigationを使われる方は，こんな事を気にされていないかもしれませんが...)

[stdio]

AndroidJavaにてActivityを新たに作るのは新たな画面を作るという意味でのすので、貴方のしたい事とは違います。
クラスのFragmentを新たに作るのが自然的な形ですが、Activityと密接に関わっているのなら仕方がないとあきらめてもその行数なら文句はありません。

＞＞一般的にどのようなやり方で対応しているのかご存じであれば教えて頂けませんか？
基本的には要素の初期化だけをActivityで行い、後の機能やレイアウトの調整はクラスを分けて実装した方が後のメンテには困りません。
一般的がどの範囲の一般的かは会社や個人間においても違うので詳しくはお話いたしませんが、分けれるクラスはさっさと分ける。要素は後に使い回す可能性がある為、専用のクラスを作っておくと便利です。