質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
RxJava

RxJavaは、リアクティブプログラミングができるReactiveXのJava向けの実装。軽量であり、その他のライブラリへの依存がなく、RxJavaのjarをパスに通せば使用できます。バージョン2.3からはAndroidもサポートしています。

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

Kotlin

Kotlinは、ジェットブレインズ社のアンドリー・ブレスラフ、ドミトリー・ジェメロフが開発した、 静的型付けのオブジェクト指向プログラミング言語です。

Q&A

解決済

2回答

2490閲覧

android: Observe

Nippun

総合スコア1147

RxJava

RxJavaは、リアクティブプログラミングができるReactiveXのJava向けの実装。軽量であり、その他のライブラリへの依存がなく、RxJavaのjarをパスに通せば使用できます。バージョン2.3からはAndroidもサポートしています。

SQLite

SQLiteはリレーショナルデータベース管理システムの1つで、サーバーではなくライブラリとして使用されている。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

Kotlin

Kotlinは、ジェットブレインズ社のアンドリー・ブレスラフ、ドミトリー・ジェメロフが開発した、 静的型付けのオブジェクト指向プログラミング言語です。

0グッド

0クリップ

投稿2020/01/03 00:27

編集2020/01/03 00:35

kotlin

1override fun onViewCreated(view: View, savedInstanceState: Bundle?) { 2 super.onViewCreated(view, savedInstanceState) 3 4// ViewModelFolder.selectFolder はMutableLiveData<String> 5 ViewModelFolder.selectFolder.observe(viewLifecycleOwner, Observer { name -> 6 name?.let { 7 taskViewModel.getTasksByFolder(it, false) 8 .observe(viewLifecycleOwner, Observer { tasks -> 9 tasks?.let { taskAdapter.setTasks(it) } 10 }) 11 } 12 }) 13}

TabLayoutのためにFragmentを使ってます。
Folder名が変更またはTaskが変更した場合にtaskViewModel.getTasksByFolder を呼びたいです。
上記のようにObserveの中にObserveを書くと中のObserveが複数回呼ばれてしまいます。
どうすれば正常に動かせますか?

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

observeをネストして記述するべきではありません。Transformations.switchMapを使いましょう。

また、ご提示のコードではView側にロジックを書いているようですが、これはMVVMの設計思想に反します。ロジックはViewModel以下に閉じ込めてView側は単純に値を受け取るのみにすべきです。

  • ViewModel

kotlin

1val tasks = selectFolder 2 .nonNull() 3 .switchMap { getTasksByFolder(it, false) } 4 .nonNull() 5 6// nullの場合は値を流さないLiveData 7fun <T> LiveData<T?>.nonNull(): LiveData<T> { 8 val mediator = MediatorLiveData<T>() 9 mediator.addSource(this) { if (it != null) { mediator.value = it } } 10 return mediator 11}
  • View

kotlin

1viewModel.tasks.observe(viewLifecycleOwner, Observer { 2 taskAdapter.setTasks(it) 3})

投稿2020/01/04 01:10

編集2020/01/05 14:27
kakajika

総合スコア3131

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Nippun

2020/01/05 19:33

observeをネストしてはいけない理由はなんですか? パーフォマンスの問題ですか?
kakajika

2020/01/07 00:12

ネストしてはいけない、というよりはネストしない方が賢明、ぐらいのニュアンスのつもりでした。(「するべきではない」というのはちょっと表現が強かったかもしれません。) observeを2重、3重とネストしていくとコードは複雑になり、メンテナンス性が低下します。また、observeを行うのは基本的にView側の役目なのでView側にロジックを露出させないためにも避けるべきです。
Nippun

2020/01/07 02:29

なるほどです。 参考になります。ありがとうございます。
guest

0

LiveDataは最新の情報を保持してアタッチされているLifecycleOwnerがアクティブになったタイミングで情報を再度通知します。なのでデータが変更されていなければ再度通知しないなどの仕組みを実現するには、作りこみが必要です。

個人的には複雑な実装をしないのであればSingleLiveEventだけで事足りるのでは、と思います。

投稿2020/01/03 08:05

nakasho_dev

総合スコア2655

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問