dubbo 2.5.3 bug 同名方法同参数个数重载ClassCastException-优快云博客

本文详细解析了Dubbo框架中InvokeTelnetHandler类findMethod方法的一个Bug，该Bug在方法重载时未正确匹配参数类型，导致方法调用错误。文章提供了修正后的代码实现，确保了方法调用的准确性。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

这次遇到的问题是com.alibaba.dubbo.rpc.protocol.dubbo.telnet.InvokeTelnetHandler.findMethod中一个bug
遍历所有方法时，首次匹配到方法名m.getName()和入参方法名method一致，且参数个数 m.getParameterTypes().length 同入参args.size()相同的方法，将invokeMethod设置为此方法，然后继续遍历方法列表。当即 invokeMethod != null 后，再次发现方法名m.getName()和参数方法名method一致，且参数个数 m.getParameterTypes().length 同入参args.size()相同的方法时，应该判断当前方法的参数m.getParameterTypes()和args是否匹配，而不是invokeMethod.getParameterTypes()和args是否匹配.

private static Method findMethod(Exporter<?> exporter, String method, List<Object> args) {
        Invoker<?> invoker = exporter.getInvoker();
        Method[] methods = invoker.getInterface().getMethods();
        Method invokeMethod = null;
        for (Method m : methods) {
            if (m.getName().equals(method) && m.getParameterTypes().length == args.size()) {
                if (invokeMethod != null) { // 重载
                    if (isMatch(invokeMethod.getParameterTypes(), args)) {
                        invokeMethod = m;
                        break;
                    }
                } else {
                    invokeMethod = m;
                }
                invoker = exporter.getInvoker();
            }
        }
        return invokeMethod;
    }
    
    private static boolean isMatch(Class<?>[] types, List<Object> args) {
        if (types.length != args.size()) {
            return false;
        }
        for (int i = 0; i < types.length; i ++) {
            Class<?> type = types[i];
            Object arg = args.get(i);
            if (ReflectUtils.isPrimitive(arg.getClass())) {
                if (! ReflectUtils.isPrimitive(type)) {
                    return false;
                }
            } else if (arg instanceof Map) {
                String name = (String) ((Map<?, ?>)arg).get("class");
                Class<?> cls = arg.getClass();
                if (name != null && name.length() > 0) {
                    cls = ReflectUtils.forName(name);
                }
                if (! type.isAssignableFrom(cls)) {
                    return false;
                }
            } else if (arg instanceof Collection) {
                if (! type.isArray() && ! type.isAssignableFrom(arg.getClass())) {
                    return false;
                }
            } else {
                if (! type.isAssignableFrom(arg.getClass())) {
                    return false;
                }
            }
        }
        return true;
    }