dotnet/roslyn

Issue with "Avoid expression statements that implicitly ignore value" code style fixer

Open

#61.467 aberto em 23 de mai. de 2022

Ver no GitHub
 (5 comments) (5 reactions) (0 assignees)C# (4.257 forks)batch import
Area-IDEBughelp wanted

Métricas do repositório

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

Description

This issue has been moved from a ticket on Developer Community.

The Avoid expression statements that implicitly ignore value code style setting can be set to Unused local or Discard:

image

However, I think that these settings do not cover all the scenarios, because there are cases in which we simply want to ignore the return values. I have attached a recording to show such a situation, that is also available at https://www.youtube.com/watch?v=ZuSf84-Mw44.

Original Comments

Feedback Bot on 5/22/2022, 07:54 PM:

(private comment, text removed)

Original Solutions

(no solutions)

Guia do colaborador

Direção de pesquisa
Inspecione o corretor de código existente para 'Evite instruções de expressão que ignoram implicitamente o valor' no repositório Roslyn (provavelmente em src/Features/Core/Portable/CodeStyle). Identifique onde o corretor atualmente lida com os cenários de descarte ' ' e variável local 'não utilizada'. Em seguida, projete uma nova opção ou estenda a lógica para permitir ignorar valores de retorno de forma mais ampla, por exemplo, quando a expressão não tem efeitos colaterais. Teste com o cenário de vídeo fornecido.
Pilha de tecnologia
csharp
Domain
developer experience
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.
Meio dia
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
GitC#
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60