Pos estava ci fent arreglos varios i m'han obligat a fer commit

source/game/gameplay/scoreboard.hpp

@@ -9,106 +9,60 @@
 class SurfaceAnimatedSprite;  // lines 10-10
 class Surface;                // lines 11-11
 
-struct ScoreboardData {
-        int items;          // Lleva la cuenta de los objetos recogidos
-        int lives;          // Lleva la cuenta de las vidas restantes del jugador
-        int rooms;          // Lleva la cuenta de las habitaciones visitadas
-        bool music;         // Indica si ha de sonar la música durante el juego
-        Uint8 color;        // Color para escribir el texto del marcador
-        Uint32 ini_clock;   // Tiempo inicial para calcular el tiempo transcurrido
-        bool jail_is_open;  // Indica si se puede entrar a la Jail
-
-        // Constructor por defecto
-        ScoreboardData()
-            : items(0),
-              lives(0),
-              rooms(0),
-              music(true),
-              color(0),
-              ini_clock(0),
-              jail_is_open(false) {}
-
-        // Constructor parametrizado
-        ScoreboardData(int items, int lives, int rooms, bool music, Uint8 color, Uint32 ini_clock, bool jail_is_open)
-            : items(items),
-              lives(lives),
-              rooms(rooms),
-              music(music),
-              color(color),
-              ini_clock(ini_clock),
-              jail_is_open(jail_is_open) {}
-};
-
 class Scoreboard {
-    private:
-        struct ClockData {
-                int hours;
-                int minutes;
-                int seconds;
-                std::string separator;
-
-                // Constructor por defecto
-                ClockData()
-                    : hours(0),
-                      minutes(0),
-                      seconds(0),
-                      separator(":") {}
-
-                // Constructor parametrizado
-                ClockData(int h, int m, int s, std::string sep)
-                    : hours(h),
-                      minutes(m),
-                      seconds(s),
-                      separator(std::move(sep)) {}
+    public:
+        // Tipos anidados
+        struct Data {
+                int items{0};              // Lleva la cuenta de los objetos recogidos
+                int lives{0};              // Lleva la cuenta de las vidas restantes del jugador
+                int rooms{0};              // Lleva la cuenta de las habitaciones visitadas
+                bool music{true};          // Indica si ha de sonar la música durante el juego
+                Uint8 color{0};            // Color para escribir el texto del marcador
+                Uint32 ini_clock{0};       // Tiempo inicial para calcular el tiempo transcurrido
+                bool jail_is_open{false};  // Indica si se puede entrar a la Jail
         };
 
-        // Objetos y punteros
-        std::shared_ptr<SurfaceAnimatedSprite> player_sprite_;  // Sprite para mostrar las vidas en el marcador
-        std::shared_ptr<Surface> item_surface_;                 // Surface con los graficos para los elementos del marcador
-        std::shared_ptr<ScoreboardData> data_;                  // Contiene las variables a mostrar en el marcador
-        std::shared_ptr<Surface> surface_;                      // Surface donde dibujar el marcador;
+        // Métodos públicos
+        explicit Scoreboard(std::shared_ptr<Data> data);  // Constructor
+        ~Scoreboard() = default;                          // Destructor
+        void render();                                    // Pinta el objeto en pantalla
+        void update(float delta_time);                    // Actualiza las variables del objeto
+        void setPaused(bool value);                       // Pone el marcador en modo pausa
+        auto getMinutes() -> int;                         // Devuelve la cantidad de minutos de juego transcurridos
+
+    private:
+        // Tipos anidados
+        struct ClockData {
+                int hours{0};                // Horas transcurridas
+                int minutes{0};              // Minutos transcurridos
+                int seconds{0};              // Segundos transcurridos
+                std::string separator{":"};  // Separador para mostrar el tiempo
+        };
 
         // Constantes de tiempo
         static constexpr float ITEMS_COLOR_BLINK_DURATION = 0.333F;  // Duración de cada estado del parpadeo (era 10 frames @ 60fps)
         static constexpr float SPRITE_WALK_CYCLE_DURATION = 0.667F;  // Duración del ciclo de caminar (era 40 frames @ 60fps)
         static constexpr int SPRITE_WALK_FRAMES = 4;                 // Número de frames de animación
 
-        // Variables
-        std::vector<Uint8> color_;        // Vector con los colores del objeto
-        bool is_paused_;                  // Indica si el marcador esta en modo pausa
-        Uint32 paused_time_;              // Milisegundos que ha estado el marcador en pausa
-        Uint32 paused_time_elapsed_;      // Tiempo acumulado en pausa
-        ClockData clock_;                 // Contiene las horas, minutos y segundos transcurridos desde el inicio de la partida
-        Uint8 items_color_;               // Color de la cantidad de items recogidos
-        SDL_FRect surface_dest_;          // Rectangulo donde dibujar la surface del marcador
-        float time_accumulator_ = 0.0F;   // Acumulador de tiempo para animaciones
-        float items_color_timer_ = 0.0F;  // Timer para parpadeo de color de items
+        // Métodos privados
+        auto getTime() -> ClockData;              // Obtiene el tiempo transcurrido de partida
+        void updateItemsColor(float delta_time);  // Actualiza el color de la cantidad de items recogidos
+        void fillTexture();                       // Dibuja los elementos del marcador en la surface
 
-        // Obtiene el tiempo transcurrido de partida
-        auto getTime() -> ClockData;
+        // Objetos y punteros
+        std::shared_ptr<SurfaceAnimatedSprite> player_sprite_;  // Sprite para mostrar las vidas en el marcador
+        std::shared_ptr<Surface> item_surface_;                 // Surface con los graficos para los elementos del marcador
+        std::shared_ptr<Data> data_;                            // Contiene las variables a mostrar en el marcador
+        std::shared_ptr<Surface> surface_;                      // Surface donde dibujar el marcador
 
-        // Actualiza el color de la cantidad de items recogidos
-        void updateItemsColor(float delta_time);
-
-        // Dibuja los elementos del marcador en la surface
-        void fillTexture();
-
-    public:
-        // Constructor
-        explicit Scoreboard(std::shared_ptr<ScoreboardData> data);
-
-        // Destructor
-        ~Scoreboard() = default;
-
-        // Pinta el objeto en pantalla
-        void render();
-
-        // Actualiza las variables del objeto
-        void update(float delta_time);
-
-        // Pone el marcador en modo pausa
-        void setPaused(bool value);
-
-        // Devuelve la cantidad de minutos de juego transcurridos
-        auto getMinutes() -> int;
+        // Variables de estado
+        std::vector<Uint8> color_;       // Vector con los colores del objeto
+        bool is_paused_{false};          // Indica si el marcador esta en modo pausa
+        Uint32 paused_time_{0};          // Milisegundos que ha estado el marcador en pausa
+        Uint32 paused_time_elapsed_{0};  // Tiempo acumulado en pausa
+        ClockData clock_{};              // Contiene las horas, minutos y segundos transcurridos desde el inicio de la partida
+        Uint8 items_color_{0};           // Color de la cantidad de items recogidos
+        SDL_FRect surface_dest_{};       // Rectangulo donde dibujar la surface del marcador
+        float time_accumulator_{0.0F};   // Acumulador de tiempo para animaciones
+        float items_color_timer_{0.0F};  // Timer para parpadeo de color de items
 };