很多人做事写代码都在对付，写好规范也没人遵守，只要能实现需求啥办法都用，重复代码一坨坨，看得人眼花缭乱。非常恶心。

[darasion]

[cyokvip]

你们代码没有审核吗,我们公司代码写完都有审核,写的不规范就返工重写!

[bullda]

定期做team code review

[bigporker]

定期review

[ultragtx]

用gerrit吧

[ofshellohicy]

写程序多，会写程序的少。

[clino]

gerrit +1,不靠谱的人只给 read+upload,靠谱的给 review+submit 权限

这样不靠谱的提交全部可以打回重新提交

[tremblingblue]

审核就是个应付，对于敝司这种项目型公司，只求短快平，是个人就招，只关注业务，不注意留人，不注意技术积累。
之前项目重复的一坨坨好不容易被我干掉了好多之后，业务代码好歹叫有半个像是代码像是html的样子，突然来个新项目组，然后又继续一坨坨，跟领导说也没用，想想也是，都2012了还在使用这种早已经被淘汰的框架来做web，还美起名什么3.0、4.0，恶心的继续。

[cyokvip]

@tremblingblue 给别人做项目的公司都这样

[chisj]

review +1

[darasion]

就是如 @tremblingblue 说的这样。

@cyokvip 我那个是给自己公司内部做的。

[welsonla]

入职就送他一本《重构》，读一星期

[aleung]

这种情况最终还是看人，包括干活的人的素质，和老板的关注点。
如果上下都有提高代码质量的共识，可以用Sonar持续检测代码质量，将质量状态可视化，然后订立质量标准。一开始起码要求代码质量不能继续下滑，然后制定目标逐步改善。
虽然这种规则检测死板了一点，但是效率比人工review高很多，而且容易定制客观标准，而不是因人而异。在团队不成熟的时候，反而比较有效。

[mitnk]

+1 @welsonla for 入职就送他一本《重构》，读一星期

[mille]

么有流程就么有生产力。

[webgeekman]

看公司的管理体制是什么样的？
制度严格，很古板的团队风格，可以强化代码review，有奖罚机制。
如果是较为人性化，很愉快的团队风格，可以靠内部交流讨论建立良好编码的风气。
总之风气很重要！更重要的是在管理上不要给混乱的代码找到借口！！！比如，不论项目时间是否紧张，都不能成为代码混乱的借口。

[darasion]

没办法阻止别人。

送书的都不现实，写代码不认真的人，根本就不会认真看书；
咱不是管理者管不着别人，是管理者也不会关心代码质量；
常常替人擦烂代码的屁股，平时工作受到赞赏的一般都是那些干活快速而无质量的人，最后出的各种稳定性问题总是留给那些认真做事的人。

也许。。。。一开始就应该做个烂人懒人奸人坏人。

[zythum]

其实感觉个公司有关。
如果工作量一堆一堆压，是个人都对付了。如果轻松惬意才会去关系代码质量

[yuelang85]

看老板喽，老板不行开掉他。

我们也是这样，天天说专注，说质量，其实是东一榔头西一棒子，一个项目不管需求多天马行空，一概两个月。。。。

也不是没有一起讨论过，也不是没有定过规范，规矩，也不是没有下过决心，技术负责人换了一个又一个，哪个来了还不是这些方式方法。你规范定了，规矩立了，第二天，老板起个项目，一个月提交测试审核，两个月上线，规范？规矩？全没啦，上线要紧啦，于是又失败了。。。。

这种公司，别要质量啦，也别要什么技术积累技术发展啦，各自忙乎各自的，尽可能让自己成长，自己舒服就得啦。

[PrideChung]

@darasion
为什么我之前一份工作跟你说的情况这么像……也是给客户做网站的。
经理啥都不管，就看你做得快不快，一来就跟我说XX一天做4个网站，我操，除非copy & paste，否则这可能么。后来XX跑路了，发现他原来干活只干一半的，有些页面直接拿前端做的静态页面来骗客户，反正客户也不懂。结果呢，就找我给他改bug。我不知道XX之前在公司呆了多久，但我直到我离职前一周起码要给他擦一次屁股。
骂了隔壁，耍滑头的拿了钱就跑，老实人还得给他打扫残局。

[dreampuf]

换个环境吧。