dotnet/roslyn

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

Ouverte

#45 232 ouverte le 16 juin 2020

Voir sur GitHubGuide contributeur
 (5 commentaires) (0 réaction) (0 personne assignée)C# (4 257 forks)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

Métriques du dépôt

Stars
 (20 414 étoiles)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
La suggestion est de créer un nouvel analyseur qui propose d'utiliser `obj is SomeConst` au lieu de `obj as string == "x"` ou `"x".Equals(obj)`. L'implémenteur doit se référer aux analyseurs de correspondance de modèles existants dans la base de code Roslyn (par exemple, IDE0019, IDE0020) et suivre des modèles similaires. L'analyseur doit gérer tous les types de constantes et couvrir également le cas d'avertissement CS0252. Un correcteur doit être fourni. Les principaux fichiers à considérer se trouvent dans les répertoires `src/Features` ou `src/Analyzers`. Consultez le fil de commentaires pour toute considération supplémentaire.
Stack technique
csharp
Domaine
developer experiencetooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#GitRoslyn
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80