/poker-hands-kata

Poker Hands Kata in Java

Primary LanguageJava

PokerHands Kata List of Poker hands

TODO

  • [R] discuss on why we should pass down the priority to the single rules in the GameRules instead of Rank

    • the actual order of the rules in GameRules is not relevant
    • DON'T put the rules in any relevant order, to avoid giving fake meanings on the relevance of that order
    • what if we want to handle different Poker versions?

  • [?] Are we able to shape our system in order to follow a more Clean Architecture-ish style?

  • [?] ThreeCards and FourCards have unused fields. Why? Do we really need them?

  • [R] try to generalize the use of Hand#selectCoupleOfTwoCardsWithTheSameValue for Hand#selectTwoCardsWithTheSameValue and Hand#selectThreeCardsWithTheSameValue

  • [R] introduce a new type CoupleOfTwoCards instead of a list of TwoCards ?

    • see Optional<List<TwoCards>> vs Optional<CoupleOfTwoCards> (now is an Optional<List<TwoCards>>)

  • [R] access modifier for Cards methods to have a coherent method visibility

  • [?] Hand#allCardsExcept seems to be at a too low level of detail

  • [R] Check asymmetry between Hand#hasAllCardsWithConsecutiveValues and Hand#hasAllCardsWithTheSameSuit

  • [R] Find a better and more descriptive name for the method Card#comesBefore (e.g. isPrevious, preceeds)

  • [R] Extract Ranks as upper level classes

  • [R] When comparing Ranks with the same priority we should compare only the value of the card.

  • [R] extract a GameParser from GameRunner to remove the parsing logic of the player from that class

  • [R] Extract priority == other.priority with a better name (are we mixing the concept of point and rank?)

  • [R] Refactor rules as composition of other rules. The StraightFlushRule is a composition of StraightRule and FlushRule.

  • [R] Remove highestCard field in all rank subclasses (boilerplate on equals())

  • [R] Use a set instead of a list for the rules

  • [R] GameResultFormatter has too many responsibilities (format the player name, format the rank and combine them together)

    • This is making tests dependent each other (e.g. if we change the formatting of the high card we have two tests that fail)

  • [R] Idea to avoid having many boilerplate equals / hashcode / ... : have a list of cards on the Rank superclass?

  • [F] CardParser add support to more values and suites?

  • Add all the possible values for the poker Cards

    • [F] add the formatting of all the remaining figures (K, Q, J)

  • [F] Should we add a validation when

    • Creating a Card (null values and suites should not be allowed) ?
    • Creating a kind of Rank (e.g. I should not be able to create a Pair with two cards with different values, or a Pair with two cards with the same suit)

  • [F] Make sure you cannot create invalid Cards.

  • Write an AT (end-to-end!) that accepts an input like Black: 2H 3D 5S 9C KD White: 2C 3H 4S 8C AH and prints the outcome of the match as a string like White wins. - with high card: Ace

  • [R] Improve the Player tests: maybe using a builder would help in improving the readability of the test

    • like aPlayer().with(highestHand())...
    • add helper assertions to "hide" the pair of symmetric assertions

  • [F] From a single-card poker hand to a 5-cards poker hand

    • full house ranking ("5 cards" world)
    • Add the missing ranks in Rank enumeration

  • [?] Are we sure we are testing all the possible combinations of hands?

    • parametrize test?

  • Building a Card is not easy (new Card(Value.Three, suit) vs some of builder / factory).

    • Waiting to have an application code that needs to create many cards in order to play a game

  • New "Texas Hold'em" rule: 2 cards in your hand + 5 common cards on the table

    • hand1: (2, 4), hand2: (8, 5), table: (3, 5, 6, 8, 8)

  • What changes if we had to expose as an HTTP service?

  • EXTRA: What would you need to change if a new game input should be supported?

  • EXTRA: What would happen if we want to support a different set of rules and priorities? (e.g. full vs flush)

  • EXTRA: find a better way to sort hand's cards Hand#sortedListOf

  • EXTRA: Do some practice with mutation testing

  • What would happen if we try to write unit tests for PlayerParser and the other two parsers?

    • What feedback would we get from those tests?

  • [F] Evaluate a straight with the high rules:

    • an ace can rank either high (as in A K Q J 10, an ace-high straight) or low (as in 5 4 3 2 A, a five-high straight)

Notes

Deck of 52 cards

Each Card has:

  • one of 4 suits (clubs, ....) => (C,D,H,S)
  • a value in (2-10, Jack, Queen, King, Ace) => (2-9,T,J,Q,K,A)

each Poker Hand has 5 cards taken from DECK

  • High Card: lowest ranking -

es: (2C, 3D, 6H, 9S, 5H) => rank: 9

  • Pair: 2 of the 5 cards in the hand have the same value.

es: (2C, 2D, 6H, 9S, 5H)

es:

  • (2C, 2D, 6H, 9S, 5H) => 2
  • (3C, 3D, 6H, 9S, 5H) => 3

es:

  • (3S, 3H, 5H, 6H, 9S) => 3 |=> WIN!

  • (3C, 3D, 4S, 6S, 7C) => 3

  • Two Pair:

es: (2C, 2D, 6H, 6S, 5H)

es

  • (2C, 2D, 6H, 6S, 5H)
  • (3C, 3D, 7H, 7S, 5H) => WIN

es

  • (2C, 2D, 3H, 3S, 7H) => WIN

  • (3C, 3D, 2S, 2S, 5H)

  • Three of a Kind

  • Straight

es: (2C, 3D, 4H, 5S, 6H)

  • Flush

es: (2C, 8C, 4C, 5C, 6C)

  • Full House = Three + Pair

es: (2C, 2D, 6H, 6S, 6D)

  • Four of a kind

es: (2C, 2D, 2H, 2S, 6C)

  • Straight flush:

es: (TS, JS, QS, KS, AS)