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

[LeroyMooney]

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

[tuomasi]

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

[Arthit]

issueStatistics = issueStatistics ?? new List<IssueStatistics>();

[wiluxy]

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

[ho121]

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

[mingtdlb]

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

[sinopf]

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

[BigBunny]

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

[chanchan]

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

[shyling]

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

[jzmws]

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

[jzmws]

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

[jzmws]

外包不如🐕 能不去做外包就不要去, 夹在中间两边都歧视 . 出了问题都是外包的原因, 用最少的代价干着最多的活 .

[jsjgjbzhang]

楼主你是他领导么？我感觉也没多大问题啊，当面沟通下，拿论坛来贴大字报反倒显得小气了点

[ldlood]

有什么当面指出，来论坛背后说别人，我觉得你也不咋样

[sakura1]

这有点搞啊，还见过这个
if(...){
return true
}else{
return false
}

[7gugu]

凑活着用吧，看到就修复一下，反正也不是什么大不了的事，总会写着写着就短路的嘛🤣

[Smash]

@shiji #14 不要在编码阶段考虑太多无意义的封装，百分之 99 的概率，这行代码都不会动了，最后成为屎山。

[oldmanong]

培训班老师：记住啊，你们写代码一定要多检查 null，这样可以让人家觉得你很有经验。楼主同事：记住了

[isnullstring]

就这？

[easylee]

非要加外包两个字，何必呢？

搞歧视么。