dotnet/roslyn

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

開放

#45,232 建立於 2020年6月16日

在 GitHub 查看貢獻者指南
 (5 則留言) (0 個反應) (0 位負責人)C# (4,257 個分叉)batch import
Area-IDEConcept-Continuous Improvementhelp wanted

倉庫指標

星標
 (20,414 顆星)
PR 合併指標
 (平均合併 6天 17小時) (30 天內合併 256 個 PR)

描述

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.

貢獻者指南

研究方向
建議建立一個新的分析器，用於建議使用 `obj is SomeConst` 而非 `obj as string == "x"` 或 `"x".Equals(obj)`。實作者應參考 Roslyn 程式碼庫中現有的模式匹配分析器（例如 IDE0019、IDE0020）並遵循類似模式。該分析器應處理所有常數類型，並涵蓋 CS0252 警告情況。應提供一個修正程式。主要考慮的檔案位於 `src/Features` 或 `src/Analyzers` 目錄中。請查看評論串以獲取任何額外注意事項。
技術棧
csharp
領域
developer experiencetooling
議題類型
功能
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-3 小時
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
活躍
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
清晰
前置要求
C#GitRoslyn
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
80