public List ge抗投诉服务器tIssueStatistics() {
    return issueStatistics == null ? null : issueStatistics;
}

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

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

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

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

原地打转！

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

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

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

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

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

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

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

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

我见过比这个还复杂的

让我想起了这个段子 

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

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

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

你们工资一样吗？

这个命名有问题，歧义

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

人家写的不好可以提醒一下，还跑论坛上 diss，外包何必为难外包，没准你在甲方眼里：看，狗咬狗

issueStatistics = issueStatistics ?? new List();

文人相轻太正常了，发到论坛里，让大伙批斗一番。
我觉得楼主要是能和这个外包同事指出问题所在，那就更好了

这种有时候会这么写，比如当 IsXXX()可以返回 null 的时候

这也要发上来？显得你牛逼吗？没什么深仇大恨，背后说人家不好....

外包项目嘛，后期不出点儿 bug，怎么心安理得的收服务费？

估计是本来有封装逻辑 后面因为什么原因去掉了

你这几张图太不痛不痒了,说实话

用嘴不能解决吗，还要发论坛批斗一下？

这种代码会在测试时候用 写一 不想动下面的代码 ,强制在上面执行 哈哈哈哈哈哈

用 null 或者 Collections.emptyList() 都是可以的 , 个人更倾向用 null , Collections.emptyList() 他说静态变量不可变的, 要不然就要先判断 null 在判断 isEmpty , 大部分人空都习惯放回 null 二不会放回 一个空对象

外包不如