NullPointerException bei Zugriff auf JNI Objekt

[mhauert]

Hallo zusammen,

ich schreibe gerade ein Programm das JNI nutzen soll um C++ Code auszuführen.
Ich nutzte dabei Eclipse und MS Visual C++ 2008 Express auf einem WindowsXP System.

Bei Testen im kleinen Rahmen bin ich auf ein Problem gestoßen, das ich nicht lösen kann.

Es tritt hier auf:

public static void main(String[] args) 
	{		
		DesktopImpl impl = new DesktopImpl();
		Snapshot x = impl.getComplete();
		System.out.println("Count: " + x.count);
		System.out.println(x.bytes[0]+", "+x.bytes[1]);
		System.out.println(x.changes[0]+", "+x.changes[1]);
         }

In der Konsole zeigt sich folgendes:

Count: 580449256
Exception in thread "main" java.lang.NullPointerException
	at de.cworxmedia.rds.server.test.main(test.java:17)

Der Konstruktor für Snapshot sieht so aus:

public Snapshot(int count, byte[] bytes, int[] changes)
	{
		this.count = count;
		this.bytes = bytes;
		this.changes = changes;
	}

Die Methode getComplete() wurde in einer anderen Klasse als native deklariert und in C++ implementiert.
Sie soll ein Snapshot- Objekt erstellen und es dann an das Javaprogramm zurückgeben

JNIEXPORT jobject JNICALL Java_de_cworxmedia_rds_natives_Win32Desktop_getComplete
  (JNIEnv *env, jobject jc)
{
jclass jclass = env->GetObjectClass(jc);
jmethodID jmid = env->GetMethodID(jclass, "<init>", "()V");
jint jcount = 345;
jbyteArray jbytes = env->NewByteArray(2);
jintArray jchanges = env->NewIntArray(2);
jbyte *jbytesbuf = env->GetByteArrayElements(jbytes, 0);
jint *jchangesbuf = env->GetIntArrayElements(jchanges, 0);
jbytesbuf[0] = (BYTE) 22;
jbytesbuf[1] = (BYTE) 23;
jchangesbuf[0] = (jint) 5;
jchangesbuf[1] = (jint) 46723;
jobject jo = env->NewObject(jclass, jmid, jcount, jbytes, jchanges);
return jo;
}

Der C++ Code wurde dann in eine DLL kompiliert.

Eigentlich sollte ja das Programm die Int- Variable und die beiden Arrays ausgeben.
Aber wie man sieht, der Int-Wert ist falsch und die Arrays nicht vorhanden.

Ich weiß aber nicht, wie ich den C++ Code abändern müsste, um das Problem zu beheben.
Das es am Java Code liegt kann ich mir nicht vorstellen.

Ich bin für jeden Lösungsvorschlag dankbar.

 

[Marco13]

Hm - kann das gerade nicht testen, aber ... es könnte sein, dass das Problem darin liegt, dass die zurückgegebene Referenz galöscht wird (in dem Moment, wo sie zurückgegeben wird)

Schau mal hier drüber
http://java.sun.com/docs/books/jni/html/refs.html

Wenns nicht hilft, sag bescheid...

 

[mhauert]

Erstmal danke für den Link. Der hat mir neue Einblicke beschert.

Nun zu meiner daraus entstandenen Idee:

JNIEXPORT jobject JNICALL Java_de_cworxmedia_rds_natives_Win32Desktop_getComplete
  (JNIEnv *env, jobject jc)
{

[...] //Der Code hier hat sich nicht geändert

jobject jo = env->NewObject(jclass, jmid, jcount, jbytes, jchanges);
return env->NewGlobalRef(jo);
}

Das einzige was sich danach änderte, war der Wert des Integers von 580449256 in 580440082.

Um die Konstruktion des Snapshot- Objekts in C++ zu umgehen, war meine nächste Idee dann, der C++ Methode ein leeres Snapshot Objekt zu übergeben, und dieses dann zu befüllen und zurück zu geben:

JNIEXPORT jobject JNICALL Java_de_cworxmedia_rds_natives_Win32Desktop_getCompleteN
  (JNIEnv *env, jobject jc, jobject dummy)
{
jclass cdummy = env->GetObjectClass(dummy);
jbyteArray jbytes  = env->NewByteArray(2);
jintArray jchanges = env->NewIntArray(2);
jbyte *jbytesbuf   = env->GetByteArrayElements(jbytes, 0);
jint *jchangesbuf  = env->GetIntArrayElements(jchanges, 0);
int jcount     = 345;
jbytesbuf[0]   = (BYTE) 22;
jbytesbuf[1]   = (BYTE) 23;
jchangesbuf[0] = (jint) 5;
jchangesbuf[1] = (jint) 46723;

jfieldID jcountID   = env->GetFieldID(cdummy, "count", "I");
jfieldID jbytesID   = env->GetFieldID(cdummy, "bytes", "[B");
jfieldID jchangesID = env->GetFieldID(cdummy, "changes", "[I");

env->SetIntField(dummy, jcountID, jcount);
env->SetObjectField(dummy, jbytesID, jbytes);
env->SetObjectField(dummy, jchangesID, jchanges);

return dummy;
}

Und siehe da, zumindest der Integer ist jetzt richtig. Die Ausgabe des Programms lautet:

345
0, 0
0, 0

Problem ist nur das die Arrays nun zwar da sind, aber als Werte nur 0 enthalten. Ich würde mal schätzen, das die folgenden Zeilen nicht den gewünschten Effekt haben:

jbyteArray jbytes  = env->NewByteArray(2);
jintArray jchanges = env->NewIntArray(2);
jbyte *jbytesbuf   = env->GetByteArrayElements(jbytes, 0);
jint *jchangesbuf  = env->GetIntArrayElements(jchanges, 0);

jbytesbuf[0]   = (BYTE) 22;
jbytesbuf[1]   = (BYTE) 23;
jchangesbuf[0] = (jint) 5;
jchangesbuf[1] = (jint) 46723;

Gibt es da noch einen Befehl, der die jarray Objekte aktualisiert? Weil ich dachte, die Zeiger zeigen in diese Objekte...

Solltes du für dieses Problem oder für das ursprüngliche Problem mit der "NullPointerException" noch eine Idee haben, würde mir das sehr helfen.

Danke im Vorraus.

 

[mhauert]

OK, alles klar.

Hatte in der Spezifikation überlesen, das die Release<Type>ArrayElements- Funktion
nicht nur den Speicher freigibt sondern auch die Werte in das Java- Array schreibt. :roll:

Der folgende Code arbeitet einwandfrei:

JNIEXPORT jobject JNICALL Java_de_cworxmedia_rds_natives_Win32Desktop_getCompleteN
  (JNIEnv *env, jobject jc, jobject dummy)
{
jclass cdummy = env->GetObjectClass(dummy);
jbyteArray jbytes  = env->NewByteArray(2);
jintArray jchanges = env->NewIntArray(2);
jbyte *jbytesbuf   = env->GetByteArrayElements(jbytes, 0);
jint *jchangesbuf  = env->GetIntArrayElements(jchanges, 0);
int jcount     = 345;
jbytesbuf[0]   = (BYTE) 22;
jbytesbuf[1]   = (BYTE) 23;
jchangesbuf[0] = (jint) 5;
jchangesbuf[1] = (jint) 46723;

jfieldID jcountID   = env->GetFieldID(cdummy, "count", "I");
jfieldID jbytesID   = env->GetFieldID(cdummy, "bytes", "[B");
jfieldID jchangesID = env->GetFieldID(cdummy, "changes", "[I");

env->ReleaseByteArrayElements(jbytes, jbytesbuf, 0);
env->ReleaseIntArrayElements(jchanges, jchangesbuf, 0);

env->SetIntField(dummy, jcountID, jcount);
env->SetObjectField(dummy, jbytesID, jbytes);
env->SetObjectField(dummy, jchangesID, jchanges);

return dummy;
}

Habe ich das auch richtig verstanden, das die Release- Funktion den Speicher wieder freigibt?
Oder muss ich die Zeiger noch extra freigeben und löschen? Nicht das mir der Speicher überläuft...

 

[Marco13]

Jo, hab nochmal drübergeschaut: Das hier ist ja so irgendwie wohl nicht richtig

jclass jclass = env->GetObjectClass(jc);
jmethodID jmid = env->GetMethodID(jclass, "<init>", "()V");
...
jobject jo = env->NewObject(jclass, jmid, jcount, jbytes, jchanges);
return jo;

Du holst dir dort die methodID vom Konstruktor der Klasse, auf der die Methode aufgerufen wird - also von DesktopImpl. Und dieser Konstruktor soll dann verwendet werden, um ein Objekt vom Typ Snapshot zu erstellen.

Hiermit sollte es gehen

#include "JniReturnTest.h"

JNIEXPORT jobject JNICALL Java_JniReturnTest_getComplete
  (JNIEnv *env, jobject jc)
{
	jint jcount = 345;
	jbyteArray jbytes = env->NewByteArray(2);
	jintArray jchanges = env->NewIntArray(2);
	jbyte *jbytesbuf = env->GetByteArrayElements(jbytes, 0);
	jint *jchangesbuf = env->GetIntArrayElements(jchanges, 0);
	jbytesbuf[0] = 22;
	jbytesbuf[1] = 23;
	jchangesbuf[0] = (jint) 5;
	jchangesbuf[1] = (jint) 46723;

	// Hier releasen / zurückschreiben
	env->ReleaseByteArrayElements(jbytes, jbytesbuf, 0);
	env->ReleaseIntArrayElements(jchanges, jchangesbuf, 0);

	// Konstruktor für "Snapshot" suchen
	jclass jcl = env->FindClass("Snapshot");
	jmethodID jmid = env->GetMethodID(jcl, "<init>", "(I[B[I)V"); // Signatur beachten!
	jobject jo = env->NewObject(jcl, jmid, jcount, jbytes, jchanges);

	return env->NewGlobalRef(jo); // Sollte auch wieder gelöscht werden können !!??!!
}

 

[Marco13]

Ach ja: Das mit dem Release sollte so OK sein. Der letzte Post bezog sich jetzt auf das Erstellen des Objektes von C++-Seite aus Und wenn du ein "dummy"-Objekt in Java erstellst und übergibst, brauchst du dir wenigstens keine Gedanken über die Referenz auf der C++-Seite zu machen - DA könnte dann nämlich Speicher flöten gehen...

 

[mhauert]

Ah ja. Da hab ich wohl was durcheinander gehauen.

Vielen Dank für die Hilfe.
Ich werds jetzt erstmal so lassen und sehen wie es sich in der Praxis macht...