为什么我的RxJava设置会阻止我的UI线程?使用BluetoothAdapter.startLeScan回调

时间:2017-12-12 22:13:23

标签: android kotlin rx-java android-bluetooth rx-java2

我正在努力找到阻止我的UI线程的特定操作,我已经尝试了几个Scheduler操作符,但我不确定如何使它工作。

我有一个带按钮的UI,其中onClicked正在启动蓝牙扫描,并使用字符串(如日志)更新textView(它显示了当前正在发生的事情)。

所以这是我的MainActivity:

 lateinit var disposable: Disposable
val textDataService = TextDataService()


override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    setContentView(R.layout.activity_scan_test)


    buttonScanTestStart.setOnClickListener {
        if (isBluetoothEnabled()) {
            textViewLog.text = ""

            buttonScanTestStop.visibility = View.VISIBLE
            buttonExportScanTestRaportSummary.visibility = View.GONE
            buttonExportScanTestRaportFull.visibility = View.GONE
            buttonScanTestStart.visibility = View.GONE

            disposable=
                    Scanner()
                            .discoverSingleDevice(this, "   ", textViewLog)
                            .doOnError {
                                setText("General error: ${it.message ?: it::class.java}", textViewLog)
                                setLogText("General error: ${it.message ?: it::class.java}")
                            }
                            .repeat(1)
                            .doOnComplete {
                                buttonScanTestStop.visibility = View.GONE
                            }
                            .doOnDispose {
                                log("TEST DISPOSED")
                            }
                            .subscribe()
        } else {
            Toast.makeText(this, "Please, enable bluetooth to start test.", Toast.LENGTH_SHORT).show()
        }
    }

    buttonScanTestStop.setOnClickListener {
        disposable.dispose()
        buttonScanTestStop.visibility = View.GONE
        buttonScanTestStart.visibility = View.VISIBLE
        buttonExportScanTestRaportSummary.visibility = View.VISIBLE
        buttonExportScanTestRaportFull.visibility = View.VISIBLE

        textDataService.generateScanGeneralStatisticsLogText(textViewLog)
    }

以下是Scanner类:

class Scanner {


private fun scan(context: Context, textView: TextView) = Observable.create<ScannedItem> { emitter ->
    val bluetoothAdapter = context.getBluetoothAdapter()
    if (bluetoothAdapter.isEnabled) {

        val scanCallback = BluetoothAdapter.LeScanCallback { bluetoothDevice, rssi, _ ->

            if(bluetoothDevice.name == null){
                    bluetoothRawLog("Scanned Item -> ${bluetoothDevice.logText()} | rssi = $rssi | time: = ${getCurrentTime()}")
                    scannedOtherDevices.add(bluetoothDevice.address)
            }
            else{
                if(!scannedDevices.contains(bluetoothDevice.name)){
                        setText("Scanned Item -> ${bluetoothDevice.logText()} | rssi = $rssi | time: = ${getCurrentTime()}\n", textView)

                    setLogText("Scanned Item -> ${bluetoothDevice.logText()} | rssi = $rssi | time: = ${getCurrentTime()}\n")
                }
                scannedDevices.add(bluetoothDevice.name)


            }
            bluetoothRawLog("Scanned Item -> ${bluetoothDevice.logText()} | rssi = $rssi")


            bluetoothDevice.name?.let {
                emitter.onNext(ScannedItem(it, bluetoothDevice.address))
            }
        }

        emitter.setCancellable { bluetoothAdapter.stopLeScan(scanCallback) }
        bluetoothAdapter.startLeScan(scanCallback)

    } else
        emitter.onError(IllegalStateException("Bluetooth turned off"))
}
        .doOnNext { log("Scanned -> $it") }
        .timeout(12, TimeUnit.SECONDS)
        .observeOn(AndroidSchedulers.mainThread())
        .doOnError { if (it is TimeoutException) setLogText("Scanner -> no named devices for 12 seconds: resetting...") }
        .retry { t -> t is TimeoutException }


  fun discoverSingleDevice(context: Context, searchName: String, textView: TextView): Observable<ScannedItem> = scan(context, textView)
        .filter { it.name.contains(searchName) }
        .take(1)

这是我的Kotlin扩展函数,我用它来设置Text:

fun setLogText(text: String) {
    logBuffer.append(text)
}

fun setText(text: String, textView: TextView) {
    textView.append(text)
}

这也是我的布局:

<?xml version="1.0" encoding="utf-8"?>

<LinearLayout
    android:layout_margin="16dp"
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_gravity="center"
    android:orientation="horizontal">

    <Button
        android:layout_weight="1"
        android:id="@+id/buttonScanTestStart"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="center"
        android:text="Start Test" />
    <Button
        android:layout_weight="1"
        android:visibility="gone"
        android:id="@+id/buttonScanTestStop"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="center"
        android:text="Stop Test" />
    <Button
        android:layout_weight="1"
        android:id="@+id/buttonExportScanTestRaportFull"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="center"
        android:text="Export raport"
        android:visibility="gone" />

    <Button
        android:layout_weight="1"
        android:id="@+id/buttonExportScanTestRaportSummary"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_gravity="center"
        android:text="Export summary"
        android:visibility="gone" />
</LinearLayout>

<LinearLayout
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical">

    <LinearLayout
        android:visibility="gone"
        android:id="@+id/scanTestStartedLinearLayout"
        android:gravity="center"
        android:orientation="horizontal"
        android:layout_width="match_parent"
        android:layout_height="wrap_content">
        <TextView
            android:text="Test running..."
            android:layout_width="wrap_content"
            android:layout_height="wrap_content" />
        <ProgressBar
            android:id="@+id/scanTestProgressBar"
            android:layout_width="16dp"
            android:layout_height="16dp" />
    </LinearLayout>

    <TextView
        android:id="@+id/textViewScanTestLog"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_margin="16dp"
        android:gravity="center" />

    <ScrollView
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_margin="16dp"
        android:gravity="center">

        <LinearLayout
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:orientation="vertical">

            <TextView
                android:id="@+id/textViewLog"
                android:layout_width="match_parent"
                android:layout_height="wrap_content" />

        </LinearLayout>

    </ScrollView>

</LinearLayout>

因此文本显示在我的textView中非常流畅,一切正常。 但是,当有更多可用设备要扫描时,我无法通过单击按钮来停止测试。当我试图点击它时,它只是冻结和反应太迟了。

discoverSingleDevice函数将永远运行,因为我过滤了我的editText中的搜索名称(我用“”硬编码)使其永远运行,我想只通过单击我的停止按钮来停止它布局。作为第二个参数,我从我的布局传递给它textViewLog(Kotlin很聪明,可以通过id找到它)

那么如何防止使用此设置阻止UI?

更新

正如文超所说,我做过这样的事情:

       val observable = Scanner()
                    .discoverSingleDevice(this, "   ", textViewLog)
                    .doOnError {
                        nexoSetText("General error: ${it.message ?: it::class.java}", textViewLog)
                        nexoSetLogText("General error: ${it.message ?: it::class.java}")
                    }
                    .repeat(1)
                    .doOnComplete {
                        buttonScanTestStop.visibility = View.GONE
                    }
                    .doOnDispose {
                        nexoLog("TEST DISPOSED")
                    }

         disposable =  observable
                    .subscribeOn(Schedulers.newThread()).subscribe()

然而,它没有帮助,它并不那么简单。我再次尝试过与其他运营商合作的几件事。 请特定于我的情况 - 我提供了代码。

就像2分钟扫描后,应用程序相当冻结。在日志当然出现I / Choreographer:跳过54帧!应用程序可能在其主线程上做了太多工作。

更新2:

正如PhoenixWang建议的那样,我已经登录以检查BluetoothAdapter运行的线程。所以我在Scanner类的扫描方法中就这样做了:

  private fun scan(context: Context, textView: TextView) = Observable.create<ScannedItem> { emitter ->
    val bluetoothAdapter = context.getBluetoothAdapter()
    if (bluetoothAdapter.isEnabled) {

        val scanCallback = BluetoothAdapter.LeScanCallback { bluetoothDevice, rssi, _ ->
            Log.v("BluetoothThread", "Running on: " + Thread.currentThread().name)
///the rest of the code

那么如何解决这个问题呢?我认为它实际上将在单独的线程(不是主要)上运行,因为我已经完成了天真的修复,这在我的第一个更新中可见。你能帮忙吗?

更新3:

所以问题出在BluetoothAdapter.leScan上 - 它在主线程上运行,即使我这样做了:

Observable.fromCallable{ bluetoothAdapter.startLeScan(scanCallback)}.subscribeOn(Schedulers.newThread()).subscribe()

正如PhoenixWang所暗示的那样,BluetoothAdapter.LeScanCallback因为他的实现而在mainthread中运行。 Observable / RxJava无法改变它。 那么如何解决我的问题?

3 个答案:

答案 0 :(得分:1)

默认情况下,RxJava是同步的。在你的情况下,它在调用者线程上运行,这是Android的主线程

您必须指定您希望订阅发生的线程。

Observable.fromCallable(whatever())
        .subscribeOn(Schedulers.newThread()).subscribe()

了解更多here

答案 1 :(得分:1)

 .subscribeOn(Schedulers.io())
 .observeOn(AndroidSchedulers.mainThread())

像上面一样添加.subscribeOn(Schedulers.io())和.observeOn(AndroidSchedulers.mainThread())。试试这个。应该工作。

答案 2 :(得分:1)

更清楚。我回答但无法解决你的问题。 你的代码:

 Observable.fromCallable{ bluetoothAdapter.startLeScan(scanCallback)}.subscribeOn(Sche‌​dulers.newThread()).‌​subscribe().

表示您的bluetoothAdapter.startLeScan(scanCallback)在newThread中运行。但不是bluetoothAdapter如何给你回电话。

所以在你的startLeScan的实现中。它可以启动一个新线程或在其他线程中运行取决于它的实现。

与Observable.create(//你的东西)相同的想法

保证在指定的调度程序上运行它们。但它也可以在其中启动一个新线程。这就是我在评论中提到检查BluetoothAdapter实施的原因。

<强>更新

更清楚一点,BluetoothAdapter的示例实现可以阻止您的UI线程。

class BluetoothAdapter {

    fun startLeScan(callback: LeScanCallback) {
        val handler = Handler(Looper.getMainLooper())
        handler.post {
            // here you back to your UI Thread. 
            callback.onSuccess(1)
        }
    }

    interface LeScanCallback {
        fun onSuccess(result: Int)
    }
}

因此,如果您的BluetoothAdapter实现将线程更改回UI线程。无论您的Observable的哪个线程,您最终都会阻止您的UI线程。我的意思 检查BluetoothAdapter的实现