fix: cannot detect interfaces' methods

Author: SmileYik

java/src/main/java/org/eu/smileyik/luajava/reflect/ReflectUtil.java

@@ -265,7 +265,16 @@ public static boolean existsMethodByName(
             return result;
         }
 
-        while (clazz != null) {
+        Set<Class<?>> checkedClasses = new HashSet<>();
+        LinkedList<Class<?>> queue = new LinkedList<>();
+        queue.add(clazz);
+        while (!queue.isEmpty()) {
+            clazz = queue.removeFirst();
+            if (!checkedClasses.add(clazz)) {
+                continue;
+            }
+            findSuperclasses(queue, clazz);
+
             for (Method method : clazz.getDeclaredMethods()) {
                 if (method.getName().equals(name)) {
                     if (!checkExecutableModifiers(method, ignoreNotPublic, ignoreStatic, ignoreNotStatic)) {
@@ -274,7 +283,6 @@ public static boolean existsMethodByName(
                     }
                 }
             }
-            clazz = clazz.getSuperclass();
         }
         CACHED_EXISTS_METHOD.putIfAbsent(cacheKey, false);
         return false;
@@ -313,9 +321,17 @@ public static LinkedList<LuaInvokedMethod<Method>> findMethodByParams(
         Set<ReflectExecutableCacheKey> checkedMethods = new HashSet<>();
 
         // find methods
-        Class<?> c = clazz;
+        Set<Class<?>> checkedClasses = new HashSet<>();
+        LinkedList<Class<?>> queue = new LinkedList<>();
+        queue.add(clazz);
         int paramsCount = params.length;
-        while (c != null) {
+        while (!queue.isEmpty()) {
+            Class<?> c = queue.removeFirst();
+            if (!checkedClasses.add(c)) {
+                continue;
+            }
+            findSuperclasses(queue, c);
+
             for (Method method : c.getDeclaredMethods()) {
                 // filters
                 if (!method.getName().equals(methodName) ||
@@ -336,7 +352,6 @@ public static LinkedList<LuaInvokedMethod<Method>> findMethodByParams(
                     if (currentPriority == FULL_MATCH) break;
                 }
             }
-            c = c.getSuperclass();
         }
         if (cacheKey != null) {
             CACHED_METHODS.get(cacheKey)
@@ -350,6 +365,18 @@ public static LinkedList<LuaInvokedMethod<Method>> findMethodByParams(
         return matchedList;
     }
 
+    private static void findSuperclasses(LinkedList<Class<?>> queue, Class<?> clazz) {
+        if (clazz.getSuperclass() != null) {
+            queue.addLast(clazz.getSuperclass());
+        }
+        Class<?>[] interfaces = clazz.getInterfaces();
+        for (Class<?> i : interfaces) {
+            if (i != null) {
+                queue.addLast(i);
+            }
+        }
+    }
+
     /**
      * return true means need skip check this method.
      */

java/src/test/java/org/eu/smileyik/luajava/javaapi/InterfaceDefaultMethodTest.java

@@ -0,0 +1,39 @@
+package org.eu.smileyik.luajava.javaapi;
+
+import org.eu.smileyik.luajava.BaseTest;
+import org.eu.smileyik.luajava.LuaStateFacade;
+import org.junit.jupiter.api.Test;
+
+public class InterfaceDefaultMethodTest extends BaseTest {
+
+    @Test
+    public void test() throws Exception {
+        LuaStateFacade luaStateFacade = newLuaState();
+        luaStateFacade.setGlobal("a", new A());
+        luaStateFacade.evalString("a:a()").justThrow();
+        luaStateFacade.evalString("a:superA()").justThrow();
+        luaStateFacade.evalString("a:b()").justThrow();
+    }
+
+    public static interface ISuperA {
+        public default void superA() {
+            System.out.println("superA");
+        }
+    }
+
+    public static interface IA extends ISuperA {
+        public default void a() {
+            System.out.println("a");
+        }
+
+        public void b();
+    }
+
+    public static class A implements IA {
+
+        @Override
+        public void b() {
+            System.out.println("b");
+        }
+    }
+}