声纳抱怨“应该使用菱形运算符（“<>”）”答案

【问题标题】：Sonar complaining "The diamond operator ("<>") should be used"声纳抱怨“应该使用菱形运算符（“<>”）”
【发布时间】：2019-02-06 15:41:17
【问题描述】：

所以我的文件中有以下代码行

List<String> nameList = new ArrayList<String>();

现在每次我运行声纳扫描时，它都会在上面的行中显示一个错误，说我应该改用菱形运算符。现在我明白了，Java7 编译器会自动检测并提供 ArrayList 的对象类型，但我的问题是，如果我在声明类时自己这样做会有害处吗？

这里是rule link from Sonar website。我真的不明白他们提供的这个规则的例子。

是否有任何性能、效率或任何其他类型的增益？将代码更改为 Sonar 的建议？

【问题讨论】：

【解决方案1】：

更少的无用和重复的代码，更多的代码是可读和可维护的。

List<String> nameList = new ArrayList<String>();

或

List<String> nameList = new ArrayList<>();

好吧，差别不大。

但假设现在你必须将泛型：String 更改为 Integer，第一种方式你必须做两个修改：

List<Integer> nameList = new ArrayList<Integer>();
      ^------               -------------^

真的不好看。

钻石需要一个：

List<Integer> nameList = new ArrayList<>();
      ^---

但是再举一个例子：

Map<String, List<Integer>> map = new HashMap<String, List<Integer>>();

或

Map<String, List<Integer>> map = new HashMap<>();

它让事情变得更清晰。

在应用程序中，您通常会声明和实例化大量的集合和泛型类。重构它真的很便宜。所以就去做吧。

【讨论】：

【解决方案2】：

如果我自己在声明类时这样做有什么害处吗？

不，这样做没有任何害处，除了代码中增加了额外的冗长。从同一个rule link我们可以读到

Java 7 引入了菱形运算符 () 以减少泛型代码

是否有任何性能、效率或任何其他类型的增益？将代码更改为 Sonar 的建议？

除了您（和您的团队）的代码不那么冗长之外，不，没有。

进一步阅读：

【讨论】：

我遇到了一个模糊的情况，其中一个字段在方法中被初始化。因此，除非您去查看现场声明，否则钻石专业化应该是什么并不明显。有人可能会争辩说，初始化一个字段应该在声明附近完成，但这不是重点

【解决方案3】：

这条规则背后的重点是：在 java 7 中，编译器能够从类型本身推断构造函数的类型。

为什么要在构造函数中省略泛型类型？

两件事：

让它保持原样有害吗？并不真地。但在您的团队中拥有一致的代码风格总是更好。因此，如果您的团队使用声纳进行代码样式检查，您应该遵守它以保持一致性。

【讨论】：