还有其他许多陈述吗?

时间:2013-08-15 22:37:10

标签: c# if-statement switch-statement

我正在阅读一个csv文件,该文件的第一行有列名,行>值为1。我需要获取列名的位置。我能想到的唯一方法是切换或ifs。我在某处读到它,在我的情况下,做ifs更快(更好)。但是该文件有很多列(~120)。只是想知道是否有替代方案。

private static void Get_Position(string line, performance p)
{
    string[] line_split = line.Split(',');
    for (int i = 0; i < line_split.Length; i++)
    {
       if (line_split[i].Contains(@"(0)\% Processor Time"))
        {
            p.percore[0] = i;
        }
        else if (line_split[i].Contains(@"(1)\% Processor Time"))
        {
            p.percore[1] = i;
        }
        else if (line_split[i].Contains("Private Bytes"))
        {}
        else if (line_split[i].contains("DPC")
        {
         }

        //on and on and on with else ifs

2 个答案:

答案 0 :(得分:6)

什么阻止您使用循环?

for (int i = 0; i < line_split.Length; i++)
{
   for(var j = 0; j < 120; j++)
   {
      if(line_split[i].Contains(@"(" + j + ")\% Processor Time"))
      {
         p.percore[j] = i;
      }
   }
...

要保持与if else if相同的功能,您可以在条件内使用break

编辑:编辑现在清楚地表明包含中的字符串没有明确的模式。尽管如此,如果您正在写出120 if / else if语句,那么您应该在某种类型的集合中存储您将要查找的内容。例如,List可以工作。然后访问循环中集合的索引j

...
var listOfSearchItems = new List<string>() { "Private Bytes", "DPC" };

for (int i = 0; i < line_split.Length; i++)
{
   for(var j = 0; j < 120; j++)
   {
      if(line_split[i].Contains(listOfSearchItems[j])
      {
         p.percore[j] = i;
      }
   }
...

答案 1 :(得分:0)

就性能而言,经常使用哪种(切换与否则为......)无关紧要。编译器和jit通常会将代码优化到最佳状态。决定因素是您的决策树是否可以作为计算goto进行提炼。如果你编写一些switch语句,那么你可以看到这个,如果是构造的话,你可以看到很大的if,然后使用像Reflector这样的东西来查看编译的程序集。

通常情况下,写出最清晰,最容易阅读的方式。编译器和jit通常会重新执行提高性能所需的操作。只有当性能至关重要且代码是瓶颈时,您才应该担心重构代码以提高性能(通常以牺牲可读性为代价)。

那就是说,我认为Dan的回答比大转换或者大转换更清晰,更可读。