之前接到一个工单,然后需要刷数据库的数据,但因为那个系统也是我刚接的系统,所以根本不熟。
结果写数据到备份表里面的时候发现数据库里面有一个 xxtype 的字段,是存储数字来标识类型的,那我刷数据库的时候总得知道这个字段每个数字代表什么吧,这时候让我抓狂的事情就出现了,数据库字段后面注释没有说明,代码里面的实体类的字段上面的注释也没表明,然后因为这只是一个数据的备份,所以没有前端页面,我也没法在页面里面找到对应的值的含义!
简直了,这种地方不写注释是要怎样,自己猜吗 :)
1
tsening OP 刚刚看到一篇吐槽注释的贴,然后我就来吐槽一下不加注释的人 hhh
|
2
tikazyq 2020-08-24 11:32:04 +08:00
真正可怕的是命名,经常被一些大神的中文拼音首字母搞疯。你这个字段还可以根据名称来猜,如果命名不规范就等于在没有路灯和车灯的夜间山路上行驶
|
3
w292614191 2020-08-24 11:52:37 +08:00 1
刚看到一个嫌弃注释太多的帖子
|
4
hyperbin 2020-08-24 11:57:38 +08:00 via Android 3
我最讨厌两件事:写注释和他人代码没注释
|
5
Aprilming 2020-08-24 12:55:04 +08:00
哈哈哈,不好意思,我就是那种基本不写注释的人,但是比较注意命名规范,不写注释我一般不会说组员,但是命名瞎搞,我就是一堆锤。挨,不写注释和注重命名都是以前跟着师傅学的。。。。
|
6
fhsan 2020-08-24 13:04:31 +08:00
群里经常看到这种人,天天 bb 别人怎么样从来不检查自己的问题,有本事你重构啊
|
7
hbolive 2020-08-24 13:07:28 +08:00
找相关人员要一份数据库设计说明书。。
|
8
xiangyuecn 2020-08-24 13:19:15 +08:00
IDEA 瑟瑟发抖,注释啥的贵在精,有事没事敲个注释看代码都要烦死人。就像这沙雕 IDEA 一样,不配置你永远看不到核心关键的提醒(滚动条上的那些高亮叫啥来着),注释写的太多也一样看着烦人
此图已无药可救 |
9
xiangyuecn 2020-08-24 13:28:17 +08:00
#8 特意去试了一下 这个沙雕提示,结果很感人😂
|
12
SaKuLa 2020-08-24 13:41:08 +08:00
搞不懂嫌注释多的人是怎么想的,要是其他人维护的模块出问题要你去找原因,没有中文注释连能不能第一时间找到代码都是问题。你代码再清晰,谁有空去一个个单词猜过去。
|
13
flowfire 2020-08-24 14:24:20 +08:00
@xiangyuecn 这个咋了。我觉得这种写法很直白啊。。。
一看就知道是干嘛的。。。 |
15
futamilk 2020-08-24 14:58:12 +08:00
emmm 一般各种字段我在最初取到值的时候都会注释一下实际意义。其实从代码里的下标去对照文档也不难找到,注释了之后如果业务有改动,还需要维护注释。这是一个不好的地方。但我还是会标一下,主要是方便自己,在开发的时候不用对着文档来回找。
|
16
luxinfl 2020-08-24 15:17:25 +08:00
最怕的是没有注释而且变量名和真实意义相反的
|
17
tsening OP @xiangyuecn 。。。你那个 DayMS2 直接返回了 int 类型那肯定超过精度了啊,你点一下那个提示的 simplify 的按钮,他会直接加一个(long)。而且为啥要特意乘个 1L,直接在后面的数上加个 L 不就好了嘛
|
19
GTD 2020-08-24 16:09:01 +08:00 1
@xiangyuecn #9 这有什么问题吗?是你自己的问题吧
|
20
unco020511 2020-08-24 17:03:23 +08:00 1
|
22
Redbeanw 2020-08-24 18:05:50 +08:00
|
24
xiangyuecn 2020-08-24 19:21:14 +08:00
|
26
jerryrib 2020-08-24 19:56:26 +08:00
注释:我也很难啊
|
27
zhady009 2020-08-24 19:58:08 +08:00
@xiangyuecn TimeUnit.DAYS.toMillis(days) Duration.ofDays(1L).toMillis()
就行了 善用工具类 |
28
tsening OP @xiangyuecn 第二点可能是我经验不多没有体会到,第一点嘛,他的提示的确没写出来,但他的方案里面确实是转成 long 的,你点一下 simplify 按钮就知道了,杠不杠的未必,理性讨论即可
|
30
aguesuka 2020-08-25 10:24:13 +08:00 via Android
@xiangyuecn 你要的是,TimeUtil.DAY.toMils(),而且看样子没有单元测试
|