dotnet/roslyn

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

Open

#61,467 opened on 2022年5月23日

GitHub で見る
 (5 comments) (5 reactions) (0 assignees)C# (4,257 forks)batch import
Area-IDEBughelp wanted

Repository metrics

Stars
 (20,414 stars)
PR merge metrics
 (平均マージ 6d 17h) (30d で 256 merged PRs)

説明

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)

コントリビューターガイド

調査方針
Roslyn リポジトリの「暗黙的に値を無視する式文を避ける」コードフィクサー（おそらく src/Features/Core/Portable/CodeStyle 内）を調査します。フィクサーが現在 ' ' 破棄と「未使用」ローカル変数のシナリオをどのように処理しているかを特定します。その後、新しいオプションを設計するかロジックを拡張して、副作用がない場合など、より広く戻り値を無視できるようにします。提供されたビデオのシナリオでテストします。
技術スタック
csharp
領域
developer experience
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
GitC#
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
60