“使用范围运算符”建议的修复会产生无效代码

Question

来自 .NET 4 的 ASP.NET 5 MVC 项目使用 Substring 是类似源代码

id = row.Categoryid?.Substring(0, 1) == "$" ? row.Categoryid?.Substring(startIndex: 1) :
                                               row.Categoryid?.Substring(0)

Visual Studio 2019 质检员抛出消息

IDE0057 子串可以简化

在

开始索引：1

应用建议的修复后

使用范围运算符

代码重构为

id = row.Categoryid?.Substring(0, 1) == "$" ? row.Categoryid?.Substring[1..] :
                                               row.Categoryid?.Substring(0)

引发编译错误

错误 CS0021 无法将带有 [] 的索引应用于类型表达式 '方法组'

如何解决这个问题，以便重构创建正确的代码？

Answer 1

忽略任何其他问题并解决手头的问题，您不能在 method 上使用 range，例如 Substring Eg Categoryid.Substring[1..] p>

应该是：

row.Categoryid?[1..] 

或将其推向合乎逻辑的结论：

var id = row.Categoryid?[0] == '$' ? row.Categoryid?[1..] : row.Categoryid;

至于为什么建议这样做完全是另一个问题。它可能是 VS 错误，也可能是 Resharper，具体取决于建议和实现的内容

注意：假设您只是想从字符串的开头删除$，并且永远只有一个（或者如果有多个，您想要它们也被删除）。您可以使用TrimStart 为自己节省一堆可打印字符。

从当前字符串中删除所有指定字符的前导匹配项。

id = row.Categoryid?.TrimStart('$');

更新

因此，进一步分析，这是与 Null-conditional operator 相关的错误。当它被删除时row.Categoryid.Substring(1) 正确替换row.Categoryid[1..]

更新

进一步的更新，这似乎是一个已知的错误，并且已经有针对此问题的拉取请求，计划发布16.9.P1

Fix "IDE0057: Invalid code fix with string.Substring and null-conditional operator" #47377