Question

我要做的是将我的播放器列表的名称和分数转储到临时数组中进行排序，以便它们可以显示在记分板上。

出于某种原因，它似乎保持对值的引用而不是值本身。排序后，记分板为每个条目读取“玩家4 - 得分4”。我知道排序方法有效，我以前在另一个游戏中使用过它。

初始值是 player1 - 1 player2 - 2 player3 - 3 player4 - 4

缺少的一件事是每次更新都会将分数转储到分数列表中。我试图重新排列东西以使其发挥作用后把它拿出来。

请帮助我理解为什么会这样。我在这里阅读这篇文章并没有多大帮助。 http://msdn.microsoft.com/en-us/library/0f66670z(v=vs.71).aspx

初始化

  ScoresList = DumpListToArray(PlayerManager.PlayerList);

更新

SortKillScores(ScoresList);

绘制

private static void DrawKillScores(SpriteBatch spriteBatch)
    {
        for (int i = 0; i < 4; i++)
        {
            spriteBatch.DrawString(font, "" + ScoresList[i].Name + " - " + ScoresList[i].KillScore, new Vector2(900, 0 + 20 * i), Color.White, 0f, Vector2.Zero, 0.5f, SpriteEffects.None, 1f);  
        }

    }

方法

private static ScoreHolder[] DumpListToArray(List<Player> tempList)
    {
        ScoreHolder[] tempArray = new ScoreHolder[4];

        for (int i = 0; i < 4; i++)
        {
            tempArray[i] = new ScoreHolder();
            tempArray[i].Name = tempList[i].PlayerName;
            tempArray[i].KillScore = tempList[i].KillScore;
            tempArray[i].TimeScore = tempList[i].TimeScore;
            tempArray[i].PointScore = tempList[i].PointScore;
        }

        return tempArray;
    }

    private static void SortKillScores(ScoreHolder[] array)
    {
        for (int i = 0; i < array.Length; i++)
        {
            for (int k = i; k > 0 && array[k].KillScore > array[k - 1].KillScore; k--)
            {
                int score = array[k].KillScore;
                string name = array[k].Name;
                array[k] = array[k - 1];
                array[k - 1].KillScore = score;
                array[k - 1].Name = name;
            }
        }       
    }

Answer 1

不是尝试将属性从一个单元格复制到另一个单元格，而是交换单元格：

private static void SortKillScores(ScoreHolder[] array)
{
    for (int i = 0; i < array.Length; i++)
    {
        for (int k = i; k > 0 && array[k].KillScore > array[k - 1].KillScore; k--)
        {
            var temp = array[k];
            array[k] = array[k - 1];
            array[k - 1] = temp;
        }
    }       
}

您所拥有的是将对同一项目的引用复制到array[k]，然后在k-1中修改是同一项目的项目！

int score = array[k].KillScore;
string name = array[k].Name;
array[k] = array[k - 1];          // cells k and k-1 are the same item!
array[k - 1].KillScore = score;   // this changes both k and k-1
array[k - 1].Name = name;         // and so does this!

Answer 2

@MattBurland似乎在这里覆盖了你，但要扩展，如果你只想尝试排序，你应该尝试使用OrderBy方法。

private static ScoreHolder[] SortKillScores(ScoreHolder[] array)
{
    return array.OrderBy(e => e.KillScore);
}

通过引用而不是值传递，为什么？

2 个答案: