made som java doc

src/main/java/edu/ntnu/idatt2001/vetlean/Controller.java

@@ -41,6 +41,10 @@ public class Controller {
         deck.refillDeck();
     }
 
+    /**
+     * Checks the hand for different values.
+     * @param event the click on the button.
+     */
     public void checkCards(ActionEvent event){
 
         try {

src/main/java/edu/ntnu/idatt2001/vetlean/DeckOfCards.java

@@ -7,6 +7,9 @@ import java.util.Random;
 
 public class DeckOfCards {
 
+    /**
+     * the constructor of the deck. makes 52 different cards.
+     */
     private final char[] suit = {'S','H','D','C'};
     private ArrayList<PlayingCard> cards;
     private Random random;
@@ -21,10 +24,19 @@ public class DeckOfCards {
         }
     }
 
+    /**
+     * removes a card from the deck.
+     * @param card PlayingCard
+     */
     public void removeCardFromDeck(PlayingCard card){
         cards.remove(card);
     }
 
+    /**
+     * deals the hand in the game
+     * @param n the amount of cards in the game
+     * @return a hand of cards
+     */
     public List<PlayingCard> dealHand(int n){
        List<PlayingCard> hand = new ArrayList<>();
        PlayingCard cardDealt = null;
@@ -36,6 +48,10 @@ public class DeckOfCards {
         return hand;
     }
 
+    /**
+     * refills the deck after use
+     * @return a  refilled dek of cards
+     */
     public List<PlayingCard> refillDeck(){
         cards.removeAll(cards);
         for (char c : suit) {

src/main/java/edu/ntnu/idatt2001/vetlean/PokerCardHand.java

@@ -11,13 +11,20 @@ public class PokerCardHand {
         this.hand = hand;
     }
 
+    /**
+     * test if the hand has a flush. 5 card og the same suit.
+     * @return a boolean. true if it has a flush
+     */
     public boolean isFlush(){
-        if(hand.stream().filter(c -> c.getSuit() == hand.get(0).getSuit()).collect(Collectors.toList()).size() >= 5){
-            return true;
-        }
-        return false;
+        return hand.stream().collect(Collectors.groupingBy(PlayingCard::getSuit, Collectors.counting()))
+                .values().stream().anyMatch(x -> x >= 5);
+
     }
 
+    /**
+     * test if the hand has a straight. 5 cards increasing.
+     * @return a boolean. true if it has a straight
+     */
     public boolean isStraight(){
          hand.sort(Comparator.comparingInt(PlayingCard::getFace));
          for (int i = 0; i<hand.size()-1;i++){
@@ -28,11 +35,19 @@ public class PokerCardHand {
          return true;
     }
 
+    /**
+     * finds the total value og the hand
+     * @return an int, the total value
+     */
     public int totalValue(){
         int sum = hand.stream().mapToInt(PlayingCard::getFace).sum();
         return sum;
     }
 
+    /**
+     * finds all the harts og the hand
+     * @return string if all the harts
+     */
     public String getAllHarts(){
         StringBuilder stringBuilder = new StringBuilder();
 
@@ -42,6 +57,10 @@ public class PokerCardHand {
         return stringBuilder.toString();
     }
 
+    /**
+     * makes a string of the hand
+     * @return a string of the hand
+     */
     public String getStringRepresentationHand(){
 
         return this.hand.stream()
@@ -49,10 +68,17 @@ public class PokerCardHand {
                 .collect(Collectors.joining(" "));
     }
 
+    /**
+     * finds out if the hand has a queen
+     * @return a boolean, true if it has a queen
+     */
     public boolean containsQueenOfSpades(){
         return hand.stream().anyMatch(playingCard -> playingCard.getFace() == 12 && playingCard.getSuit()=='S');
     }
 
+    /**
+     * @return the hand
+     */
     public List<PlayingCard> getHand() {
         return hand;
     }

src/test/java/edu/ntnu/idatt2001/vetlean/CardHandsTest.java

@@ -21,7 +21,7 @@ public class CardHandsTest {
     }
     private PokerCardHand testDataFlushWithEightCards(){
         ArrayList<PlayingCard> sameSuit = new ArrayList<>();
-        sameSuit.add( new PlayingCard('H',1));
+        sameSuit.add( new PlayingCard('S',1));
         sameSuit.add( new PlayingCard('H',2));
         sameSuit.add( new PlayingCard('H',3));
         sameSuit.add( new PlayingCard('H',4));