feat: 在 JS/TS 基础规范中优化魔法数字检查规则,添加可理解字面量的判断说明并更新示例代码
This commit is contained in:
@@ -55,18 +55,24 @@ function getuserinfo() {
|
|||||||
- console.debug 的错误信息无需审查
|
- console.debug 的错误信息无需审查
|
||||||
- console.trace 的错误信息无需审查
|
- console.trace 的错误信息无需审查
|
||||||
- 无需考虑类型是否合理匹配
|
- 无需考虑类型是否合理匹配
|
||||||
|
- 你需要自行的判断这个字面量的值开发是否可以理解,比如说参数的时间戳、毫秒时长、数量等这些如果一看就是可以理解也无需抽出常量
|
||||||
|
|
||||||
### Good
|
### Good
|
||||||
|
|
||||||
```javascript
|
```javascript
|
||||||
const MAX_COUNT = 100;
|
const ADMIN = 1;
|
||||||
|
|
||||||
|
if (user.role === ADMIN) {
|
||||||
|
// ...
|
||||||
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
### Bad
|
### Bad
|
||||||
|
|
||||||
```javascript
|
```javascript
|
||||||
const maxCount = 100;
|
if (user.role === 1) {
|
||||||
const userStatus = "active";
|
// ...
|
||||||
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
## 基于 JsTs.Base.NoMagicVar 抽离的静态常量需要放在对应文件里 `[JsTs.Base.ConstantsDefinition]`
|
## 基于 JsTs.Base.NoMagicVar 抽离的静态常量需要放在对应文件里 `[JsTs.Base.ConstantsDefinition]`
|
||||||
|
|||||||
Reference in New Issue
Block a user