新来的外包同事，这代码，我？？?

[LeroyMooney]

public List<IssueStatistics> getIssueStatistics() {
    return issueStatistics == null ? null : issueStatistics;
}

[codiy]

牛逼了

[ily433664]

if (true) {
xxx
}

[anonydmer]

@liuxey @liudaolunhuibl 我们一直是要求 null 时候返回 Collections.emptyList(), 这个方法返回的是不可变对象，无法向里面写内容；不然返回一个 ArryList() 别人往里面写内容了可能会引起难以排查的 bug

[vicalloy]

和这个比起来 怎么样
#!/bin/python3
def toJSON(self):
　 xxx = []
　...
　 return json.loads(json.dumps(xxx))

[kop1989]

get/set 在默认情况下本来也只是返回 this.xxx ，逻辑上我个人认为并没有错误。
而且更可能的情况下是这块代码有过改动或者未来有预计的改动，所以和 ide 的默认写法不同。

这就像是：同样表达去哪玩，有的人说“你去 xxx 玩”，而有的人说“你来 xxx 玩”。这俩表达的意思一致，只不过是表达者主体的情景不同。

[drydiy]

本来外包就处于鄙视链的底端，你这还强调一下。欣慰得是，评论区都很很友善。
我在现同事的代码中也看到这种脱裤子放屁的赋值，我默默得修复了。
工作中难免会有脑子短路的时刻。

[viator42]

代码写的还挺讲究，虽然没啥用

[66beta]

留下可以扩展的口子，是个高手！

[Felldeadbird]

不明白这什么有的？ 写代码总有脑抽的时候。
至少得再列出别得。

[lagoon]

考虑到我自己也经常写垃圾代码，再考虑到这人的垃圾代码写的还挺工整。
我决定默不作声。

[sutra]

@liudaolunhuibl 不应该是 Collections.emptyList() 吗？哈哈

[yamasa]

如果想规避 npe，直接 Optional.ofNullable(issueStatistics).orElse(new ArrayList<>(1)); 就好了。

[CodeGroup]

这算什么，我这还有 list.size()>=0 的判断，我寻思数组的长度也没有负的吧

[arthas2234]

@shiji +1
有可能是还不确定要如何处理，先原路返回了，在这里先留个位置等后面在改
不过应该要打个 TODO，怕后面忘记了

[lonelymarried]

这是写 oc 写多了，怕 null 崩溃所以判断一下？

[LeroyMooney]

我随便再贴几行吧
```
private String nodeName;
private int issueTotal ;
private int countersign;

xxService.execl(projectNum);

xxService.selectid();

public String s(int a, int b) {
return (Math.round((((a - b) * 1.0f) / a) * 100) + "%");
}

public String D(int a, int b) {
return (Math.round(a * 1.0f / b * 100) + "%");
}
```

[LeroyMooney]

回复没用到 Markdown，大家将就看下吧

[LeroyMooney]

其他太长的就不贴出来了

[lwlizhe]

容老夫想想怎么圆这块……

或许这块是参考双重检测的单例模式呢，只不过少了关键字和双重检测而已

好像圆不下去~~~

[liudaolunhuibl]

@sutra 对 哦，忘记了