GetResultsXXXListUpdate ,看到这个方法名,你第一时间会认为他是什么动作?是查询?是更新? 实际是作废(删除)。。。 好难受
![]() |
1
chendy 140 天前
那就进行一个简单的封装
function invalid(...){ return GetResultsXXXListUpdate(...) } |
![]() |
2
ahhui 140 天前
好难受的命名,一眼看去,动作单词是第一个 Get ,然后是 Result ,感觉就是获取结果的。后面实在搞不懂为啥这样写。
|
3
Tink 140 天前
获取结果更新
|
![]() |
4
aragakiyuii 140 天前 via iPhone
GetResultsXXXList(to)Update 🙈
|
![]() |
5
masker 140 天前 via Android
拿到 v 站又是一个可以引起讨论的知乎化主题
|
![]() |
6
Itoktsnhc 140 天前
GetXXXResultsForUpdate 或者 GetXXXUpdatedResults
|
7
shuimuzi OP 关键他是作废(删除)动作!!!
|
![]() |
8
penll 140 天前
不是 public 的话,或者确定没有其他地方引用该方法的话,直接改名呗
|
![]() |
9
liuzhaowei55 140 天前 via iPhone
不如给这个函数再加一个模棱两可的注释,为屎山的崛起贡献一点自己的力量
|