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

[LeroyMooney]

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

[Leonard]

哈哈哈哈哈哈

[quan01994]

别问，问就是按代码量计费的。

[legiorange]

？

[mazai]

脱裤子放屁，多此一举😂

[Guidoo]

哈哈 这代码。。 强行封装

[totoro52]

笑死我了 这就是脱裤子放屁啊

[bzw875]

可能是不知不觉摸了 1 天鱼，随便写几行代码提交吧

[liuxey]

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

这样还能说得过去一点

[zhuangzhuang1988]

正常啊，这代码

[wxsm]

谁都有脑抽的时候，如果只是一行还好

[huage2580]

问一下他，怎么写出这么秀的代码的

[murmur]

getter/setter 而已。函数的封装并没有问题

[CalistaK]

这代码不报错呀，逻辑上好像也没问题（手动狗头

[shiji]

有的时候也许 null 的返回值还没有沟通好，先这么放着，等定下来了直接改。

[jwenjian]

反正也会被 JVM 优化掉，不影响😁

[AA5DE3F034ACCB9E]

学到了

[zhuweiyou]

有可能之前还有别的条件和代码, 然后删掉了, 只剩下这一行.

[liudaolunhuibl]

@liuxey 1 、新建 ArrayList 的时候最好指定容量，这里就表示一个空的所以 new Arraylsit （ 1 ）就可以了，2 、new 一个 arrayList 的时候最好静态方法，用谷歌的 guava：Lists.newArrayListwithCatity(1)

[wangwei1025]

我还写过
if (condition){
A=B;
}else{
A=B
}

哈哈哈哈，有时候确实会脑子抽了

[zhuweiyou]

比如 return issueStatistics == null || issueStatistics.length == 0 ? null : issueStatistics;
后来不需要 issueStatistics.length == 0 了, 只删掉了一部分. 忘记改成返回了