这是一个创建于 446 天前的主题，其中的信息可能已经有所发展或是发生改变。
好几次因为疏忽导致的 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 的替代语言😂
|
Select Language