search

spin6lock / spin6lock.github.io

2 stars 0 forks source link

代码扫描 #18

Open utterances-bot opened 2 years ago

utterances-bot commented 2 years ago

代码扫描 | spin6lock’s blog

最近犯了一个很蠢的错，Code Review 的时候觉得非常丢人，决心不再犯第二次了，于是写了个小工具来检查。但是实际跑起来，才发现 Lua 这里设计得不太好，留下了一个合法但很容易犯错的坑

https://spin6lock.github.io/2022/08/12/%E4%BB%A3%E7%A0%81%E6%89%AB%E6%8F%8F.html

rangercyh commented 2 years ago

我们也经常这么写，感觉还是写的人不用心：D

hanxi commented 2 years ago

我记不住返回一个变量时是key还是value，我都会用返回2个的写法，不用的那个就用下划线。

spin6lock commented 2 years ago

主要是用ipairs还觉得自己的key是某个key，实际上是序号……而且序号有可能和真实key重叠，自己开发的时候不会发现问题。。

fanyh commented 2 weeks ago

kk；来晚了， 吃口冷饭， 不是，就这个代码而已，直接 math.random(#cfg) 不就可以了；走远了吧 佩服你和@rangercyh 检查写blog