Android同步onSensorChanged？

这是我的问题的后续：Android线程可运行性能 https://stackoverflow.com/questions/36395440/android-thread-runnable-performance

我在理解应用程序的同步方法时遇到一些困难

我正在轮询传感器并将传感器值存储到数组中，只要它们发生变化

float[] accelerometerMatrix = new float[3];
float[] accelerometerWorldMatrix = new float[3];
float[] gyroscopeMatrix = new float[3];
float[] gravityMatrix = new float[3];
float[] magneticMatrix = new float[3];
float[] rotationMatrix = new float[9];

class InsertHandler implements Runnable {
        public void run() {
            //get values from arrays and insert into db
        }
    }

public void onSensorChanged(SensorEvent event) {
        sensor = event.sensor;

        int i = sensor.getType();
        if (i == MainActivity.TYPE_ACCELEROMETER) {
            accelerometerMatrix = event.values;
        } else if (i == MainActivity.TYPE_GYROSCOPE) {
            gyroscopeMatrix = event.values;
        } else if (i == MainActivity.TYPE_GRAVITY) {
            gravityMatrix = event.values;
        } else if (i == MainActivity.TYPE_MAGNETIC) {
            magneticMatrix = event.values;
        }

        long curTime = System.currentTimeMillis();
        long diffTime = (curTime - lastUpdate);

        // only allow one update every POLL_FREQUENCY.
        if(diffTime > POLL_FREQUENCY) {
            lastUpdate = curTime;

            //insert into database in background thread
            executor.execute(insertHandler);
        }
    }

每 10 毫秒，我的应用程序将获取当前传感器值（来自数组）并使用单线程执行器将它们插入数据库中。所以onSensorChanged方法既可以写入数组，也可以从数组中读取以写入数据库

我的问题是，应该onSensorChanged方法要同步吗？

最重要的是我不会错过任何数据。每 10 毫秒我需要存储当前的传感器值 - 任何一个都不能错过。

因此，根据我的理解，同步方法意味着 UI 线程将持有锁，并将传感器值写入数组。在此期间，执行程序线程由于锁定而无法从这些数组中读取数据。然后锁被解除，执行器线程然后加锁，从数组读取，写入数据库，释放锁

我可能会误解这里同步方法的使用，特别是考虑到onSensorChanged是事件驱动的，我不确定它是如何发挥作用的

但似乎在这种情况下，我可能不会每 10 毫秒插入一次最新值。当 UI 线程建立锁时，执行程序线程无法将这些值写入数据库。当执行器线程可以写入时，这些值现在已经是几毫秒的旧值并且不准确了

另一方面，同步意味着我不会遇到 UI 线程更改数组值，同时执行程序线程将一半更改的值插入数据库的情况

因此，对于这种需要每 10 毫秒插入最新/准确的传感器数据的情况，我应该使用同步方法吗？

您当前的代码不是线程安全的Runnable使用 UI 线程写入的相同数组。一旦你打电话executor.execute(insertHandler);无法保证 UI 线程不会收到另一个传感器事件并更改之前的数组值之一Runnable将它们写入数据库。看来你明白这部分了。

为了解决这个问题，我根本不建议使用同步块，因为您似乎只想写出数组中存储的任何值diffTime > POLL_FREQUENCY. The onSensorChanged(...)方法本身只会在代码中的 UI 线程上调用，因此您不必担心另一个线程在此方法中更改数组的值。

综上所述，您可以做的是将数组的当前值存储在您的新实例中Runnable班级。我知道您在上一篇文章中建议使用相同的实例，但这不会产生明显的差异。您甚至可以通过打开 Android Monitor 进行验证，并在应用程序运行时检查内存使用情况。通过存储当前值，现在不再重要onSensorChanged()在写出数据之前再次调用，因为您已经拥有所需数据的副本，并且不会更改。

这是我在代码中建议的内容：

class InsertHandler implements Runnable {
    final float[] accelerometerMatrix;
    final float[] accelerometerWorldMatrix;
    final float[] gyroscopeMatrix;
    final float[] gravityMatrix;
    final float[] magneticMatrix;
    final float[] rotationMatrix;

    public InsertHandler(float[] accelerometerMatrix, float[] accelerometerWorldMatrix,
            float[] gyroscopeMatrix, float[] gravityMatrix,
            float[] magneticMatrix, float[] rotationMatrix) {
        this.accelerometerMatrix = accelerometerMatrix;
        this.accelerometerWorldMatrix = accelerometerWorldMatrix;
        this.gyroscopeMatrix = gyroscopeMatrix;
        this.gravityMatrix = gravityMatrix;
        this.magneticMatrix = magneticMatrix;
        this.rotationMatrix = rotationMatrix;
    }

    public void run() {
        // use class field arrays values and insert into db
    }
}

然后当你添加Runnable to the executor use:

Runnable insertHandler = new InsertHandler(accelerometerMatrix, accelerometerWorldMatrix,
        gyroscopeMatrix, gravityMatrix, magneticMatrix, rotationMatrix);
executor.execute(insertHandler);