社区首页 >问答首页 >findbugs反对匿名内部类。

问findbugs反对匿名内部类。
EN

Stack Overflow用户

提问于 2014-02-24 10:48:01

回答 4查看 19.9K关注 0票数 11

此代码：

Set<Map.Entry<String, SSGSession>> theSet =  new TreeSet<Map.Entry<String, SSGSession>>(new Comparator<Map.Entry<String, SSGSession>>() {

        @Override
        public int compare(final Map.Entry<String, SSGSession> e1, final Map.Entry<String, SSGSession> e2) {
            return e2.getValue().getStartTime().compareTo(e1.getValue().getStartTime());
        }
    }));

触发Sonar中的违规行为，触发具有以下描述的findbug规则"SIC_INNER_SHOULD_BE_STATIC_ANON“：

这个类是一个内部类，但不使用它对创建它的对象的嵌入引用。此引用使类的实例更大，并可能使对创建者对象的引用比必要的时间更长。如果可能的话，这个类应该变成一个静态的内部类。由于匿名内部类不能被标记为静态的，为此需要对内部类进行重构，使其成为一个命名的内部类。

真的？这不是很挑剔吗？我真的应该在匿名内部类中重构一行方法以节省额外引用的成本吗？在这种情况下，它不可能将引用保存得超过必要的时间。

我不介意这样做，因为我们严格执行的编码标准是“零声纳违反”，但我强烈想在这里提出//NOSONAR的理由，因为imho将一行方法提取到静态内部会使代码更难理解。

java纯粹主义者是怎么想的？

java

findbugs

sonarqube

回答 4

Stack Overflow用户

回答已采纳

发布于 2014-02-24 11:22:57

将评论转化为回答，首先，我可以相信，把它作为匿名的内部类是有道理的，即使这有一个明确的技术理由来挑剔。

尽管如此，我还是要说：遵循您设置的规则。规则创建一致性，当所有代码都以相同的方式编写时，代码库作为一个整体更容易理解。如果某些规则是坏的，在任何地方都禁用它。

当有例外的时候，也需要解释为什么会有例外:对阅读代码的人来说是额外的精神负担，在代码评审中需要讨论的额外项目等等。只有在个别情况下，如果你认为这是一种例外情况，才需要禁用规则。

此外，我也不确定将它作为静态类来执行会更难理解，即使它添加了更多的样板(如果下面不是100%正确的代码，那么我的Java就有点生疏了，尽管可以建议编辑)：

Set<Map.Entry<String, SSGSession>> theSet 
    = new TreeSet<Map.Entry<String, SSGSession>>(new SSGSessionStartTimeComparator());

然后在文件中的其他地方，连同其他静态类：

static class SSGSessionStartTimeComparator extends Comparator<Map.Entry<String, SSGSession>>() {
    @Override
    public int compare(final Map.Entry<String, SSGSession> e1, final Map.Entry<String, SSGSession> e2) {
        return e2.getValue().getStartTime().compareTo(e1.getValue().getStartTime());
    }
}

票数 11

Stack Overflow用户

发布于 2014-02-25 01:54:42

为了完整起见，我想在已经提供的优秀答案的基础上添加另一个变体。为Comparator定义一个常量，并使用它：

private static final Comparator<Map.Entry<String, SSGSession>> BY_STARTTIME =
        new Comparator<Map.Entry<String, SSGSession>>() {
    @Override
    public int compare(final Map.Entry<String, SSGSession> e1,
            final Map.Entry<String, SSGSession> e2) {
        return e2.getValue().getStartTime().compareTo(e1.getValue().getStartTime());
    }
};

private void foo() {
    Set<Map.Entry<String, SSGSession>> theSet =
        new TreeSet<Map.Entry<String, SSGSession>>(BY_STARTTIME);
}