如果一个程序员提交的代码连格式化对齐都不做,敲代码是怀着怎样的心情....

2019-06-11 15:22:09 +08:00
 abbenyyy

没错,我说的就是我的同事,我也不要求你有多牛逼,提交的代码不能有任何警告,我只是要求提交代码前按一下快捷键格式化对齐代码,每次都说 okok,然后依然我行我素。

重点是有时候这个同事可能没空,我要修改他的代码,一堆没对齐代码附加一大堆警告,作为一个强迫症看得想死。

提交的代码连格式化对齐都不做,到底是什么让他走上码农的道路...

6178 次点击
所在节点    程序员
58 条回复
Meltdown
2019-06-11 23:07:10 +08:00
我遇到的情况是公司没有统一规定代码风格,然后有些人可能用了 IDE 的自动格式化,有些文件明明明大家都没做什么修改,这些 IDE 造成的格式修改却老是被提交来提交去,随便一个提交就搞成了几千行
ginjedoad
2019-06-11 23:31:32 +08:00
golang 从来不需要对齐,直接保存对齐了....
SingeeKing
2019-06-11 23:34:08 +08:00
配个 git pre-commit hook
chen2019
2019-06-11 23:35:10 +08:00
ide 直接格式化就行了 管他对不对齐
zhuzhibin
2019-06-11 23:59:42 +08:00
代码规范意识很重要 团队里可以定义一套代码规范 大家都要遵循这么一套规范 就没那么多鸟事了
lincanbin
2019-06-12 00:02:22 +08:00
你需要做的是加 ci,ci 里跑 lint 做格式检查,格式出问题系统自动发邮件给他然后通报全组。
hastyfish
2019-06-12 00:06:49 +08:00
vim gg=G
dangyuluo
2019-06-12 00:30:23 +08:00
CI 是干什么的?你管他对齐对不齐?
elfive
2019-06-12 00:49:35 +08:00
@galikeoy 我初中就开始参加计算机奥林匹克竞赛了……
FrankHB
2019-06-12 00:56:20 +08:00
KPI 掌嘴。
FrankHB
2019-06-12 00:57:17 +08:00
@chen2019 污染 commit history 可能引起更严重的强迫症。
(我最烦的是 autocrlf 之类一时半会儿还未必马上能发现的……)
nanxiaobei
2019-06-12 01:00:20 +08:00
上 Prettier 完事
russian
2019-06-12 01:04:05 +08:00
@linvaux 哈哈哈。我以前维护一个 nodejs 工程,接受一个人的活,然后这个人根本不知道什么叫 async,后调函数一直硬刚,最多好像嵌套了 7,8 个括号
galikeoy
2019-06-12 01:04:20 +08:00
@elfive #49 好吧,见识太少,小时候条件 环境太差,并没有经历这些
MonoLogueChi
2019-06-12 01:13:58 +08:00
@hydrionz 你去改他的代码,格式化,可能会造成冲突
magicdawn
2019-06-12 12:46:47 +08:00
嗯知道 node 的话用这个

npm i -g yo generator-magicdawn
yo magicdawn:add-config


会添加 husky prettier lint-staged
HangoX
2019-06-12 13:10:48 +08:00
我在 ci 上加了逻辑,格式不过不让合并,然后共享自动化格式的配置文件。问题解决了
buhi
2019-06-12 18:05:28 +08:00
1. 有这么多时间折腾代码哪里应该有一个空格, 哪里应该换行, 花同样的时间在其他地方不是更好吗?
2. 这个格式化本来就没有一个四海皆准的规范, 都是见仁见智, 举两个例子比如函数体的左大括号应不应该换行, 缩进应该用 tab 还是空格. 假如你定了一个标准, 然后强制同事去遵守, 然后因为跟同事的习惯不符影响了开发效率, 会不会有一种钦定的意思?

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

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

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

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

© 2021 V2EX