- 主题:有个老外code review不让过
说起来是对的，但是绝大多数库里不这么干，所以看起来很奇怪。
【 在 freyoneby (freyoneby) 的大作中提到: 】
: 一定要函数参数加个const，比如conf（const char＊ path）改成conf（const char＊ const path），我说参数的值改不了，没必要加，他说能加const地方都要加，想问一下参数要么进栈要么进寄存器，const是怎么作用到寄存器上的？
:
: - 来自 水木社区APP v3.5.7
: --
--
FROM 183.179.53.*
那你们的所有参数不得是 const int, const double
【 在 eyepeer (eyepeer) 的大作中提到: 】
: 太常见了，我上家公司，你不这么写，基本review不过。
:
: 【 在 z16166 的大作中提到: 】
: : 搞两个const来声明一个变量的代码，很少见啊。
--
FROM 183.179.53.*
可读性提高了
【 在 mopo 的大作中提到: 】
: 这个好像没啥意义吧，函数里就算给path重新赋值了又咋样，对外部毫无影响
: --
发自「今日水木 on iPhone 12」
--
FROM 125.33.206.*
指针。。。。。。
【 在 fanci 的大作中提到: 】
:
: 那你们的所有参数不得是 const int, const double
: --
: 发自xsmth (iOS版)
: --
:
发自「今日水木 on iWatch」
--
FROM 123.121.195.*
没啥区别吧，都是值传递。
【 在 eyepeer 的大作中提到: 】
: 指针。。。。。。
: 发自「今日水木 on iWatch」
--
FROM 123.123.42.*
我知道意义不同啊，所以说review人说的是对的
但是搞法啰嗦，不值得推广（除非是写航天器之类的代码的？）。公司里如果有强制规定这么搞，那就老老实实按规定来呗
如果不是强制规定，那就商量，或者PK，看谁更强势了
【 在 hongyan2022 的大作中提到: 】
: 这两个的 意义不同啊
:
--
修改:z16166 FROM 221.218.163.*
FROM 221.218.163.*
你们对code review这么抵触吗。。。我遇到这种情况一般就点个like然后顺手改了就完了。。。
- 来自 水木社区APP v3.5.7
【 在 z16166 的大作中提到: 】
: 我知道意义不同啊，所以说review人说的是对的
:
: 但是搞法啰嗦，不值得推广（除非是写航天器之类的代码的？）。公司里如果有强制规定这么搞，那就老老实实按规定来呗
:
: 如果不是强制规定，那就商量，或者PK，看谁更强势了
--
FROM 107.77.228.*
不是对code review这个活动抵触，而是style的问题
我觉得这种双const的style是属于政zhi上正确，但是不值得提倡的，除非是写一些要求极为严格的产品（通常是拿造原子弹来比拟）
从另外一个角度喷，可以喷C的设计不好，把下面三种情况分开表达，可能更好，而不是统一弄个const来表达：
仅仅指针不能改、仅仅指针指向的数据不能改、指针不能改而且指针指向的数据也不能改
【 在 Brightblade9 的大作中提到: 】
: 你们对code review这么抵触吗。。。我遇到这种情况一般就点个like然后顺手改了就完了。。。
:
: - 来自 水木社区APP v3.5.7
--
修改:z16166 FROM 221.218.163.*
FROM 221.218.163.*
现在clang lint会给他这种包警告吧，直接拿这个怼他
【 在 freyoneby 的大作中提到: 】
:
: 一定要函数参数加个const，比如conf（const char＊ path）改成conf（const char＊ const path），我说参数的值改不了，没必要加，他说能加const地方都要加，想问一下参数要么进栈要么进寄存器，const是怎么作用到寄存器上的？
:
: - 来自 水木社区APP v3.5.7
#发自zSMTH-v-@钛星
--
FROM 223.104.41.*
你没理解他回你的问题
【 在 eyepeer 的大作中提到: 】
:
: 指针。。。。。。
: 【 在 fanci 的大作中提到: 】
: :
: : 那你们的所有参数不得是 const int, const double
#发自zSMTH-v-@钛星
--
FROM 222.129.53.*