サブスクライブの結果は使用されません

今日、Android Studio 3.1にアップグレードしました。lintチェックがいくつか追加されたようです。これらのlintチェックの1つsubscribe()は、変数に格納されていないワンショットのRxJava2 呼び出しに対するものです。たとえば、私の部屋のデータベースからすべてのプレーヤーのリストを取得します。

Single.just(db)
            .subscribeOn(Schedulers.io())
            .subscribe(db -> db.playerDao().getAll());

大きな黄色のブロックとこのツールチップが表示されます：

の結果subscribeは使用されません

このようなワンショットRx呼び出しのベストプラクティスは何ですか？を完全に保持する必要がDisposableありdispose()ますか？それとも@SuppressLint、次に進むべきですか？

これはRxJava2（io.reactivex）にのみ影響するようです。RxJava（rx）にはこのリントがありません。

IDEは、サブスクリプションが破棄されない場合にサブスクリプションがどのような影響を与える可能性があるかを認識していないため、安全でない可能性があるものとして扱います。たとえばSingle、ネットワークコールが含まれている場合があり、Activityその実行中に放棄されるとメモリリークが発生する可能性があります。

大量のを管理する便利な方法Disposableは、CompositeDisposableを使用することです。CompositeDisposable囲むクラスで新しいインスタンス変数を作成してから、すべてのDisposableをCompositeDisposableに追加します（RxKotlinを使用addTo(compositeDisposable)すると、すべてのDisposableに追加できます）。最後に、インスタンスが完了したら、を呼び出しますcompositeDisposable.dispose()。

これは糸くずの警告を取り除き、Disposables適切に管理されていることを確認します。

この場合、コードは次のようになります。

CompositeDisposable compositeDisposable = new CompositeDisposable();

Disposable disposable = Single.just(db)
        .subscribeOn(Schedulers.io())
        .subscribe(db -> db.get(1)));

compositeDisposable.add(disposable); //IDE is satisfied that the Disposable is being managed. 
disposable.addTo(compositeDisposable); //Alternatively, use this RxKotlin extension function.


compositeDisposable.dispose(); //Placed wherever we'd like to dispose our Disposables (i.e. in onDestroy()).

アクティビティが破棄される瞬間、使い捨てアイテムのリストがクリアされ、私たちは元気です。

io.reactivex.disposables.CompositeDisposable mDisposable;

    mDisposable = new CompositeDisposable();

    mDisposable.add(
            Single.just(db)
                    .subscribeOn(Schedulers.io())
                    .subscribe(db -> db.get(1)));

    mDisposable.dispose(); // dispose wherever is required

DisposableSingleObserverでサブスクライブできます。

Single.just(db)
    .subscribeOn(Schedulers.io())
    .subscribe(new DisposableSingleObserver<Object>() {
            @Override
            public void onSuccess(Object obj) {
                // work with the resulting todos...
                dispose();
            }

            @Override
            public void onError(Throwable e) {
                // handle the error case...
                dispose();
            }});

Singleオブジェクトを直接破棄する必要がある場合（たとえば、オブジェクトが発生する前に）onSubscribe(Disposable d)、Disposable参照を取得して使用するメソッドを実装できます。

SingleObserver独自のインターフェイスを実現したり、他の子クラスを使用したりすることもできます。

提案されたようCompositeDisposableに、サブスクライブ操作の結果をそこに追加するために、いくつかのグローバルを使用することができます。

RxJava2Extensionsのライブラリが自動的に作成さから使い捨て削除する便利なメソッドが含まれCompositeDisposable、それが完了したとき。subscribeAutoDisposeセクションを参照してください。

あなたの場合、このように見えるかもしれません

SingleConsumers.subscribeAutoDispose(
    Single.just(db)
            .subscribeOn(Schedulers.io()),
    composite,
    db -> db.playerDao().getAll())

Uber AutoDisposeとrxjavaを使用できます.as

        Single.just(db)
            .subscribeOn(Schedulers.io())
            .as(AutoDispose.autoDisposable(AndroidLifecycleScopeProvider.from(this)))
            .subscribe(db -> db.playerDao().getAll());

ScopeProviderに基づいてサブスクリプションを解除するときを理解していることを確認してください。

私は何度も何度も、サブスクリプションを正しく破棄する方法の問題、特にこの投稿に戻ってきます。いくつかのブログや講演では、呼び出しに失敗するとdispose必ずメモリリークが発生すると主張していますが、これは一般的すぎると思います。私の理解では、結果を保存しないことに関する糸くずの警告subscribeは、場合によっては問題ではありません。

• すべてのオブザーバブルがAndroidアクティビティのコンテキストで実行されるわけではありません
• オブザーバブルは同期可能です
• Disposeは、オブザーバブルが完了した場合に暗黙的に呼び出されます

lint警告を抑制したくないので、最近、同期オブザーバブルの場合に次のパターンを使用し始めました。

var disposable: Disposable? = null

disposable = Observable
   .just(/* Whatever */)
   .anyOperator()
   .anyOtherOperator()
   .subscribe(
      { /* onSuccess */ },
      { /* onError */ },
      {
         // onComplete
         // Make lint happy. It's already disposed because the stream completed.
         disposable?.dispose()
      }
   )

それが正しさの確認であろうと、抜け穴の発見であろうと、私はこれについてのコメントに興味があります。

Disposableを手動で使用しないようにする別の方法があります（サブスクリプションの追加と削除）。

Observableを定義すると、そのObservableはSubjectBehaviourからコンテンツを受信します（RxJavaを使用している場合）。そして、そのオブザーバブルをLiveDataに渡すことで、動作するはずです。最初の質問に基づいて、次の例を確認してください。

private val playerSubject: Subject<Player> = BehaviorSubject.create()

private fun getPlayer(idPlayer: String) {
        playerSubject.onNext(idPlayer)
}

private val playerSuccessful: Observable<DataResult<Player>> = playerSubject
                        .flatMap { playerId ->
                            playerRepository.getPlayer(playerId).toObservable()
                        }
                        .share()

val playerFound: LiveData<Player>
    get() = playerSuccessful
        .filterAndMapDataSuccess()
        .toLiveData()

val playerNotFound: LiveData<Unit>
    get() = playerSuccessful.filterAndMapDataFailure()
        .map { Unit }
        .toLiveData()

// These are a couple of helpful extensions

fun <T> Observable<DataResult<T>>.filterAndMapDataSuccess(): Observable<T> =
filter { it is DataResult.Success }.map { (it as DataResult.Success).data }

fun <T> Observable<DataResult<T>>.filterAndMapDataFailure(): Observable<DataResult.Failure<T>> =
filter { it is DataResult.Failure }.map { it as DataResult.Failure<T> }

たとえば、doOnSubscribe（）演算子を使用して、使い捨てが正しく処理されていることが確実な場合は、これをGradleに追加できます。

android {
lintOptions {
     disable 'CheckResult'
}}