- 主题:review代码哭笑不得
将错就错也没啥问题吧,只要整个系统都用这个错误的单词,那就是没问题的
我这有个人一直把status变量名写为statue,崩溃念成奔溃,剪贴板说成剪切板
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 221.218.161.*
看文档里有没事先明确约定要用status作为节点名。
真正危害大的是码农用各种只有自己知道含义的缩写,换个人维护就类似天书了
【 在 seablue 的大作中提到: 】
: 这属于typo,不能将错就错。
: 一旦失守,其它地方也到处都是将错就错,随着时间的推移,软件慢慢会变得难以阅读、维护。长此以往,软件慢慢变得一团乱麻,得不偿失。
:
--
FROM 221.218.161.*
不valid,哈哈
invalid
【 在 coocodelie 的大作中提到: 】
: 这个什么意思?buvalid
:
--
FROM 221.218.161.*
“能用”是猪对猪食的需求,不是人对代码的需求,哈哈
【 在 stub 的大作中提到: 】
: 又不是不能用
--
FROM 221.218.161.*
魅族黄章的
【 在 stub 的大作中提到: 】
: 这是罗永浩那个梗
--
FROM 221.218.161.*
别人能听懂就行,或者要求不是那么严格的场合也无所谓。
阈值,你要是说个阀值,别人估计大概率能听懂,哈哈
【 在 eosiocaptain 的大作中提到: 】
: 哥,我也习惯说剪切板
:
--
FROM 221.218.161.*
Internal Server Error,应该翻译成“内部服务器错误”,还是“服务器内部错误”?哈哈
--
FROM 221.218.161.*
确实。
如果是cpp的话,json解析代码用nlohmann json要好很多,把挨个提取json节点的代码都隐藏在宏里了。
有个工程,充斥这种字符串和0、1。
这种字符串算是其中某个开关设置管理模块的实现细节,不需要暴露到每个其他模块中。真的可怕,因为每个相关的功能函数都要判断这个功能的开关。
而0、1明明可以用true、false的。
说了不改,已经太多了。我也没兴趣给他擦屁股。
CXXXX::Instance()->GetStatus() != L"activated"
CXXXX::Instance()->GetModuleEn("enable_general") != 1
【 在 Hongjinlong 的大作中提到: 】
: string 常量乱飞
: 是垃圾代码的一个标志
--
修改:z16166 FROM 221.218.161.*
FROM 221.218.161.*
为啥?英语就是这个限定词的顺序吧,internal在前,而且大家都知道这是http协议里的标准错误文本。
【 在 JulyClyde 的大作中提到: 】
: 这个可能连英语都是错的
--
FROM 114.241.228.*
中文、英文针对同一个东西的限定词顺序可能有差异
类似"人民内部矛盾"还是“内部人民矛盾"
【 在 PaoloMaldini 的大作中提到: 】
: 如果楞要翻译的话,前者吧,跟external对应的
:
--
修改:z16166 FROM 114.241.228.*
FROM 114.241.228.*