非静态比较器内部类

Question

我想知道是否可以制作实现比较器接口的非静态内部类。

例如：。

public class A {
   private Map<String, Integer> scoreMap;

   private final class myComparator implements Comparator<MyOtherClass> {

       private int calScore(MyOtherClass ob) {
           int score = ob.someValue * ob.otherValue;
           scoreMap.put(ob.getName(), score);
       }

       public int compare(MyOtherClass ob1, MyOtherCLass ob2) {
           return Integer.compare(calScore(ob1), calScore(ob2));
       }   

   }

}

我想使用比较器类非静态，因为我想使用非静态字段“num”并想修改它的值。有非静态比较器内部类可以吗？

附加信息

对于每个对象，我都在计算内部的分数，并进行相应的比较和排序。我需要将这些分数保存在我在comparator内计算的地图中，并希望在外部类中使用该地图进行进一步计算。

Answer 1

从技术上讲，没有什么能阻止你。然而从道德上来说。。。

只要您保持幂等性（运行多次并获得相同的结果），那么它就不是不正确的。然而，这是一个副作用。（使用此比较器，某些类的值会发生变化。运行另一个比较器，什么都不会改变）。副作用不一定是坏的，但它们会使程序更加复杂，更难理解和调试。

试着通过一些可怜的灵魂的眼睛来看待它，他们必须维护你的代码。一些价值观发生了变化，但他们不知道为什么。他们所做的只是比较一份无关联的名单。

不管怎样，你的例子非常抽象，所以很难判断上下文，但通常没有必要做你想做的事情。除非你有一个非常好的理由，否则这通常是要避免的。在我看来，这个非常好的理由不应该是“因为我不想再次遍历数据集”。

从您的编辑：

你试图通过不需要再次从声音中重新计算来节省自己的工作。你实际上只节省了自己少量的精力。为什么不先计算分数，存储结果，然后对结果进行排序（在分数字段上）

例如：

public static class ScoredEntry {
    private SomeGameData data; //or something
    private int score;
    // constructor takes data + score,  imagine getters, setters, I am lazy ok.
}

public List<ScoredEntry> scoreEntries(List<SomeGameData> gameData) {
    List<ScoredEntry> result = new ArrayList<ScoredEntry>();
    for (SomeGameData data : gameData) {
        int score = calculateScore(data);
        result.add(new SCoredEntry(score, data);
    }
    return result;
}

public void sortBySCore(List<ScoredEntry> list) {
    Collections.sort(list, new Comparator<ScoredEntry>() {
        public int compare(SCoredEntry a, ScoredEntry b) {
            // etcetera.
        }
    }
}