如何避免副作用（Javascript）[关闭]

Question

我目前正在构建一个二十一点游戏，我的一个课程叫做“Deck”。这个套牌需要做两件事：

1. 返回一组其他对象可以使用的卡片（这相当于分发这些卡片）
2. 从牌堆中取出这些牌（一旦发牌，就应该将其取出）。

目前，我通过创建两个方法实现了这两个功能：selectCards() 和removeIndexFromDeck。我首先调用selectCards()，它返回选定的卡片，但也调用removeIndexFromDeck()。

这种方法是否违反最佳做法？看起来我的selectCards() 函数既有返回值又有副作用。

如果它确实违反了最佳做法，我将如何更改这些方法，但要确保我仍然能够归还所选卡片并将它们从牌组中移除。

谢谢！

class Deck {
  constructor() {
    this.deck = [];
    ['♦', '♣', '♥', '♠'].forEach(suit => {
      ['A', '2', '3', '4', '5', '6', '7', '8', '9', '10', 'J', 'Q', 'K'].forEach(value => {
        this.deck.push(`${value}${suit}`);
      });
    });
  }

  selectCards(numCards) {
    let selectedCards = [];

    while (numCards > 0) {
      let randIndex = Math.floor(Math.random() * this.deck.length);
      selectedCards.push(this.deck[randIndex]);
      this.removeIndexFromDeck(randIndex);
      numCards -= 1;
    }

    return selectedCards;
  }

  removeIndexFromDeck(index) {
    this.deck.splice(index, 1);
  }
}

Answer 1

确实，您通常应该避免同时改变和返回值，但是好的规则总是有例外的。甚至本机 Java Script 也有例外，Array#pop 可能是最广为人知的一个。然而，许多人会同意pop 非常有用，因为它目前有效。此外，您在脚本中调用的Array#splice 会改变并返回信息。

除非您想放弃 OOP 并转向函数式编程，否则这种模式很好。我只是确保方法的名称尽可能少地对这种双重效果产生怀疑。出于这个原因，我会打电话给selectCards 而不是extractCards 或pullCards。这更强烈地暗示了套牌已经变异。

我还建议实现shuffle 方法，而不是在选择卡片时使用随机索引。如果您支持私有属性，则将deck 数组定义为私有，以便对外界隐藏打乱后的内容。

这就是我的意思（没有私人）：

class Deck {
  constructor() {
    this.deck = Array.from('♦♣♥♠', suit =>
      ['A','2','3','4','5','6','7','8','9','10','J','Q','K'].map(
        value => `${value}${suit}`
      )
    ).flat();
  }

  shuffle() { // mutates the deck, much like Array#sort mutates an array
    let deck = this.deck;
    for (let i = deck.length - 1; i > 0; i--) {
      let j = Math.floor(Math.random() * (i + 1));
      let temp = deck[i];
      deck[i] = deck[j];
      deck[j] = temp;
    }
  }

  extractCards(numCards) {  // Better name. Mutates & returns. 
    // Perform a controlled Array#splice
    if (typeof numCards !== "number" || numCards <= 0) throw "Invalid argument";
    if (this.deck.length < numCards) throw "Deck does not have enough cards for this operation";
    return this.deck.splice(-numCards);
  }
}

let deck = new Deck();
deck.shuffle();
console.log(...deck.extractCards(4));

Answer 2

如果您使用类，这通常意味着存在副作用，因为数据是类的一部分。 Aka 你得到的方法没有返回值（通常意味着它们有副作用），它们会改变类状态。您可以做的最接近的事情是尽可能多地创建方法返回新的牌组值，而不是使用变异/副作用数组方法，然后在您的父方法中执行this.deck = removeIndexFromDeck(index) 来限制您的位置有副作用。

如果你放弃类，转而创建操作数据的函数，并将 Deck 仅视为数据而不是智能类，则函数本身将不会产生副作用。

您还可以在进行突变时使用返回新的Deck 类实例的模式，这意味着Deck 中的数据将始终是不可变的。不过，在这种情况下，我没有看到这样做的好处。