java - 在多线程应用程序中使用 JNI 的正确方法

Question

描述

在我的 C++ 应用程序类 JNIXMLDocument 中，它进行了一些 JAVA 方法调用。在 JNIXMLDocument 类的构造函数中，我附加了当前线程并将其设置为我的类成员 JNIEnv* m_JavaEnv，然后在所有方法中使用它。同样在构造函数中，我试图找到我的 JAVA 类 com/fido/android/framework/service/XMLDOMDocument 并将其设置为类成员 m_XMLDocumentClass 并获取该类对象从类并将其设置为类成员 m_XMLDocumentObject。

C++代码

class JNIXMLDocument
{
    /* Constructor **/
    JNIXMLDocument()
    {
        /* Get JNI right version and set it. **/
        jint interface_id = JNI_VERSION_1_2;
        #ifdef JNI_VERSION_1_2
            interface_id = JNI_VERSION_1_2;
        #else
            interface_id = JNI_VERSION_1_1;
        #endif

        /* Trying to attach current thread. **/
        int res = g_JavaVirtualMachine->GetEnv(&m_JavaEnv, interface_id);
        if (res == JNI_EDETACHED || res == JNI_EVERSION) {
            res = g_JavaVirtualMachine->AttachCurrentThread(&m_JavaEnv, NULL);
        }

        /* Get Class from Java **/
        m_XMLDocumentClass = m_JavaEnv->FindClass("com/fido/android/framework/service/XMLDOMDocument");
        if (m_XMLDocumentClass != NULL) {
            /* Call java class constructor. **/
            jmethodID constructor = m_JavaEnv->GetMethodID(m_XMLDocumentClass , "<init>", "()V");
            m_XMLDocumentObject = m_JavaEnv->NewObject(m_XMLDocumentClass , constructor);    

        }

    }

    bool Initialize()
    {
        jmethodID method = m_JavaEnv->GetMethodID(m_XMLDocumentClass, "Initialize", "()Lorg/w3c/dom/Document;");
        jobject document = m_JavaEnv->CallObjectMethod(m_XMLDocumentObject , method);

    }

    private:
        JNIEnv* m_JavaEnv;
        jclass  m_XMLDocumentClass;
        jobject m_XMLDocumentObject;


};

C++代码(正确的方式)

class JNIXMLDocument
{
    /* Constructor **/
    JNIXMLDocument()
    {
        /* Get JNI right version and set it. **/
        jint interface_id = JNI_VERSION_1_2;
        #ifdef JNI_VERSION_1_2
            interface_id = JNI_VERSION_1_2;
        #else
            interface_id = JNI_VERSION_1_1;
        #endif

        JNIEnv* env;
        /* Trying to attach current thread. **/
        int res = g_JavaVirtualMachine->GetEnv(&env, interface_id);
        if (res == JNI_EDETACHED || res == JNI_EVERSION) {
            res = g_JavaVirtualMachine->AttachCurrentThread(&env, NULL);
        }

        /* Get Class from Java **/
        jclass localClass = env->FindClass("com/fido/android/framework/service/XMLDOMDocument");
        if (localClass != NULL) {
            m_XMLDocumentClass = env->NewGlobalRef(localClass);
            /* Call java class constructor. **/
            jmethodID constructor = env->GetMethodID(localClass, "<init>", "()V");
            jobject localObject = env->NewObject(m_XMLDocumentClass , constructor);    
            m_XMLDocumentObject = env->NewGlobalRef(localObject );
        }

    }

    bool Initialize()
    {

    }

    private:
        jclass  m_XMLDocumentClass;
        jobject m_XMLDocumentObject;


};

问题

1. 在构造函数中设置一次 JNI 接口(interface)指针(JNIEnv* m_JavaEnv)并在整个代码中使用它是否正确？
2. 在构造函数中设置 jclass m_XMLDocumentClass 并在所有方法中使用该变量是否正确？
3. 以这种方式在构造函数中设置 jobject m_XMLDocumentObject 是否正确 m_JavaEnv->NewObject(m_XMLDocumentClass , constructor); 或者我必须调用 NewGlobalRef。
4. 如果我的应用程序不在同一个线程(使用多个线程)中工作，会出现什么问题？

Answer 1

Is it right to set JNI interface pointer (JNIEnv* m_JavaEnv) in the constructor once and use it in the whole code ?

没有。它是特定于线程的。这就是 Attach/DetachCurrentThread 的用途。唯一可行的方法是在同一线程中构造和销毁 C++ 对象。

Is it right to set jclass m_XMLDocumentClass in the constructor and than use that variable in the all methods ?

没有。它是一个本地引用，当获取它的 JNI 方法返回时它就过期了。您必须保存为全局引用或弱引用，除非它只会在单个 JNI 方法中使用。

Is it right to set jobject m_XMLDocumentObject in the constructor in this way m_JavaEnv->NewObject(m_XMLDocumentClass , constructor);

否:见上文。

or maybe I must call NewGlobalRef.

是的，如上所述。

What problems can appear if my application work not in the same thread (use many threads) ?

主要是 JVM 崩溃。 JVM 假定您遵循 JNI 规范中的所有规则。那就这样做吧。