- 主题:有个老外code review不让过
这个好像没啥意义吧,函数里就算给path重新赋值了又咋样,对外部毫无影响
--
FROM 221.224.15.*
并不会,第一反应是为啥要加第二个const,是不是有什么坑,真要可读性的话应该用个typedef或者using,起个别名,这样别人也不用特别在意里面写了啥
【 在 Icanread 的大作中提到: 】
: 可读性提高了
:
: 发自「今日水木 on iPhone 12」
--
FROM 221.224.15.*
道理是这个道理,也看整体的coding style,如果一个项目99%的参数都没加const,这一个const就很扎眼了,对于洁癖leader来说,是改这一个呢,还是全都改了呢,就留这一个的话可能每个新来的维护者都会问一次:为啥这个函数多了一个const,是有啥考虑吗?
当然这个有点夸张了,但类似的黑天鹅代码确实见过不少,别问,问就是“历史原因”,别改就对了。
【 在 Icanread 的大作中提到: 】
: 我个人而已,都加const了,我就可以放心的认为它就是一个常量了,不用考虑函数里它会不会变
: 发自「今日水木 on iPhone 12」
--
FROM 221.224.15.*