优化C#中的字符串比较

Question

我在 asp.net 按钮上有以下代码，单击

if(str == ipaddr1 || ipaddr2 || ipaddr3 || ipaddr4 || ipaddr5 || ipaddr6 || ipaddr7)
// do this
else
//cancel click event

如何优化这段代码？

更新：向大家道歉！我并不是想将它与文字字符串 ipaddr 进行比较。我的意思是将其与 ipaddr1、ipaddr2 所持有的值进行比较，依此类推

Answer 1

替换为：

Regex.IsMatch(str, "^ipaddr[1-7]$")

针对可读性进行优化，不一定是针对性能进行优化。

Answer 2

HashSet 是检查包含的最佳容器：

var ips = new HashSet<string> { "ip1", "ip2", "ip3", "ip4", "ip5" };
if (ips.Contains(input))
{
    // do stuff
}

---

对于任何类型：

var ips = new HashSet<IPHostEntry> { ip1, ip2, ip3, ip4, ip5 };
if (ips.Contains(input))
{
    // do stuff
}

---

Was:

if(str = qry.StartsWith("23.55") || str = qry.StartsWith("xuz") || str = qry.StartsWith("i3") || str = qry.StartsWith("i444") || str = qry.StartsWith("ki5") || str = qry.StartsWith("65fr6")) // do this else // do this

变成：

var arr = new[] { "23.55", "xuz", "i3",  "i444", "ki5", "65fr6") };
if (arr.Any(str => input.StartsWith(str, StringComparison.Ordinal))
{
    // do stuff
}

StringComparison.Ordinal 或 StringComparison.OrdinalIgnoreCase 非常对性能很重要。

Answer 3

供您参考

if(str.Substring(0,6) == "ipaddr" && str[6] >= '1' && str[6] <= '7')

，您的原始代码甚至无法编译。 所以

if(str == "ipaddr1" || "ipaddr2" || "ipaddr3" || "ipaddr4" || "ipaddr5" || "ipaddr6" || "ipaddr7")

需要替换成this才能编译

if(str == "ipaddr1" || str == "ipaddr2" || str == "ipaddr3" || str == "ipaddr4" || str == "ipaddr5" || str == "ipaddr6" || str == "ipaddr7")

原来的代码其实比你想象的还要繁琐。

更新

根据您更新的问题，最好的选择是将字符串变量放入名为的List中，例如ipaddr。然后要查看是否包含字符串 str，只需执行以下操作：

if( ipaddr.Contains( str ) )
{
   //contained in the list
}

Answer 4

我会做类似的事情：

str.Length == 7 && str.StartsWith("ipaddr") && str[6] > '0' && str[6] < '8'

编辑：

更新后，我会做类似的事情：

string[] validStrings = { ipaddr1, ipaddr2, ... };
bool isStrValid = validStrings.Contains(str);

为了获得更好的性能，请考虑使用 HashSet 而不是数组，特别是如果有效字符串列表不会改变。

Answer 5

更具可读性和更高的性能将是：（

switch(str)
{
case "ipaddr1": 
case "ipaddr2":
case "ipaddr3":
case "ipaddr4":
case "ipaddr5":
case "ipaddr6":
case "ipaddr7":
    //do something
    break;
default:
    //do something else
    break;
}

尽管，不可否认的是非常冗长......）

Answer 6

我会做一个

List<string> variables = new List<string> { "ip1","ip2","ip3","ip4","ip5" };

if (variables.Contains(inputstring))
  ...