dotnet/roslyn

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

Open

#61 467 ouverte le 23 mai 2022

Voir sur GitHub
 (5 commentaires) (5 réactions) (0 assignés)C# (4 257 forks)batch import
Area-IDEBughelp wanted

Métriques du dépôt

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

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)

Guide contributeur

Direction de recherche
Inspectez le correcteur de code existant pour 'Éviter les instructions d'expression qui ignorent implicitement la valeur' dans le dépôt Roslyn (probablement dans src/Features/Core/Portable/CodeStyle). Identifiez où le correcteur gère actuellement les scénarios de rejet ' ' et de variable locale 'inutilisée'. Concevez ensuite une nouvelle option ou étendez la logique pour permettre d'ignorer les valeurs de retour plus largement, par exemple lorsque l'expression n'a pas d'effets secondaires. Testez avec le scénario vidéo fourni.
Stack technique
csharp
Domaine
developer experience
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.
Une demi journée
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
GitC#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60