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

[loliordie]

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

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

[dustinth]

.gitignore + git precommit hook(lint integration), 既然要求那么高,为什么不对自己要求高点, 把这套工具给整齐了?

[tenca]

代码洁癖算是个好事，除非你认为代码洁癖是个好事情。

[opengps]

刚开始是好事，后来逐步到了管理团队的时候就暴露弊端了

[afx]

是好事

[inhzus]

ci 上加上 pep8 的检查，过不了直接自动 close

[jay4497]

洁癖没什么好坏，别强加给别人就好。不过像工作环境这种有利益关系的场合，当然是谁权利大谁就为所欲为了。。。

[StarUDream]

我觉得这完全不是洁癖啊，这不是最基本要求吗？

[StarUDream]

提出来觉得不适合，直接写 ci lint，强制改。

[lamy]

git add 完还要再 diff 一次自己 review

[ibegyourpardon]

我觉得，你累……活该。

这和洁癖没关系。

道理千万条，但你选择靠人肉和规范来解决问题。

人是靠不住的，只有工具可以。

你应该用工具来解决。

所以目标上，我相信大多数人都认同你。

但这个思路上，完完全全你就错了。

或者说有规范也可以，但必须有对应的工具配套。

也即，如果你不按规范来，你的代码提交不了，你的系统给你报错，总之你就是不行。而不需要人肉来打回，核实。

[nightwitch]

洁癖可以，但是你应该引入自动化工具来审核。

[yuanfnadi]

说说前端
1. 项目有 eslint ,代码不规范无法 commit
2. 项目有 CI，CI 不通过拒绝 CR
3. gitignore 解决临时文件问题




python 应该也有类似工具

[yousabuk]

好事，做软件就得有尽善尽美的意识。

胡日鬼倒棒槌可不行

[bintianbaihua]

算好事，可以在团队分享这样的好处，让大家知道一些规范， 推广肯定是需要时间的。

[Flourite]

不算，因为可以用工具解决你的问题，你的关注点应该在其他地方

[shunconf]

git add . 省事，或者 git add *.py 咯

[liunian1004]

看处于什么环境和价值观，如果大家都不在乎，你会觉得很累

[Leonard]

对自己算好事，要求别人就不好说

[cominghome]

我有洁癖，而且是比较严重的那种。IDE 上见不得一点提示，哪怕是拼写检查。而且热衷于前期 review，功能实现完只要没到 deadline，就开始瞎鸡儿优化代码结构（其实一开始就一两个类，整个鸡毛 mixin ），出活效率奇低。

[littleylv]

洁癖没问题，但“揪出来重点批评”，“逼”，“发火”……
发火，还是对一个实习生发火……好好讲不行吗？