Kartenspiel in Javascript, Konstruktor Funktion Fehler

Question

Ich erstelle ein grundlegendes Kartenspiel (21) in JavaScript, wo jeder Spieler eine Hand der Karten hat. Die Spieler beginnen mit 2 Karten, aber ich möchte abhängig vom Gesamtwert der Karten neue Karten hinzufügen und anpassen, ob ein As vorhanden ist oder nicht.

Leider bekomme ich diesen Fehler: FATAL ERROR: CALL_AND_RETRY_LAST Zuweisung fehlgeschlagen - JavaScript-Heap nicht genügend Arbeitsspeicher. Ich denke das Problem muss mit der this.newCard Methode, ich kann einfach nicht verstehen, warum es nicht funktioniert.

function Hand() { 
    this.cards = [] 
    this.cardsAtStart = function() { 
     this.cards.push(new DealCards, new DealCards) 
     return this.cards 
    } 
    this.checkValue = function() { 
     let cardsInHand = this.cardsAtStart() 

     let ace = false 
     let value = 0 
     for (let i = 0; i < cardsInHand.length; i++) { 
      if (cardsInHand[i].cardnumber === 'A' && !ace) { 
       ace = true 
       value = value + 13 
      } 
      value = value + cardsInHand[i].cardvalue 
     } 
     if (ace && value > 21) { 
      value = value - 13 
     } 
     console.log(value) 

     return value 
    } 
    this.newCard = function() { 
     let value = this.checkValue() 
     console.log(value) 
     while (value < 15) { 
      this.cards.push(new DealCards) 
     } 
     if (value > 15) { 
      endGame() 
     } 
     console.log(this.cards) 
    } 
} 

Eine Karte Objekt sieht wie folgt aus: { suit: '♦', cardnumber: 'K', cardvalue: 13 }

Answer 1

Sie überprüfen den Wert nicht jede Schleife:

this.newCard = function() { 
    let value = this.checkValue() 
    console.log(value) 
    while (value < 15) { 
     this.cards.push(new DealCards); 
     // check value again here! 
     value = this.checkValue() 
    } 
    if (value > 15) { 
     endGame() 
    } 
    console.log(this.cards) 
} 

Aber auch Checkvalue ändern, um die cardsAtStart zu bewegen draußen aus und in den Konstruktor.

Alternativ können Sie den Wert wie folgt aus:

while (value < 15) { 
    this.cards.push(new DealCards); 
    value = this.cards.reduce((all, current) => all + current.cardvalue, 0), 
} 

Scratch, dass die Ace Logik Verwirrungen mit meinem Minderer. Bleib beim ursprünglichen Plan. Verschieben Sie den Aufruf von cardsAtStart in die Konstruktormethode. Verwenden Sie checkValue, um den Wert zu überprüfen.

Answer 2

Sie werden nie die while Schleife verlassen, weil value wird sich nicht ändern:

while (value < 15) { 
    this.cards.push(new DealCards) 
} 

Sie benötigen die value Variable innerhalb der sich ändern while Schleife so kann der Zustand sein false.

Etwas wie folgt aus:

while (value < 15) { 
    this.cards.push(new DealCards); 
    value++; 
} 

Answer 3

Sie diesen Fehler, weil Sie hier in einer Endlosschleife an dieser Linie stecken,

while (value < 15) { 
    this.cards.push(new DealCards) 
} 

Das Problem ist, der Wert ist nie größer als 15 wenn diese Funktion läuft, wiederholt sie sich einfach.

Wenn Sie diese eine Karte zu behandeln, wenn die Karte weniger als 15 dann etwas anderes tun Sie so etwas tun könnte,

if (value < 15) { 
     this.cards.push(new DealCards) 
} 

So, Ihr Problem zu beheben, müssen Sie den Wert jedes aktualisieren Zeit es Schleifen durch den Aufruf Ihrer this.checkValue() Funktion.

while (value < 15) { 
    this.cards.push(new DealCards); 
    value = this.checkValue(); 
} 

Answer 4

Diese for-Schleife wird nie

while (value < 15) { 
      this.cards.push(new DealCards) 
     } 

Sie beenden müssen Wert anrufen ++ oder ++ Wert zu

machen Ende

while (value < 15) { 
      this.cards.push(new DealCards); 
      value++; 
     }