s-aska / Justaway-for-Android-Original

Lightweight Twitter Client for Android
http://justaway.info/
MIT License
15 stars 8 forks source link

とにかく setOnClickListener が増えすぎて汚い #139

Closed s-aska closed 10 years ago

s-aska commented 10 years ago

とにかく setOnClickListener が増えすぎて汚いので整理しよう。

例) http://screen.chihaya-pm.org/22f8a1caa3f185fe29137de124f937bc.png http://screen.chihaya-pm.org/0fea344b1ff91c88555024b4b482128a.png

Activityのソース自体は短くならないがonCreateどこまで続くんだみたいな状況は改善される。

B. クラスを分ける

info.justaway.listener.main.* にMainActivity用のリスナーを次々と放り込む

AよりスッキリするがMainActivityのメンバ変数に直接アクセスできないので、リスナーを生成するコンストラクタ(new)とは別のnewInstanceみたいなメソッドを生やしてfinalしたmAdapterを...という具合にややこしくなるかもしれない。

s-aska commented 10 years ago

http://screen.chihaya-pm.org/20f93618816ad94678c3e2c840132622.png 同時にこの滅茶苦茶な定義順のStructureも整理しよう。

  1. onCreate / onDestory など FragmentActivity のメソッドをオーバーライドしたもの、ライフサイクル順
  2. onEventMainThread
  3. 自分で生やしたプライベートメソッド

の順で並び替える

リスナーの定義は関連するViewの表示順位に合わせると見やすい、上部または左側あるViewをソースコード上でも上(先に)で扱う。