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

[LeroyMooney]

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

[vicalloy]

@fenghengzhi 我开始也以为是深拷贝，确认后发现不是。结合其他代码，应当只是单纯的认为 Python 里的“Json 对象”和 dict/list 不是同一种东西。

[Colis]

可能之前想返回一个空的 List，后来需求变更为返回一个 null 。保留这么写方便以后再改回去。

[KarmaWu]

Optional.ofNullable(issueStatistics).orElse(Collections.EMPTY_LIST);

[godloveplay]

@thtznet #69 单纯对这句话，你手里拿个锤子，看什么都是钉子。。

[QingStone]

原地打转！

[lakehylia]

反正编译器会优化的，编译器又不傻

[JasonTsang]

其实还好 ，可能暂时 状态 ，一般这种会多处调用 ，封闭成方法，如果这里被调用 1000 次，下次这里逻辑有变。只要改这里就可以了。当然 现在的代码是没意义的 直接 return issueStatistics 就可以了。但习惯是很好的。

[digimoon]

可能原来是 A，后来改成 B，突然又要改成 C，想着以后估计还得弄成 D

[ssssstest]

虽然但是，没必要强调外包。非得加重歧视吗

[belin520]

内卷，还要强调外包，这也不是不能用

[PopRain]

一周 600 行代码，搁到国外算高效了。。。。5 个错误，你选了一个最没有必要、不是错误的错误吐槽。。。。

[smilenceX]

谁还没写过几行烂代码了，这点小事，没必要专门发个帖。

[shifttacn]

抄的代码
第二个 null 是后改的


我见过比这个还复杂的

[dingdangnao]

让我想起了这个段子 

[msaionyc]

也许有一天会有人贴出你的代码发帖：
新来的专科 /本科同事，这代码，我？？?
新来的非 985 同事，这代码，我？？?
新来的非博士同事，这代码，我？？?
新来的大龄同事，这代码，我？？?

[Jinnn]

新来的外包同事: 你歧视外包?
新来的同事: 你歧视新来的?
同事: 你歧视同事?

[zeropercenthappy]

@dingdangnao 哈哈哈哈哈笑得我忘了代码写到哪了

[yousabuk]

你们工资一样吗？

[comoyi]

这个命名有问题，歧义

[EscYezi]

这种写法 ide 会有黄色提示的，如果是之前有别的逻辑删了一些代码，不把告警消除了也说不过去啊