dotnet/roslyn

Suggestion to use `obj is SomeConst` instead of safe cast or direct Equals(object) call

Aperta

#45.232 aperta il 16 giu 2020

Vedi su GitHubGuida contributor
 (5 commenti) (0 reazioni) (0 assegnatari)C# (4257 fork)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

Metriche repository

Star
 (20.414 stelle)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

obj as string == "x" and "x".Equals(obj) would be replaced with obj is "x".

This could apply to all constant types, so obj as bool? == SomeBoolConstant and SomeBoolConstant.Equals(obj) would be replaced with obj is SomeBoolConstant.

When CS0252 "Possible unintended reference comparison; to get a value comparison, cast the left hand side to type 'string'" is reported, this fix could offer to change obj == "x" to obj is "x".

Happy to implement.

Guida contributor

Direzione di ricerca
Il suggerimento è di creare un nuovo analizzatore che proponga l'uso di `obj is SomeConst` invece di `obj as string == "x"` o `"x".Equals(obj)`. L'implementatore dovrebbe fare riferimento agli analizzatori di pattern matching esistenti nella codebase di Roslyn (ad es. IDE0019, IDE0020) e seguire modelli simili. L'analizzatore dovrebbe gestire tutti i tipi di costante e coprire anche il caso di avviso CS0252. Dovrebbe essere fornito un correttore. I file principali da considerare si trovano nelle directory `src/Features` o `src/Analyzers`. Controllare il thread dei commenti per eventuali considerazioni aggiuntive.
Tech stack
csharp
Dominio
developer experiencetooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#GitRoslyn
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80