以下两种方法哪一种效率更高?(或许还有第三种更好的选择?)

string val = "AStringValue";

if (val.Equals("astringvalue", StringComparison.InvariantCultureIgnoreCase))

OR

if (val.ToLowerCase() == "astringvalue")

?


当前回答

对于这类关于“效率”的问题,我的一般答案几乎总是,哪个版本的代码最易读,是最有效的。

话虽如此,我认为(val.ToLowerCase() == "astringvalue")对大多数人来说一眼就能理解。

我所说的效率并不一定是指代码的执行,而是指所讨论的代码的维护和可读性。

其他回答

tolowercase版本不会更快——它涉及额外的字符串分配 (以后必须收集)等等。

就我个人而言,我会使用

string.Equals(val, "astringvalue",  StringComparison.OrdinalIgnoreCase)

这避免了区域性敏感字符串的所有问题,但结果也避免了区域性敏感字符串的所有问题。只有你自己知道这在你的语境中是否合适。

使用字符串。Equals static方法避免了val为null的任何问题。

你可能还想看看那个已经回答过的问题 c#中字符串比较方法的差异

我敢说最安全的方法是使用静态方法String。Equals可以减少val为null时出现NullReferenceException的可能性。

对于这类关于“效率”的问题,我的一般答案几乎总是,哪个版本的代码最易读,是最有效的。

话虽如此,我认为(val.ToLowerCase() == "astringvalue")对大多数人来说一眼就能理解。

我所说的效率并不一定是指代码的执行,而是指所讨论的代码的维护和可读性。

第一个是正确的,以我之见,也是更有效的,因为第二个“解决方案”实例化了一个新的字符串实例。