举几个例子我看见过的:
红帽是 redhad ,乌班图是 ubutu ,
加密是 encript ,解密是 discrypt ,
前台是 forground ,后台进程是 damon 。
最初是项目里编译出 lib/redhad/lib*.so
,这个 redhad 让我震惊了一整天。
刚接触代码时,我是想去改掉的。但是了解到这个目录 N 年前就是这样了,而且也不清楚到底有多少依赖,便作罢。
后来我也佛系了,毕竟不关我事,能跑就行。
直到我某天写代码,要用到一个大写的常量 某 DEFALUT ,怎么看怎么怪,定睛一瞧还是拼错的。一搜发现有三十多处。
如果是你,该怎么办?
...我最终还是选择了 2 ,因为前者会让 git blame 全变成了我,而我并不想背这个锅...
![]() |
1
lithiumii 2024-02-16 03:12:47 +08:00 via Android ![]() 没什么办法,HTTP 头里面的 referer 都错了这么多年了
|
![]() |
2
conge 2024-02-16 04:04:05 +08:00
哈哈哈
我也老出这样的错…… sorry |
![]() |
3
wildnode 2024-02-16 06:22:23 +08:00
在对应 IDE 市场里面找找 code spell check 相关的插件,组会的时候委婉推荐团队装一下呗,或者相关人员平时关系好的话直接提醒下让他改了...
|
![]() |
4
naminokoe 2024-02-16 06:33:29 +08:00 via iPhone
/usr 这个拼写不也是错误的吗?
|
![]() |
7
littlecreek 2024-02-16 08:25:20 +08:00 ![]() 如果是我, 这种事多了我会跑路, 因为这一般说明同事的技术水平包括态度都不怎么样
庆幸的是工作这么多年从来没接触过这么离谱的同事, 倒是在跟第三方对接的时候见识过其他公司的工程师有这样的情况, 并且验证了我上面看似武断不讲理的断言. |
![]() |
9
littlecreek 2024-02-16 08:31:32 +08:00 ![]() @littlecreek 为啥我这么说呢? 因为一个工程师偶尔拼错正常, 谁都有不小心的时候, 但是经常拼错, 就说明了一点问题. 这种代码偶尔 merge 进主干, 可能他/她包括 reviewer 都没注意, 也没啥. 但是经常出现低级拼写错误 merge to master 没人提出异议, 说明整个团队都多少有点问题.
|
![]() |
10
DTCPSS 2024-02-16 08:37:24 +08:00
装拼写检查插件,并在 CI 里警告错误拼写
|
![]() |
11
JavenXiao 2024-02-16 08:53:25 +08:00
cspell 直接在 mr 的 ci 强卡点
|
![]() |
12
Fish1024 2024-02-16 09:35:40 +08:00
说明工程师的学历都不高,建议跑路。
|
![]() |
14
pursuer 2024-02-16 09:57:31 +08:00
如果是模块内部不是导出的名字拼错,就改了,但如果是导出部分且被其他模块引用了,就只能忍了,可能会再加个别名。
|
![]() |
15
LavaC 2024-02-16 10:12:10 +08:00 ![]() 我寻思这些人的 ide 不可能没波浪线啊,为什么他们就总是无视它
|
![]() |
16
acerphoenix 2024-02-16 10:31:49 +08:00
是很坑, 搜索时都搜不到
|
![]() |
17
wolfie 2024-02-16 10:38:42 +08:00
是个 IDE 都有 spell check 啊,有时候忘了某个单词写的对不对 写个大概 靠 提示纠正。
|
18
lsry 2024-02-16 10:41:26 +08:00
這就算了,項目裏面用拼音,而且拼音還打錯的
|
19
unregister 2024-02-16 10:59:35 +08:00
我遇到这种错误都会骂以前的人写的代码,同时会给他 rename 一下。
|
20
unregister 2024-02-16 11:04:54 +08:00
如果这块代码你来改的话就 rename 一下,不然影响你工作
|
![]() |
21
nzbin 2024-02-16 11:11:31 +08:00
这个是代码习惯问题,我一般写变量都会刻意多看几眼,偶尔写错也可以理解,数量不多就改过来,历史代码能不改就不改
吐槽:同事代码 newOption 能写成 nowOption ,我愣是研究了大半天才明白是啥,可以想象这种代码得多难维护 |
22
yanyao233 2024-02-16 11:24:23 +08:00 via Android
ide 不是会有提示吗,很多缩写识别不了也提示,都给我整麻了
|
23
boris1993Jr 2024-02-16 12:32:28 +08:00 via iPhone
@yanyao233 #22 可以添加到个人词典吧?
|
![]() |
25
adoal 2024-02-16 12:41:49 +08:00
以前有个段子,股市行情好的时候坐台小姐到证券公司当交易员,行情不好的时候交易员坐台当小姐。
红利行业就是这样,对人的需求大于供给,各种阿狗阿猫都会冲进来。 (程序)又不是不能跑、(人)又不是不能跑。 你能做的就是尽量往高水平的团队去。 |
![]() |
26
raptor 2024-02-16 13:37:32 +08:00 via Android
我碰到 reason 拼成 resaon 的,我让他改了,别的团队的同事,我直接在公司群里说这个单词拼写错了,他下个版本就改了。反正都是公司内部对接的东西,大家一起改一下就是了。
|
27
wangkun025 2024-02-16 14:03:52 +08:00
能改就改。
不能改就注释。 不能注释,你妹夫的,跟你拼了。 |
![]() |
28
churchill 2024-02-16 14:28:39 +08:00
说实话大部分拼写错误放在上下文中都不影响阅读
怕乱起名的,怕 define FALSE true 的 |
29
lykhero 2024-02-16 14:33:35 +08:00 ![]() 这个就是不认真 不要洗什么学历什么的 不认真就是不认真 完全可以查了之后自己背一下的
又不是考托福雅思 就这么几个单词 |
![]() |
30
cctv1005s927 2024-02-16 14:46:38 +08:00
安全的地方尽量改掉,并在 review 的时候,引导身边的人尽量注意拼写的问题。
正确的事情应该坚持,你有大大方方做的理由,不要犹豫。 |
![]() |
31
lambdaq 2024-02-16 14:49:11 +08:00
前鼻音 后鼻音 平舌翘舌搞混的多得很。
|
![]() |
32
shuimugan 2024-02-16 15:41:09 +08:00
外部的不管,自己团队的坚持开拼写检查,碰到写错的坚决改,碰到离谱的还要拿出来鞭尸。妈的以前有个功能在开发文档里叫去重,有个同事查了下翻译然后字段写了个 go_heavy ,震惊我一整年,被我长期拿来做反面例子,教育他们一定要看 IDE 的告警,提交前要保证所有告警消失。
|
33
celeron533 2024-02-16 16:01:32 +08:00
说明之前的团队不怎么样
- 英文水平不佳 - 不认真 - 没有 code review |
![]() |
34
twofox 2024-02-16 16:52:27 +08:00
对于我这种英语水平一般的人,copilot 和 gpt 真的帮了大忙。。很多变量不知道怎么命名合适,就靠这两个
对于这种历史的拼写错误,如果是我写的,我就会改。不是我写的,凑合用 |
![]() |
35
FlytoSirius 2024-02-16 17:14:35 +08:00
现在来看, 改动之前库里这些错别单词确实不现实,
但新的代码, 至少可以通过 "静态代码检查" 类的工具大幅减少这类问题的出现, 降低 human review 的压力, 毕竟人去 review 代码主要还是集中在 代码逻辑 上吧. |
![]() |
36
FlytoSirius 2024-02-16 17:17:28 +08:00
我个人 不太觉着靠引导别人能让他人解决英文拼写问题, 直接把 "拼写检查" 放在 CI 的 pipeline 里, 每次提交代码都必须 pipeline 运行通过, 你看他们都自己改不改拼写...
|
![]() |
37
Felldeadbird 2024-02-16 17:33:30 +08:00
影响不大,只是作用域内,可以改。
一旦超出了作用域,你要慎重修改了。 |
![]() |
38
iqoo 2024-02-16 19:58:35 +08:00 ![]() #define mian main
#define ture true |
![]() |
39
Sprdance 2024-02-16 20:58:45 +08:00
老代码基本没办法再更改了,不过可以拿出来当反面教材学习学习
新代码合入检视的时候,在 IDEA 里看看哪里有黄色波浪线,大概率就是拼写错了 |
40
julyclyde 2024-02-16 22:03:53 +08:00
你是不是腾讯的?
|
![]() |
41
netabare 2024-02-16 22:03:58 +08:00 via Android ![]() 感觉和使用非英文变量名(例如 xxglxt 之类的)一样是很不标准的做法
|
43
julyclyde 2024-02-16 22:06:43 +08:00
想当年为了解决同事们经常给拼写接近的别家域名发信,导致退信堵死服务器的破事
我直接把另外几个错误拼写都 alias 成自家了…… (美团发给面团的邮件会转回本域) |
![]() |
44
StevenRCE0 2024-02-16 22:15:18 +08:00
@julyclyde 美团变面团哈哈哈哈哈(我屎一样的笑点
|
![]() |
45
StevenRCE0 2024-02-16 22:17:10 +08:00
我会放弃挣扎,因为不知道别人是不是在同时改动,重构简单但是合并难以预测
|
49
Mmahaha 2024-02-16 23:54:35 +08:00
引用到了就顺便 refactor 一下,没有就算了
|
![]() |
50
asmoker 2024-02-17 00:43:05 +08:00 via Android
b_super
B 超 😂 |
![]() |
51
dif 2024-02-17 06:58:42 +08:00
将错就错,屎山代码不要动。
|
![]() |
52
sugarsalt 2024-02-17 07:41:04 +08:00
别人的一般不管。。以前有同学 button 和 bottom 写混了,debug 排查了半天😅
|
53
vincentzhu 2024-02-17 09:39:36 +08:00
你这种情况还是不改了吧,很容易出问题
|
![]() |
54
corcre 2024-02-17 11:46:51 +08:00 via Android
之前数据库有个字段名拼错了,做了一翻思想斗争决定不管,不然以前的代码要一起改掉…
|
![]() |
55
lesismal 2024-02-17 14:35:42 +08:00 ![]() 另一个角度:因为有 typo ,所以让更多人相对容易地成为了 contributor ,虽然这种 pr 有水分,但至少培养了兴趣、提高了参与度,对社区未来也是有好处的
|
![]() |
56
voy 2024-02-17 15:22:14 +08:00
```
int &redhad = redhat; int &ubuntu = ubuntu; ``` ok |
![]() |
57
r6cb 2024-02-17 19:50:28 +08:00
#define DEFAULT DEFALUT
然后自己就能用正确的拼法了 |
58
zogwosh 2024-02-18 01:29:12 +08:00 ![]() 1.对于大头兵,纠结拼写没意义,因为不算产出
2.对于 leader,纠结拼写没意义,因为结果导向 3.对于个人开发者,纠结拼写没意思,因为成不成功全看你的点子 4.对于开源专家,纠结拼写没意义,因为如果你的项目有人蹭 pr 会帮你改的 总结:代码洁癖爱好者改,其他人不改 |
![]() |
60
wangtian2020 2024-02-18 09:07:51 +08:00
vscode 的 cSpell 插件会波浪线提示我不常见单词拼写的
|
61
polo3584 2024-02-18 09:14:52 +08:00
自己写的检查出来就改一改,别人的坚决不改,不知道有多少莫名其妙的依赖。。。
|
62
raymond2020 2024-02-18 09:40:26 +08:00
能跑就行朋友,这都无所谓的
|
![]() |
63
cBlank 2024-02-18 11:15:33 +08:00
代码写的你能看懂就行了,毕竟机器执行的时候已经不是这样子了
|
64
julyclyde 2024-02-18 12:43:05 +08:00
@polo3584 之前我用 pyCharm 的时候看到他们有个 refactor 功能可以关联修改。不过也仅限于 project 管理范围内
shit 一旦放出去,确实就很难翻身了 |
65
dongzhuo777 2024-02-18 16:21:53 +08:00
老外都有写错拼错的。
|