OOP Design for Card Game Classes

Question

¿Cuál sería el mejor enfoque al diseñar las siguientes clases aplicando patrones de diseño?

• cubierta-addCard, trato, shuffle, getTopCard, removeTopCard, removeAllCards
• mano-addCard, eliminarCarta, getCard, removeAllCards
• DiscardPile-addCard, getTopCard , eliminarTopCard, eliminar TodasCartas
• MeldPile-addCard, removeAllCards

(El MeldPile tiene todos los melds en la mesa.)

Para mí, creo que el getTopCard y removeTopCard son sólo un envoltorio de getCard y removeCard, como simplemente obtiene la posición superior de una tarjeta y luego la pasa a getCard o removeCard.

¿Debo usar composición? patrón de estrategia? o simplemente crea otra clase llamada CardPile y úsala como la clase base de la clase anterior? Realmente aprecio si pudieras proporcionar un código de muestra sobre esto.

Answer 1

Creo que puede lograr lo que quiere con una sola clase de cubierta como la que se muestra a continuación, que es esencialmente una envoltura alrededor de Stack, no veo por qué una determinada baraja/pila/mano no querría la mayoría si no todas las mismas métodos.

class Deck { 
    private Stack<Card> cards = new Stack<Card>(); 

    public Deck() { } 

    public Deck(int numberOfCards) { 
     for (int i=0; i<numberOfCards; i++) { 
      cards.push(CardFactory.createCard(i)); 
     } 
    } 

    private void shuffle() { 
     Collections.shuffle(this.cards); 
    } 

    public void sort() { 
     Collections.sort(this.cards); 
    } 

    public void removeAllCards() { 
     this.cards.removeAllElements(); 
    } 

    public void removeCard(Card c) { 
     int i = this.cards.search(c); 
     this.cards.remove(i);    
    } 

    public Card getCard(Card c) { 
     int i = this.cards.search(c); 
     return this.cards.get(i); 
    } 

    public Card getTopCard() { 
     return this.cards.pop(); 
    } 

    public Card getNthCard(int i) { 
     return this.cards.get(i); 
    } 

    public Card addCard(Card c) { 
     this.cards.push(c); 
    } 

} 

El único problema real que veo es con el método deal() y si esta debe ser la responsabilidad de una cubierta? Personalmente no lo creo, esto me lleva a pensar que tal vez tendría una clase de jugador y una clase de distribuidores que se extiende jugador e implementa la lógica de tratar una cubierta

class Player() { 
    protected String name; 
    protected Deck hand = new Deck(); 

    public void addCard(Card c) { 
     this.hand.addCard(c); 
    } 

    // ..... 
} 

class Dealer() extends Player { 
    private Deck deck; 

    public Dealer(int deckSize) { 
     this.deck = new Deck(deckSize); 
    } 

    public void deal(Player[] players, int numberOfCards) { 
     for (player in players) { 
      for (int i=0; i<numberOfCards; i++) { 
       player.addCard(this.deck.getTopCard()); 
      } 
     } 
    } 

    // ..... 
}