本文转载自微信公众号「小姐姐味道」，作者小姐姐养的狗02号。转载本文请联系小姐姐味道公众号。
如果不加上代码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了，谁又在乎长远的利益?
作者简介：小姐姐味道 (xjjdog)，一个不允许程序员走弯路的公众号。聚焦基础架构和Linux。十年架构，日百亿流量，与你探讨高并发世界，给你不一样的味道。我的个人微信xjjdog0，欢迎添加好友，进一步交流。
