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

[loliordie]

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

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

[loliordie]

@dearmymy 一般项目我会给团队争取 10%-20%的缓冲时间 这个是硬条件我从来没妥协 大 BOSS 也知道我的团队不怎么出错 容许这个行为 另外我也尽量减少开会这种浪费时间的行为 所以一般我们都会有比较充裕的时间来改改代码整理一下文档

[zjsxwc]

看公司，以前上上家公司，写烂代码（随意命名）会被开会批斗，因为 40 多人维护一套代码，还有开源版本的二开，之后几家内部系统就几个人自己万就比较随意了，逃

[zhuangzhuang1988]

不是.

[cweijan]

git add * 这语句没有任何问题, git 项目应当在 gitignore 过滤临时文件,
至于代码洁癖, 当然是好事, 但你需要为你的规范定一个标准明确告知.

[Acoolda]

你自己严格要求自己没啥问题，但是同事之间，应该设定一个最低标准(不影响项目进度)。每个人都有自己的习惯，没必要干涉。

[sockpuppet9527]

看到楼上已经有老哥提及了 hook + ci 了
我个人还是非常方案这种又拿洁癖说事，又不加上这些工具的人的。

[sockpuppet9527]

@sockpuppet9527 #66 方案 -> 反感

[cnbattle]

普及一下 git 相关知识点，写入规范文档不就好了

为什么要用最坏的方式处理，发现问题要先想怎么处理及避免，而不是生气批评，弄得大家都不舒服 0.0

[yuchenyang1994]

1. 无论什么东西，一旦过于偏执就不是好事
2. 干的就是脏活，代码不可能干净
3. 适当留一点脏代码，给实习生练手，不是核心业务不挣钱的业务给他们做，让他们慢慢理解脏代码为什么脏，但你发火，一解决不了问题，二，大家都不舒服。当然看到不爽在这吐槽没啥问题

[liuqiang1357]

git add .不香吗，整个图像化工具，提交前还能 review 一下，不能放到 ignore 里面，具体啥原因能说一说吗

[Tenlp]

有必要，代码整洁性应该是从入门编程就该一直注意的一点，不过很多大学或者培训机构反倒忽视了这一点

[qiumaoyuan]

哈哈哈加油

[hallDrawnel]

我的建议是不要手动去做这些工作，通过 CI 、IDE 插件、统一配置、文档来控制，合并代码不过 CI 不合并，这样避免你直接和合并代码的人冲突。从技术上杜绝不符合你的要求的代码合并到主干，规则都写清楚，这样谁不遵守那就是谁的锅了， 不让他合并理由也十分充分。
发现漏网之鱼补上规则检查就好。这些例子就可以整理一个文档贴出来，周围人犯的错有很好示范作用。
都是来搬砖的，好好说话别发火。人家实习生跑了，回学校一说，可能这个学校的人都主动避开你这个部门了。

[qiumaoyuan]

代码统一风格仅仅是树立权威而已，没别的用处。

代码的可读性来自于命名、DRY，没了。当然，多数人不这么认为。

[ai277014717]

站在填坑人的角度考虑应该是好事

[ruyu]

我也有点代码洁癖. 但是我慢慢发现, 编程是一件理性的事, 但是 "洁癖" 太感性了.

[1109599636]

我觉得 任何强制性要求应该有基础设置作保证，
比如代码格式 可以用 pre-commit hook 在 git 提交代码的时候验证格式，不符合不能 commit，自然也不能 push，有的语言还可以自动格式化。 这样比人为操作不仅方便还更高效。

[1109599636]

https://github.com/pre-commit/pre-commit-hooks

楼主可以考虑下

[Mark24]

你应该用 lint 、commit hook 、gitignore 去约束。

而不是要教导每一个人。那样没意义呀。

[love]

是好事，除非水平跟不上眼界的时候还硬要跟，眼界都是大于实际水平，做好平衡就可。