dotnet/roslyn

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

Open

#61.467 geöffnet am 23. Mai 2022

Auf GitHub ansehen
 (5 Kommentare) (5 Reaktionen) (0 zugewiesene Personen)C# (4.257 Forks)batch import
Area-IDEBughelp wanted

Repository-Metriken

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

Beschreibung

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)

Contributor Guide

Research-Richtung
Überprüfen Sie den vorhandenen Codefixer für 'Ausdrucksanweisungen, die implizit Werte ignorieren' im Roslyn Repository (wahrscheinlich unter src/Features/Core/Portable/CodeStyle). Identifizieren Sie, wo der Fixer derzeit die Szenarien ' ' verwerfen und 'nicht verwendete' lokale Variable behandelt. Entwerfen Sie dann eine neue Option oder erweitern Sie die Logik, um Rückgabewerte breiter zu ignorieren, z. B. wenn der Ausdruck keine Nebenwirkungen hat. Testen Sie mit dem bereitgestellten Videoszenario.
Tech Stack
csharp
Domain
developer experience
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.
Halber Tag
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
GitC#
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60