好几次因为疏忽导致的 bug 都没发现,类似这样的疏忽,而且没有警告,往往很久后才发现。
function someBol(): boolean {
return true
}
if (someBol) {
doSomething()
}
1
codehz 2023-09-11 18:25:58 +08:00 1
eslint 有规则 @typescript-eslint/no-unnecessary-condition
|
2
ChrisFreeMan OP @codehz 多谢多谢,看了逃不了 eslint 了。
|
3
chnwillliu 2023-09-11 18:41:12 +08:00
打开 strictNullChecks 试试?
This condition will always return true since this function is always defined. Did you mean to call it instead?ts(2774) |
4
thinkershare 2023-09-11 18:41:13 +08:00
你这个 SomeBol 就不应该声音返回类型位 boolean, 而是应该声明位 true 类型。
|
5
Wxh16144 2023-09-11 18:52:58 +08:00
+1 ,我也遇到过好几次这个问题。 不知道 1 楼的 eslint 有没有效果。先去看看
|
6
intmax2147483647 2023-09-11 21:54:12 +08:00 1
确实容易疏忽掉,第一眼竟然没看出问题🤣老了
``` if (someBol()) { doSomething() } ``` |
7
gogogo1203 2023-09-12 01:48:42 +08:00
最好还是
``` const isSomething= someBol() if(isSomething){ doSomething() } ``` 多一个变量就多一个吧,更好阅读理解。 |
8
yinmin 2023-09-12 01:52:03 +08:00 via iPhone
if (someBol() === true) {
doSomething(); } |
9
zhangjiancheng 2023-09-12 09:09:08 +08:00 1
@yinmin 直接 if(!!someBol())就行了
|
10
liuidetmks 2023-09-12 10:22:41 +08:00
借楼
ts 没有条件编译,提议被否了,我觉得挺遗憾的 似乎前端都喜欢用 minijs 之类的把测试代码删掉 但是,有时候代码测试代码比较复杂,是 async await 写法,编译出来的 js 是个状态机跳转,minifyjs 无能为力。 |
11
ChanKc 2023-09-12 10:30:57 +08:00
|
12
XXWHCA 2023-09-12 16:21:53 +08:00
TS 不是说强类型语言吗,怎么还能这么写,编译不会报错吗
|
13
ChrisFreeMan OP @XXWHCA 我也纳闷 TS 为什么允许这种判断,我觉得这种情况应该报错的。
|
14
ChanKc 2023-09-12 17:39:10 +08:00
默认是可以检查出来的。楼主应该是关掉了 strictNullChecks
[Playground Link]( https://www.typescriptlang.org/play?#code/GYVwdgxgLglg9mABAZzgWwKYCE4BsAUAlAFyIBGceGAhkgN4CwAUIogE4ZQhtJRsgZmAX2bMYwRPlSYcuQokYtEAEzgBldJwAWMMAHMiwoA) |
15
ChrisFreeMan OP @ChanKc 我开着呢,如果是判断 true 类型确实是有警告,但是前面加个!判断否定的时候就没有😅很奇怪
|
16
ChrisFreeMan OP @chnwillliu 这个在判断 true 类型的时候确实可以,但是前面加个!取否的时候就失效了。
|
18
ChrisFreeMan OP @ChanKc 嗯嗯,现在暂时先用 eslint 顶着,不过我还是不喜欢太多插件,以后我应该会强迫自己写 fuc() === true 这类的语法吧,好过把项目环境搞的太过复杂。
|
19
ChrisFreeMan OP @ChanKc 最好是出现一个 js 的替代语言😂
|