Override JObject's toString with appropriate Java's toString (#337)

* add toString * remove extra hashCode and == as JObject already has it
dart-archive · Jul 31, 2023 · 71425b0 · 71425b0
1 parent be7a930
commit 71425b0
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 32 deletions.
diff --git a/jni/lib/src/jobject.dart b/jni/lib/src/jobject.dart
@@ -342,6 +342,15 @@ class JObject extends JReference {
     return Jni.accessors.callMethodWithArgs(reference, _equalsId,
         JniCallType.booleanType, [other.reference]).boolean;
   }
+
+  static final _toStringId = Jni.accessors.getMethodIDOf(
+      _objectClass.reference, r"toString", r"()Ljava/lang/String;");
+  @override
+  String toString() {
+    return JString.fromRef(Jni.accessors.callMethodWithArgs(
+            reference, _toStringId, JniCallType.objectType, []).object)
+        .toDartString(deleteOriginal: true);
+  }
 }
 
 /// A high level wrapper over a JNI class reference.

diff --git a/jni/lib/src/util/jlist.dart b/jni/lib/src/util/jlist.dart
@@ -267,21 +267,6 @@ class JList<$E extends JObject> extends JObject with ListMixin<$E> {
   JSet<$E> toSet() {
     return toJSet(E);
   }
-
-  static final _hashCodeId =
-      Jni.accessors.getMethodIDOf(_class.reference, r"hashCode", r"()I");
-  @override
-  int get hashCode => Jni.accessors.callMethodWithArgs(
-      reference, _hashCodeId, JniCallType.intType, []).integer;
-
-  static final _equalsId = Jni.accessors
-      .getMethodIDOf(_class.reference, r"equals", r"(Ljava/lang/Object;)Z");
-  @override
-  bool operator ==(Object other) {
-    if (other is! JObject) return false;
-    return Jni.accessors.callMethodWithArgs(reference, _equalsId,
-        JniCallType.booleanType, [other.reference]).boolean;
-  }
 }
 
 extension ToJavaList<E extends JObject> on Iterable<E> {

diff --git a/jni/lib/src/util/jset.dart b/jni/lib/src/util/jset.dart
@@ -190,23 +190,6 @@ class JSet<$E extends JObject> extends JObject with SetMixin<$E> {
     return super.retainAll(elements);
   }
 
-  static final _hashCodeId =
-      Jni.accessors.getMethodIDOf(_class.reference, r"hashCode", r"()I");
-  @override
-  int get hashCode => Jni.accessors.callMethodWithArgs(
-      reference, _hashCodeId, JniCallType.intType, []).integer;
-
-  static final _equalsId = Jni.accessors
-      .getMethodIDOf(_class.reference, r"equals", r"(Ljava/lang/Object;)Z");
-  @override
-  bool operator ==(Object other) {
-    if (other is! JObject) {
-      return false;
-    }
-    return Jni.accessors.callMethodWithArgs(reference, _equalsId,
-        JniCallType.booleanType, [other.reference]).boolean;
-  }
-
   @override
   $E? lookup(Object? element) {
     if (contains(element)) return element as $E;

diff --git a/jni/test/jstring_test.dart b/jni/test/jstring_test.dart
@@ -43,4 +43,9 @@ void run({required TestRunnerCallback testRunner}) {
       testStringBackAndForth('');
     });
   });
+
+  testRunner('Inherited toString', () {
+    final s = 'hello'.toJString();
+    expect(s.toString(), 'hello');
+  });
 }