ไม่ได้ใช้ผลลัพธ์ของการสมัครสมาชิก

วันนี้ฉันอัปเกรดเป็น Android Studio 3.1 แล้วซึ่งดูเหมือนว่าจะมีการตรวจสอบผ้าสำลีเพิ่มขึ้นอีกเล็กน้อย หนึ่งในการตรวจสอบขุยเหล่านี้มีไว้สำหรับการsubscribe()เรียกRxJava2 แบบ one-shot ที่ไม่ได้เก็บไว้ในตัวแปร ตัวอย่างเช่นการรับรายชื่อผู้เล่นทั้งหมดจากฐานข้อมูลห้องของฉัน:

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

ผลลัพธ์เป็นบล็อกสีเหลืองขนาดใหญ่และคำแนะนำเครื่องมือนี้:

subscribeไม่ได้ใช้ผลลัพธ์ของ

แนวทางปฏิบัติที่ดีที่สุดสำหรับการโทรแบบ one-shot Rx เช่นนี้คืออะไร? ฉันควรถือDisposableและdispose()ทำให้เสร็จหรือไม่? หรือฉันควรจะ@SuppressLintเดินต่อไป?

ดูเหมือนว่าจะมีผลเฉพาะ RxJava2 ( io.reactivex), RxJava ( rx) ไม่มีผ้าสำลีนี้

IDE ไม่ทราบว่าการสมัครสมาชิกของคุณจะมีผลกระทบใดบ้างเมื่อไม่ได้รับการกำจัดดังนั้นจึงถือว่าการสมัครรับข้อมูลอาจไม่ปลอดภัย ตัวอย่างเช่นคุณSingleอาจมีการโทรผ่านเครือข่ายซึ่งอาจทำให้หน่วยความจำรั่วไหลหากคุณActivityถูกละทิ้งระหว่างการดำเนินการ

วิธีที่สะดวกในการจัดการจำนวนมากDisposableคือการใช้CompositeDisposable ; เพียงสร้างCompositeDisposableตัวแปรอินสแตนซ์ใหม่ในคลาสที่ปิดล้อมของคุณจากนั้นเพิ่ม Disposables ทั้งหมดของคุณไปยัง CompositeDisposable (ด้วย RxKotlin คุณสามารถผนวกaddTo(compositeDisposable)เข้ากับ Disposables ทั้งหมดของคุณได้) 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()).

ทันทีที่กิจกรรมจะถูกทำลายรายชื่อ Disposables จะถูกล้างและเราก็พร้อม

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อินเทอร์เฟซของคุณเองหรือใช้ชั้นเรียนเด็กอื่น ๆ

ตามที่แนะนำไว้คุณอาจใช้ global 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 เรียกว่าโดยปริยายหากการสังเกตเสร็จสมบูรณ์

เนื่องจากฉันไม่ต้องการระงับคำเตือนผ้าสำลีฉันเพิ่งเริ่มใช้รูปแบบต่อไปนี้สำหรับกรณีที่มีการสังเกตแบบซิงโครนัส:

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()
      }
   )

ฉันสนใจความคิดเห็นใด ๆ เกี่ยวกับเรื่องนี้ไม่ว่าจะเป็นการยืนยันความถูกต้องหรือการค้นพบช่องโหว่ก็ตาม

มีวิธีอื่นที่ใช้ได้ซึ่งหลีกเลี่ยงการใช้ Disposables ด้วยตนเอง (เพิ่มและลบการสมัครสมาชิก)

คุณสามารถกำหนด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'
}}