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

2020-06-30 22:58:46 +08:00
 loliordie

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

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

12486 次点击
所在节点    Python
142 条回复
yolee599
2020-07-01 14:14:59 +08:00
代码洁癖是好事
yuankui
2020-07-01 14:31:28 +08:00
怎么都好,发飙不好
thulof
2020-07-01 14:47:39 +08:00
实习生提 MR 之前自己不看 diff 吗……
WytheHuang
2020-07-01 14:55:25 +08:00
@ericls #15 中肯
charlie21
2020-07-01 15:00:51 +08:00
扣钱阿?
yaphets666
2020-07-01 15:03:47 +08:00
我工作两年多了,除了技术我觉得我最有收获的就是深刻的认识到了工作绝对不带情绪.
notejava
2020-07-01 15:30:49 +08:00
对自己的代码洁癖,别人的代码随意。
HenryWang0723
2020-07-01 15:35:21 +08:00
如果你是实权领导,可以理解,不合适就辞退。如果不是,你的行为接近于癖了,显然这个词不是个好的意思。除了制造矛盾让别人不爽外,你自己不是也烦躁了吗。把情绪的精力放在从细节实际上培训和统一代码规范上,有意义得多
fenyu
2020-07-01 17:36:02 +08:00
我吐槽一下,如果说洁癖我也有也很支持这么做
但是为什么不把那些加.gitignore,
如果你是实权或者负责 review 代码,
另外新人嘛,警告几次,拒绝几次就好
miniwade514
2020-07-01 18:07:27 +08:00
代码规范是好事。但是沟通带情绪就不太好了,尤其是对下属。想想,如果是你的 leader 提交了不符合规范的代码,你会对他发火吗?离开这份工作,大家都是一样的普通人,对人 nice 一点,以后也许还能一起撸个串。
salmon5
2020-07-01 18:16:31 +08:00
@miniwade514 如果“不符合规范的代码”,经常引起故障或者以后可能会引起故障,让你背锅呢?还能心平气和吗
aguesuka
2020-07-01 18:48:27 +08:00
洁癖没问题,发火有。但是最大的问题是你没有意识到这一点
blackshow
2020-07-01 18:55:40 +08:00
cco
2020-07-01 18:57:23 +08:00
唉,一样的事,只不过不配合而已。
d0wnl0ad
2020-07-01 19:09:12 +08:00
我觉得,这个更多是 SOP 的定义及流程落地的问题。

规范有白纸黑字写下来吗?有确保这个规范在入职培训里面吗?有确认培训的有效,到位吗?有设立奖励机制来确保大家有动机去做这个?系统有足够的防呆来抑制出错嘛?有强制执行规范的决心嘛?

说真的,单靠宣导不一定能够达到目的,特别是无明显好处+增加工作量的事情。

必须有更多的制度去诱导这个行为。
zypy333
2020-07-01 19:34:38 +08:00
我也有代码洁癖

同事们数据库建表,表名,大小写混着写,还有拼音首字母缩写当表名,横杠还有两个连到一起的

前台控制台里报不影响功能的错也不管,类名不写注释干啥用的

有同事还有写那种写死数据的假代码(原型产品,离上线测试还很远)...

太多了,我提醒了也不管用,不鸟我,私下估计不恨我都不错了

关键是这团队技术负责人,不做代码审查,之前定的规范也不管大家执行不,每个周末只看功能,界面看起来可以就能混过去
amundsen
2020-07-01 19:38:22 +08:00
@yuanfnadi 我们还加上了 ts 和 code review,还有定期里程碑复查
chenyi
2020-07-01 20:00:59 +08:00
洁不洁癖不重要,反正我觉得代码风格统一挺重要的,而统一代码风格最好的方式就是用工具
gunavy
2020-07-01 20:07:42 +08:00
嗯……
如果你是大哥,是个好事。
如果你不是大哥,降低了你的不可替代性?,(逃
octalempyrean
2020-07-01 21:43:57 +08:00
视情况而定吧

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/686069

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX