Confusing Variable Naming: "step" & "move"
未关闭
还没有人认领这个 Issue。
- 主要语言
- JavaScript
- 星标
- 11.8k
- 派生
- 7.9k
- 平均合并
- 1 天 11 小时
- 30 天内合并 PR
- 11
描述
At line 1139, jumpTo() requires an argument named step.
However, on lines 1017, and 1106, it passes the variable, move, into jumpTo function, which is confusing.
I think it's better either change the naming of the argument of jumpTo() (:1139),
or switch the naming of currentValue and index of map() (:1017, :1106)
The below PR goes the second way since I think move is more like representing the "movement", on the other hand, step is understood as a "number" in most cases.
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
调研方向
首先检查第 1139 行的 jumpTo() 以及第 1017 行和第 1106 行的 map() 调用。确认每个位置使用了哪些名称，然后使命名与预期含义保持一致，并验证受影响的代码仍然清晰易读。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- javascript
- 领域
- documentation
- Issue 类型
- 重构
- 难度
- 1/5
- 预计耗时
- 1 小时以内
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 38/100