发一个本人写的鸡肋的jni代码,你能看出有什么问题吗？

/*
 * 执行对象中的方法
 */
jvalue callMethod(JNIEnv *env, jboolean& hasException, jobject obj, const char *methodName, const char *descriptor, ...)
{
//	printf("Method");
	jvalue result;

	jobject refobj;

	jclass targetcls;

	jmethodID mid;

	const char *p = descriptor;

	//判断本地版本
//	if(env->EnsureLocalCapacity(100) != JNI_OK)
//	{
//		printf("EnsureLocalCapacity == null");
//		goto error;
//	}

	printf("Method11");
	refobj = env->NewLocalRef(obj);
	printf("Method33");
	targetcls = env->GetObjectClass(refobj);
	printf("Method1");
	mid = env->GetMethodID(targetcls, methodName, descriptor);

	if(mid == NULL){
		printf("mid == null");
		env->DeleteLocalRef(targetcls);
		env->DeleteLocalRef(refobj);
		goto error;
	}

	while(*p != ')') p++;

	p++;

	va_list ap;

	va_start(ap, descriptor);
//	printf("Method2");
	switch(*p){
	case 'V':
		env->CallVoidMethodV(refobj, mid, ap);
		break;
	case '[':
	case 'L':
		result.l = env->CallObjectMethodV(refobj, mid, ap);
		break;
	case 'Z':
		result.z = env->CallBooleanMethodV(refobj, mid, ap);
		break;
	case 'B':
		result.b = env->CallByteMethodV(refobj, mid, ap);
		break;
	case 'C':
		result.c = env->CallCharMethodV(refobj, mid, ap);
		break;
	case 'S':
		result.s = env->CallShortMethodV(refobj, mid, ap);
		break;
	case 'I':
		result.i = env->CallIntMethodV(refobj, mid, ap);
		break;
	case 'J':
		result.j = env->CallLongMethodV(refobj, mid, ap);
		break;
	case 'F':
		result.f = env->CallFloatMethodV(refobj, mid, ap);
		break;
	case 'D':
		result.d = env->CallDoubleMethodV(refobj, mid, ap);
		break;

	}

	va_end(ap);
//	printf("Method3");
	env->DeleteLocalRef(targetcls);
	env->DeleteLocalRef(refobj);

	error:
		if(env->ExceptionOccurred())
		{
			printf("Method4");
			hasException = env->ExceptionCheck();
//			env->ExceptionDescribe();
//			env->ExceptionClear();
//			env->ThrowNew(targetcls, "error");
		}
	return result;
}