发现公司项目里有个类的名字拼错了,要改吗?

2015-12-11 13:24:08 +08:00
 langzizx39

在加一个功能时发现这个类(不是我写的)的名字拼错了两个字母,但这个类牵扯到的东西很多(前后端都有),我要修改吗?还是将错就错?

8365 次点击
所在节点    程序员
98 条回复
liujiangbei
2015-12-11 16:46:17 +08:00
为什么大家都建议不改,毫无疑问,跟写这个代码的人(如果还在)简单沟通一下果断要改。
sacuba
2015-12-11 16:52:13 +08:00
@liujiangbei 牵扯很多的话,几乎没人愿意去改的,毕竟无关大雅 我觉得 易懂简练有规则的命名方式就是好的命名,拼错了个单词无伤大雅
xinyewdz
2015-12-11 16:58:15 +08:00
改完之后,是要背黑锅的.
yoa1q7y
2015-12-11 17:08:23 +08:00
最好别动,多一事不如少一事
merlinran
2015-12-11 17:13:10 +08:00
一个连改名都担心背黑锅的地方,代码只会越来越糟糕,还值得呆么?版本管理、自动化测试和构建,轻而易举就可以防住可能的风险。
jsfaint
2015-12-11 17:21:16 +08:00
@vanxining unix 时代 creat 是故意简写的,后来 Ken 大爷后悔了,再写 go 的时候亲自改成了 create
xiaowei4895
2015-12-11 17:32:32 +08:00
当然改啊,改个名字都搞不定,你还能干点啥?
liberize
2015-12-11 17:38:09 +08:00
不作死就不会。。。
young
2015-12-11 17:40:08 +08:00
之前碰到过一个表中某个字段 channel 写成了 channle
就这样用了很长时间... 没人敢动
crayygy
2015-12-11 17:46:53 +08:00
适配器或者外观模式?
server
2015-12-11 17:52:20 +08:00
这事干过 一个常量名错了,改了很多地方。
知道错了不该,以后慢慢的也成这样了。
new_bee
2015-12-11 18:06:03 +08:00
同为代码洁癖者表示不能忍。不过哥们吃过这个亏。建议大项目慎重,要及时和其他人沟通,不要相信 IDE 自动化重构。可以 wrapper 或另写。当然小项目随意。
bdbai
2015-12-11 19:17:37 +08:00
@vanxining 还有 HTTP Referer
akagi
2015-12-11 19:40:41 +08:00
除非作者授意,或者项目特简单,不然两个月之后的某一天,也许你就要开启一个满是 error 的平行世界……
luoluoluo
2015-12-11 20:00:06 +08:00
It doesn't matter at all.
n6DD1A640
2015-12-11 20:18:32 +08:00
这是一个深坑, LZ 请三思。。
incompatible
2015-12-11 20:25:49 +08:00
不要改。原因见 26 楼。

另外,如果觉得用 IDE 的重构功能改可以高枕无忧,那就太 naive 了。
@loshine1992
@xbb7766
@Wangxf
@cxbig
你永远不会知道是否有人通过反射调用了这个类的某个方法,你也不知道是否有人把这个代码的源码或者编译结果作为依赖引入他自己的工程。这些都是无法通过 IDE 的 AST 检测出来的。
jukka
2015-12-11 20:33:06 +08:00
我以前给 label 写成 lable 一直到项目上线俩月大家才发现。。。。。要知道基本上每个 UI 文件里都出现了这个单词啊。
sagnitude
2015-12-11 20:40:35 +08:00
要慎重,我们在 model 里改动变量名字或者增删一个变量有时候会需要提前一周通知,相关开发人员一起动手改,新功能暂停开发,所有项目同时更改同时上线。
除非你的所有项目都有完备的自动化测试和构建体系,否则一定要慎重。

服务端改动一个 model 的变量名, iOS , javascript , Android , Java 桌面端,所有的项目都要改,
尤其是前端! javascript 这种语言,服务端改动 model 之后,前端页面里不跑到特定的代码那里根本不知道会出错

我们有很多小项目,半年或者一两年没维护,基本都是跑不了的,因为其他的公共代码修改了,哪怕只是修改了一两个域,也很难查。基本要用的时候都需要安排人修复
virusdefender
2015-12-11 20:43:04 +08:00
Java 或者 c 语言系列的,相信 ide 估计可以都自动改过来, Python 或者 js 类的,很可能改不全。

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

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

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

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

© 2021 V2EX