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

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

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

8386 次点击
所在节点    程序员
98 条回复
xbb7766
2015-12-11 21:17:28 +08:00
@vitovan 第一眼看成:setup, 再想想难道是 step?

.201511051109 这个后缀更坑,“ 11 月 5 号到 9 号”?还是“ 11 月 5 号 11 点 09 分”……
sobigfish
2015-12-11 21:43:56 +08:00
除非测试 100% code coverage 的代码加 CI ,不然还是加 todo 以后重构的时候改的好
zander
2015-12-11 21:48:44 +08:00
同 # 8 ,不要改,但是可以把正确名字的坑占了,然后引用原类。
你可以用正确的,他们用旧的也好新的也好无所谓。
mringg
2015-12-11 21:53:02 +08:00
改完没准问题就大了,将错就错吧
lmaq
2015-12-11 21:56:29 +08:00
使用没有问题的话最好不要改
cxbig
2015-12-11 21:57:10 +08:00
@incompatible 在我看来只是成本问题
修正 typo 以前看看有多少关联,有多复杂,评估一下。至于怕漏掉反射调用之类,基于人员素质的问题,都是可以纳入成本核算的。
incompatible
2015-12-11 22:32:26 +08:00
@cxbig 抱歉,没有仔细读清楚你的观点就 @了你
你说的对,这个事情的关键点是成本。 IDE 只是降低成本的必要手段。
yjxjn
2015-12-11 22:33:54 +08:00
按照系统 risk 的方面来考虑。不改,为啥呢,如果这个系统你非常熟悉架构,调查之后,能 100%确保那些地方用这个类了,就改,如果不确定,那就 no zuo no die 吧,放着吧。不碍事的。
ibireme
2015-12-11 22:36:54 +08:00
我自己的代码,作死也要改。
公司项目中遇到的话,看心情了。。
yjxjn
2015-12-11 22:43:41 +08:00
我反对楼上说拿 ide 一次性替换的! too young !你除非系统特别小。。。。业务量支撑非常大的系统,成本是非常重要的,无论是经济成本还是时间成本。
我就说说我组怎么做一个完整的 devops 。
1.做设计书,确定各个变量名啊,方法名,接口之类的。
2.做详细设计书,把流,处理等等写好。
3.开始 coding , coding 之后,将设计书和详细设计的内容比较一番,看有没有漏的,再自己测试一下。最后团队的大拿来 review 。。。。然后定版,并且把每一步的开发和测试 log 留好,
4.user access test 俗称 UAT 测试,每发现一个问题,提一个 ticket ,然后 track 问题。
5.uat 没啥问题了,直接上 prod 测试。测完之后,留测试 log ,最后把设计书改一改,完成最终版本。
googlefans
2015-12-11 22:47:20 +08:00
先搞清楚不改对你的影响大吗?
monkeymonkey
2015-12-12 00:41:56 +08:00
这种错误写的时候就该避免,装一个 Spell Checker 插件吧。
不只是 typo ,连单词是否合适都应该首先考虑清楚再写。
knightdf
2015-12-12 01:08:17 +08:00
我还把项目名写错过呢 - -, 还好项目名只是个名字,改改没事
BigDecimal
2015-12-12 02:11:50 +08:00
第一,因为不是你写的,所以改后可能存在潜在的未知问题。
第二,因为这不算 bug ,改后可能会造出新的 bug ,所以最好不要冒这个风险。
第三,如果实在想改,那么改完之后就得把这个类所涉及到的功能都统统测试一遍,以确保没有造成新的 bug ,这是没事找事的做法。


综上所述,还是将错就错,不改为妙。
FrankFang128
2015-12-12 04:21:45 +08:00
破窗理论
POPOEVER
2015-12-12 05:23:13 +08:00
维护一个 typo 文档吧
bbx
2015-12-12 07:47:00 +08:00
dependency 太多,搞了就等着背锅吧
letv
2015-12-12 08:19:30 +08:00
遇到同样的情况,用 vs 看了下,只有两处引用,所以改了。。。
pynix
2015-12-12 08:36:13 +08:00
能改的时候尽快改了吧。再不改就等着成为历史遗留问题吧。。。。
happyz90
2015-12-12 08:53:03 +08:00
反正要混淆掉。。。

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

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

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

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

© 2021 V2EX