KeepChatGPT icon indicating copy to clipboard operation
KeepChatGPT copied to clipboard

变量定义和声明都用了 var

Open Iey4iej3 opened this issue 1 year ago • 4 comments

我发现这个脚本中所有变量定义和声明都用了关键词 var。根据 https://stackoverflow.com/questions/47465515/why-is-var-not-deprecated 和 https://softwareengineering.stackexchange.com/questions/274342/is-there-any-reason-to-use-the-var-keyword-in-es6 ,似乎(在 ECMAScript6 以后)一般情况下应该用 letconst(尤其是例如循环变量这类局部变量)。不清楚这是有意设计还是无意使用?

Iey4iej3 avatar Apr 23 '23 14:04 Iey4iej3

谢谢反馈,很好的提议

xcanwin avatar Apr 23 '23 14:04 xcanwin

另外代码里面的 == 很多大概都能改成(类型)严格相等 ===

代码里面有很多 var foo = function () {};,但似乎 function foo() {} 是更常见的写法,不过这应该是风格不同。

Iey4iej3 avatar Apr 23 '23 15:04 Iey4iej3

名气大了,都有人检查语法了,哈哈

PandyYang avatar Apr 24 '23 06:04 PandyYang

名气大了,都有人检查语法了,哈哈

hhhh

xcanwin avatar Apr 24 '23 07:04 xcanwin