有代码洁癖算不算是个好事

[loliordie]

我就有比较强的洁癖 要求实习生写 python 不符合 pep8 直接 close PR. 要求团队内部统一代码风格, 用拼音或者风格不一致的我会揪出来重点批评, 复杂一点的函数逼每个人都写文档 并且随改随更新.

今天上午又因为一个实习生使用git add *, 结果加进来一大堆临时文件发火了...感觉其他人都不理解自己好累...

[RicardoY]

应该写 gitignore 解决临时文件的问题吧...这应该不能怪到实习生头上...

[Jooooooooo]

算好事, 风格统一易于维护, 而且你有实权要求是最好

不过能用系统去解决的问题不要用流程解决

比如不符合你说的那个规范的代码用插件在开发时就提醒, 不要到了提 pr 阶段再打回去, 多浪费时间呀

还有 git, 这个用 gitignore 去解决呀, 手动 git add 具体文件很傻

[jinsongzhao]

你已经反问自己了，还问啥呢？物极必反，平时可以多多要求，但赶时间时，就别干扰推进速度，反正吃了苦头，就会改进，有难同当，拿得起放得下。

[namelosw]

挺好的，但是这个东西首先得给他们足够的资源，时间，如果因为这些做不完加班加到组员都上肯定都是不想做的。

其次就是文化上的，光跟别人吼没用，得培养出来这种文化，把超过半数的人都培养成代码洁癖，以后就容易保持了。只有不到半数的人坚持，就是很吃力。有点像文明系列的宗教，有传染性，主流宗教先手就比较好过。

[loliordie]

@RicardoY 我写过了 但是因为项目文件很杂没法用白名单 每个人开发环境又不同 总有漏网之鱼 我说了很多次了又不是 ios 这种自动生成一大堆文件的项目 就一个后端系统 改的不多 改了啥 add 啥就行了.

[namelosw]

如果有一些实权，可以先把规则都定下来，试着让组员轮流值日，比如每周轮换，到了这周这个人负责这些风格问题，你尽量少插手。

总有别人帮他做决定的时候，他就不会去想这些问题，就会觉得这些都是规定，你们就成了阶级敌人了。

反过来，一个人自己完全负责某些事情的时候，他的思想就容易从这个事情上开始出发，这就是所谓的屁股决定脑袋。

[seliote]

深有体会。但是规则是用来约束自己的。

[newtype0092]

我也有点，每次看新人提的代码都有点头疼，有些不合适的零零碎碎的语法、风格、不规范命名都很难受，我一般开始会严一点，把问题集中说几次，之后如果问题多了就让他们改去，偶尔漏了的就忍了，提醒下让下次注意。

就我当初实习的时候被 review 的死去活来的经验来说，越严格的 review 并一直坚持，再坏的习惯都能改过来，如果还改不过来那就是真的不上心了，这种人趁早踢掉，因为这种心态什么事都做不好。

不过说实话这种东西到底自动化才是正道，能用插件用插件，特殊的自己写 hook 检查，这样起码不会再因为这些杂事浪费自己的时间了，只要自动化检查做的好“代码洁癖”和“不拘小节”的人也是能和睦相处的。

[linvon]

git add * 也太有问题了

[lenqu]

@linvon git add * 没有问题，是不会 gitignore 的问题，或者是太懒，以及不使用 ide 自带的 git workflow

[RicardoY]

@loliordie 你可以维护几套常用的开发环境的 gitignore，比如 pycharm&vscode 的..然后不允许实习生用别的开发环境吧...pep8 的问题应该用钩子解决...我感觉你说的这些问题靠 review 解决起来会非常痛苦，应该引入自动化的强制约束...

[msg7086]

@Jooooooooo 手动很傻的可以用工具解决，比如 GUI 。

@loliordie 为什么要关 pr 而不是要求继续改进直到通过？

[ericls]

作为一个 lead，你应该清楚工具比人靠谱。

PR 本来的作用就是给你 review 的，你就好好 review 就行了，有什么 feedback 发表就行了，别带那么多情绪。

[D0n9]

@ericls 在理

[ericls]

如果说你工作目标的一部分就是保证代码风格，那你就应该做工作去保证代码风格，不管你是用自动化工具也好，集体开会说明也好，一对一开会也好，或者建立什么别的机制都行。但是你内心上不能把这件事情当成别人理所应当承担的责任，这是你的工作。

[ericgui]

你要是组长，你想怎么玩就可以

但你也要负责这个后果：组员恨你，离职，进度跟不上，自己压力大等等

[supermoonie]

约束大于规范，别人说我就听，虽然和我的规范不太一样，慢慢也就适应了

[zhengjing]

@loliordie 你这种对没有心的人没用，反而会把自己搞的很累。同意楼上的，自己把系统搞好，gitignore 应该由你整理好，把所有情况都考虑进来，如果自己偷懒，那怪不得别人哦~

[zhengjing]

@ericls 完全同意

[zwater]

不是好事，除了自己暗爽以外。
我有类似的公文格式洁癖，正文仿宋 GB2312，一级标题黑体二级标题楷体加粗三级标题仿宋 GB2312 加粗行间距 30……
每次写稿都特别慢。