PMD规则访问器方法生成

Question

有人能详细解释一下这条规则吗。规则：AccessorMethodGeneration优先级：3避免自动生成的方法访问私有字段和内部/外部类的方法，并提供字段和方法的具体示例？

Answer 1

PMD文档提供了AccessorMethodGeneration规则的摘要，并添加了我的重点：

当从另一个类访问私有字段/方法时，Java编译器将生成具有包私有可见性的访问器方法。这增加了开销，并在Android上对dex方法计数。这种情况可以通过将字段/方法的可见性从私有更改为包私有来避免。

所以PMD建议您，如果您将这些成员和方法包设置为私有而不是私有，您将避免通过编译器生成的访问器方法访问它们的开销。（我不是Android开发者，所以我不能对“dex方法计数”问题发表评论。）

这是PMD针对私有成员提供的代码示例：

public class OuterClass {
    private int counter;
    /* package */ int id;

    public class InnerClass {
        InnerClass() {
            OuterClass.this.counter++; // wrong accessor method will be generated
        }

        public int getOuterClassId() {
            return OuterClass.this.id; // id is package-private, no accessor method needed
        }
    }
}

下面是一个SO示例，编译器将自动生成代码来访问私有方法。注意海报上的评论（加上我的重点）：

编译器获取内部类并将其转换为顶级类。由于私有方法只对内部类可用，编译器必须添加新的“合成”方法，这些方法具有包级访问权限，以便顶级类能够访问它。

总之，PMD是一种标记代码，在某些情况下，您可以通过修改对私有成员和私有方法的访问来提高性能（我认为这通常是很小的）。

另一个值得注意的点是，AccessorMEDGeneration上有一些PMD错误报告无法正常工作（例如https://github.com/pmd/pmd/issues/274和https://github.com/pmd/pmd/issues/342）。因此，如果您不明白为什么PMD用AccessorMEDGeneration警告标记您的代码，请检查错误报告。