2024 Clojure 状态调查! 中分享你的想法。

欢迎!请参阅 关于 页面以获取更多有关这个工作方式的信息。

+3
Java 互操作

Clojure 编译器无法解析从包私有基类继承的公开泛型方法。

重现说明

  • 在包 P1 中
    • 定义一个具有泛型类型参数的包私有类 A
    • 在 A 中定义一个使用泛型类型(其参数或返回值)的公共方法 M
      **** 定义一个扩展 A 的公共类 B
  • 在包 P2 中
    • 创建 B 的一个实例
      **** 调用 B.M()

这在 Java 中是有效的。在 Clojure 中,调用 B.M 会产生一个反射警告,然后是错误 "java.lang.IllegalArgumentException: Can't call public method of non-public class。"无论多少类型提示都无法防止警告或错误。

附件 clj-1243-demo1.tar.gz 包含示例代码和演示问题的脚本。

使用包私有类中的公开方法的 Java 项目的示例

14 个答案

0

评论由:stuart.sierra

Java 中也无法通过反射调用该方法。

这可能是 Java 反射中的一个漏洞:(链接:http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4283544 文字:JDK-4283544)

但为什么它只会在泛型方法中出现呢?

错误。这是一个合法的方法调用,因此可以完全通过 Java 反射调用它。您甚至不需要使用setAccessible。请参见我的答案示例:[链接](https://ask.clojure.org/index.php/4255/cannot-resolve-public-generic-method-package-private-class?show=12483#a12483)
0

评论由:stuart.sierra

根据 Rich Hickey,桥接方法的实现是不指定并且在不同 JDK 版本之间不一致。

一个可能的解决方案是使用 ASM 检查第三方 Java 类的字节码,而不是使用反射 API。这样 Clojure 编译器就可以访问与 Java 编译器相同的信息。

0

评论者:jafingerhut

CLJ-1183 被关闭,作为此问题的重复。

0

评论者:noamb

我现在使用的一个当前绕过方法是定义一个新的 Java 类,添加一个静态方法来完成我需要的功能,然后从 Clojure 中调用它。

0

评论者:noamb

此外,我发现在 1.6 和 1.7(alpha5) 中也存在这个问题,但问题中提到的是只有 1.5 的问题。

0

评论人:adamtait

在使用 Google 的 Cloud API 时遇到了这个问题。
要使用 Google 的 Cloud Datastore,您需要访问受保护泛型子类 (BaseKey) 上的 (链接: https://github.com/GoogleCloudPlatform/gcloud-java/blob/v0.1.6/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/BaseKey.java#L96 文本: .kind) 方法,该子类通过 (链接: https://github.com/GoogleCloudPlatform/gcloud-java/blob/v0.1.6/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/KeyFactory.java#L25 文本: KeyFactory) 扩展。

在 Clojure 1.7 和 1.8 运行时测试过,以下异常仍然存在;
{quote}
IllegalArgumentException 无法调用非公共类的公开方法:public com.google.gcloud.datastore.BaseKey$Builder com.google.gcloud.datastore.BaseKey$Builder.kind(java.lang.String) clojure.lang.Reflector.invokeMatchingMethod (Reflector.java:88)
{quote}

0

评论人:kstrempel

我遇到了与 Google Cloud API 完全相同的类似问题。

在 1.8 和 1.9.0-alpha7 上测试过。问题相同。

0

评论人:kstrempel

我遇到了与 Google Cloud API 完全相同的类似问题。

在 1.8 和 1.9.0-alpha7 上测试过。问题相同。

0

评论人:mruza

我遇到了相同的问题。(链接: ^invocation_target_selection.patch 文本: 附加的补丁)为我解决了这个问题。
项目的所有测试仍然通过,但迫切需要有人进行审查。

0

评论人:alexmiller

嗨,Michal,

谢谢您查看。

  1. 请在这里遵循创建补丁的正确格式的说明:http://dev.clojure.org/display/community/Developing Patches
  2. 如果您能提供一些解释说明更改,以便于审查,这将非常有帮助。否则,审查员不得不从头开始重新构建您的思维过程。
  3. 在审查之前,这次更改也需要进行一些测试(虽然编写起来并不特别有趣,但我觉得在这里这是必要的)
0

评论人:mruza

我已经根据指示添加了测试并更新了补丁。

下面是其中的合理理由。以下是从 src/jvm/clojure/lang/Compiler.java 文件中摘录的内容

1462: if(target.hasJavaClass() && target.getJavaClass() != null) 1463: { 1464: List methods = Reflector.getMethods(target.getJavaClass(), args.count(), methodName, false); 1465: if(methods.isEmpty()) 1466: { 1467: method = null; 1468: if(RT.booleanCast(RT.WARN_ON_REFLECTION.deref())) 1469: { 1470: RT.errPrintWriter() 1471: .format("Reflection warning, %s:%d:%d - call to method %s on %s can't be resolved (no such method).\n", 1472: SOURCE_PATH.deref(), line, column, methodName, target.getJavaClass().getName()); 1473: } 1474: } 1475: else 1476: { 1477: int methodidx = 0; 1478: if(methods.size() > 1) 1479: { 1480: ArrayList<Class[]> params = new ArrayList(); 1481: ArrayList<Class> rets = new ArrayList(); 1482: for(int i = 0; i < methods.size(); i++) 1483: { 1484: java.lang.reflect.Method m = (java.lang.reflect.Method) methods.get(i); 1485: params.add(m.getParameterTypes()); 1486: rets.add(m.getReturnType()); 1487: } 1488: methodidx = getMatchingParams(methodName, params, args, rets); 1489: } 1490: java.lang.reflect.Method m = 1491: (java.lang.reflect.Method) (methodidx >= 0 ? methods.get(methodidx) : null); 1492: if(m != null && !Modifier.isPublic(m.getDeclaringClass().getModifiers())) 1493: { 1494: //public method of non-public class, try to find a public descendant 1495: if((type=Reflector.getDeepestPublicDescendant(m.getDeclaringClass(), target.getJavaClass())) == null) 1496: //if descendant not found, try to find an ancestor 1497: m = Reflector.getAsMethodOfPublicBase(m.getDeclaringClass(), m); 1498: } 1499: method = m; 1500: if(method == null && RT.booleanCast(RT.WARN_ON_REFLECTION.deref())) 1501: { 1502: RT.errPrintWriter() 1503: .format("Reflection warning, %s:%d:%d - call to method %s on %s can't be resolved (argument types: %s).\n", 1504: SOURCE_PATH.deref(), line, column, methodName, target.getJavaClass().getName(), getTypeStringForArgs(args)); 1505: } 1506: } 1507: }

  • 行1462上的条件确保了目标类型的类型/类已知
  • 行1464上调用的 {{clojure.lang.Reflector.getMethods()}} 方法返回针对目标类型定义的给定名称的所有 public 方法列表
  • 然后在行1477-1491中选择最佳要调用的方法
  • 如果所选方法的声明类不是 public 的,那么将尝试找到一个既是目标类型的超类又是声明所选方法的类的子类的 public 类 - 这个功能在 {{clojure.lang.Reflector.getDeepestPublicDescendant()}} 方法中实现
  • 如果找到这样的类,则用它代替方法的声明类来生成调用方法的字节码
  • 如果找不到这样的类,则尝试在声明所选方法的类的 public 祖先中找到一个兼容的方法

请注意,更改可能会导致调用的方法与更改前不同,如 {{selecting-method-on-nonpublic-interface}} 测试所示。我认为这是一个可以接受的变化,因为它
导致了更好的方法匹配(相对于参数类型),可以调用的方法

0
by

评论由:stuart.sierra

CLJ-126 描述了在 Java 5 上的一个类似问题。

0
by
参考:[https://clojure.atlassian.net/browse/CLJ-1243](https://clojure.atlassian.net/browse/CLJ-1243)(由 stuart.sierra 报告)
0
by

以下是一个Java代码可以正常工作但在Clojure上失败的示例。示例类来自“[org.opensearch.client/opensearch-java "2.1.0"]”库。

此Java代码没有问题可以工作

package example;

import org.opensearch.client.opensearch.core.bulk.IndexOperation;

import java.io.IOException;
import java.util.Map;

public class Example {
    public static void main(String[] args) throws IOException {
        new IndexOperation.Builder<Map<?, ?>>()
                .index("test-index")
                .id("test-id")
                .document(Map.of("test-document", 123))
                .build();
    }
}

但Clojure版本不能使用Java互操作书写。解决方法是手动使用反射调用这些方法

(ns example
  (:import (org.opensearch.client.opensearch.core.bulk IndexOperation$Builder)))

(defn bug []
  (-> (IndexOperation$Builder.)
      (.index "test-index")
      (.id "test-id")
      (.document {"test-document" 123})
      (.build)))

(defn workaround []
  (let [bulk-builder-index (-> (Class/forName "org.opensearch.client.opensearch.core.bulk.BulkOperationBase$AbstractBuilder")
                               (.getDeclaredMethod "index" (into-array Class [String])))
        bulk-builder-id (-> (Class/forName "org.opensearch.client.opensearch.core.bulk.BulkOperationBase$AbstractBuilder")
                            (.getDeclaredMethod "id" (into-array Class [String])))]
    (-> (IndexOperation$Builder.)
        ^IndexOperation$Builder ((fn [builder index]
                                   (.invoke bulk-builder-index builder (into-array Object [index])))
                                 "test-index")
        ^IndexOperation$Builder ((fn [builder id]
                                   (.invoke bulk-builder-id builder (into-array Object [id])))
                                 "test-id")
        (.document {"test-document" 123})
        (.build))))

(comment
  (bug)
  (workaround))

当使用Clojure 1.11.1加载此命名空间时,会出现以下反射警告

Reflection warning, .../example.clj:6:7 - call to method index on org.opensearch.client.opensearch.core.bulk.IndexOperation$Builder can't be resolved (argument types: java.lang.String).
Reflection warning, .../example.clj:7:7 - call to method id can't be resolved (target class is unknown).
Reflection warning, .../example.clj:8:7 - call to method document can't be resolved (target class is unknown).
Reflection warning, .../example.clj:9:7 - reference to field build can't be resolved.

当执行bug函数时,会抛出异常

Execution error (IllegalArgumentException) at example/bug (example.clj:6).
Can't call public method of non-public class: public final org.opensearch.client.opensearch.core.bulk.BulkOperationBase$AbstractBuilder org.opensearch.client.opensearch.core.bulk.BulkOperationBase$AbstractBuilder.index(java.lang.String)

IndexOperation$Builder 是一个公开类,它扩展了受保护的抽象类 WriteOperation$AbstractBuilder,该类又扩展了受保护的抽象类 BulkOperationBase$AbstractBuilderindexid 方法的返回类型是一个类级类型参数,但方法参数是非泛型的。

以下是库代码中的相关位置

...