REFACTORING: rimossi i TODO. Specificato la classe nel comparable

src/main/java/it/unimi/di/sweng/briscola/Player.java

@@ -7,11 +7,7 @@ import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
 
-public class Player implements Iterable<Card>, Comparable {
-
-  // TODO rendere la classe Comparable con altri Player confrontando i punteggi
-  // TODO rendere la classe Iterable sulle carte che ha in mano
-
+public class Player implements Iterable<Card>, Comparable<Player> {
   private @NotNull final String name;
   private @NotNull final List<Card> cards = new ArrayList<>();
   private @NotNull final List<Card> personalDeck = new ArrayList<>();
@@ -92,7 +88,7 @@ public class Player implements Iterable<Card>, Comparable {
   }
 
   @Override
-  public int compareTo(@NotNull Object o) {
-    return getPoints() - ((Player) o).getPoints();
+  public int compareTo(@NotNull Player player) {
+    return getPoints() - player.getPoints();
   }
 }

src/test/java/it/unimi/di/sweng/briscola/PlayerTest.java

@@ -11,22 +11,22 @@ import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.Mockito.mock;
 
 public class PlayerTest {
-
 	@Test
 	void iterablePlayerTest(){
 		Player pl1 = new Player("Gio");
+
 		List<Card> cards = List.of(
 				Card.get(Rank.ASSO,Suit.BASTONI),
 				Card.get(Rank.ASSO,Suit.COPPE),
 				Card.get(Rank.TRE,Suit.BASTONI));
-		for (Card card : cards) {
+
+		for (Card card : cards)
 			pl1.giveCard(card);
-		}
+
 		Iterator<Card> it = pl1.iterator();
-		for (Card card : cards) {
+		for (Card card : cards)
 			assertThat(card).isEqualTo(it.next());
 	}
-	}
 
 	@Test
 	void comparePlayerTest(){