怎么看待代码不准加注释的规则

2019-01-19 07:57:01 +08:00
 pkookp8
第一次提交代码时有人 review 让我去掉注释,我以为是注释写的不好
后来我复制同一项目内别人的代码顺带复制了注释,也让我去掉,而且还不是正式代码,是测试代码

好奇怪的规定,但又不是明文规定
3216 次点击
所在节点    问与答
24 条回复
zhazi
2019-01-19 08:00:29 +08:00
先问是不是,再问为什么
规定靠猜的吗?
binux
2019-01-19 08:14:29 +08:00
你确定不是注释掉的代码?
然后,你干嘛要复制代码?而且是一段需要注释的代码。
一段需要注释的代码说明它复杂到应该被重用,而不是被复制。
pkookp8
2019-01-19 08:28:01 +08:00
@zhazi 当然是说公司内部的规则,有什么好问是不是的

@binux 不是,就是很普通的解释性的。因为是测试代码,改一个变量就行,要跑两次。代码结构上就不允许我做循环,不复制我敲一遍也没意义
yidinghe
2019-01-19 09:08:08 +08:00
Linux 内核代码不照样有很多注释。
leekafai
2019-01-19 09:16:08 +08:00
外包项目吗?正常的。
zhazi
2019-01-19 09:20:09 +08:00
不是明文规定你怎么知道是内部规定,
是内部规定不问问为什么这么规定,
跟你这人咋沟通费劲呢
loading
2019-01-19 09:22:32 +08:00
提出这种要求的公司就是太菜了,对应这种这种需求,就应该有专门的清洗工具。
dong3580
2019-01-19 09:22:36 +08:00
我们有个项目也是的,看领导要求了,review 时候要我删除注释,另外一个项目我就写了大量注释。前一个然后发现前几天维护的人看不懂之前写的代码的意思了,有趣。
abcbuzhiming
2019-01-19 09:43:52 +08:00
@pkookp8 胡扯,内部规则?让它把规则明文拿出来,你说的那个人是谁,它有什么资格要你去掉注释?搞技术的还玩起暗箱操作来了?
greatghoul
2019-01-19 10:02:18 +08:00
@zhazi 同感
coolcfan
2019-01-19 10:20:08 +08:00
比如你的 reviewer 觉得这段代码没有注释我也看得懂,说明代码逻辑清晰,self-explaining,所以就不用加注释了呗?
Solace202
2019-01-19 10:36:18 +08:00
刚来公司也是,试用期那会儿他没事 review 我代码,就说让我把注视删掉以后别写注释,我去,这完全不合理啊,不过也没怎么问,现在那主管走了换人了又说尽量写注释
pkookp8
2019-01-19 10:40:40 +08:00
原意是问问大家对加注释,或者不加注释怎么看
删了就删了,我也没觉得啥。只是多了一次重新提交代码的步骤,又不是不给工资

@leekafai 不是,公司内部项目
pkookp8
2019-01-19 10:46:45 +08:00
@abcbuzhiming 怎么这么纠结规则什么的呢,而且还升级到暗箱操作了。。。。
可能我应该用规律,不应该用规则
本来就没有这方面的规定,有的组就在加,我的组加了就得删,这样一个现象
所以问大家对注释怎么看
learnshare
2019-01-19 10:48:15 +08:00
难道是因为占了行数,会造成虚假工作量?
Flicker
2019-01-19 12:26:09 +08:00
我公司 leader 比较注重代码的整洁,就规定最好别写注释,通过类,函数,变量的命名,函数抽象来确定代码的工作。
但是我觉得复杂的场景还是要写点注释,注释也不一定正确,很多人也不会坚持维护注释。
liuwei666666
2019-01-19 12:47:30 +08:00
现在沟通都得靠猜吗?
猜来猜去,为什么不直接真诚的沟通呢?
ResidualWind
2019-01-19 12:54:40 +08:00
@zhazi 就应该这样做,直接沟通,猜来猜去的 玩吗!
micean
2019-01-19 13:03:08 +08:00
不靠注释难道靠命名?
abcbuzhiming
2019-01-19 13:07:57 +08:00
@pkookp8 加了就得删?谁定的,凭什么?没规则,听谁的?你这不是暗箱操作是啥呢?

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

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

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

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

© 2021 V2EX