dotnet/roslyn

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

Aberta

#45.232 aberto em 16 de jun. de 2020

Ver no GitHubGuia do colaborador
 (5 comentários) (0 reação) (0 responsável)C# (4.257 forks)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

Métricas do repositório

Stars
 (20.414 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 17h) (256 fundiu PRs em 30d)

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.

Guia do colaborador

Direção de pesquisa
A sugestão é criar um novo analisador que sugira usar `obj is SomeConst` em vez de `obj as string == "x"` ou `"x".Equals(obj)`. O implementador deve consultar os analisadores de correspondência de padrões existentes na base de código do Roslyn (por exemplo, IDE0019, IDE0020) e seguir padrões semelhantes. O analisador deve lidar com todos os tipos de constantes e também cobrir o caso de aviso CS0252. Um fixador deve ser fornecido. Os principais arquivos a considerar estão nos diretórios `src/Features` ou `src/Analyzers`. Verifique o tópico de comentários para quaisquer considerações adicionais.
Pilha de tecnologia
csharp
Domain
developer experiencetooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#GitRoslyn
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80