有没有更好的方法来跟踪列表索引,同时迭代列表

Question

目前,该方法使用for()循环和foreach()循环来迭代列表中的每个数组元素,以比较数组元素内的值.如果符合条件,我需要for循环来记录列表的当前索引值,以便我知道之后重建列表的位置.

// update high score list
public void updateScores()
{
    // if the list of scores isnt empty compare score
    if (list.Count() != 0)
    {
        // loop through each array in the list
        for (int i = 0; i <= list.Count(); i++)
        {
            // look into each linedata array in the list
            foreach (string[] ld in list)
            {
                // only look at scores that are the same difficulty
                if (lineData[1] == difficulty)
                {
                    // if score is greater than this score in list
                    if (score > Convert.ToInt32(lineData[0]))
                    {
                        // record which index the score was higher than
                        scoreIndex = i;
                    }
                }
            }
        }

        // if the score is higher than one of the saved highscores remove all lower score down one index
        for (int i = list.Count() - 1; i > scoreIndex; i--)
        {
            // continue
        }
    }
}

这是列表的构建方式:

// load highscores
public bool LoadScores()
{
    // attempt to load the highscore list
    if (File.Exists("highScores"))
    {
        // loop through each line in the file
        while ((line = file.ReadLine()) != null)
        {
            // seperate the score from the difficulty
            lineData = line.Split(',');

            // add each line to the list
            list.Add(lineData);
        }
        return true;
    }
    else{return false;}
}

高分文本文件:

1231,1
1231232,2
4664,2
252,1
475532,1

问题:在迭代列表时是否有更好的方法来跟踪列表索引.

编辑:基本上我将球员得分与高分榜单进行比较.如果分数足够高,则较低的分数将向下移动以允许新分数进入列表

Answer 1

首先,谢尔盖是正确的; 这段代码是大量的错误和不良做法.谢尔盖没有提到的两个是:

• C#中的标准是PascalCase您的方法名称,而不是camelCase它们.
• 不要调用扩展方法Count()上IList.使用该Count属性.

我建议您养成编写相互调用的较小方法的习惯,然后仔细测试每个方法.

正如一些评论者指出的那样,你可以通过维护一个排序列表,取出旧列表和新项目的联合,排序该序列,将其截断为仅包含前n个,将其转回列表,更容易解决这个问题,你完成了

这些都没有回答你的问题

在迭代列表时是否有更好的方法来跟踪列表索引？

我向你提出,这不是一个非常正确的问题.一个更好的问题是:

如何确定列表中与给定条件匹配的第一项的索引？

在静态类中创建扩展方法:

    public static int? FirstIndex<T>(this IEnumerable<T> items, Func<T, bool> predicate) 
    {
      if (items == null) 
        throw new ArgumentNullException("items");
      if (predicate == null) 
        throw new ArgumentNullException("predicate");
      int index = 0;
      foreach (var item in items) 
      {
        if (predicate(item))
          return index;
        index += 1;
      }
      return null;
    }

此扩展方法接受序列和谓词,并返回与谓词匹配的第一个项的索引;如果没有项与谓词匹配,则返回null.请注意,这适用于任何序列,而不仅仅是IList- 不使用[]索引器和Count属性.