issues
search
kirikirisu
/
kanazawaApp-2019
stockSupporter
0
stars
1
forks
source link
DIを考慮してリファクタリング & ボタンの外をタップしたらFragmentを閉じるように変更
#68
Closed
tomu28
closed
4 years ago
tomu28
commented
4 years ago
Issueの番号 / Title
8 の実装内容のリファクタリング
ボタンの外をタップしたらFragmentを閉じるように変更
どのように解決したか / How
コールバック先をOptionalで持つように変更
コンストラクタではなく、フィールドで宣言するように変更
なぜそのような解決策を取ったのか / Why
Adapterはコールバックを注入しなくても稼働できるため
Adapter利用側は、とりあえずAdapterを生成しておいて、自分の準備ができたらコールバック先を自分にするように後からセットできるようにするため
メリットとして、コールバック先を
null
にすれば容易かつ明確にイベントの受け取りを遮断できるようになる
その他 / スクリーンショットなど
kirikirisu
commented
4 years ago
LGTM!!
Issueの番号 / Title
8 の実装内容のリファクタリング
どのように解決したか / How
なぜそのような解決策を取ったのか / Why
null
にすれば容易かつ明確にイベントの受け取りを遮断できるようになるその他 / スクリーンショットなど