GetResultsXXXListUpdate，看到这个方法名，你第一时间会认为他是什么动作？
shuimuzi · 2022-09-14 15:36:04 +08:00 · 1099 次点击这是一个创建于 803 天前的主题，其中的信息可能已经有所发展或是发生改变。
GetResultsXXXListUpdate ，看到这个方法名，你第一时间会认为他是什么动作？是查询？是更新？ 实际是作废（删除）。。。 好难受
 |
1
chendy 2022-09-14 15:39:46 +08:00
那就进行一个简单的封装
function invalid(...){ return GetResultsXXXListUpdate(...) } |
 |
2
0TSH60F7J2rVkg8t 2022-09-14 15:40:09 +08:00
好难受的命名，一眼看去，动作单词是第一个 Get ，然后是 Result ，感觉就是获取结果的。后面实在搞不懂为啥这样写。
|
 |
3
Tink 2022-09-14 15:56:27 +08:00
获取结果更新
|
 |
4
aragakiyuii 2022-09-14 15:59:19 +08:00 via iPhone
GetResultsXXXList(to)Update 🙈
|
 |
5
masker 2022-09-14 16:01:56 +08:00 via Android
拿到 v 站又是一个可以引起讨论的知乎化主题
|
 |
6
Itoktsnhc 2022-09-14 16:02:17 +08:00
GetXXXResultsForUpdate 或者 GetXXXUpdatedResults
|
 |
7
shuimuzi OP 关键他是作废（删除）动作！！！
|
 |
8
penll 2022-09-14 16:32:21 +08:00
不是 public 的话，或者确定没有其他地方引用该方法的话，直接改名呗
|
 |
9
liuzhaowei55 2022-09-14 16:42:26 +08:00 via iPhone
不如给这个函数再加一个模棱两可的注释，为屎山的崛起贡献一点自己的力量
|
Select Language