`!.` after `?.` should be warned

Description

This is an anti-pattern on the current type system. And the compiler must not do quick fix so. !. after ?. is simply replaceable with ?. and should do so.

TypeScript Version: 3.7.x-dev.20191105

Search Terms:

Code

document.querySelector('_')!.textContent!.split('') // old code
document.querySelector('_')?.textContent!.split('') ?? 0 // string[]
document.querySelector('_')?.textContent?.split('') ?? 0 // string[] | 0

Expected behavior:

document.querySelector('_')!.textContent!.split('') // old code
document.querySelector('_')?.textContent!.split('') ?? 0 // unsafe, warning
document.querySelector('_')?.textContent?.split('') ?? 0 // safe

Actual behavior:

document.querySelector('_')!.textContent!.split('') // old code
document.querySelector('_')?.textContent!.split('') ?? 0 // unsafe, no warning
document.querySelector('_')?.textContent?.split('') ?? 0 // safe

Playground Link:

Related Issues:

Par où commencer

Lisez l'issue en entier, puis le guide de contribution du projet.

Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.

Forkez le dépôt et travaillez sur une branche.

Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par reproduire les exemples dans le TypeScript nightly playground et examinez la gestion des diagnostics du compilateur pour optional chaining et non-null assertions. La modification est terminée lorsque !. après ?. produit un avertissement, qu'aucun quick fix n'est proposé et que les exemples dangereux et sûrs sont couverts par un test de régression.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique

typescript

Domaine

compilers

Type d'issue

Fonctionnalité

Difficulté

4/5

Temps estimé

3-5 jours

Activité

À l'abandon

Clarté

Plutôt claire

Accessibilité débutants

35/100

Description

Guide de contribution

Par où commencer

Piste de recherche

Évaluation

Recevez les nouvelles issues par e-mail