dotnet/roslyn

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

Offen

#45.232 geöffnet am 16.06.2020

Auf GitHub ansehenContributor Guide
 (5 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

Repository-Metriken

Stars
 (20.414 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 6T 17h) (256 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Der Vorschlag ist, einen neuen Analyzer zu erstellen, der die Verwendung von `obj is SomeConst` anstelle von `obj as string == "x"` oder `"x".Equals(obj)` vorschlägt. Der Implementierer sollte sich auf vorhandene Pattern Matching Analyzer in der Roslyn Codebasis (z. B. IDE0019, IDE0020) beziehen und ähnliche Muster befolgen. Der Analyzer sollte alle Konstantentypen behandeln und auch den CS0252 Warnfall abdecken. Ein Fixer sollte bereitgestellt werden. Die Hauptdateien, die zu berücksichtigen sind, befinden sich in den Verzeichnissen `src/Features` oder `src/Analyzers`. Überprüfen Sie den Kommentarthread auf zusätzliche Überlegungen.
Tech Stack
csharp
Domain
developer experiencetooling
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#GitRoslyn
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80