From 6d0235c04611c18cdaa12085cd1035dd58d25d21 Mon Sep 17 00:00:00 2001
From: GabrieleRadice <265572328+GabrieleRadice@users.noreply.github.com>
Date: Fri, 17 Apr 2026 16:04:43 +0200
Subject: [PATCH] Add: Added toString To Player.java + Respective Tests And
 Fixed BuildingCard.java's toString.

---
 .../polimi/ingsw/gc14/Model/Cards/BuildingCard.java   |  2 +-
 src/main/java/it/polimi/ingsw/gc14/Model/Player.java  | 11 ++++-------
 .../java/it/polimi/ingsw/gc14/Model/PlayerTest.java   | 10 +++++++++-
 3 files changed, 14 insertions(+), 9 deletions(-)

diff --git a/src/main/java/it/polimi/ingsw/gc14/Model/Cards/BuildingCard.java b/src/main/java/it/polimi/ingsw/gc14/Model/Cards/BuildingCard.java
index 04e68b0..9b632f7 100644
--- a/src/main/java/it/polimi/ingsw/gc14/Model/Cards/BuildingCard.java
+++ b/src/main/java/it/polimi/ingsw/gc14/Model/Cards/BuildingCard.java
@@ -89,6 +89,6 @@ public class BuildingCard extends PlayableCard implements Cloneable , BuildingEf
 
     @Override
     public String toString() {
-        return "Era:"+String.valueOf(getEra())+"\nPrice:"+String.valueOf(getPrice())+"\nPrestige:"+String.valueOf(getPrestigeValue()+"\n");
+        return "Era:"+String.valueOf(getEra())+" Price:"+String.valueOf(getPrice())+" Prestige:"+String.valueOf(getPrestigeValue());
     }
 }
diff --git a/src/main/java/it/polimi/ingsw/gc14/Model/Player.java b/src/main/java/it/polimi/ingsw/gc14/Model/Player.java
index 64a3338..345bc04 100644
--- a/src/main/java/it/polimi/ingsw/gc14/Model/Player.java
+++ b/src/main/java/it/polimi/ingsw/gc14/Model/Player.java
@@ -227,19 +227,16 @@ public class Player {
     @Override
     public String toString() {
         String toPrint = "";
-        toPrint = this.getUserName() + ":\n\tFood: " + this.getFoodValue()
+        toPrint = this.getUserName()
+                + ":\n\tFood: " + this.getFoodValue()
                 + "\n\tPrestige: " + this.getPrestigeValue()
                 + "\n\tArtists: " + this.artists.toString()
                 + "\n\tBuilders: " + this.builders.toString()
                 + "\n\tGatherers: " + this.gatherers.toString()
                 + "\n\tShamans: " + this.shamans.toString()
                 + "\n\tHunters: " + this.hunters.toString()
-                + "\n\tBuildings: [\n";
-
-        for (BuildingCard buildingCard : this.buildingCards) {
-            toPrint += buildingCard.toString().indent(8);
-        }
-        toPrint += "\t\t]";
+                + "\n\tBuildings: "
+                + this.buildingCards.toString();
         return toPrint;
     }
     // endregion functions
diff --git a/src/test/java/it/polimi/ingsw/gc14/Model/PlayerTest.java b/src/test/java/it/polimi/ingsw/gc14/Model/PlayerTest.java
index b82579f..c3984d5 100644
--- a/src/test/java/it/polimi/ingsw/gc14/Model/PlayerTest.java
+++ b/src/test/java/it/polimi/ingsw/gc14/Model/PlayerTest.java
@@ -203,6 +203,14 @@ class PlayerTest {
         p.artists.add(new Artist(2));
         p.buildingCards.add(new BuildingCard(1,2, 5));
         p.buildingCards.add(new BuildingCard(2,3, 6));
-        System.out.println(p.toString());
+        assertEquals("test_usr:"+ "\n\t" +
+                "Food: 7" + "\n\t" +
+                "Prestige: 15" + "\n\t" +
+                "Artists: [Era:1 ARTIST, Era:2 ARTIST]" +  "\n\t" + "Builders: []" +  "\n\t" +
+                "Gatherers: []" +  "\n\t" +
+                "Shamans: []" +  "\n\t" +
+                "Hunters: []" +  "\n\t" +
+                "Buildings: [Era:1 Price:2 Prestige:5, Era:2 Price:3 Prestige:6]",
+                p.toString());
     }
 }
\ No newline at end of file