原创：小姐姐味道（微信公众号ID：xjjdog），欢迎分享，转载请保留出处。任何不保留此声明的转载都是抄袭。

如果不加上代码review环节，你真的不知道小伙伴们能在代码里干出什么事情来。不管是有意的还是无意的，缺少这个环节，你的项目都会陷入可能的风险之中。

说一件真事。

你能想象么，在给客户上线的jar包中，我们发现了让人啼笑皆非的小电影，这件事一直在公司的内部流传。虽然从来没有在经验分享会上分享过，但几乎每个人都知道这件事。

始作俑者早已经离开了公司，但他的传说一直在流传。

事情是这样的。这个项目属于早期团队的作品，技术经验都不是很足，他们使用的Java系的技术，用的版本管理器是svn。

某一天之后，大家发现上线的时间变的特别的长。

上线没有高大上的持续集成，大家需要使用远程管理软件，把打好的jar包传到客户的机器上，然后手工执行。这都是老掉牙的处理方式了。但后来，光拷贝文件，就需要接近十几分钟的时间。但很神奇的是，大家都能忍受，所以很长一段时间，上线人员就坐在电脑前看着进度条在那里缓慢的前进，顺便能喝一壶茶。

PS：不要diss这种上线方式，目前仍然有大部分做项目的前线队员们，在用原始古老的远程拷贝法。

直到有一天有个上线人员因为网速原因，传了几次都没传上去，实在受不了了，打算看看jar包里到底有什么。

他使用winrar解压了jar包，然后就发现了宝藏。里面有一个奇奇怪怪的文件夹，打开一看竟然是小电影。

即使是svn，也有提交记录的。有懂行的终于追溯到了原因，那是一次午夜的提交。

有一段时间，项目加班，经常到半夜。大多数成员在公司百无聊赖，于是有个小伙伴打算分享一下自己的资源，但是传到哪里好呢？想了半天，就传到svn服务器上吧。反正代码都能拉下来，电影怎么能拉不下来？一顿猛如虎的操作之后，svn留下了他的印记。

真是让人尴尬呀。领导默默的把svn仓库删掉重建，抹光了所有的提交记录。

开发人员的水平能差到什么程度，你是永远无法想象的。世界上并不全都是BAT，无数的项目在使用非常初级的开发人员堆人力，尤其是一些大量使用外包的项目，他们的唯一目标就是项目按时完工。所以甲方的系统到底是什么鬼样子，可想而知。

我还见过很多这样的案例。

• 系统的debug日志，4，5GB大小，也能够打到springboot的执行文件里；

• 文件上传的测试图片，数千个文件，也能够跟随着war包到服务器上风光一把；

• 直接把编译后的class文件拖到服务器上上线。直到最后源代码都搞丢了，只能通过反编译找出逻辑的蛛丝马迹；

• 直接把开发环境在服务器重建一套，服务跑在eclipse上；

• 一个文件几万行，到最后IDE拒绝编译；

这背后隐藏的现实，是无数面试造火箭,入职拧螺丝的抱怨，是无脑招聘人海战术的终极演进，是什么价位配什么系统的尴尬现状。

但现实情况大多数是，宁缺毋滥是 拼不过人海战术的，那些差的团队，因为人多，会获得额外的关注，Leader也会迅速成为管理型人才。什么代码review，出现问题只不过是概率问题，几年之后早就和现在的团队byebye了，谁又在乎长远的利益？

推荐阅读：

戳「阅读原文」，查看上期精彩文章