2

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

出于某种原因，它似乎保留了对值的引用，而不是值本身。对记分牌进行排序后，每个条目都会显示“玩家 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;
            }
        }       
    }
4

2 回答 2

5

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

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!
于 2013-03-14T17:24:18.943 回答
1

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

private static ScoreHolder[] SortKillScores(ScoreHolder[] array)
{
    return array.OrderBy(e => e.KillScore);
}
于 2013-03-14T17:32:27.363 回答