C++ 对Java的回调 -> 为什么当来自不同的线程时，我无法在回调类中检索JNIEnv？

Question

我正试图在Android上构建一个回调类，以便从我的本地代码中的不同线程调用Java方法。我读了很多关于如何做到这一点的书，只要我在同一条线上，一切都会奏效。但从另一个线程中，我无法正确检索JNIEnv，也无法找出我做错了什么。

我对C++和JNI不是很有经验，所以这很可能是初学者的问题。。。但是我已经讲了好几天了，看不清它是什么。

这是我的回调类、.h和.cpp文件：

class AudioCallback {
public:
explicit AudioCallback(JavaVM&, jobject);
void playBackProgress(int progressPercentage);
private:
JavaVM& g_jvm;
jobject g_object;
};

jclass target = NULL;
jmethodID id = NULL;
AudioCallback::AudioCallback(JavaVM &jvm, jobject object) : g_jvm(jvm), g_object(object) {
JNIEnv *g_env;
int getEnvStat = g_jvm.GetEnv((void **) &g_env, JNI_VERSION_1_6);
if (g_env != NULL) {
target = g_env->GetObjectClass(g_object);
id = g_env->GetMethodID(target, "integerCallback", "(I)V");
//This is a test call to see if I can call my java method. It works.
g_env->CallVoidMethod(g_object, id, (jint) 103);
}
}
// this method is calles from other threads, so I want to attach to the current thread once I got my JNIEnv, but I can't since it's null...
void AudioCallback::playBackProgress(int progressPercentage) {
JNIEnv *g_env;
// This is null and I don't know why!
int getEnvStat = g_jvm.GetEnv((void **) &g_env, JNI_VERSION_1_6);

if (g_env == NULL) {
LOGE("JNIEnv in callback method is null");
} else {
LOGD("Env Stat: %d", getEnvStat);
JavaVMAttachArgs vmAttachArgs;
if (getEnvStat == JNI_EDETACHED) {
LOGD("GetEnv: not attached - attaching");
if (g_jvm.AttachCurrentThread(&g_env, &vmAttachArgs) != 0) {
LOGD("GetEnv: Failed to attach");
}
} else if (getEnvStat == JNI_OK) {
LOGD("GetEnv: JNI_OK");
} else if (getEnvStat == JNI_EVERSION) {
LOGD("GetEnv: version not supported");
}
g_env->CallVoidMethod(g_object, id, (jint) progressPercentage);
//thread gets detached elsewhere
}
}

这是我的native_lib，在这里我获得JavaVM并实例化回调类：

std::unique_ptr<AudioEngine> audioEngine;
std::unique_ptr<AudioCallback> callback;
JavaVM *g_jvm = nullptr;
static jobject myJNIClass;
jint JNI_OnLoad(JavaVM *pJvm, void *reserved) {
g_Jvm = pJvm;
return JNI_VERSION_1_6;
}
JNIEXPORT void JNICALL
Java_com_my_appy_common_jni_JniBridge_playFromJNI(JNIEnv *env, jobject instance,jstring URI) {
myJNIClass = env->NewGlobalRef(instance);
callback = std::make_unique<AudioCallback>(*gJvm, myJNIClass);
// this test call to my callback works
callback->playBackProgress(104);
const char *uri = env->GetStringUTFChars(URI, NULL);
//... urelated code is left out here ...
//audioEngine gets the callback and uses it from threads it creates
audioEngine = std::make_unique<AudioEngine>(*extractor, *callback);
audioEngine->setFileName(uri);
audioEngine->start();
}

我已经缩短了代码并删除了所有不相关/不必要的部分。如果缺少关键内容，请发表评论，我会添加它。

解决方案：根据@Michael在回答中提出的建议，我在回调类中对playbackProgress方法进行了以下编辑，使其发挥作用：

void AudioCallback::playBackProgress(int progressPercentage) {
JNIEnv *g_env;
int getEnvStat = g_jvm.GetEnv((void **) &g_env, JNI_VERSION_1_6);
if (getEnvStat == JNI_EDETACHED) {
LOGD("GetEnv: not attached - attaching");
if (g_jvm.AttachCurrentThread(&g_env, NULL) != 0) {
LOGD("GetEnv: Failed to attach");
}
} else if (getEnvStat == JNI_OK) {
LOGD("GetEnv: JNI_OK");
} else if (getEnvStat == JNI_EVERSION) {
LOGD("GetEnv: version not supported");
}
g_env->CallVoidMethod(g_object, id, (jint) progressPercentage);
//    mJvm.DetachCurrentThread();
}

现在它直接检查getEnvStat的值，以前g_env的空检查是错误的。我还必须用NULL替换JavaVMAttachArgs才能使其工作。

Answer 1

playBackProgress中的逻辑错误。

尝试附加当前线程的唯一时间是g_env为非NULL时。但是如果g_env不是NULL，那么GetEnv可能成功了(当然也应该检查getEnvStat == JNI_OK(，而AttachCurrentThread不是必需的。

需要调用AttachCurrentThread的情况是当g_env为NULL并且getEnvStat为JNI_EDETACHED时。

您还需要跟踪是否调用了AttachCurrentThread，因为在这种情况下，您应该在某个时刻调用DetachCurrentThread。请参阅此答案以了解更多信息。