android - while循环中的一个变量没有改变，ndk/android

Question

我是 ndk 的新手。你能解释一下下面的案例吗？ 这是我的简单代码示例:

loop.c

#include "loop.h"
#include <android/log.h>

static int flag = 1;

void test_loop()
{
    while(flag)
    {
        __android_log_print(ANDROID_LOG_DEBUG, "TEST", "in the loop");
    }
    __android_log_print(ANDROID_LOG_DEBUG, "TEST", "exit");
}

void set_exit_flag()
{
    flag = 0;
}

loop.h

#ifndef LOOP_H
#define LOOP_H

#ifdef __cplusplus
extern "C" {
#endif

void test_loop();
void set_exit_flag();

#ifdef __cplusplus
}
#endif
#endif

测试.cpp

#include "test.h"
#include "LoopFolder/loop.h"

JNIEXPORT void JNICALL Java_com_example_changevariablevalues_MainActivity_Start
  (JNIEnv* pEnv, jobject pObj)
{
    test_loop();
}

JNIEXPORT void JNICALL Java_com_example_changevariablevalues_MainActivity_Stop
  (JNIEnv* pEnv, jobject pObj)
{
    set_exit_flag();
}

在 Java 端我有两个带有点击监听器的按钮:

mStart = (Button) findViewById(R.id.button1);
mStop = (Button) findViewById(R.id.button2);

mStart.setOnClickListener(new OnClickListener() {
    @Override
    public void onClick(View v) {
        exec.execute(new Runnable() {
            @Override
            public void run() {
                Start();
            }
        });
    }
});

mStop.setOnClickListener(new OnClickListener() {
    @Override
    public void onClick(View v) {
        exec.execute(new Runnable() {
            @Override
            public void run() {
                Stop();
            }
        });
    }
});

和服务执行器:

private Executor exec = Executors.newCachedThreadPool();

我在 mStart 按下后执行开始循环。 之后，我单击 mStop 并等待循环停止。 但是没有任何反应。 在 adb 控制台中我只看到

D/TEST    (17456): in the loop
D/TEST    (17456): in the loop
D/TEST    (17456): in the loop
..............................
D/TEST    (17456): in the loop
D/TEST    (17456): in the loop

消息。

有人可以帮助我吗？如何停止循环？

Answer 1

您的两个功能未正确同步。它们由不同的线程调用，一个通过单击 mStart 启动，另一个通过单击 mStop 启动。这本身不是问题。两个线程相互并行调用本地函数也不成问题。但当 native 函数都尝试同时访问 flag 时，它确实成为一个问题。

您需要同步。否则，最(看似)最基本的事情可能会出错。例如，根本不能保证线程 2 会看到线程 1 中变量的更改。即使像 flag++ 这样的简单表达式也不再是原子的，并且会导致各种奇怪的效果。如果没有同步，您实际上已经随机化您的应用并引发崩溃。

同步可以在 C++ 和 Java 中完成，但对于 JNI 用例，通常最好将所有同步留给 Java 端。这只是 JNI 的通用软件工程指南:

尽可能使用 Java，如果必须，则使用 native !

针对您的情况，一个可能的解决方案是将循环和标志移至 Java，并将 JNI 函数减少为真正必须在 C++ 中完成的功能。 Stop 函数将完全用 Java 编写，cancel 标志将变为 AtomicBoolean。 :

private static AtomicBoolean cancel = new AtomicBoolean(false);

private void stop() {
    cancel.set(true);
}

Start 函数也将成为一个 Java 方法，并包装一个更简单的 JNI 函数:

private native void print();

private void start() {
    while (!cancel.get()) {
        print();
    }
} 

这是简化的 JNI 函数:

void test_loop() {
    __android_log_print(ANDROID_LOG_DEBUG, "TEST", "in the loop");
}

这里有一个重要的最终提示:您可能应该使用 Android SDK 的 AsyncTask类，专门为可取消的后台操作而设计。

---

编辑:原来的答案有一个错误。 Java 代码现在使用 AtomicBoolean，这应该可以解决问题。