javascript - 如何使 javascript 显示随机名称而不重复？

Question

我需要能够单击一个按钮，然后以随机顺序出现几个不同的名称，而不会重复任何名称。显示随机名称很简单，但我无法避免重复。这是我所拥有的，但我不明白为什么它不起作用。

<script>

var players = []
players[0] = "Billy";
players[1] = "Alex";
players[2] = "Kevin";
players[3] = "Fred";

function tournament(amount) {
    var a = 0, i = 0, place = 1;
    randomlyselect:
    while (a<amount) {
        var randomplayer = Math.floor(Math.random()*(players.length));
        if (players[randomplayer] === document.getElementById('player1').innerHTML || document.getElementById('player2').innerHTML || document.getElementById('player3').innerHTML) {
            continue randomlyselect;
            } 
        else {
            document.getElementById('player'+place).innerHTML = players[randomplayer];
            place++;
            a++;
        }
    }

}

</script>

<button onClick="tournament(4)">Tournament Results</button>

<p id="player1"></p>
<p id="player2"></p>
<p id="player3"></p>
<p id="player4"></p>

我知道问题出在 if 语句上，但我不知道还有什么方法可以检查名称是否已显示。如果您知道我检查是否已选择某些东西的方法有什么问题或避免重复的更好方法，请告诉我。

Answer 1

var players = []
players[0] = "Billy";
players[1] = "Alex";
players[2] = "Kevin";
players[3] = "Fred";


function tournament(amount) {
  var randomOrder = shuffle(players)
  var count = Math.min(amount, randomOrder.length)
  for (var i = 0; i < count; ++i) {
    document.getElementById("player" + (i + 1)).textContent = randomOrder[i]
  }
}

// http://stackoverflow.com/a/2450976/4339170
function shuffle(array) {
  var currentIndex = array.length,
    temporaryValue, randomIndex;

  // While there remain elements to shuffle...
  while (0 !== currentIndex) {

    // Pick a remaining element...
    randomIndex = Math.floor(Math.random() * currentIndex);
    currentIndex -= 1;

    // And swap it with the current element.
    temporaryValue = array[currentIndex];
    array[currentIndex] = array[randomIndex];
    array[randomIndex] = temporaryValue;
  }

  return array;
}

<button onClick="tournament(4)">Tournament Results</button>

<p id="player1"></p>
<p id="player2"></p>
<p id="player3"></p>
<p id="player4"></p>