Comments (8)
Quote from the other issue, emphasis mine:
Background: The control flow graph creator can't resolve the types of expressions (doing so would create a circular dependency between passes of the compiler), but does have some logic for detecting
if (true) {
. Basically, TS thinksif (value === null) {
might not always execute, because during the construction of the control flow graph, it is not yet known that__TRUE__
istrue
.
That's a pretty fundamental design constraint, and I highly doubt anything has changed in that regard.
Then again, TS can do control flow for asserts T
and never
returns as long as there's an explicit type annotation on the function, so who knows...
from typescript.
@kungfooman I'm only a passerby so I can't tell. You have to wait for a TS team member to answer it.
from typescript.
The lint rule has a lot of false positives, e.g.
const n = [1, 2];
if (n[4] !== undefined) { // <- thinks this is always true
}
from typescript.
Duplicate of #29323.
from typescript.
Thank you @whzx5byb, that issue is from 2019, is it still a design limitation? I want to use some advanced type programming which figures out true/false values and have unreachability detection of it.
from typescript.
Everything is of course in-principle solvable, but there aren't a lot of cases where people intentionally write if (expr)
where expr
is a complex expression which statically provable to be true
(and then need the corresponding code to change color).
from typescript.
@RyanCavanaugh I investigated type checks on ranges a bit through type programming doing arithmetic, e.g. a function could return a random number from 20 to 100. If later there is an condition which checks that this number is bigger than... lets say 200, it can only be false (and ideally be pointed out through dead-code detection).
You already planned range based type flow in another PR and I just wanted to see if it's possible already without any extra PR's. But this issue makes range based type checks not too helpful as-is.
OTOH it seems to already exist, but with an extra linting step: https://typescript-eslint.io/rules/no-unnecessary-condition/
from typescript.
This issue has been marked as "Duplicate" and has seen no recent activity. It has been automatically closed for house-keeping purposes.
from typescript.
Related Issues (20)
- object getters return type error HOT 12
- Poor autocomplete inside object literal as the return value of a getter HOT 1
- possibly 'undefined'. emitted after Boolean constructor check in Visual Studio HOT 2
- Error when using different types of return which should have the same result HOT 2
- [isolatedDeclarations][5.5] Disagreement between transpileDeclaration API and typechecker on Symbol.iterator as computed property HOT 7
- Handle JSDoc {@link ... } inline tags for OS
- Polymorphic this in static methods is not resolving correctly HOT 4
- Regression in detecting unused parameters in 5.5.0-beta
- Computed string literal can't be used as generic argument HOT 8
- [transpileDeclaration API] What to do about "has or is using private name" diagnostics HOT 1
- Event `type` (name string) doesn't get properly checked or suggestions when using addEventListener() on an Union of multiple HTMLElement types HOT 5
- [Feat] support extending current class interface?
- TS 5.5 ConfigDir With Include Bug HOT 3
- value of type never can be created using document.all HOT 9
- Marking a constructor as `@internal` emits no constructor declaration at all instead of a private constructor HOT 2
- [NewErrors] 5.5.0-dev.20240512 vs 5.4.5 HOT 57
- [ServerErrors][TypeScript] 5.5.0-dev.20240512 HOT 12
- [ServerErrors][JavaScript] 5.5.0-dev.20240512 HOT 13
- Discriminated union with an optional discriminator is not being resolved as expected HOT 2
- Find All References should include composed return value
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from typescript.