【问题标题】:A bug in Java type system?Java类型系统中的错误?
【发布时间】:2012-08-28 14:33:03
【问题描述】:

想象有一个主类 -- Simulator -- 它使用另外两个类 -- ProducersEvaluators,它们实现了接口 IProducer 和IEvaluator,分别。

IProducer 实现产生结果,而 IEvaluator 实现评估这些结果。模拟器通过查询 IProducer 实现然后将结果传送到 IEvaluator 实例来控制执行流程。

Producer 和 Evaluator 的实际实现在运行时是已知的,在编译时我只知道它们的接口。检查下面的示例。

package com.test;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;

/**
 * Producers produce results. I do not care what their actual type is, but the
 * values in the map have to be comparable amongst themselves.
 */
interface IProducer<T extends Comparable<T>> {
    public Map<Integer, T> getResults();
}

/**
 * This example implementation ranks items in the map by using Strings.
 */
class ProducerA implements IProducer<String> {
    @Override
    public Map<Integer, String> getResults() {
        Map<Integer, String> result = new HashMap<Integer, String>();
        result.put(1, "A");
        result.put(2, "B");
        result.put(3, "B");

        return result;
    }
}

/**
 * This example implementation ranks items in the map by using integers.
 */
class ProducerB implements IProducer<Integer> {
    @Override
    public Map<Integer, Integer> getResults() {
        Map<Integer, Integer> result = new HashMap<Integer, Integer>();
        result.put(1, 10);
        result.put(2, 30);
        result.put(3, 30);

        return result;
    }
}

/**
 * Evaluator evaluates the results against the given groundTruth. All it needs
 * to know about results, is that they are comparable amongst themselves.
 */
interface IEvaluator {
    public <T extends Comparable<T>> double evaluate(Map<Integer, T> results,
            Map<Integer, Double> groundTruth);
}

/**
 * This is example of an evaluator, metric Kendall Tau-B. Don't bother with
 * semantics, all that matters is that I want to be able to call
 * r1.compareTo(r2) for every (r1, r2) that appear in Map<Integer, T> results.
 */
class KendallTauB implements IEvaluator {
    @Override
    public <T extends Comparable<T>> double evaluate(Map<Integer, T> results,
            Map<Integer, Double> groundTruth) {
        int concordant = 0, discordant = 0, tiedRanks = 0, tiedCapabilities = 0;

        for (Entry<Integer, T> rank1 : results.entrySet()) {
            for (Entry<Integer, T> rank2 : results.entrySet()) {
                if (rank1.getKey() < rank2.getKey()) {
                    final T r1 = rank1.getValue();
                    final T r2 = rank2.getValue();
                    final Double c1 = groundTruth.get(rank1.getKey());
                    final Double c2 = groundTruth.get(rank2.getKey());

                    final int ranksDiff = r1.compareTo(r2);
                    final int actualDiff = c1.compareTo(c2);

                    if (ranksDiff * actualDiff > 0) {
                        concordant++;
                    } else if (ranksDiff * actualDiff < 0) {
                        discordant++;
                    } else {
                        if (ranksDiff == 0)
                            tiedRanks++;

                        if (actualDiff == 0)
                            tiedCapabilities++;
                    }
                }
            }
        }

        final double n = results.size() * (results.size() - 1d) / 2d;

        return (concordant - discordant)
                / Math.sqrt((n - tiedRanks) * (n - tiedCapabilities));
    }
}

/**
 * The simulator class that queries the producer and them conveys results to the
 * evaluator.
 */
public class Simulator {
    public static void main(String[] args) {
        // example of a ground truth
        Map<Integer, Double> groundTruth = new HashMap<Integer, Double>();
        groundTruth.put(1, 1d);
        groundTruth.put(2, 2d);
        groundTruth.put(3, 3d);

        // dynamically load producers
        List<IProducer<?>> producerImplementations = lookUpProducers();

        // dynamically load evaluators
        List<IEvaluator> evaluatorImplementations = lookUpEvaluators();

        // pick a producer
        IProducer<?> producer = producerImplementations.get(0);

        // pick an evaluator
        IEvaluator evaluator = evaluatorImplementations.get(0);

        // evaluate the result against the ground truth
        double score = evaluator.evaluate(producer.getResults(), groundTruth);

        System.out.printf("Score is %.2f\n", score);
    }

    // Methods below are for demonstration purposes only. I'm actually using
    // ServiceLoader.load(Clazz) to dynamically discover and load classes that
    // implement interfaces IProducer and IEvaluator
    public static List<IProducer<?>> lookUpProducers() {
        List<IProducer<?>> producers = new ArrayList<IProducer<?>>();
        producers.add(new ProducerA());
        producers.add(new ProducerB());

        return producers;
    }

    public static List<IEvaluator> lookUpEvaluators() {
        List<IEvaluator> evaluators = new ArrayList<IEvaluator>();
        evaluators.add(new KendallTauB());

        return evaluators;
    }
}

此代码在没有警告的情况下编译,并且还按应有的方式运行。这是solution to the question that I asked before,所以这是一个后续问题。

使用上面的代码,假设您希望将 producer.getResults() 调用的结果存储在一个变量中(稍后将在对 evaluator.evaluate(results, groundTruth) 调用的调用中使用该变量) . 该变量的类型是什么?

Map, Map>?使main方法泛型并使用泛型类型?到目前为止,我尝试过的任何方法都不起作用。编译器会抱怨我提出的每种类型。

public static void main(String[] args) {
    // example of a ground truth
    Map<Integer, Double> groundTruth = new HashMap<Integer, Double>();
    groundTruth.put(1, 1d);
    groundTruth.put(2, 2d);
    groundTruth.put(3, 3d);

    // dynamically load producers
    List<IProducer<?>> producerImplementations = lookUpProducers();

    // dynamically load evaluators
    List<IEvaluator> evaluatorImplementations = lookUpEvaluators();

    // pick a producer
    IProducer<?> producer = producerImplementations.get(0);

    // pick an evaluator
    IEvaluator evaluator = evaluatorImplementations.get(0);

    // evaluate the result against the ground truth
    Map<Integer, ?> data = producer.getResults(); // this type works
    double score = evaluator.evaluate(data, groundTruth); // but now this call does not


    System.out.printf("Score is %.2f\n", score);
}

似乎 producer.getResults() 返回的东西不能用 Java 静态表达。这是一个错误,还是我错过了什么?

【问题讨论】:

  • 为什么不能直接使用evaluator.evaluate(producer.getResults(), groundTruth)
  • @Jeffrey,我可以使用它,但是,假设您想在几个不同的评估器实例上调用评估(数据,goundTruth)具有相同的数据(即从某个生产者实例返回)。每次这样的调用都调用 getResult 是不明智的,因为 getResults 通常包含繁重的计算
  • 你有没有试过只使用Map,没有泛型类型参数?我知道这种回避了这里的问题,但它至少应该编译和运行。那么确保Map 中包含正确类型的内容就是您的责任。
  • @David 很公平,我发布了一种可能的解决方法作为您的答案。

标签: java generics types wildcard


【解决方案1】:

这不是一个错误,而是一个限制。在类型系统社区中众所周知,带有通配符的 Java 具有无法用 Java 语法表示的类型。您的示例展示了一种这样的情况,并证明通配符本质上与 F 有界多态性(即 T extends Something&lt;T&gt; 形式的类型参数)不兼容。

坦率地说,通配符是一种可怕的类型系统黑客攻击。它们不应该被放入 Java 中。真正想要的,以及使您的示例可表达的是正确的存在类型(其中通配符是一种有限的临时变体)。不幸的是,Java 没有它们(尽管 Scala 有)。

【讨论】:

    【解决方案2】:

    在我回答之前请注意:您所有的T extends Comparable&lt;T&gt; 语句可能都应该是T extends Comparable&lt;? super T&gt;,它允许更大的灵活性(您为什么要关心它是否比较Ts 或Objects?),并且它是必需的让我的解决方案发挥作用。

    这并不是 Java 类型系统的真正“错误”,它只是给它带来了不便。 Java 并不特别喜欢将交集类型作为类型声明的一部分。

    我发现解决此问题的一种方法是创建一个在正常情况下永远不应使用的“不安全”方法:

    @SuppressWarnings("unchecked")
    private static <T extends Comparable<? super T>> Map<Integer, T> cast(Map<Integer, ?> map) {
        return (Map<Integer, T>) map;
    }
    

    只需确保使用实际上是 Map&lt;Integer, T extends Comparable&lt;? super T&gt;&gt;Map 调用此方法(就像返回的 IProducers 一样)。

    使用此方法,您可以执行以下操作:

    IProducer<?> producer = ...
    IEvaluator evaluator = ...
    Map<Integer, ?> product = producer.getResults();
    evaluator.evaluate(cast(product), truth);
    

    然后Java会自动为你推断出正确的类型参数。

    另外,I 前缀在 Java 社区中是 generally frowned upon

    【讨论】:

    • 谢谢。一些 cmets: - 而不是 @SuppressWarnings("unused") 它应该是 @SuppressWarnings("unchecked")。 - 我有类似的东西,但 SuppressWarnings 指令困扰着我 - 没有明显的原因,只是觉得我做的不对。所以我试图找到更好的东西。我很惊讶我偶然发现了语言限制/不便。 - PS:也感谢您指出他的 IInterface 约定。我会放弃它。
    • @David 啊,哎呀。我正在徒手输入此代码。只要您可以验证强制转换确实正确,抑制未经检查的警告就完全可以了。编译器不够聪明,无法为您执行此操作,因为它会丢弃所有类型信息。作为程序员,你足够聪明来验证转换,因为你不必通过类型擦除来保持二进制兼容性。
    猜你喜欢
    • 2020-12-02
    • 1970-01-01
    • 1970-01-01
    • 2011-07-16
    • 1970-01-01
    • 2012-09-27
    • 2023-03-22
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多