如何应对一个项目中其他组员写的烂代码？

[enlight]

实在看不下去了，思维毫无逻辑想到哪写到哪，代码不格式化，变量命名大小写毫无规则，model controller 混着写，各种不规范的方式导致整个代码质量降低很多经常出一些奇怪的 bug，但是碍于面子没有出 bug 的地方又不好意思改动别人写的代码。 大家有什么方法应对这种问题吗？

[uxstone]

跳槽离职

[whileFalse]

一般而言我推荐团队新人的代码风格应和现有代码相符。

你也这么写吖🐶

[kiritoxf]

正常应该有人 review 的

[xianxiaobo]

直接告诉他，让他注意点就行了

[FindByMe]

制定代码风格规范、配置静态检查工具、有不符合规范的代码、直接抛 error

[danyi]

没遇到过这样的组员，不过我同事遇到过，看样子他们挺想打我的。

[coderluan]

你是负责人就制定代码规范和进行 code review，你不是负责人就建议负责人这么做。负责人明确拒绝建议离职，不懂代码质量重要性的人基本也没啥技术水平。负责人认可但是因实际情况难以推行，比如工期太紧张，这种就没啥好办法了，只能尽量明确职责，分配好模块，让自己尽量不去看别人代码，不负责别人的 bug。

[minglanyu]

1.以身作则
2.成为他们的 leader
3.code review

[tt67wq]

code review 不给通过

[itskingname]

依靠文档不如依靠代码。用文档或者 txt 规范来约束人是没有效果的。

举个例子，例如 Python 项目，你多次强调写代码要遵守 PEP8 规范，可就是有人不遵守。怎么办？在 git 上面加个流程，必需 Pylint 扫描无报错，无警报了才能 merge 代码。这样就能迫使对方必需遵守。

[sagaxu]

跳槽到好 team，让你自己成为被嫌弃的那个

[carpediem777]

其他组员也许也有跟你一样的想法

[hoyixi]

团队没有代码规范吗？
试用期是干嘛用的？

管理人员是干嘛吃的，正事不管，在管啥？ 哈哈

[meteor957]

把你的烂代码给他写，然后你就心理平衡了

[nmap]

严格模块级隔离，出了问题可追溯，就跟你没啥关系

[Smash]

代码提交要先规范检查，检查不通过不会让合入的，规范后的代码，我都能忍。不然就六字真言。

[shm7]

向甲方提这些事情

[learnshare]

接手的项目，该扛起来就扛起来。代码规范还是要有的

我接手的项目没办法下手写代码，就完全换了 lint 规则，处理了几百个文件、几千处问题
不过逻辑、模块划分啥的还是没有办法

[gowa]

你当 reviewer，代码打回去。

[enlight]

@learnshare 说的很对，我这边代码已经完全不是 lint 能解决的了，逻辑思维感觉很混乱。我本来搭好了一个架子，model controller services 划分得非常清晰。结果对方直接把 model 写到 services 里去了，怎么省事怎么来。而甲方只看能不能跑通和开发速度不管代码质量，本来还是因为我一个人开发速度不够才又找了个人，所以我也很为难