if-statements allow multiple arguments and don't check for constant conditions after the first argument

Offen

#60,822 erstellt am 19.12.2024 8 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 25/100

Issue-Typ: Bug
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: typescript
Bereich: compilers

Rechercherichtung

Reproduziere das bereitgestellte Beispiel im verlinkten TypeScript Playground und vergleiche die beiden im Bericht beschriebenen Formen von if-Anweisungen. Überprüfe das Compilerverhalten für durch Kommas getrennte Ausdrücke und konstante Bedingungen; abgeschlossen ist die Aufgabe, wenn die vorgesehenen Diagnosen oder das akzeptierte Verhalten für diese Beispiele definiert und abgedeckt sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Awaiting More Feedback Suggestion

🔎 Search Terms

"if statement", "constant condition"

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about common bugs

⏯ Playground Link

https://www.typescriptlang.org/play/?#code/MYewdgzgLgBBIFsCmBhAFk4BrAlmA5gGICuYwUO4MAvDABQAOAhgE5MICMAXHFC3vgA0MZmwQAmHtH4FhIBhXBMANgAVW7AMw0YfYkgCUNAHwwA3gCgY1mAHpbMACYhexAGZurNlkijEWYLos+gDcFgC+Fhb2MCoA7kwAnhAwbioQSMI+EMTKsCBucIioGNgCJGSKgfFJKTj4YCA+jhY4bnTwyOiYuAQV5JRgdABEouwcw8KjGhLDBsJpyhlGljYwEVExjkgMSGDbYPmB2bn5hZ0lPeWkA+Ct7RfdZX03VSNjnJMw02LiX4vLFZeawRIA

💻 Code

const someCheckingFunction = (param1: string, param2: string, optionalParam3 = true) => {
    // do stuff
    return true;
}

// always false, result of someCheckingFunction always ignored
if(someCheckingFunction("param1", "param2"), false) {
    
}

// dependent on result of someCheckingFunction
if(someCheckingFunction("param1", "param2", false)) {
    
}

🙁 Actual behavior

Due to a mistake in my bracket placement as seen in the example, I accidentally learned that multiple arguments in an if statement are possible.

Currently that means:

only the last argument is used for the if statement check
even for constant conditions there is not error visible

🙂 Expected behavior

What I would expect:

don't allow multiple arguments, this only happens because someone made an error
at least show an error if there is a constant condition

Additional information about the issue

No response

Vorherrschende Sprache: Go
Sterne: 111k
Forks: 14.4k
Ø Merge: 1 T. 19 Std.
Gemergte PRs (30 T.): 117

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/TypeScript

[Bug] PrivateIdentifier nodes are omitted from 2020 semantic classifications Offen

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100

microsoft/TypeScript#64322 · 2 Kommentare · 1 Reaktion · 2 zugewiesene Personen · Erstellt am 18.9.2026
enhance: Add tests for premature caching of contextual parameter types Offen

Possible Improvement

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100

microsoft/TypeScript#64278 · 1 Kommentar · 1 Reaktion · Erstellt am 15.9.2026
dom.generated.d.ts is missing from this repo while still referenced in the docs Offen

Docs

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100

microsoft/TypeScript#64118 · 1 Kommentar · Erstellt am 01.9.2026
typescript-language-server does not work with @typescript/typescript6 Offen

Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100

microsoft/TypeScript#64094 · Erstellt am 29.8.2026
The new `--lsp` flag is not mentioned in `tsc --help`'s output Offen

Docs

Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100

microsoft/TypeScript#63959 · 5 Kommentare · Erstellt am 22.8.2026

Alle Issues in microsoft/TypeScript