Android Studio中这段onSensorChanged代码存在什么问题？

让我帮你拆解这段代码里的几个关键问题：

1. 未判断传感器类型，数据处理逻辑混乱 #

传感器回调onSensorChanged会触发所有你注册过的传感器事件（比如同时注册了陀螺仪和加速度计的话），但你直接把所有事件的values都当成角速度数据处理（单位写了rad/s，这是陀螺仪的单位），如果收到其他传感器（比如加速度计，单位是m/s²）的数据，显示的内容就完全不对了。

修复建议：先判断当前事件对应的传感器类型，再处理数据：

public void onSensorChanged(SensorEvent event) {
    // 先判断是不是陀螺仪传感器
    if (event.sensor.getType() == Sensor.TYPE_GYROSCOPE) {
        float x = event.values[0];
        float y = event.values[1];
        float z = event.values[2];
        textX.setText("X : " + (int) x + " rad/s");
        textY.setText("Y : " + (int) y + " rad/s");
        textZ.setText("Z : " + (int) z + " rad/s");
    }
}

2. 强制转int丢失大量精度 #

传感器返回的是浮点型数据，直接用(int)强制转换会把小数部分完全丢弃，比如实际值是2.8rad/s会被显示成2rad/s，丢失了关键的精度信息，对于传感器数据来说这很影响准确性。

修复建议：用格式化字符串保留小数位，比如保留1位或2位小数：

textX.setText(String.format("X : %.1f rad/s", x));

3. 缺少空指针检查 #

如果textX、textY、textZ这几个TextView对象没有被正确初始化，或者在回调触发时已经被销毁（比如Activity/Fragment已经走到生命周期后期），直接调用setText会抛出NullPointerException导致崩溃。

修复建议：添加空指针判断：

if (textX != null) {
    textX.setText(String.format("X : %.1f rad/s", x));
}
// textY、textZ同理

4. 代码末尾存在语法错误 #

你提供的代码最后有多余的};，这会导致编译失败，应该去掉这个多余的符号，保证代码语法正确。

内容的提问来源于stack exchange，提问作者Gueni Mohamed