JNI 环境指针

Question

我有一个 Java 类，其中有一个必须从我的 C 代码调用的函数。 函数如下：

public void endTrial(){
    //Code
}

所以我在我的 C 文件中创建了以下代码：

JNIEXPORT void JNICALL package_endTrialJava();
JNIEXPORT void JNICALL package_endTrialJava(){
    jobject javaObjectRef = env->NewObject(javaClassRef, javaMethodRef);
    env->CallVoidMethod(javaObjectRef, javaMethodRef);
}

但是为了能够使用env 变量调用这个函数，我创建了这个函数，每当我启动我的程序时都会调用它。

JNIEXPORT void JNICALL package_initJNI(JNIEnv* en, jobject ob)
{
    try {
        // LOGD("(JNI) [FluidMechanics] loadVelocityDataSet()");

        if (!App::getInstance())
            throw std::runtime_error("init() was not called");

        if (App::getType() != App::APP_TYPE_FLUID)
            throw std::runtime_error("Wrong application type");


        FluidMechanics* instance = dynamic_cast<FluidMechanics*>(App::getInstance());
        android_assert(instance);
        jclass dataClass = en->FindClass("fr/limsi/ARViewer/MainActivity");
        javaClassRef = (jclass) env->NewGlobalRef(dataClass);
        javaMethodRef = env->GetMethodID(javaClassRef, "endTrial", "()V");

        env = en ;
        obj = ob ;        

    } catch (const std::exception& e) {
        throwJavaException(env, e.what());
    }
}

我的代码中也声明了这些全局变量，因此对 initJNI() 的调用可以存储 JNIEnv 变量。

JNIEnv* env ;
jobject obj ;

但是我的程序崩溃了，所以存储JNIEnv 变量似乎不起作用。那么有什么解决办法吗？

Answer 1

您不能/不应该存储JNIEnv 指针。 Is 仅对当前线程有效。
但是您可以使用AttachCurrentThread 为您当前的线程获取JNIEnv指针。
或者，如果您知道它已经附加，您可以使用GetEnv。
除此之外，您没有提及如何使用全局 jobject obj，但请记住，您需要注意这些引用保持足够长的有效时间。 NewGlobalRef 是要走的路。

未经测试：

JavaVM* g_jvm = 0;
JNIEXPORT void JNICALL package_initJNI(JNIEnv* en, jobject ob)
{
    ....
    // insted of the env store the VM
    en->GetJavaVM(&g_jvm);
    obj = en->NewGlobalRef(ob); // I don't think you need this
    // and at some point you must delete it again
    ....
}

JNIEXPORT void JNICALL package_endTrialJava(){
    JNIEnv* env;
    g_jvm->AttachCurrentThread(&env, NULL); // check error etc
    jobject javaObjectRef = env->NewObject(javaClassRef, javaMethodRef);
    // this line makes not much sense. I think you don't need it if you use the global
    // with the global it would be more like this
    env->CallVoidMethod(obj javaMethodRef);
}