JNI int 方法从异常返回

Question

假设我有一个像这样的 Java 类：

    public class Test
    {
        static { System.loadLibrary("test"); }
        public native int foo();
    }

假设 foo() 方法正在执行一些 JNI 调用，并且其中一个调用失败（IE，抛出异常）。然后我想从 JNI 代码返回并在 Java 中抛出异常。例如：

    jint Java_Test_foo(JNIEnv* env, jobject thiz)
    {
        jstring foobar = (*env)->NewStringUTF(env, "Hello from JNI !");
        if(foobar == NULL) // Not enough memory. OutOfMemoryError is thrown
            return NULL; // Return immediately to get exception thrown in Java
        // Do other stuff
        ...
        return some_computed_jint;
    }

问题是 return NULL 不是 jint。以 Android 为例，我在编译时会收到此警告： 警告：返回从指针生成整数，无需强制转换。

现在的问题是：如果在返回 jint 的 JNI 方法中抛出异常，我应该返回什么？

Answer 1

如果您的代码（或库）在 Java 中引发Exception，那么无论您返回什么值，Java 都会忽略它。显然它需要是兼容的类型 - 因此在您的示例中返回 0 似乎是有意义的，或者您喜欢的任何类型。当您的代码返回时，Java 运行时将注意到已引发Exception 并继续传播它并忽略函数的返回值。

当然，您需要返回兼容的类型。不要简单地返回NULL，因为当函数未声明为返回指针时，它将被转换为int，这可能不合适。

但显然，当您调用 C 函数时，它们不会引发 Exception。因此，您可以将整数映射到错误条件（例如 -1），然后在 Java 中抛出 Exception，或者您可以花时间构建一个 >JNI 中的异常。

Answer 2

编辑：另请参阅这个优雅的答案，使用函数而不是底部预处理器宏。

---

我提供了一个示例来完成爱德华·汤姆森的回答。

在此示例中，JNI 非 void 函数return 0;

JNIEXPORT jlong JNICALL Java_group_package_class_function1(
                           JNIEnv *env, jobject object, jlong value)
{
  try 
  {
    /* ... my processing ... */
    return jlong(result);
  }
  CATCH_CPP_EXCEPTION_AND_THROW_JAVA_EXCEPTION
  return 0;
}

JNIEXPORT jstring JNICALL Java_group_package_class_function2(
                            JNIEnv *env, jobject object, jlong value)
{
  try 
  {
    /* ... my processing ... */
    jstring jstr = env->NewStringUTF("my result");
    return  jstr;
  }
  CATCH_CPP_EXCEPTION_AND_THROW_JAVA_EXCEPTION
  return 0;
}

JNIEXPORT void JNICALL Java_group_package_class_function3(
                           JNIEnv *env, jobject object, jlong value)
{
  try 
  {
    /* ... my processing ... */
  }
  CATCH_CPP_EXCEPTION_AND_THROW_JAVA_EXCEPTION
}  // void function => no "return 0;" statement

C 预处理器宏CATCH_CPP_EXCEPTION_AND_THROW_JAVA_EXCEPTION 出现在所有上述 JNI 函数的末尾。

#define CATCH_CPP_EXCEPTION_AND_THROW_JAVA_EXCEPTION              \
                                                                  \
  catch (const package::Exception& e)                             \
  {                                                               \
    jclass jc = env->FindClass("group/package/Exception");        \
    if(jc) env->ThrowNew (jc, e.what());                          \
    /* if null => NoClassDefFoundError already thrown */          \
  }                                                               \
  catch (const std::bad_alloc& e)                                 \
  {                                                               \
    /* OOM exception */                                           \
    jclass jc = env->FindClass("java/lang/OutOfMemoryError");     \
    if(jc) env->ThrowNew (jc, e.what());                          \
  }                                                               \
  catch (const std::ios_base::failure& e)                         \
  {                                                               \
    /* IO exception */                                            \
    jclass jc = env->FindClass("java/io/IOException");            \
    if(jc) env->ThrowNew (jc, e.what());                          \
  }                                                               \
  catch (const std::exception& e)                                 \
  {                                                               \
    /* unknown exception */                                       \
    jclass jc = env->FindClass("java/lang/Error");                \
    if(jc) env->ThrowNew (jc, e.what());                          \
  }                                                               \
  catch (...)                                                     \
  {                                                               \
    /* Oops I missed identifying this exception! */               \
    jclass jc = env->FindClass("java/lang/Error");                \
    if(jc) env->ThrowNew (jc, "unexpected exception");            \
  }