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

[LeroyMooney]

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

[Labs]

都是打工人

[talen666]

你是没见过我们公司外包写的代码。。想哭

[JerryCha]

除了看起来比较好笑，并没有什么问题。

[jdandelion573]

这算啥 还见过 if(!(!a&&!b)) 的自嗨代码 反正没有 review 当然放飞自我

[proletarius]

对自己又有信心了

[snowlyg]

代码没有错误就行，纠结那个多没啥用

[auh]

实际上，不光外包的存在这样的问题。某些本公司的也是垃圾。
所以说，描述的时候，别带外包。直接说同事垃圾就行了。
我都见垃圾见多了。你还是太年轻了。你见过哪个高手死扣别人不放的。
要么是有强迫症，要么就是没有社会经验，不懂变通。

[auh]

嘲讽没有错，谁没有这样的垃圾同事。不过谁又总能遇到比自己还优秀，代码写的还钓的人。
心态放平。该喷还喷，千万别认真。仅此而已。

[coolesting]

抄底的人笑没有抄底的人，谁知道接下来是跌还是涨！？

[nowto]

可能是他想强调这个方法有可能返回 null

[a7dog]

这是什么语言

[Fengyesuixiang]

外包怎么了，养活多少人你知道吗

[Hapon]

代码这么写，确实不妥，就像这帖子一样

[SmartKeyerror]

“每当你觉得想要批评什么人的时候，你切要记着，这个世界上的人并非都具备你禀有的条件。”
大家都是同事，如果你觉得他写的不好，你可以选择帮他改。

[hankai17]

感觉是改出来的结果

[James369]

不用纠结这些细节，查查代码的历史记录。这段代码应该是有故事的。

[54skyer]

写差代码跟是不是外包没有直接关系，可能几率大点而已。

我以前在腾讯互娱心悦做外包，有一次活动对接 QQ 音乐，别人提供的 js 库对接出了问题，我排查发现是个 bug，联系对接方的正职员工并给出了解决方案，别人最后校验也认可了。但是，让我直接 copy 一份改好之后上传到到互娱心悦的 CDN，相当于一个官方库，让我修复 bug 本地使用，官方库 bug 保留。

另外一个，有个正职大佬带我外包做一个内部项目，让 TX 各个手游接入到自己的游戏内活动页，领取哪些每日每周每月礼包之类的，传个 QQ 或者微信 openid 之类的参数即可，接入网关时候，有个加密校验，游戏客户端因为加密不对导致网关半天接不进去，各个语言的加密代码文档都备好了，包括天美这样的工作室下的游戏都有开发人员半天接不进来需要我协助的。

做到最后，我把自己做得像个客服一样，天天给大佬们沟通问题。

这些问题难吗，我觉得不难，我就一培训出来当时几千块月薪的外包。但这些正职大佬就是有一部分混子，或者不认真的。然后我干了快两年就升了一次外包职级涨了一千块，这个外包升职听说外包一年评估一次，每次通过率在整个外包人员是 10%-15%，涨薪就千把来块，由正职导师推荐。镀金可以，薪资潜力不高，后来我就离职了。

外包也别把自己看轻，别人怎么看不重要，好好提升自己的技术，技术强了，你是外包又怎么样呢，如果比正职还厉害，可能羞愧的是别人。

有想做外包的同事去大厂长下见识还是可以的，可以待个一两年，转正基本不要想，想转职不如好好提升技能到能说服导师的程度，然后让其帮你走社招进入；工作起始比较清闲和稳定，但工作内容可能有点枯燥。空闲的时候抓紧时间学习。正职员工大多都是跟我们一样的普通人，没那么多优越感，抓紧机会多多讨教学习。

提出指出问题容易，解决问题指导别人更难能可贵。

站在顶峰的人就那么一些，我们怎么看下面，上面就可能怎么看我们。

歧视必然存在，尽量不内卷吧；

[hantsy]

可能写的时候手抖了吧。

判断为 NULL， 返回一个 Collections.emptyList()，这是比较常见的操作。

在 Caller 调用保证所有的 Collection 不为 NULL 。


@54skyer 有些人就喜欢标题来吸引人眼球。

我自己 Freelancer 这么多年，也是接外包项目。没有独立做过外包项目的人，我觉得知识面和动手能力不是一般的差（我帮朋友公司面试过很多人，之前已经举过很多例子）。

[hantsy]

@LeroyMooney

一个星期如果写的 600 行高质量的代码已经不错了，比如包含在 CI 上跑过完整的测试，达到要求的 Coverage， 并且代码质量报告评估通过（没有引入新的 bad smell, 没有 duplicate codes, 通过 Sonar，Coday，Code Climate 等工具检测），最终合并到 Master 。

很多时候我一个星期都是写不了 600 行代码的。

如果你写的代码仅仅在你机器能够跑起来，这个标准写出来再多的代码不就是屎山，谁见谁怕。

[hantsy]

@PopRain 相当高效的啦。