Pasaeta de include-what-you-use

Afegits destructors virtuals en les classes Sprite
Acabat amb cppcheck
2024-10-13 21:58:36 +02:00 · 2024-10-13 21:23:15 +02:00 · 2024-10-13 21:00:33 +02:00 · 2024-10-13 19:39:43 +02:00 · 2024-10-13 19:26:27 +02:00 · 2024-10-13 14:25:05 +02:00
60 changed files with 787 additions and 885 deletions
--- a/8
+++ b/8
@@ -25,8 +25,8 @@ INCLUDES:= -I$(DIR_SOURCES)
 ifeq ($(OS),Windows_NT)
    FixPath = $(subst /,\,$1)
 	SOURCES := source/*.cpp
-	CXXFLAGS:= -std=c++14 -Wall -Os -ffunction-sections -fdata-sections -Wl,--gc-sections -static-libstdc++ -Wl,-subsystem,windows
-	CXXFLAGS_DEBUG:= -std=c++14 -Wall -g
+	CXXFLAGS:= -std=c++20 -Wall -Os -ffunction-sections -fdata-sections -Wl,--gc-sections -static-libstdc++ -Wl,-subsystem,windows
+	CXXFLAGS_DEBUG:= -std=c++20 -Wall -g
 	LDFLAGS := -lmingw32 -lws2_32 -lSDL2main -lSDL2 -lopengl32
 	RM = del /Q
 	MKD:= mkdir
@@ -34,8 +34,8 @@ else
    FixPath = $1
 	SOURCES := $(shell find $(DIR_SOURCES) -name '*.cpp')
 	SOURCES := source/*.cpp
-	CXXFLAGS:= -std=c++14 -Wall -Os -ffunction-sections -fdata-sections
-	CXXFLAGS_DEBUG:= -std=c++14 -Wall -g
+	CXXFLAGS:= -std=c++20 -Wall -Os -ffunction-sections -fdata-sections
+	CXXFLAGS_DEBUG:= -std=c++20 -Wall -g
 	LDFLAGS := -lSDL2
 	RM = rm -f
 	MKD:= mkdir -p
--- a/linux-utils/check-includes.sh
+++ b/linux-utils/check-includes.sh
@@ -4,7 +4,7 @@ SOURCEPATH=../source/

 for i in "$SOURCEPATH"/*.cpp
 do
-    include-what-you-use -D DEBUG -D VERBOSE -std=c++14 -Wall "$i"
+    include-what-you-use -D DEBUG -D VERBOSE -std=c++20 -Wall "$i"
    read -p "Presiona cualquier tecla para continuar..."
    clear
 done
--- a/source/animated_sprite.cpp
+++ b/source/animated_sprite.cpp
@@ -1,6 +1,8 @@
 #include "animated_sprite.h"
+#include <algorithm> // for copy
 #include <fstream>	 // for basic_ostream, operator<<, basic_istream, basic...
 #include <iostream>	 // for cout
+#include <iterator>	 // for back_insert_iterator, back_inserter
 #include <sstream>	 // for basic_stringstream
 #include "texture.h" // for Texture

@@ -37,7 +39,7 @@ AnimatedFile loadAnimationFromFile(std::shared_ptr<Texture> texture, std::string
 				buffer.counter = 0;
 				buffer.current_frame = 0;
 				buffer.completed = false;
-				buffer.name = "";
+				buffer.name.clear();
 				buffer.speed = 5;
 				buffer.loop = 0;
 				buffer.frames.clear();
@@ -159,18 +161,15 @@ AnimatedFile loadAnimationFromFile(std::shared_ptr<Texture> texture, std::string

 // Constructor
 AnimatedSprite::AnimatedSprite(std::shared_ptr<Texture> texture, const std::string &file, std::vector<std::string> *buffer)
-	: MovingSprite(texture)
+	: MovingSprite(texture),
+	  current_animation_(0)
 {
 	// Carga las animaciones
-	if (file != "")
+	if (!file.empty())
 	{
 		AnimatedFile as = loadAnimationFromFile(texture, file);

 		// Copia los datos de las animaciones
-		/*for (auto animation : as.animations)
-		{
-			animations_.push_back(animation);
-		}*/
 		std::copy(as.animations.begin(), as.animations.end(), std::back_inserter(animations_));
 	}

@@ -178,18 +177,13 @@ AnimatedSprite::AnimatedSprite(std::shared_ptr<Texture> texture, const std::stri
 	{
 		loadFromVector(buffer);
 	}
-
-	// Inicializa variables
-	current_animation_ = 0;
 }

 // Constructor
 AnimatedSprite::AnimatedSprite(const AnimatedFile *animation)
-	: MovingSprite(animation->texture)
+	: MovingSprite(animation->texture),
+	  current_animation_(0)
 {
-	// Inicializa variables
-	current_animation_ = 0;
-
 	// Copia los datos de las animaciones
 	std::copy(animation->animations.begin(), animation->animations.end(), std::back_inserter(animations_));
 }
@@ -366,7 +360,7 @@ bool AnimatedSprite::loadFromVector(std::vector<std::string> *source)
 			buffer.counter = 0;
 			buffer.current_frame = 0;
 			buffer.completed = false;
-			buffer.name = "";
+			buffer.name.clear();
 			buffer.speed = 5;
 			buffer.loop = 0;
 			buffer.frames.clear();
--- a/source/animated_sprite.h
+++ b/source/animated_sprite.h
@@ -2,11 +2,11 @@

 #include <SDL2/SDL_rect.h>   // for SDL_Rect
 #include <SDL2/SDL_stdinc.h> // for Uint8
-#include <string>            // for string, basic_string
+#include <memory>            // for shared_ptr
+#include <string>            // for string
 #include <vector>            // for vector
 #include "moving_sprite.h"   // for MovingSprite
-#include "texture.h"
-#include <memory>
+class Texture;

 struct Animation
 {
@@ -37,11 +37,11 @@ protected:

 public:
    // Constructor
-    explicit AnimatedSprite(std::shared_ptr<Texture> texture = nullptr, const std::string &file = "", std::vector<std::string> *buffer = nullptr);
+    explicit AnimatedSprite(std::shared_ptr<Texture> texture = nullptr, const std::string &file = std::string(), std::vector<std::string> *buffer = nullptr);
    explicit AnimatedSprite(const AnimatedFile *animation);

    // Destructor
-    ~AnimatedSprite();
+    virtual ~AnimatedSprite();

    // Calcula el frame correspondiente a la animación actual
    void animate();
--- a/source/background.cpp
+++ b/source/background.cpp
@@ -1,10 +1,12 @@
 #include "background.h"
-#include <SDL2/SDL_blendmode.h> // for SDL_BLENDMODE_BLEND
-#include <SDL2/SDL_pixels.h>    // for SDL_PIXELFORMAT_RGBA8888
-#include <algorithm>            // for max, min
-#include <string>               // for basic_string
-#include "asset.h"              // for Asset
-#include "param.h"              // for param
+#include <SDL2/SDL_blendmode.h>  // for SDL_BLENDMODE_BLEND
+#include <SDL2/SDL_pixels.h>     // for SDL_PIXELFORMAT_RGBA8888
+#include <algorithm>             // for clamp, max
+#include "asset.h"               // for Asset
+#include "moving_sprite.h"       // for MovingSprite
+#include "param.h"               // for param
+#include "sprite.h"              // for Sprite
+#include "texture.h"             // for Texture

 // Constructor
 Background::Background(SDL_Renderer *renderer)
@@ -103,18 +105,19 @@ void Background::update()
    // Incrementa el contador
    counter_++;

+    // Compone todos los elementos del fondo en la textura
    fillCanvas();
 }

 // Dibuja el gradiente de fondo
 void Background::renderGradient()
 {
-    // Dibuja el gradiente 2
+    // Dibuja el gradiente de detras
    gradients_texture_->setAlpha(255);
    gradient_sprite_->setSpriteClip(gradient_rect_[(gradient_number_ + 1) % 4]);
    gradient_sprite_->render();

-    // Dibuja el gradiente 1 con una opacidad cada vez menor
+    // Dibuja el gradiente de delante con una opacidad cada vez menor
    gradients_texture_->setAlpha(alpha_);
    gradient_sprite_->setSpriteClip(gradient_rect_[gradient_number_]);
    gradient_sprite_->render();
@@ -123,36 +126,36 @@ void Background::renderGradient()
 // Dibuja las nubes de arriba
 void Background::renderTopClouds()
 {
-    // Dibuja el primer conjunto de nubes
+    // Dibuja el primer conjunto de nubes, las de detras
    top_clouds_texture_->setAlpha(255);
    top_clouds_sprite_a_->setSpriteClip(top_clouds_rect_[(gradient_number_ + 1) % 4]);
-    top_clouds_sprite_a_->render();
    top_clouds_sprite_b_->setSpriteClip(top_clouds_rect_[(gradient_number_ + 1) % 4]);
+    top_clouds_sprite_a_->render();
    top_clouds_sprite_b_->render();

-    // Dibuja el segundo conjunto de nubes
+    // Dibuja el segundo conjunto de nubes, las de delante
    top_clouds_texture_->setAlpha(alpha_);
    top_clouds_sprite_a_->setSpriteClip(top_clouds_rect_[gradient_number_]);
-    top_clouds_sprite_a_->render();
    top_clouds_sprite_b_->setSpriteClip(top_clouds_rect_[gradient_number_]);
+    top_clouds_sprite_a_->render();
    top_clouds_sprite_b_->render();
 }

 // Dibuja las nubes de abajo
 void Background::renderBottomClouds()
 {
-    // Dibuja el primer conjunto de nubes
+    // Dibuja el primer conjunto de nubes, las de detras
    bottom_clouds_texture_->setAlpha(255);
    bottom_clouds_sprite_a_->setSpriteClip(bottom_clouds_rect_[(gradient_number_ + 1) % 4]);
-    bottom_clouds_sprite_a_->render();
    bottom_clouds_sprite_b_->setSpriteClip(bottom_clouds_rect_[(gradient_number_ + 1) % 4]);
+    bottom_clouds_sprite_a_->render();
    bottom_clouds_sprite_b_->render();

-    // Dibuja el segundo conjunto de nubes
+    // Dibuja el segundo conjunto de nubes, las de delante
    bottom_clouds_texture_->setAlpha(alpha_);
    bottom_clouds_sprite_a_->setSpriteClip(bottom_clouds_rect_[gradient_number_]);
-    bottom_clouds_sprite_a_->render();
    bottom_clouds_sprite_b_->setSpriteClip(bottom_clouds_rect_[gradient_number_]);
+    bottom_clouds_sprite_a_->render();
    bottom_clouds_sprite_b_->render();
 }

@@ -217,9 +220,7 @@ void Background::setGradientNumber(int value)
 // Ajusta el valor de la variable
 void Background::setTransition(float value)
 {
-    value = std::min(value, 1.0f);
-    value = std::max(value, 0.0f);
-    transition_ = value;
+    transition_ = std::clamp(value, 0.0f, 1.0f);
 }

 // Establece la posición del objeto
@@ -265,13 +266,10 @@ void Background::setColor(Color color)
 void Background::setAlpha(int alpha)
 {
    // Evita que se asignen valores fuera de rango
-    alpha_ = std::min(alpha, 255);
-    alpha_ = std::max(alpha, 0);
+    alpha_ = std::clamp(alpha, 0, 255);

-    // Guarda el valor actual
+    // Guarda el valor actual y establece el nuevo valor
    alpha_color_text_temp_ = alpha_color_text_;
-
-    // Establece el nuevo valor
    alpha_color_text_ = alpha_;
 }

--- a/source/background.h
+++ b/source/background.h
@@ -1,12 +1,12 @@
 #pragma once

-#include <SDL2/SDL_rect.h>   // for SDL_Rect
-#include <SDL2/SDL_render.h> // for SDL_Renderer, SDL_Texture
-#include "utils.h"           // for Color
-#include "moving_sprite.h"
-#include "sprite.h"
-#include "texture.h"
-#include <memory>
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_render.h>  // for SDL_Renderer, SDL_Texture
+#include <memory>             // for unique_ptr, shared_ptr
+#include "utils.h"            // for Color
+class MovingSprite;
+class Sprite;
+class Texture;

 /*
    Esta clase es la encargada de dibujar el fondo que aparece durante la sección
--- a/source/balloon.cpp
+++ b/source/balloon.cpp
@@ -1,5 +1,5 @@
 #include "balloon.h"
-#include <math.h>			 // for abs
+#include <cmath>			 // for abs
 #include "animated_sprite.h" // for AnimatedSprite
 #include "moving_sprite.h"	 // for MovingSprite
 #include "param.h"			 // for param
--- a/source/balloon.h
+++ b/source/balloon.h
@@ -1,12 +1,12 @@
 #pragma once

 #include <SDL2/SDL_stdinc.h> // for Uint8, Uint16, Uint32
+#include <memory>			 // for shared_ptr, unique_ptr
 #include <string>			 // for string
 #include <vector>			 // for vector
-#include <memory>
-#include "utils.h" // for Circle
-#include "animated_sprite.h"
-#include "texture.h"
+#include "animated_sprite.h" // for AnimatedSprite
+#include "utils.h"			 // for Circle
+class Texture;

 // Cantidad de elementos del vector con los valores de la deformación del globo al rebotar
 constexpr int MAX_BOUNCE = 10;
--- a/source/balloon_formations.cpp
+++ b/source/balloon_formations.cpp
@@ -428,7 +428,6 @@ void BalloonFormations::initBalloonFormations()
 	// #24 - Treinta enemigos BALLOON1. Del centro hacia los extremos. Juntos. Simetricos
 	j = 24;
 	balloon_formation_[j].number_of_balloons = 30;
-	inc_x = 0;
 	inc_time = 5;
 	for (int i = 0; i < balloon_formation_[j].number_of_balloons; i++)
 	{
--- a/source/bullet.cpp
+++ b/source/bullet.cpp
@@ -1,9 +1,9 @@
 #include "bullet.h"
-#include "param.h"  // for param
-#include "sprite.h" // for Sprite
-#include <memory>   // for std::unique_ptr
+#include <memory>    // for unique_ptr, make_unique, shared_ptr
+#include "param.h"   // for param
+#include "sprite.h"  // for Sprite
+class Texture;

-// Constantes evaluables en tiempo de compilación
 constexpr int BULLET_WIDTH = 12;
 constexpr int BULLET_HEIGHT = 12;
 constexpr int BULLET_VELY = -3;
@@ -23,10 +23,9 @@ Bullet::Bullet(int x, int y, BulletType kind, bool powered_up, int owner, SDL_Re
      owner_(owner),
      play_area_(play_area)
 {
-    vel_x_ = (kind_ == BulletType::LEFT)    ? BULLET_VELX_LEFT
-             : (kind_ == BulletType::RIGHT) ? BULLET_VELX_RIGHT
-                                            : 0;
-    
+    vel_x_ = (kind_ == BulletType::LEFT) ? BULLET_VELX_LEFT : (kind_ == BulletType::RIGHT) ? BULLET_VELX_RIGHT
+                                                                                           : 0;
+
    auto sprite_offset = powered_up ? 3 : 0;
    auto kind_index = static_cast<int>(kind);
    sprite_->setSpriteClip((kind_index + sprite_offset) * width_, 0, sprite_->getWidth(), sprite_->getHeight());
@@ -67,12 +66,12 @@ BulletMoveStatus Bullet::move()

 bool Bullet::isEnabled() const
 {
-    return kind_ != BulletType::NULL_TYPE;
+    return kind_ != BulletType::NONE;
 }

 void Bullet::disable()
 {
-    kind_ = BulletType::NULL_TYPE;
+    kind_ = BulletType::NONE;
 }

 int Bullet::getPosX() const
--- a/source/bullet.h
+++ b/source/bullet.h
@@ -1,11 +1,11 @@
 #pragma once

-#include <SDL2/SDL_rect.h>	 // for SDL_Rect
-#include <SDL2/SDL_stdinc.h> // for Uint8
-#include <memory>			 // for unique_ptr
-#include "sprite.h"			 // for Sprite
-#include "utils.h"			 // for Circle
-#include "texture.h"		 // lines 9-9
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_stdinc.h>  // for Uint8
+#include <memory>             // for shared_ptr, unique_ptr
+#include "sprite.h"           // for Sprite
+#include "utils.h"            // for Circle
+class Texture;

 // Enumeración para los diferentes tipos de balas
 enum class BulletType
@@ -13,7 +13,7 @@ enum class BulletType
 	UP,
 	LEFT,
 	RIGHT,
-	NULL_TYPE
+	NONE
 };

 // Enumeración para los resultados del movimiento de la bala
--- a/source/define_buttons.cpp
+++ b/source/define_buttons.cpp
@@ -1,10 +1,12 @@
 #include "define_buttons.h"
-#include "lang.h"    // for getText
-#include "options.h" // for options
-#include "param.h"   // for param
-#include "section.h" // for name, SectionName, options, SectionOptions
-#include "text.h"    // for Text
-#include "utils.h"   // for OptionsController, Options, Param, ParamGame
+#include <utility>    // for move
+#include "input.h"    // for Input, InputType
+#include "lang.h"     // for getText
+#include "options.h"  // for options
+#include "param.h"    // for param
+#include "section.h"  // for Name, Options, name, options
+#include "text.h"     // for Text
+#include "utils.h"    // for OptionsController, Options, Param, ParamGame

 // Constructor
 DefineButtons::DefineButtons(std::unique_ptr<Text> text_)
--- a/source/define_buttons.h
+++ b/source/define_buttons.h
@@ -1,12 +1,13 @@
 #pragma once

-#include <SDL2/SDL_events.h>         // for SDL_ControllerButtonEvent
-#include <SDL2/SDL_gamecontroller.h> // for SDL_GameControllerButton
-#include <string>                    // for string, basic_string
-#include <vector>                    // for vector
-#include "input.h"                   // for inputs_e
-#include "text.h"
-#include <memory>
+#include <SDL2/SDL_events.h>          // for SDL_ControllerButtonEvent
+#include <SDL2/SDL_gamecontroller.h>  // for SDL_GameControllerButton
+#include <memory>                     // for shared_ptr, unique_ptr
+#include <string>                     // for string
+#include <vector>                     // for vector
+class Input;
+class Text;
+enum class InputType : int;

 struct DefineButtonsButton
 {
--- a/source/director.cpp
+++ b/source/director.cpp
@@ -5,43 +5,44 @@
 #include <SDL2/SDL_error.h>			 // for SDL_GetError
 #include <SDL2/SDL_gamecontroller.h> // for SDL_CONTROLLER_BUTTON_B, SDL_CO...
 #include <SDL2/SDL_hints.h>			 // for SDL_SetHint, SDL_HINT_RENDER_DR...
-#include <SDL2/SDL_scancode.h>		 // for SDL_SCANCODE_DOWN, SDL_SCANCODE_E
-#include <SDL2/SDL_stdinc.h>		 // for Uint32
+#include <SDL2/SDL_scancode.h>		 // for SDL_SCANCODE_0, SDL_SCANCODE_DOWN
+#include <SDL2/SDL_stdinc.h>		 // for SDL_bool, Uint32
 #include <SDL2/SDL_timer.h>			 // for SDL_GetTicks
-#include <errno.h>					 // for errno, EACCES, EEXIST, ENAMETOO...
-#include <stdio.h>					 // for printf, perror, size_t
+#include <errno.h>					 // for errno, EEXIST, EACCES, ENAMETOO...
+#include <stdio.h>					 // for printf, perror
 #include <string.h>					 // for strcmp
-#include <sys/stat.h>				 // for stat, mkdir, S_IRWXU
+#include <sys/stat.h>				 // for mkdir, stat, S_IRWXU
 #include <unistd.h>					 // for getuid
 #include <cstdlib>					 // for exit, EXIT_FAILURE, rand, srand
 #include <iostream>					 // for basic_ostream, operator<<, cout
-#include <string>					 // for basic_string, operator+, allocator
-#include "asset.h"					 // for Asset, assetType
+#include <memory>					 // for make_unique, unique_ptr
+#include <string>					 // for operator+, allocator, char_traits
+#include "asset.h"					 // for Asset, AssetType
 #include "dbgtxt.h"					 // for dbg_init
 #include "game.h"					 // for Game, GAME_MODE_DEMO_OFF, GAME_...
-#include "global_inputs.h"
-#include "hiscore_table.h"		  // for HiScoreTable
-#include "input.h"				  // for inputs_e, Input
-#include "instructions.h"		  // for Instructions
-#include "intro.h"				  // for Intro
-#include "jail_audio.h"			  // for JA_DeleteMusic, JA_DeleteSound
-#include "logo.h"				  // for Logo
-#include "manage_hiscore_table.h" // for ManageHiScoreTable
-#include "on_screen_help.h"		  // for OnScreenHelp
-#include "options.h"			  // for options, loadOptionsFile, saveO...
-#include "param.h"				  // for param, loadParamsFromFile
-#include "screen.h"				  // for Screen
-#include "section.h"			  // for SectionName, name, options, SectionOptions
-#include "title.h"				  // for Title
-#include "utils.h"				  // for MusicFile, SoundFile, opt...
-#include <memory>
+#include "global_inputs.h"			 // for init
+#include "hiscore_table.h"			 // for HiScoreTable
+#include "input.h"					 // for Input, InputType
+#include "instructions.h"			 // for Instructions
+#include "intro.h"					 // for Intro
+#include "jail_audio.h"				 // for JA_LoadMusic, JA_LoadSound, JA_...
+#include "lang.h"					 // for Code, loadFromFile
+#include "logo.h"					 // for Logo
+#include "manage_hiscore_table.h"	 // for ManageHiScoreTable
+#include "on_screen_help.h"			 // for OnScreenHelp
+#include "options.h"				 // for options, loadOptionsFile, saveO...
+#include "param.h"					 // for param, loadParamsFromFile
+#include "screen.h"					 // for Screen
+#include "section.h"				 // for Name, name, Options, options
+#include "title.h"					 // for Title
+#include "utils.h"					 // for MusicFile, SoundFile, Options

 #ifndef _WIN32
 #include <pwd.h> // for getpwuid, passwd
 #endif

 // Constructor
-Director::Director(int argc, char *argv[])
+Director::Director(int argc, const char *argv[])
 {
 #ifdef RECORDING
 	section::name = section::Name::GAME;
@@ -160,8 +161,8 @@ void Director::initInput()
 	Input::get()->bindKey(InputType::RESET, SDL_SCANCODE_F10);

 	// Asigna botones a inputs
-	const int numGamePads = Input::get()->getNumControllers();
-	for (int i = 0; i < numGamePads; ++i)
+	const int num_gamepads = Input::get()->getNumControllers();
+	for (int i = 0; i < num_gamepads; ++i)
 	{
 		// Mando - Movimiento del jugador
 		Input::get()->bindGameControllerButton(i, InputType::UP, SDL_CONTROLLER_BUTTON_DPAD_UP);
@@ -188,7 +189,7 @@ void Director::initInput()
 	}

 	// Mapea las asignaciones a los botones desde el archivo de configuración, si se da el caso
-	for (int i = 0; i < numGamePads; ++i)
+	for (int i = 0; i < num_gamepads; ++i)
 		for (int index = 0; index < (int)options.controller.size(); ++index)
 			if (Input::get()->getControllerName(i) == options.controller[index].name)
 			{
@@ -200,7 +201,7 @@ void Director::initInput()
 			}

 	// Asigna botones a inputs desde otros inputs
-	for (int i = 0; i < numGamePads; ++i)
+	for (int i = 0; i < num_gamepads; ++i)
 	{
 		Input::get()->bindGameControllerButton(i, InputType::EXIT, InputType::START);
 		Input::get()->bindGameControllerButton(i, InputType::RESET, InputType::FIRE_CENTER);
@@ -213,7 +214,7 @@ void Director::initInput()
 	}

 	// Guarda las asignaciones de botones en las opciones
-	for (int i = 0; i < numGamePads; ++i)
+	for (int i = 0; i < num_gamepads; ++i)
 	{
 		options.controller[i].name = Input::get()->getControllerName(i);
 		for (int j = 0; j < (int)options.controller[i].inputs.size(); ++j)
@@ -340,7 +341,7 @@ bool Director::setFileList()
 #ifdef MACOS_BUNDLE
 	const std::string prefix = "/../Resources";
 #else
-	const std::string prefix = "";
+	const std::string prefix;
 #endif

 	// Ficheros de configuración
@@ -484,15 +485,15 @@ void Director::loadParams(const std::string &file_path)
 }

 // Comprueba los parametros del programa
-void Director::checkProgramArguments(int argc, char *argv[])
+void Director::checkProgramArguments(int argc, const char *argv[])
 {
 	// Establece la ruta del programa
 	executable_path_ = argv[0];

 	// Valores por defecto
-	param_file_argument_ = "";
+	param_file_argument_.clear();

-	// Comprueba el resto de parametros
+	// Comprueba el resto de parámetros
 	for (int i = 1; i < argc; ++i)
 	{
 		if (strcmp(argv[i], "--320x240") == 0)
@@ -566,17 +567,15 @@ void Director::createSystemFolder(const std::string &folder)
 void Director::loadSounds()
 {
 	// Obtiene la lista con las rutas a los ficheros de sonidos
-	std::vector<std::string> list = Asset::get()->getListByType(AssetType::SOUND);
+	auto list = Asset::get()->getListByType(AssetType::SOUND);
 	sounds_.clear();

 	for (const auto &l : list)
 	{
-		const size_t lastIndex = l.find_last_of("/") + 1;
-		const std::string name = l.substr(lastIndex, std::string::npos);
-		SoundFile temp;
-		temp.name = name;					 // Añade el nombre del fichero
-		temp.file = JA_LoadSound(l.c_str()); // Carga el fichero de audio
-		sounds_.push_back(temp);
+		auto last_index = l.find_last_of('/') + 1;
+		auto name = l.substr(last_index);
+
+		sounds_.emplace_back(SoundFile{name, JA_LoadSound(l.c_str())});
 	}
 }

@@ -584,78 +583,69 @@ void Director::loadSounds()
 void Director::loadMusics()
 {
 	// Obtiene la lista con las rutas a los ficheros musicales
-	std::vector<std::string> list = Asset::get()->getListByType(AssetType::MUSIC);
+	auto list = Asset::get()->getListByType(AssetType::MUSIC);
 	musics_.clear();

 	for (const auto &l : list)
 	{
-		const size_t lastIndex = l.find_last_of("/") + 1;
-		const std::string name = l.substr(lastIndex, std::string::npos);
-		MusicFile temp;
-		temp.name = name;					 // Añade el nombre del fichero
-		temp.file = JA_LoadMusic(l.c_str()); // Carga el fichero de audio
-		musics_.push_back(temp);
+		auto last_index = l.find_last_of('/') + 1;
+		auto name = l.substr(last_index);
+
+		musics_.emplace_back(MusicFile{name, JA_LoadMusic(l.c_str())});
 	}
 }

 // Ejecuta la sección con el logo
 void Director::runLogo()
 {
-	auto logo = new Logo();
+	auto logo = std::make_unique<Logo>();
 	logo->run();
-	delete logo;
 }

 // Ejecuta la sección con la secuencia de introducción
 void Director::runIntro()
 {
-	auto intro = new Intro(getMusic(musics_, "intro.ogg"));
+	auto intro = std::make_unique<Intro>(getMusic(musics_, "intro.ogg"));
 	intro->run();
-	delete intro;
 }

-// Ejecuta la sección con el titulo del juego
+// Ejecuta la sección con el título del juego
 void Director::runTitle()
 {
-	auto title = new Title(getMusic(musics_, "title.ogg"));
+	auto title = std::make_unique<Title>(getMusic(musics_, "title.ogg"));
 	title->run();
-	delete title;
 }

 // Ejecuta la sección donde se juega al juego
 void Director::runGame()
 {
-	const auto playerID = section::options == section::Options::GAME_PLAY_1P ? 1 : 2;
-	constexpr auto currentStage = 0;
-	auto game = new Game(playerID, currentStage, GAME_MODE_DEMO_OFF, getMusic(musics_, "playing.ogg"));
+	const auto player_id = section::options == section::Options::GAME_PLAY_1P ? 1 : 2;
+	constexpr auto current_stage = 0;
+	auto game = std::make_unique<Game>(player_id, current_stage, GAME_MODE_DEMO_OFF, getMusic(musics_, "playing.ogg"));
 	game->run();
-	delete game;
 }

 // Ejecuta la sección donde se muestran las instrucciones
 void Director::runInstructions()
 {
-	auto instructions = new Instructions(getMusic(musics_, "title.ogg"));
+	auto instructions = std::make_unique<Instructions>(getMusic(musics_, "title.ogg"));
 	instructions->run();
-	delete instructions;
 }

 // Ejecuta la sección donde se muestra la tabla de puntuaciones
 void Director::runHiScoreTable()
 {
-	auto hiScoreTable = new HiScoreTable(getMusic(musics_, "title.ogg"));
-	hiScoreTable->run();
-	delete hiScoreTable;
+	auto hi_score_table = std::make_unique<HiScoreTable>(getMusic(musics_, "title.ogg"));
+	hi_score_table->run();
 }

 // Ejecuta el juego en modo demo
 void Director::runDemoGame()
 {
-	const auto playerID = (rand() % 2) + 1;
-	constexpr auto currentStage = 0;
-	auto game = new Game(playerID, currentStage, GAME_MODE_DEMO_ON, nullptr);
+	const auto player_id = (rand() % 2) + 1;
+	constexpr auto current_stage = 0;
+	auto game = std::make_unique<Game>(player_id, current_stage, GAME_MODE_DEMO_ON, nullptr);
 	game->run();
-	delete game;
 }

 int Director::run()
@@ -702,8 +692,8 @@ int Director::run()
 		}
 	}

-	const int returnCode = section::options == section::Options::QUIT_NORMAL ? 0 : 1;
-	return returnCode;
+	const int return_code = section::options == section::Options::QUIT_NORMAL ? 0 : 1;
+	return return_code;
 }

 // Obtiene una fichero a partir de un lang::Code
--- a/source/director.h
+++ b/source/director.h
@@ -1,11 +1,12 @@
 #pragma once

-#include <SDL2/SDL_render.h> // for SDL_Renderer
-#include <SDL2/SDL_video.h>	 // for SDL_Window
-#include <string>			 // for string, basic_string
-#include <vector>			 // for vector
-#include "lang.h"			 // for lang_e
-#include "utils.h"			 // for MusicFile, SoundFile
+#include <SDL2/SDL_render.h>  // for SDL_Renderer
+#include <SDL2/SDL_video.h>   // for SDL_Window
+#include <string>             // for string
+#include <vector>             // for vector
+namespace lang { enum class Code : int; }
+struct MusicFile;
+struct SoundFile;

 // Textos
 constexpr char WINDOW_CAPTION[] = "Coffee Crisis Arcade Edition";
@@ -46,7 +47,7 @@ private:
 	void loadMusics();

 	// Comprueba los parametros del programa
-	void checkProgramArguments(int argc, char *argv[]);
+	void checkProgramArguments(int argc, const char *argv[]);

 	// Crea la carpeta del sistema donde guardar datos
 	void createSystemFolder(const std::string &folder);
@@ -77,7 +78,7 @@ private:

 public:
 	// Constructor
-	Director(int argc, char *argv[]);
+	Director(int argc, const char *argv[]);

 	// Destructor
 	~Director();
--- a/source/explosions.cpp
+++ b/source/explosions.cpp
@@ -1,6 +1,7 @@
 #include "explosions.h"
+#include <utility>           // for move
 #include "animated_sprite.h" // for AnimatedSprite
-class Texture;
+class Texture;               // lines 3-3

 // Constructor
 Explosions::Explosions()
--- a/source/explosions.h
+++ b/source/explosions.h
@@ -1,10 +1,10 @@
 #pragma once

-#include <string> // for string
-#include <vector> // for vector
-#include "animated_sprite.h"
-#include <memory>
-#include "texture.h"
+#include <memory>  // for shared_ptr, unique_ptr
+#include <string>  // for string
+#include <vector>  // for vector
+class AnimatedSprite;
+class Texture;

 struct ExplosionTexture
 {
--- a/source/game.cpp
+++ b/source/game.cpp
@@ -1,38 +1,41 @@
 #include "game.h"
 #include <SDL2/SDL_blendmode.h>	  // for SDL_BLENDMODE_BLEND
-#include <SDL2/SDL_keycode.h>	  // for SDLK_1, SDLK_2, SDLK_3, SDLK_h
+#include <SDL2/SDL_events.h>	  // for SDL_PollEvent, SDL_Event, SDL_KEYDOWN
+#include <SDL2/SDL_keycode.h>	  // for SDLK_1, SDLK_2, SDLK_3, SDLK_4
 #include <SDL2/SDL_pixels.h>	  // for SDL_PIXELFORMAT_RGBA8888
 #include <SDL2/SDL_rwops.h>		  // for SDL_RWFromFile, SDL_RWclose, SDL_R...
 #include <SDL2/SDL_timer.h>		  // for SDL_GetTicks
 #include <SDL2/SDL_video.h>		  // for SDL_WINDOWEVENT_FOCUS_GAINED, SDL_...
 #include <stdlib.h>				  // for rand
-#include <algorithm>			  // for min
-#include <fstream>				  // for basic_ifstream
-#include <iostream>				  // for char_traits, basic_istream, ifstream
+#include <algorithm>			  // for min, remove_if
+#include <fstream>				  // for basic_ostream, operator<<, basic_i...
+#include <iostream>				  // for cout
+#include <numeric>				  // for accumulate
+#include <utility>				  // for move
 #include "asset.h"				  // for Asset
 #include "background.h"			  // for Background
-#include "balloon.h"			  // for Balloon, BALLOON_SPEED_1, BALLOON_...
-#include "bullet.h"				  // for Bullet, BulletType::LEFT, BulletType::RIGHT
-#include "balloon_formations.h"	  // for Stage, EnemyFormations, enemyIni...
+#include "balloon.h"			  // for Balloon, BALLOON_SCORE_1, BALLOON_...
+#include "balloon_formations.h"	  // for Stage, BalloonFormationParams, Bal...
+#include "bullet.h"				  // for Bullet, BulletType, BulletMoveStatus
 #include "explosions.h"			  // for Explosions
-#include "fade.h"				  // for Fade, FadeType::RANDOM_SQUARE, FADE_VEN...
-#include "global_inputs.h"		  // for globalInputs::check
-#include "input.h"				  // for inputs_e, Input, INPUT_DO_NOT_ALLO...
+#include "fade.h"				  // for Fade, FadeType
+#include "global_inputs.h"		  // for check
+#include "input.h"				  // for InputType, Input, INPUT_DO_NOT_ALL...
 #include "item.h"				  // for Item, ITEM_COFFEE_MACHINE, ITEM_CLOCK
 #include "jail_audio.h"			  // for JA_PlaySound, JA_DeleteSound, JA_L...
 #include "lang.h"				  // for getText
 #include "manage_hiscore_table.h" // for ManageHiScoreTable
 #include "options.h"			  // for options
 #include "param.h"				  // for param
-#include "player.h"				  // for Player, PlayerStatus::PLAYING, PLA...
-#include "scoreboard.h"			  // for Scoreboard, scoreboard_modes_e
+#include "player.h"				  // for Player, PlayerStatus
+#include "scoreboard.h"			  // for Scoreboard, ScoreboardMode, SCOREB...
 #include "screen.h"				  // for Screen
+#include "section.h"			  // for Name, name, Options, options
 #include "smart_sprite.h"		  // for SmartSprite
 #include "text.h"				  // for Text, TEXT_CENTER
 #include "texture.h"			  // for Texture
-#include "dbgtxt.h"
-struct JA_Music_t;
-struct JA_Sound_t;
+struct JA_Music_t;				  // lines 35-35
+struct JA_Sound_t;				  // lines 36-36

 // Constructor
 Game::Game(int player_id, int current_stage, bool demo, JA_Music_t *music)
@@ -133,13 +136,13 @@ void Game::init(int player_id)
 	players_.push_back(std::move(player2));

 	// Obtiene mediante "playerID" el jugador que va a empezar jugar
-	auto player = getPlayer(player_id);
+	auto main_player = getPlayer(player_id);

 	// Cambia el estado del jugador seleccionado
-	player->setStatusPlaying(PlayerStatus::PLAYING);
+	main_player->setStatusPlaying(PlayerStatus::PLAYING);

 	// Como es el principio del juego, empieza sin inmunidad
-	player->setInvulnerable(false);
+	main_player->setInvulnerable(false);

 	// Variables relacionadas con la dificultad
 	switch (difficulty_)
@@ -243,9 +246,9 @@ void Game::init(int player_id)
 		// Activa o no al otro jugador
 		if (rand() % 2 == 0)
 		{
-			const auto otherPlayer = player_id == 1 ? 2 : 1;
-			auto player = getPlayer(otherPlayer);
-			player->setStatusPlaying(PlayerStatus::PLAYING);
+			const auto other_player_id = player_id == 1 ? 2 : 1;
+			auto other_player = getPlayer(other_player_id);
+			other_player->setStatusPlaying(PlayerStatus::PLAYING);
 		}

 		for (auto &player : players_)
@@ -707,7 +710,7 @@ bool Game::saveDemoFile(const std::string &file_path)
 #endif // RECORDING

 // Crea una formación de enemigos
-void Game::deployEnemyFormation()
+void Game::deployBalloonFormation()
 {
 	// Solo despliega una formación enemiga si ha pasado cierto tiempo desde la última
 	if (balloon_deploy_counter_ == 0)
@@ -1587,14 +1590,8 @@ void Game::killPlayer(std::shared_ptr<Player> &player)
 // Calcula y establece el valor de amenaza en funcion de los globos activos
 void Game::evaluateAndSetMenace()
 {
-	menace_current_ = 0;
-	for (auto &balloon : balloons_)
-	{
-		if (balloon->isEnabled())
-		{
-			menace_current_ += balloon->getMenace();
-		}
-	}
+	menace_current_ = std::accumulate(balloons_.begin(), balloons_.end(), 0, [](int sum, const auto &balloon)
+									  { return sum + (balloon->isEnabled() ? balloon->getMenace() : 0); });
 }

 // Obtiene el valor de la variable
@@ -1645,11 +1642,11 @@ void Game::updateTimeStoppedCounter()
 }

 // Actualiza la variable enemyDeployCounter
-void Game::updateEnemyDeployCounter()
+void Game::updateBalloonDeployCounter()
 {
 	if (balloon_deploy_counter_ > 0)
 	{
-		balloon_deploy_counter_--;
+		--balloon_deploy_counter_;
 	}
 }

@@ -1750,7 +1747,7 @@ void Game::update()

 			// Actualiza los contadores de estado y efectos
 			updateTimeStoppedCounter();
-			updateEnemyDeployCounter();
+			updateBalloonDeployCounter();

 			// Actualiza el ayudante
 			updateHelper();
@@ -1872,7 +1869,7 @@ void Game::updateMenace()
 	if (menace_current_ < menace_threshold_)
 	{
 		// Crea una formación de enemigos
-		deployEnemyFormation();
+		deployBalloonFormation();

 		// Recalcula el nivel de amenaza con el nuevo globo
 		evaluateAndSetMenace();
@@ -2169,7 +2166,7 @@ void Game::renderMessages()
 	if (stage_bitmap_counter_ < STAGE_COUNTER)
 	{
 		const auto stage_number = balloon_formations_->getStage(current_stage_).number;
-		std::string text = "";
+		std::string text;

 		if (stage_number == 10)
 		{ // Ultima fase
@@ -2248,28 +2245,14 @@ void Game::run()
 // Indica si se puede crear una powerball
 bool Game::canPowerBallBeCreated()
 {
-	if ((!power_ball_enabled_) && (calculateScreenPower() > POWERBALL_SCREENPOWER_MINIMUM) && (power_ball_counter_ == 0))
-	{
-		return true;
-	}
-
-	return false;
+	return (!power_ball_enabled_) && (calculateScreenPower() > POWERBALL_SCREENPOWER_MINIMUM) && (power_ball_counter_ == 0);
 }

 // Calcula el poder actual de los globos en pantalla
 int Game::calculateScreenPower()
 {
-	auto power = 0;
-
-	for (auto &balloon : balloons_)
-	{
-		if (balloon->isEnabled())
-		{
-			power += balloon->getPower();
-		}
-	}
-
-	return power;
+	return std::accumulate(balloons_.begin(), balloons_.end(), 0, [](int sum, const auto &balloon)
+						   { return sum + (balloon->isEnabled() ? balloon->getPower() : 0); });
 }

 // Inicializa las variables que contienen puntos de ruta para mover objetos
@@ -2354,7 +2337,7 @@ void Game::updateHelper()
 	// Solo ofrece ayuda cuando la amenaza es elevada
 	if (menace_current_ > 15)
 	{
-		for (auto &player : players_)
+		for (const auto &player : players_)
 		{
 			helper_.need_coffee = (player->getCoffees() == 0);
 			helper_.need_coffee_machine = (!player->isPowerUp());
@@ -2370,7 +2353,7 @@ void Game::updateHelper()
 bool Game::allPlayersAreWaitingOrGameOver()
 {
 	auto success = true;
-	for (auto &player : players_)
+	for (const auto &player : players_)
 	{
 		success &= player->isWaiting() || player->isGameOver();
 	}
@@ -2382,7 +2365,7 @@ bool Game::allPlayersAreWaitingOrGameOver()
 bool Game::allPlayersAreGameOver()
 {
 	auto success = true;
-	for (auto &player : players_)
+	for (const auto &player : players_)
 	{
 		success &= player->isGameOver();
 	}
@@ -2394,7 +2377,7 @@ bool Game::allPlayersAreGameOver()
 bool Game::allPlayersAreNotPlaying()
 {
 	auto success = true;
-	for (auto &player : players_)
+	for (const auto &player : players_)
 	{
 		success &= !player->isPlaying();
 	}
@@ -2566,7 +2549,7 @@ void Game::reloadTextures()
 // Actualiza el marcador
 void Game::updateScoreboard()
 {
-	for (auto &player : players_)
+	for (const auto &player : players_)
 	{
 		scoreboard_->setScore(player->getScoreBoardPanel(), player->getScore());
 		scoreboard_->setMult(player->getScoreBoardPanel(), player->getScoreMultiplier());
@@ -2590,7 +2573,7 @@ void Game::pause(bool value)
 }

 // Añade una puntuación a la tabla de records
-void Game::addScoreToScoreBoard(std::string name, int score)
+void Game::addScoreToScoreBoard(const std::string &name, int score)
 {
 	const auto entry = (HiScoreEntry){trim(name), score};
 	auto manager = std::make_unique<ManageHiScoreTable>(&options.game.hi_score_table);
@@ -2635,7 +2618,7 @@ void Game::checkPlayersStatusPlaying()
 // Obtiene un jugador a partir de su "id"
 std::shared_ptr<Player> Game::getPlayer(int id)
 {
-	for (auto &player : players_)
+	for (const auto &player : players_)
 	{
 		if (player->getId() == id)
 		{
--- a/source/game.h
+++ b/source/game.h
@@ -1,31 +1,29 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
 #include <SDL2/SDL_render.h> // for SDL_Renderer, SDL_Texture
 #include <SDL2/SDL_stdinc.h> // for Uint32
+#include <memory>			 // for shared_ptr, unique_ptr
 #include <string>			 // for string
 #include <vector>			 // for vector
-#include "section.h"		 // for SectionOptions
+#include "balloon.h"		 // for Balloon
+#include "player.h"			 // for Player
 #include "utils.h"			 // for DemoKeys, Color, HiScoreEntry
-#include <memory>
-#include "asset.h"				// lines 11-11
-#include "background.h"			// lines 12-12
-#include "balloon.h"			// lines 13-13
-#include "bullet.h"				// lines 14-14
-#include "balloon_formations.h" // lines 15-15
-#include "explosions.h"			// lines 16-16
-#include "fade.h"				// lines 17-17
-#include "input.h"				// lines 18-18
-#include "item.h"				// lines 19-19
-#include "player.h"				// lines 20-20
-#include "scoreboard.h"			// lines 21-21
-#include "screen.h"				// lines 22-22
-#include "smart_sprite.h"		// lines 23-23
-#include "text.h"				// lines 24-24
-#include "texture.h"			// lines 24-24
-enum class BulletType;
-struct JA_Music_t; // lines 26-26
-struct JA_Sound_t; // lines 27-27
+class Asset;
+class Background;
+class BalloonFormations;
+class Bullet;
+class Explosions;
+class Fade;
+class Input;
+class Item;
+class Scoreboard;
+class Screen;
+class SmartSprite;
+class Text;
+class Texture;
+enum class BulletType; // lines 26-26
+struct JA_Music_t;	   // lines 27-27
+struct JA_Sound_t;	   // lines 28-28

 // Modo demo
 constexpr bool GAME_MODE_DEMO_OFF = false;
@@ -233,7 +231,7 @@ private:
 	bool saveDemoFile(const std::string &file_path);
 #endif
 	// Crea una formación de enemigos
-	void deployEnemyFormation();
+	void deployBalloonFormation();

 	// Aumenta el poder de la fase
 	void increaseStageCurrentPower(int power);
@@ -362,7 +360,7 @@ private:
 	void incTimeStoppedCounter(int value);

 	// Actualiza la variable EnemyDeployCounter
-	void updateEnemyDeployCounter();
+	void updateBalloonDeployCounter();

 	// Actualiza y comprueba el valor de la variable
 	void updateTimeStoppedCounter();
@@ -431,7 +429,7 @@ private:
 	void checkMusicStatus();

 	// Añade una puntuación a la tabla de records
-	void addScoreToScoreBoard(std::string name, int score);
+	void addScoreToScoreBoard(const std::string &name, int score);

 	// Saca del estado de GAME OVER al jugador si el otro está activo
 	void checkAndUpdatePlayerStatus(int active_player_index, int inactive_player_index);
--- a/source/game_logo.cpp
+++ b/source/game_logo.cpp
@@ -1,6 +1,6 @@
 #include "game_logo.h"
+#include <SDL2/SDL_render.h> // for SDL_FLIP_HORIZONTAL
 #include <algorithm>         // for max
-#include <string>            // for basic_string
 #include "animated_sprite.h" // for AnimatedSprite
 #include "asset.h"           // for Asset
 #include "jail_audio.h"      // for JA_DeleteSound, JA_LoadSound, JA_PlaySound
@@ -17,17 +17,17 @@ GameLogo::GameLogo(int x, int y)
      coffee_texture_(std::make_shared<Texture>(Screen::get()->getRenderer(), Asset::get()->get("title_coffee.png"))),
      crisis_texture_(std::make_shared<Texture>(Screen::get()->getRenderer(), Asset::get()->get("title_crisis.png"))),
      arcade_edition_texture_(std::make_shared<Texture>(Screen::get()->getRenderer(), Asset::get()->get("title_arcade_edition.png"))),
-      
+
      dust_left_sprite_(std::make_unique<AnimatedSprite>(dust_texture_, Asset::get()->get("title_dust.ani"))),
      dust_right_sprite_(std::make_unique<AnimatedSprite>(dust_texture_, Asset::get()->get("title_dust.ani"))),
-      
+
      coffee_sprite_(std::make_unique<SmartSprite>(coffee_texture_)),
      crisis_sprite_(std::make_unique<SmartSprite>(crisis_texture_)),
-      
+
      arcade_edition_sprite_(std::make_unique<Sprite>((param.game.width - arcade_edition_texture_->getWidth()) / 2, param.title.arcade_edition_position, arcade_edition_texture_->getWidth(), arcade_edition_texture_->getHeight(), arcade_edition_texture_)),
-      
+
      crash_sound_(JA_LoadSound(Asset::get()->get("title.wav").c_str())),
-      
+
      x_(x),
      y_(y)
 {
--- a/source/game_logo.h
+++ b/source/game_logo.h
@@ -1,13 +1,11 @@
 #pragma once

-#include <SDL2/SDL_render.h> // for SDL_Renderer
-#include <memory>
-#include "texture.h"
-#include "animated_sprite.h"
-#include "smart_sprite.h"
-#include "sprite.h"
-
-struct JA_Sound_t;
+#include <memory> // for unique_ptr, shared_ptr
+class AnimatedSprite;
+class SmartSprite;
+class Sprite;
+class Texture;
+struct JA_Sound_t; // lines 10-10

 // Clase GameLogo
 class GameLogo
--- a/source/hiscore_table.cpp
+++ b/source/hiscore_table.cpp
@@ -1,5 +1,6 @@
 #include "hiscore_table.h"
 #include <SDL2/SDL_blendmode.h> // for SDL_BLENDMODE_BLEND
+#include <SDL2/SDL_events.h>	// for SDL_PollEvent, SDL_Event, SDL_QUIT
 #include <SDL2/SDL_pixels.h>	// for SDL_PIXELFORMAT_RGBA8888
 #include <SDL2/SDL_timer.h>		// for SDL_GetTicks
 #include <SDL2/SDL_video.h>		// for SDL_WINDOWEVENT_SIZE_CHANGED
@@ -7,15 +8,17 @@
 #include <vector>				// for vector
 #include "asset.h"				// for Asset
 #include "background.h"			// for Background
-#include "global_inputs.h"		// for globalInputs::check
+#include "fade.h"				// for Fade, FadeMode, FadeType
+#include "global_inputs.h"		// for check
 #include "input.h"				// for Input
 #include "jail_audio.h"			// for JA_GetMusicState, JA_Music_state
 #include "lang.h"				// for getText
 #include "options.h"			// for options
 #include "param.h"				// for param
 #include "screen.h"				// for Screen
-#include "text.h"				// for Text, TEXT_CENTER, TEXT_SHADOW, TEXT_COLOR
-#include "utils.h"				// for Param, ParamGame, HiScoreEntry
+#include "section.h"			// for Name, name, Options, options
+#include "text.h"				// for Text, TEXT_CENTER, TEXT_SHADOW, TEXT...
+#include "utils.h"				// for Param, ParamGame, Color, HiScoreEntry

 // Constructor
 HiScoreTable::HiScoreTable(JA_Music_t *music)
@@ -129,7 +132,7 @@ void HiScoreTable::fillTexture()
 		const auto score = format(options.game.hi_score_table[i].score);
 		const auto score_lenght = score.size();
 		const auto num_dots = 25 - name_lenght - score_lenght;
-		std::string dots = "";
+		std::string dots;
 		for (int j = 0; j < (int)num_dots; ++j)
 		{
 			dots = dots + ".";
@@ -255,7 +258,7 @@ std::string HiScoreTable::format(int number)
 	const std::string score = std::to_string(number);

 	auto index = (int)score.size() - 1;
-	std::string result = "";
+	std::string result;
 	auto i = 0;
 	while (index >= 0)
 	{
--- a/source/hiscore_table.h
+++ b/source/hiscore_table.h
@@ -1,17 +1,15 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
 #include <SDL2/SDL_rect.h>	 // for SDL_Rect
 #include <SDL2/SDL_render.h> // for SDL_Renderer, SDL_Texture
-#include <SDL2/SDL_stdinc.h> // for Uint16, Uint32
-#include <memory>
-#include <string> // for string
-#include "fade.h"
-#include "section.h" // for SectionOptions
-#include "background.h"
-#include "text.h"
-
-struct JA_Music_t;
+#include <SDL2/SDL_stdinc.h> // for Uint16, Uint32, Uint8
+#include <memory>			 // for unique_ptr
+#include <string>			 // for string
+class Background;			 // lines 8-8
+class Fade;					 // lines 9-9
+class Text;					 // lines 10-10
+enum class FadeMode : Uint8; // lines 11-11
+struct JA_Music_t;			 // lines 12-12

 /*
 	Esta clase gestiona un estado del programa. Se encarga de mostrar la tabla con las puntuaciones
--- a/source/instructions.cpp
+++ b/source/instructions.cpp
@@ -1,25 +1,26 @@
 #include "instructions.h"
 #include <SDL2/SDL_blendmode.h> // for SDL_BLENDMODE_BLEND
+#include <SDL2/SDL_events.h>	// for SDL_PollEvent, SDL_Event, SDL_QUIT
 #include <SDL2/SDL_pixels.h>	// for SDL_PIXELFORMAT_RGBA8888
 #include <SDL2/SDL_timer.h>		// for SDL_GetTicks
 #include <SDL2/SDL_video.h>		// for SDL_WINDOWEVENT_SIZE_CHANGED
 #include <algorithm>			// for max
-#include <string>				// for basic_string
+#include <utility>				// for move
 #include "asset.h"				// for Asset
-#include "fade.h"				// for Fade, FadeType::FULLSCREEN, FadeMode::IN
-#include "global_inputs.h"		// for globalInputs::check
+#include "fade.h"				// for Fade, FadeMode, FadeType
+#include "global_inputs.h"		// for check
 #include "input.h"				// for Input
 #include "jail_audio.h"			// for JA_GetMusicState, JA_Music_state
 #include "lang.h"				// for getText
 #include "param.h"				// for param
 #include "screen.h"				// for Screen
-#include "section.h"			// for name, SectionName, options, SectionOptions
+#include "section.h"			// for Name, name, Options, options
 #include "sprite.h"				// for Sprite
-#include "text.h"				// for Text, TEXT_CENTER, TEXT_COLOR, TEXT_SHADOW
+#include "text.h"				// for Text, TEXT_CENTER, TEXT_COLOR, TEXT_...
 #include "texture.h"			// for Texture
 #include "tiled_bg.h"			// for Tiledbg, TILED_MODE_STATIC
-#include "utils.h"				// for Param, ParamGame, Color, shdwT...
-struct JA_Music_t;
+#include "utils.h"				// for Param, ParamGame, Color, shdw_txt_color
+struct JA_Music_t;				// lines 22-22

 // Constructor
 Instructions::Instructions(JA_Music_t *music)
--- a/source/instructions.h
+++ b/source/instructions.h
@@ -1,17 +1,16 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
-#include <SDL2/SDL_rect.h>	 // for SDL_Point, SDL_Rect
-#include <SDL2/SDL_render.h> // for SDL_Texture, SDL_Renderer
-#include <SDL2/SDL_stdinc.h> // for Uint32
-#include <vector>			 // for vector
-#include <memory>
-#include "fade.h"
-#include "sprite.h"
-#include "text.h"
-#include "texture.h"
-#include "tiled_bg.h"
-struct JA_Music_t;
+#include <SDL2/SDL_rect.h>    // for SDL_Point, SDL_Rect
+#include <SDL2/SDL_render.h>  // for SDL_Texture, SDL_Renderer
+#include <SDL2/SDL_stdinc.h>  // for Uint32
+#include <memory>             // for unique_ptr, shared_ptr
+#include <vector>             // for vector
+class Fade;
+class Sprite;
+class Text;
+class Texture;
+class Tiledbg;
+struct JA_Music_t;  // lines 14-14

 /*
 	Esta clase gestiona un estado del programa. Se encarga de poner en pantalla
--- a/source/intro.cpp
+++ b/source/intro.cpp
@@ -1,22 +1,22 @@
 #include "intro.h"
-#include <SDL2/SDL_render.h> // for SDL_Renderer
-#include <SDL2/SDL_timer.h>	 // for SDL_GetTicks
-#include <SDL2/SDL_video.h>	 // for SDL_WINDOWEVENT_SIZE_CHANGED
-#include <string>			 // for basic_string
-#include "asset.h"			 // for Asset
-#include "global_inputs.h"	 // for globalInputs::check
-#include "input.h"			 // for Input
-#include "jail_audio.h"		 // for JA_StopMusic, JA_PlayMusic
-#include "lang.h"			 // for getText
-#include "param.h"			 // for param
-#include "screen.h"			 // for Screen
-#include "section.h"		 // for name, SectionName, options, SectionOptions
-#include "smart_sprite.h"	 // for SmartSprite
-#include "text.h"			 // for Text
-#include "texture.h"		 // for Texture
-#include "utils.h"			 // for ParamGame, Param, Zone, BLOCK
-#include "writer.h"			 // for Writer
-struct JA_Music_t;
+#include <SDL2/SDL_events.h>  // for SDL_PollEvent, SDL_Event, SDL_QUIT, SDL...
+#include <SDL2/SDL_timer.h>   // for SDL_GetTicks
+#include <SDL2/SDL_video.h>   // for SDL_WINDOWEVENT_SIZE_CHANGED
+#include <utility>            // for move
+#include "asset.h"            // for Asset
+#include "global_inputs.h"    // for check
+#include "input.h"            // for Input
+#include "jail_audio.h"       // for JA_StopMusic, JA_PlayMusic
+#include "lang.h"             // for getText
+#include "param.h"            // for param
+#include "screen.h"           // for Screen
+#include "section.h"          // for Name, name, Options, options
+#include "smart_sprite.h"     // for SmartSprite
+#include "text.h"             // for Text
+#include "texture.h"          // for Texture
+#include "utils.h"            // for Param, ParamGame, Zone, BLOCK, Color
+#include "writer.h"           // for Writer
+struct JA_Music_t;  // lines 19-19

 // Constructor
 Intro::Intro(JA_Music_t *music)
--- a/source/intro.h
+++ b/source/intro.h
@@ -1,14 +1,13 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
-#include <SDL2/SDL_stdinc.h> // for Uint32, Uint8
-#include <vector>			 // for vector
-#include <memory>
-#include "smart_sprite.h"
-#include "texture.h"
-#include "text.h"
-#include "writer.h"
-struct JA_Music_t;
+#include <SDL2/SDL_stdinc.h>  // for Uint32, Uint8
+#include <memory>             // for unique_ptr, shared_ptr
+#include <vector>             // for vector
+#include "smart_sprite.h"     // for SmartSprite
+#include "writer.h"           // for Writer
+class Text;
+class Texture;
+struct JA_Music_t;  // lines 11-11

 /*
 	Esta clase gestiona un estado del programa. Se encarga de mostrar la secuencia
--- a/source/item.h
+++ b/source/item.h
@@ -1,13 +1,13 @@
 #pragma once

-#include <SDL2/SDL_rect.h>   // for SDL_Rect
-#include <SDL2/SDL_stdinc.h> // for Uint16
-#include <string>            // for string
-#include <vector>            // for vector
-#include <memory>
-#include "utils.h" // for Circle
-#include "animated_sprite.h"
-#include "texture.h"
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_stdinc.h>  // for Uint16
+#include <memory>             // for shared_ptr, unique_ptr
+#include <string>             // for string
+#include <vector>             // for vector
+#include "animated_sprite.h"  // for AnimatedSprite
+#include "utils.h"            // for Circle
+class Texture;

 // Tipos de objetos
 constexpr int ITEM_POINTS_1_DISK = 1;
--- a/source/logo.cpp
+++ b/source/logo.cpp
@@ -1,17 +1,18 @@
 #include "logo.h"
-#include <SDL2/SDL_render.h> // for SDL_Renderer
-#include <SDL2/SDL_timer.h>	 // for SDL_GetTicks
-#include <SDL2/SDL_video.h>	 // for SDL_WINDOWEVENT_SIZE_CHANGED
-#include <string>			 // for basic_string
-#include "asset.h"			 // for Asset
-#include "global_inputs.h"	 // for globalInputs::check
-#include "input.h"			 // for Input
-#include "jail_audio.h"		 // for JA_StopMusic
-#include "param.h"			 // for param
-#include "screen.h"			 // for Screen
-#include "section.h"		 // for name, SectionName, options, SectionOptions
-#include "sprite.h"			 // for Sprite
-#include "texture.h"		 // for Texture
+#include <SDL2/SDL_events.h>  // for SDL_PollEvent, SDL_Event, SDL_QUIT, SDL...
+#include <SDL2/SDL_render.h>  // for SDL_Renderer
+#include <SDL2/SDL_timer.h>   // for SDL_GetTicks
+#include <SDL2/SDL_video.h>   // for SDL_WINDOWEVENT_SIZE_CHANGED
+#include <utility>            // for move
+#include "asset.h"            // for Asset
+#include "global_inputs.h"    // for check
+#include "input.h"            // for Input
+#include "jail_audio.h"       // for JA_StopMusic
+#include "param.h"            // for param
+#include "screen.h"           // for Screen
+#include "section.h"          // for Name, name, Options, options
+#include "sprite.h"           // for Sprite
+#include "texture.h"          // for Texture

 // Constructor
 Logo::Logo()
@@ -114,12 +115,12 @@ void Logo::updateJAILGAMES()
 	{
 		for (int i = 0; i < (int)jail_sprite_.size(); ++i)
 		{
-			if (jail_sprite_[i]->getPosX() != dest_.x)
+			if (jail_sprite_[i]->getIntPosX() != dest_.x)
 			{
 				if (i % 2 == 0)
 				{
 					jail_sprite_[i]->incPosX(-SPEED);
-					if (jail_sprite_[i]->getPosX() < dest_.x)
+					if (jail_sprite_[i]->getIntPosX() < dest_.x)
 					{
 						jail_sprite_[i]->setPosX(dest_.x);
 					}
@@ -127,7 +128,7 @@ void Logo::updateJAILGAMES()
 				else
 				{
 					jail_sprite_[i]->incPosX(SPEED);
-					if (jail_sprite_[i]->getPosX() > dest_.x)
+					if (jail_sprite_[i]->getIntPosX() > dest_.x)
 					{
 						jail_sprite_[i]->setPosX(dest_.x);
 					}
--- a/source/logo.h
+++ b/source/logo.h
@@ -1,13 +1,12 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
-#include <SDL2/SDL_rect.h>	 // for SDL_Point
-#include <SDL2/SDL_stdinc.h> // for Uint32
-#include <vector>			 // for vector
-#include <memory>
-#include "utils.h" // for Color
-#include "sprite.h"
-#include "texture.h"
+#include <SDL2/SDL_rect.h>    // for SDL_Point
+#include <SDL2/SDL_stdinc.h>  // for Uint32
+#include <memory>             // for unique_ptr, shared_ptr
+#include <vector>             // for vector
+#include "sprite.h"           // for Sprite
+#include "utils.h"            // for Color
+class Texture;

 /*
 	Esta clase gestiona un estado del programa. Se encarga de dibujar por pantalla el
--- a/source/main.cpp
+++ b/source/main.cpp
@@ -7,25 +7,22 @@ Actualizando a la versión "Arcade Edition" en 08/05/2024

 */

-#include <iostream>    // for basic_ostream, char_traits, operator<<, cout
-#include <string>      // for basic_string, operator<<, string
+#include <iostream>    // for char_traits, basic_ostream, operator<<, cout
+#include <memory>      // for make_unique, unique_ptr
 #include "director.h"  // for Director

 int main(int argc, char *argv[])
 {
-	std::cout << "Game start" << std::endl;
+    std::cout << "Game start" << std::endl;

-	// Crea el objeto Director
-	Director *director = new Director(argc, argv);
+    // Crea el objeto Director
+    auto director = std::make_unique<Director>(argc, const_cast<const char **>(argv));

-	// Bucle principal
-	const auto exit = director->run();
+    // Bucle principal
+    const auto exit = director->run();

-	// Destruye el objeto Director
-	delete director;
+    const auto endType = exit == 0 ? "keyboard" : "controller";
+    std::cout << "\nGame end with " << endType << std::endl;

-	const auto endType = exit == 0 ? "keyboard" : "controller";
-	std::cout << "\nGame end with " << endType << std::endl;
-
-	return exit;
+    return exit;
 }
--- a/source/manage_hiscore_table.cpp
+++ b/source/manage_hiscore_table.cpp
@@ -50,7 +50,7 @@ void ManageHiScoreTable::sort()
 {
    struct
    {
-        bool operator()(HiScoreEntry &a, HiScoreEntry &b) const { return a.score > b.score; }
+        bool operator()(const HiScoreEntry &a, const HiScoreEntry &b) const { return a.score > b.score; }
    } custom_less;

    std::sort(table_->begin(), table_->end(), custom_less);
--- a/source/moving_sprite.h
+++ b/source/moving_sprite.h
@@ -1,11 +1,11 @@
 #pragma once

-#include <SDL2/SDL_rect.h>   // for SDL_Rect, SDL_Point
-#include <SDL2/SDL_render.h> // for SDL_RendererFlip
-#include <SDL2/SDL_stdinc.h> // for Uint16
-#include <memory>
-#include "sprite.h" // for Sprite
-#include "texture.h"
+#include <SDL2/SDL_rect.h>    // for SDL_Rect, SDL_Point
+#include <SDL2/SDL_render.h>  // for SDL_RendererFlip
+#include <SDL2/SDL_stdinc.h>  // for Uint16
+#include <memory>             // for shared_ptr
+#include "sprite.h"           // for Sprite
+class Texture;

 // Clase MovingSprite. Añade posicion y velocidad en punto flotante
 class MovingSprite : public Sprite
@@ -41,6 +41,9 @@ public:
    explicit MovingSprite(float x = 0, float y = 0, int w = 0, int h = 0, float velx = 0, float vely = 0, float accelx = 0, float accely = 0, std::shared_ptr<Texture> texture = nullptr);
    explicit MovingSprite(std::shared_ptr<Texture> texture = nullptr);

+    // Destructor
+    virtual ~MovingSprite() = default;
+
    // Mueve el sprite
    void move();

--- a/source/notify.cpp
+++ b/source/notify.cpp
@@ -1,16 +1,15 @@
 #include "notify.h"
-#include <SDL2/SDL_blendmode.h> // for SDL_BLENDMODE_BLEND
-#include <SDL2/SDL_pixels.h>    // for SDL_PIXELFORMAT_RGBA8888
-#include <string>               // for basic_string, char_traits, string
-#include "jail_audio.h"         // for JA_DeleteSound, JA_LoadSound, JA_Pla...
-#include "options.h"            // for options
-#include "param.h"
-#include "sprite.h"  // for Sprite
-#include "text.h"    // for Text
-#include "texture.h" // for Texture
+#include <SDL2/SDL_blendmode.h>  // for SDL_BLENDMODE_BLEND
+#include <SDL2/SDL_pixels.h>     // for SDL_PIXELFORMAT_RGBA8888
+#include <string>                // for string
+#include "jail_audio.h"          // for JA_DeleteSound, JA_LoadSound, JA_Pla...
+#include "param.h"               // for param
+#include "sprite.h"              // for Sprite
+#include "text.h"                // for Text
+#include "texture.h"             // for Texture

 // Constructor
-Notify::Notify(SDL_Renderer *renderer, std::string iconFile, std::string bitmapFile, std::string textFile, std::string soundFile)
+Notify::Notify(SDL_Renderer *renderer, std::string iconFile, std::string bitmapFile, std::string textFile, const std::string &soundFile)
    : renderer(renderer),
      text(std::make_unique<Text>(bitmapFile, textFile, renderer)),
      bgColor(param.notification.color),
@@ -147,18 +146,19 @@ void Notify::clearFinishedNotifications()
 void Notify::showText(std::string text1, std::string text2, int icon)
 {
    // Cuenta el número de textos a mostrar
-    const int numTexts = (text1 != "") + (text2 != "");
+    const int numTexts = !text1.empty() + !text2.empty();

    // Si no hay texto, acaba
    if (numTexts == 0)
    {
        return;
    }
+    
    // Si solo hay un texto, lo coloca en la primera variable
-    else if (numTexts == 1)
+    if (numTexts == 1)
    {
-        text1 = text1 + text2;
-        text2 = "";
+        text1 += text2;
+        text2.clear();
    }

    // Si las notificaciones no se apilan, elimina las anteriores
--- a/source/notify.h
+++ b/source/notify.h
@@ -1,15 +1,15 @@
 #pragma once

-#include <SDL2/SDL_rect.h>	 // for SDL_Rect
-#include <SDL2/SDL_render.h> // for SDL_Renderer
-#include <string>			 // for basic_string, string
-#include <vector>			 // for vector
-#include <memory>
-#include "utils.h" // for Color
-#include "text.h"
-#include "texture.h"
-#include "sprite.h"
-struct JA_Sound_t;
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_render.h>  // for SDL_Renderer
+#include <memory>             // for shared_ptr, unique_ptr
+#include <string>             // for string, basic_string
+#include <vector>             // for vector
+#include "utils.h"            // for Color
+class Sprite;
+class Text;
+class Texture;
+struct JA_Sound_t;  // lines 12-12

 class Notify
 {
@@ -77,7 +77,7 @@ private:

 public:
 	// Constructor
-	Notify(SDL_Renderer *renderer, std::string iconFile, std::string bitmapFile, std::string textFile, std::string soundFile);
+	Notify(SDL_Renderer *renderer, std::string iconFile, std::string bitmapFile, std::string textFile, const std::string &soundFile);

 	// Destructor
 	~Notify();
@@ -89,7 +89,7 @@ public:
 	void update();

 	// Muestra una notificación de texto por pantalla;
-	void showText(std::string text1 = "", std::string text2 = "", int icon = -1);
+	void showText(std::string text1 = std::string(), std::string text2 = std::string(), int icon = -1);

 	// Indica si hay notificaciones activas
 	bool active();
--- a/source/on_screen_help.cpp
+++ b/source/on_screen_help.cpp
@@ -1,16 +1,15 @@
 #include "on_screen_help.h"
-#include <SDL2/SDL_blendmode.h> // for SDL_BLENDMODE_BLEND
-#include <SDL2/SDL_pixels.h>    // for SDL_PIXELFORMAT_RGBA8888
-#include <memory>               // for make_unique, unique_ptr
-#include <string>               // for basic_string
-#include "asset.h"              // for Asset
-#include "lang.h"               // for getText
-#include "param.h"              // for param
-#include "screen.h"             // for Screen
-#include "sprite.h"             // for Sprite
-#include "text.h"               // for Text
-#include "texture.h"            // for Texture
-#include "utils.h"              // for easeInOutSine, ParamGame, Param
+#include <SDL2/SDL_blendmode.h>  // for SDL_BLENDMODE_BLEND
+#include <SDL2/SDL_pixels.h>     // for SDL_PIXELFORMAT_RGBA8888
+#include <memory>                // for make_unique, make_shared, unique_ptr
+#include "asset.h"               // for Asset
+#include "lang.h"                // for getText
+#include "param.h"               // for param
+#include "screen.h"              // for Screen
+#include "sprite.h"              // for Sprite
+#include "text.h"                // for Text
+#include "texture.h"             // for Texture
+#include "utils.h"               // for easeInOutSine, Param, ParamGame

 // [SINGLETON] Hay que definir las variables estáticas, desde el .h sólo la hemos declarado
 OnScreenHelp *OnScreenHelp::onScreenHelp = nullptr;
--- a/source/on_screen_help.h
+++ b/source/on_screen_help.h
@@ -1,11 +1,10 @@
 #pragma once

-#include <SDL2/SDL_rect.h>   // for SDL_Rect
-#include <SDL2/SDL_render.h> // for SDL_Texture
-#include <vector>            // for vector
-#include <memory>
-#include "sprite.h"                // lines 10-10
-#include "text.h"
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_render.h>  // for SDL_Texture
+#include <vector>             // for vector
+class Sprite;
+class Text;

 enum class OnScreenHelpStatus
 {
--- a/source/options.cpp
+++ b/source/options.cpp
@@ -13,7 +13,7 @@
 Options options;

 // Declaraciones
-bool setOptions(std::string var, const std::string &value);
+bool setOptions(const std::string &var, const std::string &value);

 // Inicializa las opciones del programa
 void initOptions()
@@ -244,7 +244,7 @@ bool saveOptionsFile(std::string file_path)
 }

 // Asigna variables a partir de dos cadenas
-bool setOptions(std::string var, const std::string &value)
+bool setOptions(const std::string &var, const std::string &value)
 {
 	// Indicador de éxito en la asignación
 	auto success = true;
@@ -393,7 +393,7 @@ bool setOptions(std::string var, const std::string &value)
 	}

 	// Lineas vacias o que empiezan por comentario
-	else if (var == "" || var.substr(0, 1) == "#")
+	else if (var.empty() || var.starts_with("#"))
 	{
 	}

--- a/source/param.cpp
+++ b/source/param.cpp
@@ -1,7 +1,10 @@
 #include "param.h"
 #include <fstream>	// for char_traits, basic_ostream, basic_ifstream, basi...
 #include <iostream> // for cout
-#include "utils.h"	// for Param, ParamGame, Zone, ParamBalloon
+#include <sstream>
+#include <string>
+#include <stdexcept>
+#include "utils.h" // for Param, ParamGame, Zone, ParamBalloon

 Param param;

@@ -40,7 +43,7 @@ void initParam()
 	param.title.title_c_c_position = 11;

 	// BACKGROUND
-	param.background.attenuate_color = {255, 255, 255};
+	param.background.attenuate_color = (Color){255, 255, 255};
 	param.background.attenuate_alpha = 32;

 	// BALLOONS
@@ -52,10 +55,18 @@ void initParam()
 	param.balloon_3.grav = 0.10f;
 	param.balloon_4.vel = 4.95f;
 	param.balloon_4.grav = 0.10f;
+
+	// NOTIFICATION
+	param.notification.pos_v = NotifyPosition::TOP;
+	param.notification.pos_h = NotifyPosition::LEFT;
+	param.notification.sound = false;
+	param.notification.color.r = 48;
+	param.notification.color.g = 48;
+	param.notification.color.b = 48;
 }

 // Establece valores para los parametros a partir de un fichero de texto
-void loadParamsFromFile(std::string file_path)
+/*void loadParamsFromFile(std::string file_path)
 {
 	// Pone valores por defecto a las variables
 	initParam();
@@ -80,8 +91,15 @@ void loadParamsFromFile(std::string file_path)
 			param1.clear();
 			param2.clear();

-			// Elimina los comentarios
-			line = line.substr(0, line.find("#"));
+			// Elimina los comentarios al final de una linea
+			{
+				// line = line.substr(0, line.find("#"));
+				auto pos = line.find("#");
+				if (pos != std::string::npos)
+				{
+					line.resize(pos);
+				}
+			}

 			// Ignora los espacios en blanco
 			int pos = 0;
@@ -148,6 +166,50 @@ void loadParamsFromFile(std::string file_path)
 		std::cout << "Failed to load file: " << file_path << std::endl;
 #endif

+	precalculateZones();
+}*/
+void loadParamsFromFile(const std::string &file_path)
+{
+	// Inicializa los parámetros con valores por defecto
+	initParam();
+
+	// Abre el archivo
+	std::ifstream file(file_path);
+	if (!file.is_open())
+	{
+#ifdef VERBOSE
+		std::cerr << "Error: No se pudo abrir el archivo " << file_path << std::endl;
+#endif
+		throw std::runtime_error("No se pudo abrir el archivo: " + file_path);
+	}
+
+#ifdef VERBOSE
+	const std::string file_name = file_path.substr(file_path.find_last_of("\\/") + 1);
+	std::cout << "Reading file: " << file_name << std::endl;
+#endif
+
+	std::string line, param1, param2;
+	while (std::getline(file, line))
+	{
+		// Elimina comentarios
+		auto comment_pos = line.find('#');
+		if (comment_pos != std::string::npos)
+		{
+			line.resize(comment_pos);
+		}
+
+		// Usa un stream para separar palabras
+		std::istringstream iss(line);
+		if (iss >> param1 >> param2)
+		{
+			setParams(param1, param2);
+		}
+	}
+
+	// Cierra el archivo
+	file.close();
+
+	// Realiza cálculos adicionales después de cargar los parámetros
 	precalculateZones();
 }

--- a/source/param.h
+++ b/source/param.h
@@ -6,4 +6,4 @@ struct Param;
 extern Param param;

 // Establece valores para los parametros a partir de un fichero de texto
-void loadParamsFromFile(std::string file_path);
+void loadParamsFromFile(const std::string &file_path);
--- a/source/player.cpp
+++ b/source/player.cpp
@@ -12,30 +12,29 @@

 // Constructor
 Player::Player(int id, float x, int y, bool demo, SDL_Rect *play_area, std::vector<std::shared_ptr<Texture>> texture, std::vector<std::vector<std::string> *> animations)
+	: player_sprite_(std::make_unique<AnimatedSprite>(texture[0], "", animations[0])),
+	  power_sprite_(std::make_unique<AnimatedSprite>(texture[1], "", animations[1])),
+	  enter_name_(std::make_unique<EnterName>()),
+	  play_area_(play_area),
+	  id_(id),
+	  pos_x_(x),
+	  pos_y_(y),
+	  default_pos_x_(x),
+	  default_pos_y_(y),
+	  status_playing_(PlayerStatus::WAITING),
+	  scoreboard_panel_(0),
+	  name_(std::string()),
+	  controller_index_(0),
+	  demo_(demo)
 {
 	// Reserva memoria para los objetos
-	player_sprite_ = std::make_unique<AnimatedSprite>(texture[0], "", animations[0]);
-	power_sprite_ = std::make_unique<AnimatedSprite>(texture[1], "", animations[1]);
 	power_sprite_->getTexture()->setAlpha(224);
-	enter_name_ = std::make_unique<EnterName>();
-
-	// Rectangulo con la zona de juego
-	play_area_ = play_area;
-
-	// Establece la posición inicial del jugador
-	default_pos_x_ = pos_x_ = x;
-	default_pos_y_ = pos_y_ = y;

 	// Establece los offsets para el sprite de PowerUp
 	power_up_desp_x_ = (power_sprite_->getWidth() - player_sprite_->getWidth()) / 2;
 	power_sprite_->setPosY(y - (power_sprite_->getHeight() - player_sprite_->getHeight()));

 	// Inicializa variables
-	id_ = id;
-	demo_ = demo;
-	status_playing_ = PlayerStatus::WAITING;
-	scoreboard_panel_ = 0;
-	name_ = "";
 	setRecordName(enter_name_->getName());
 	init();
 }
@@ -237,21 +236,13 @@ void Player::render()
 // Establece el estado del jugador cuando camina
 void Player::setWalkingStatus(PlayerStatus status)
 {
-	// Si cambiamos de estado, reiniciamos la animación
-	if (status_walking_ != status)
-	{
-		status_walking_ = status;
-	}
+	status_walking_ = status;
 }

 // Establece el estado del jugador cuando dispara
 void Player::setFiringStatus(PlayerStatus status)
 {
-	// Si cambiamos de estado, reiniciamos la animación
-	if (status_firing_ != status)
-	{
-		status_firing_ = status;
-	}
+	status_firing_ = status;
 }

 // Establece la animación correspondiente al estado
@@ -694,7 +685,7 @@ void Player::shiftColliders()
 }

 // Pone las texturas del jugador
-void Player::setPlayerTextures(std::vector<std::shared_ptr<Texture>> texture)
+void Player::setPlayerTextures(const std::vector<std::shared_ptr<Texture>> &texture)
 {
 	player_sprite_->setTexture(texture[0]);
 	power_sprite_->setTexture(texture[1]);
@@ -736,7 +727,7 @@ int Player::getScoreBoardPanel() const
 void Player::decContinueCounter()
 {
 	continue_ticks_ = SDL_GetTicks();
-	continue_counter_--;
+	--continue_counter_;
 	if (continue_counter_ < 0)
 	{
 		setStatusPlaying(PlayerStatus::GAME_OVER);
@@ -744,13 +735,13 @@ void Player::decContinueCounter()
 }

 // Establece el nombre del jugador
-void Player::setName(std::string name)
+void Player::setName(const std::string &name)
 {
 	name_ = name;
 }

 // Establece el nombre del jugador para la tabla de mejores puntuaciones
-void Player::setRecordName(std::string record_name)
+void Player::setRecordName(const std::string &record_name)
 {
 	record_name_ = record_name.substr(0, 8);
 }
--- a/source/player.h
+++ b/source/player.h
@@ -1,15 +1,16 @@
 #pragma once

-#include <SDL2/SDL_rect.h>	 // for SDL_Rect
-#include <SDL2/SDL_stdinc.h> // for Uint32
-#include <memory>			 // for unique_ptr
-#include <string>			 // for string, basic_string
-#include <vector>			 // for vector
-#include "animated_sprite.h" // for AnimatedSprite
-#include "enter_name.h"		 // for EnterName
-#include "utils.h"			 // for Circle
-#include "texture.h"		 // lines 12-12
-enum class ScoreboardMode;
+#include <SDL2/SDL_rect.h>    // for SDL_Rect
+#include <SDL2/SDL_stdinc.h>  // for Uint32
+#include <memory>             // for unique_ptr, shared_ptr
+#include <string>             // for string
+#include <vector>             // for vector
+#include "animated_sprite.h"  // for AnimatedSprite
+#include "enter_name.h"       // for EnterName
+#include "utils.h"            // for Circle
+class Texture;
+enum class InputType : int;
+enum class ScoreboardMode;  // lines 12-12

 // Estados del jugador
 enum class PlayerStatus
@@ -118,7 +119,7 @@ public:
 	void render();

 	// Pone las texturas del jugador
-	void setPlayerTextures(std::vector<std::shared_ptr<Texture>> texture);
+	void setPlayerTextures(const std::vector<std::shared_ptr<Texture>> &texture);

 	// Actua en consecuencia de la entrada recibida
 	void setInput(InputType input);
@@ -271,10 +272,10 @@ public:
 	void decContinueCounter();

 	// Establece el nombre del jugador
-	void setName(std::string name);
+	void setName(const std::string &name);

 	// Establece el nombre del jugador para la tabla de mejores puntuaciones
-	void setRecordName(std::string record_name);
+	void setRecordName(const std::string &record_name);

 	// Obtiene el nombre del jugador
 	std::string getName() const;
--- a/source/scoreboard.cpp
+++ b/source/scoreboard.cpp
@@ -34,42 +34,39 @@ Scoreboard *Scoreboard::get()

 // Constructor
 Scoreboard::Scoreboard(SDL_Renderer *renderer)
-	: renderer_(renderer)
+	: renderer_(renderer),
+	  
+	  game_power_meter_texture_(std::make_shared<Texture>(renderer, Asset::get()->get("game_power_meter.png"))),
+	  power_meter_sprite_(std::make_unique<Sprite>(game_power_meter_texture_)),
+	  text_scoreboard_(std::make_unique<Text>(Asset::get()->get("8bithud.png"), Asset::get()->get("8bithud.txt"), renderer)),
+	  
+	  stage_(1),
+	  hi_score_(0),
+	  power_(0),
+	  hi_score_name_(std::string()),
+	  color_({0, 0, 0}),
+	  rect_({0, 0, 320, 40}),
+	  ticks_(SDL_GetTicks()),
+	  counter_(0)
 {
-	// Inicializa punteros
-	game_power_meter_texture_ = nullptr;
-	power_meter_sprite_ = nullptr;
-	text_scoreboard_ = nullptr;
-
 	// Inicializa variables
 	for (int i = 0; i < SCOREBOARD_MAX_PANELS; ++i)
 	{
-		name_[i] = "";
-		record_name_[i] = "";
+		name_[i].clear();
+		record_name_[i].clear();
 		selector_pos_[i] = 0;
 		score_[i] = 0;
 		mult_[i] = 0;
 		continue_counter_[i] = 0;
 	}
-	stage_ = 1;
-	hi_score_ = 0;
-	power_ = 0;
-	hi_score_name_ = "";
-	color_ = {0, 0, 0};
-	rect_ = {0, 0, 320, 40};
+
 	panel_[SCOREBOARD_LEFT_PANEL].mode = ScoreboardMode::SCORE;
 	panel_[SCOREBOARD_RIGHT_PANEL].mode = ScoreboardMode::SCORE;
 	panel_[SCOREBOARD_CENTER_PANEL].mode = ScoreboardMode::STAGE_INFO;
-	ticks_ = SDL_GetTicks();
-	counter_ = 0;

 	// Recalcula las anclas de los elementos
 	recalculateAnchors();
-
-	// Crea objetos
-	game_power_meter_texture_ = std::make_shared<Texture>(renderer_, Asset::get()->get("game_power_meter.png"));
-	power_meter_sprite_ = std::make_unique<Sprite>(slot4_2_.x - 20, slot4_2_.y, 40, 7, game_power_meter_texture_);
-	text_scoreboard_ = std::make_unique<Text>(Asset::get()->get("8bithud.png"), Asset::get()->get("8bithud.txt"), renderer_);
+	power_meter_sprite_->setPos({slot4_2_.x - 20, slot4_2_.y, 40, 7});

 	// Crea la textura de fondo
 	background_ = nullptr;
@@ -130,13 +127,13 @@ void Scoreboard::render()
 }

 // Establece el valor de la variable
-void Scoreboard::setName(int panel_, std::string name_)
+void Scoreboard::setName(int panel_, const std::string &name_)
 {
 	this->name_[panel_] = name_;
 }

 // Establece el valor de la variable
-void Scoreboard::setRecordName(int panel_, std::string record_name_)
+void Scoreboard::setRecordName(int panel_, const std::string &record_name_)
 {
 	this->record_name_[panel_] = record_name_;
 }
@@ -184,7 +181,7 @@ void Scoreboard::setPower(float power_)
 }

 // Establece el valor de la variable
-void Scoreboard::setHiScoreName(std::string name_)
+void Scoreboard::setHiScoreName(const std::string &name_)
 {
 	hi_score_name_ = name_;
 }
@@ -323,7 +320,7 @@ void Scoreboard::fillPanelTextures()

 			// ENTER NAME
 			text_scoreboard_->writeCentered(slot4_3_.x, slot4_3_.y, lang::getText(106));
-			SDL_Rect rect_ = {enter_name_pos_.x, enter_name_pos_.y, 5, 7};
+			SDL_Rect rect = {enter_name_pos_.x, enter_name_pos_.y, 5, 7};
 			SDL_SetRenderDrawColor(renderer_, 0xFF, 0xFF, 0xEB, 255);
 			for (int j = 0; j < (int)record_name_[i].size(); ++j)
 			{
@@ -331,16 +328,16 @@ void Scoreboard::fillPanelTextures()
 				{ // La letra seleccionada se pinta de forma intermitente
 					if (counter_ % 3 > 0)
 					{
-						SDL_RenderDrawLine(renderer_, rect_.x, rect_.y + rect_.h, rect_.x + rect_.w, rect_.y + rect_.h);
-						text_scoreboard_->write(rect_.x, rect_.y, record_name_[i].substr(j, 1));
+						SDL_RenderDrawLine(renderer_, rect.x, rect.y + rect.h, rect.x + rect.w, rect.y + rect.h);
+						text_scoreboard_->write(rect.x, rect.y, record_name_[i].substr(j, 1));
 					}
 				}
 				else
 				{
-					SDL_RenderDrawLine(renderer_, rect_.x, rect_.y + rect_.h, rect_.x + rect_.w, rect_.y + rect_.h);
-					text_scoreboard_->write(rect_.x, rect_.y, record_name_[i].substr(j, 1));
+					SDL_RenderDrawLine(renderer_, rect.x, rect.y + rect.h, rect.x + rect.w, rect.y + rect.h);
+					text_scoreboard_->write(rect.x, rect.y, record_name_[i].substr(j, 1));
 				}
-				rect_.x += 7;
+				rect.x += 7;
 			}
 			break;
 		}
--- a/source/scoreboard.h
+++ b/source/scoreboard.h
@@ -1,15 +1,15 @@
 #pragma once

-#include <SDL2/SDL_rect.h>	 // for SDL_Point, SDL_Rect
-#include <SDL2/SDL_render.h> // for SDL_Renderer, SDL_Texture
-#include <SDL2/SDL_stdinc.h> // for Uint32
-#include <memory>			 // for unique_ptr
-#include <string>			 // for string, basic_string
-#include <vector>			 // for vector
-#include "utils.h"			 // for Color
-#include "sprite.h"			 // lines 11-11
-#include "text.h"			 // lines 12-12
-#include "texture.h"		 // lines 13-13
+#include <SDL2/SDL_rect.h>    // for SDL_Point, SDL_Rect
+#include <SDL2/SDL_render.h>  // for SDL_Renderer, SDL_Texture
+#include <SDL2/SDL_stdinc.h>  // for Uint32
+#include <memory>             // for unique_ptr, shared_ptr
+#include <string>             // for string
+#include <vector>             // for vector
+#include "utils.h"            // for Color
+class Sprite;
+class Text;
+class Texture;

 // Defines
 constexpr int SCOREBOARD_LEFT_PANEL = 0;
@@ -50,9 +50,9 @@ private:

 	std::shared_ptr<Texture> game_power_meter_texture_; // Textura con el marcador de poder de la fase
 	std::unique_ptr<Sprite> power_meter_sprite_;		// Sprite para el medidor de poder de la fase
-	std::unique_ptr<Text> text_scoreboard_;			// Fuente para el marcador del juego
+	std::unique_ptr<Text> text_scoreboard_;				// Fuente para el marcador del juego

-	SDL_Texture *background_;				 // Textura para dibujar el marcador
+	SDL_Texture *background_;				   // Textura para dibujar el marcador
 	std::vector<SDL_Texture *> panel_texture_; // Texturas para dibujar cada panel

 	// Variables
@@ -103,7 +103,7 @@ private:
 	// [SINGLETON] Ahora el constructor y el destructor son privados

 	// Constructor
-	Scoreboard(SDL_Renderer *renderer);
+	explicit Scoreboard(SDL_Renderer *renderer);

 	// Destructor
 	~Scoreboard();
@@ -125,10 +125,10 @@ public:
 	void render();

 	// Establece el valor de la variable
-	void setName(int panel, std::string name);
+	void setName(int panel, const std::string &name);

 	// Establece el valor de la variable
-	void setRecordName(int panel, std::string record_name);
+	void setRecordName(int panel, const std::string &record_name);

 	// Establece el valor de la variable
 	void setSelectorPos(int panel, int pos);
@@ -152,7 +152,7 @@ public:
 	void setPower(float power);

 	// Establece el valor de la variable
-	void setHiScoreName(std::string name);
+	void setHiScoreName(const std::string &name);

 	// Establece el valor de la variable
 	void setColor(Color color);
--- a/source/screen.cpp
+++ b/source/screen.cpp
@@ -44,12 +44,27 @@ Screen *Screen::get()

 // Constructor
 Screen::Screen(SDL_Window *window, SDL_Renderer *renderer)
-    : window_(window), renderer_(renderer)
+    : window_(window),
+      renderer_(renderer),
+
+      notify_(std::make_unique<Notify>(renderer_, std::string(), Asset::get()->get("8bithud.png"), Asset::get()->get("8bithud.txt"), Asset::get()->get("notify.wav"))),
+      game_canvas_(SDL_CreateTexture(renderer_, SDL_PIXELFORMAT_RGBA8888, SDL_TEXTUREACCESS_TARGET, param.game.width, param.game.height)),
+      shader_canvas_(SDL_CreateTexture(renderer_, SDL_PIXELFORMAT_RGBA8888, SDL_TEXTUREACCESS_TARGET, param.game.width, param.game.height)),
+
+      src_rect_({0, 0, param.game.width, param.game.height}),
+      dst_rect_({0, 0, param.game.width, param.game.height}),
+      border_color_({0x00, 0x00, 0x00}),
+      attenuate_effect_(false),
+      fps_ticks_(0),
+      fps_counter_(0),
+      fps_(0),
+#ifdef DEBUG
+      show_info_(true)
+#else
+      show_info_(false)
+#endif
 {
    // Inicializa variables
-    src_rect_ = {0, 0, param.game.width, param.game.height};
-    dst_rect_ = {0, 0, param.game.width, param.game.height};
-    border_color_ = {0, 0, 0};
    flash_effect_.enabled = false;
    flash_effect_.counter = 0;
    flash_effect_.lenght = 0;
@@ -62,29 +77,10 @@ Screen::Screen(SDL_Window *window, SDL_Renderer *renderer)
    shake_effect_.remaining = 0;
    shake_effect_.originalPos = 0;
    shake_effect_.originalWidth = param.game.width;
-    attenuate_effect_ = false;
-    fps_ticks_ = 0;
-    fps_counter_ = 0;
-    fps_ = 0;
-#ifdef DEBUG
-    show_info_ = true;
-#else
-    show_info_ = false;
-#endif
    SDL_DisplayMode DM;
    SDL_GetCurrentDisplayMode(0, &DM);
    info_resolution_ = std::to_string(DM.w) + " X " + std::to_string(DM.h) + " AT " + std::to_string(DM.refresh_rate) + " HZ";

-    // Crea los objetos
-    notify_ = std::make_unique<Notify>(renderer_, "", Asset::get()->get("8bithud.png"), Asset::get()->get("8bithud.txt"), Asset::get()->get("notify.wav"));
-
-    // Define el color del borde para el modo de pantalla completa
-    border_color_ = {0x00, 0x00, 0x00};
-
-    // Crea las textura donde se dibujan los graficos del juego
-    game_canvas_ = SDL_CreateTexture(renderer_, SDL_PIXELFORMAT_RGBA8888, SDL_TEXTUREACCESS_TARGET, param.game.width, param.game.height);
-    shader_canvas_ = SDL_CreateTexture(renderer_, SDL_PIXELFORMAT_RGBA8888, SDL_TEXTUREACCESS_TARGET, param.game.width, param.game.height);
-
    // Establece el modo de video
    setVideoMode(options.video.mode);

@@ -183,9 +179,10 @@ void Screen::blit()
 // Establece el modo de video
 void Screen::setVideoMode(ScreenVideoMode videoMode)
 {
-    options.video.mode = videoMode;
 #ifdef ARCADE
    options.video.mode = ScreenVideoMode::WINDOW;
+#else
+    options.video.mode = videoMode;
 #endif

    switch (options.video.mode)
@@ -455,7 +452,7 @@ void Screen::attenuate(bool value)
 }

 // Muestra una notificación de texto por pantalla;
-void Screen::showNotification(std::string text1, std::string text2, int icon)
+void Screen::showNotification(const std::string &text1, const std::string &text2, int icon)
 {
    notify_->showText(text1, text2, icon);
 }
--- a/source/screen.h
+++ b/source/screen.h
@@ -152,7 +152,7 @@ public:
 	void attenuate(bool value);

 	// Muestra una notificación de texto por pantalla;
-	void showNotification(std::string text1 = "", std::string text2 = "", int icon = -1);
+	void showNotification(const std::string &text1 = std::string(), const std::string &text2 = std::string(), int icon = -1);

 	// Indica si hay alguna notificación activa en pantalla
 	bool notificationsAreActive() const;
--- a/source/smart_sprite.h
+++ b/source/smart_sprite.h
@@ -1,8 +1,8 @@
 #pragma once

-#include "animated_sprite.h" // for AnimatedSprite
-#include "texture.h"
-#include <memory>
+#include <memory>             // for shared_ptr
+#include "animated_sprite.h"  // for AnimatedSprite
+class Texture;

 // Clase SmartSprite
 class SmartSprite : public AnimatedSprite
--- a/source/sprite.cpp
+++ b/source/sprite.cpp
@@ -21,13 +21,13 @@ void Sprite::render()
 }

 // Obten el valor de la variable
-int Sprite::getPosX() const
+int Sprite::getIntPosX() const
 {
 	return pos_.x;
 }

 // Obten el valor de la variable
-int Sprite::getPosY() const
+int Sprite::getIntPosY() const
 {
 	return pos_.y;
 }
--- a/source/sprite.h
+++ b/source/sprite.h
@@ -20,14 +20,14 @@ public:
    explicit Sprite(std::shared_ptr<Texture> texture = nullptr);

    // Destructor
-    ~Sprite() = default;
+    virtual ~Sprite() = default;

    // Muestra el sprite por pantalla
    virtual void render();

    // Obten el valor de la variable
-    int getPosX() const;
-    int getPosY() const;
+    int getIntPosX() const;
+    int getIntPosY() const;
    int getWidth() const;
    int getHeight() const;

--- a/source/text.cpp
+++ b/source/text.cpp
@@ -1,9 +1,10 @@
 #include "text.h"
-#include <fstream>	 // for char_traits, basic_ostream, basic_ifstream, ope...
-#include <iostream>	 // for cout
-#include "sprite.h"	 // for Sprite
-#include "texture.h" // for Texture
-#include "utils.h"	 // for Color
+#include <SDL2/SDL_rect.h>  // for SDL_Rect
+#include <fstream>          // for basic_ostream, basic_ifstream, basic_istream
+#include <iostream>         // for cout
+#include "sprite.h"         // for Sprite
+#include "texture.h"        // for Texture
+#include "utils.h"          // for Color

 // Llena una estructuta TextFile desde un fichero
 TextFile LoadTextFile(std::string file_path)
@@ -21,7 +22,9 @@ TextFile LoadTextFile(std::string file_path)
 	}

 	// Abre el fichero para leer los valores
+#ifdef VERBOSE
 	const std::string file_name = file_path.substr(file_path.find_last_of("\\/") + 1).c_str();
+#endif
 	std::ifstream file(file_path);

 	if (file.is_open() && file.good())
@@ -55,7 +58,7 @@ TextFile LoadTextFile(std::string file_path)

 // Cierra el fichero
 #ifdef VERBOSE
-		std::cout << "Text loaded: " << file_name.c_str() << std::endl;
+		std::cout << "Text loaded: " << file_name << std::endl;
 #endif
 		file.close();
 	}
@@ -64,7 +67,7 @@ TextFile LoadTextFile(std::string file_path)
 	else
 	{
 #ifdef VERBOSE
-		std::cout << "Warning: Unable to open " << file_name.c_str() << " file" << std::endl;
+		std::cout << "Warning: Unable to open " << file_name << " file" << std::endl;
 #endif
 	}

@@ -79,7 +82,7 @@ TextFile LoadTextFile(std::string file_path)
 }

 // Constructor
-Text::Text(std::string bitmap_file, std::string text_file, SDL_Renderer *renderer)
+Text::Text(const std::string &bitmap_file, const std::string &text_file, SDL_Renderer *renderer)
 {
 	// Carga los offsets desde el fichero
 	auto tf = LoadTextFile(text_file);
@@ -103,7 +106,7 @@ Text::Text(std::string bitmap_file, std::string text_file, SDL_Renderer *rendere
 }

 // Constructor
-Text::Text(std::string text_file, std::shared_ptr<Texture> texture)
+Text::Text(const std::string &text_file, std::shared_ptr<Texture> texture)
 {
 	// Carga los offsets desde el fichero
 	auto tf = LoadTextFile(text_file);
@@ -146,7 +149,7 @@ Text::Text(TextFile *text_file, std::shared_ptr<Texture> texture)
 }

 // Escribe texto en pantalla
-void Text::write(int x, int y, std::string text, int kerning, int lenght)
+void Text::write(int x, int y, const std::string &text, int kerning, int lenght)
 {
 	auto shift = 0;

@@ -169,7 +172,7 @@ void Text::write(int x, int y, std::string text, int kerning, int lenght)
 }

 // Escribe el texto con colores
-void Text::writeColored(int x, int y, std::string text, Color color, int kerning, int lenght)
+void Text::writeColored(int x, int y, const std::string &text, Color color, int kerning, int lenght)
 {
 	sprite_->getTexture()->setColor(color.r, color.g, color.b);
 	write(x, y, text, kerning, lenght);
@@ -177,7 +180,7 @@ void Text::writeColored(int x, int y, std::string text, Color color, int kerning
 }

 // Escribe el texto con sombra
-void Text::writeShadowed(int x, int y, std::string text, Color color, Uint8 shadow_distance, int kerning, int lenght)
+void Text::writeShadowed(int x, int y, const std::string &text, Color color, Uint8 shadow_distance, int kerning, int lenght)
 {
 	sprite_->getTexture()->setColor(color.r, color.g, color.b);
 	write(x + shadow_distance, y + shadow_distance, text, kerning, lenght);
@@ -186,14 +189,14 @@ void Text::writeShadowed(int x, int y, std::string text, Color color, Uint8 shad
 }

 // Escribe el texto centrado en un punto x
-void Text::writeCentered(int x, int y, std::string text, int kerning, int lenght)
+void Text::writeCentered(int x, int y, const std::string &text, int kerning, int lenght)
 {
 	x -= (Text::lenght(text, kerning) / 2);
 	write(x, y, text, kerning, lenght);
 }

 // Escribe texto con extras
-void Text::writeDX(Uint8 flags, int x, int y, std::string text, int kerning, Color textColor, Uint8 shadow_distance, Color shadow_color, int lenght)
+void Text::writeDX(Uint8 flags, int x, int y, const std::string &text, int kerning, Color textColor, Uint8 shadow_distance, Color shadow_color, int lenght)
 {
 	const auto centered = ((flags & TEXT_CENTER) == TEXT_CENTER);
 	const auto shadowed = ((flags & TEXT_SHADOW) == TEXT_SHADOW);
@@ -235,7 +238,7 @@ void Text::writeDX(Uint8 flags, int x, int y, std::string text, int kerning, Col
 }

 // Obtiene la longitud en pixels de una cadena
-int Text::lenght(std::string text, int kerning) const
+int Text::lenght(const std::string &text, int kerning) const
 {
 	auto shift = 0;

@@ -267,7 +270,7 @@ void Text::setFixedWidth(bool value)
 }

 // Carga una paleta de colores para el texto
-void Text::addPalette(std::string path)
+void Text::addPalette(const std::string &path)
 {
 	texture_->addPalette(path);
 }
--- a/source/text.h
+++ b/source/text.h
@@ -2,11 +2,11 @@

 #include <SDL2/SDL_render.h> // for SDL_Renderer
 #include <SDL2/SDL_stdinc.h> // for Uint8
+#include <memory>			 // for shared_ptr, unique_ptr
 #include <string>			 // for string
-#include <memory>
-#include "utils.h"
-#include "sprite.h"
-#include "texture.h"
+#include "sprite.h"			 // for Sprite
+#include "utils.h"			 // for Color
+class Texture;

 constexpr int TEXT_COLOR = 1;
 constexpr int TEXT_SHADOW = 2;
@@ -33,41 +33,41 @@ class Text
 {
 private:
 	// Objetos y punteros
-	std::unique_ptr<Sprite> sprite_;	  // Objeto con los graficos para el texto
+	std::unique_ptr<Sprite> sprite_;   // Objeto con los graficos para el texto
 	std::shared_ptr<Texture> texture_; // Textura con los bitmaps del texto

 	// Variables
-	int box_width_;			// Anchura de la caja de cada caracter en el png
-	int box_height_;			// Altura de la caja de cada caracter en el png
-	bool fixed_width_;		// Indica si el texto se ha de escribir con longitud fija en todas las letras
+	int box_width_;			 // Anchura de la caja de cada caracter en el png
+	int box_height_;		 // Altura de la caja de cada caracter en el png
+	bool fixed_width_;		 // Indica si el texto se ha de escribir con longitud fija en todas las letras
 	TextOffset offset_[128]; // Vector con las posiciones y ancho de cada letra

 public:
 	// Constructor
-	Text(std::string bitmap_file, std::string text_file, SDL_Renderer *renderer);
-	Text(std::string text_file, std::shared_ptr<Texture> texture);
+	Text(const std::string &bitmap_file, const std::string &text_file, SDL_Renderer *renderer);
+	Text(const std::string &text_file, std::shared_ptr<Texture> texture);
 	Text(TextFile *text_file, std::shared_ptr<Texture> texture);

 	// Destructor
 	~Text() = default;

 	// Escribe el texto en pantalla
-	void write(int x, int y, std::string text, int kerning = 1, int lenght = -1);
+	void write(int x, int y, const std::string &text, int kerning = 1, int lenght = -1);

 	// Escribe el texto con colores
-	void writeColored(int x, int y, std::string text, Color color, int kerning = 1, int lenght = -1);
+	void writeColored(int x, int y, const std::string &text, Color color, int kerning = 1, int lenght = -1);

 	// Escribe el texto con sombra
-	void writeShadowed(int x, int y, std::string text, Color color, Uint8 shadow_distance = 1, int kerning = 1, int lenght = -1);
+	void writeShadowed(int x, int y, const std::string &text, Color color, Uint8 shadow_distance = 1, int kerning = 1, int lenght = -1);

 	// Escribe el texto centrado en un punto x
-	void writeCentered(int x, int y, std::string text, int kerning = 1, int lenght = -1);
+	void writeCentered(int x, int y, const std::string &text, int kerning = 1, int lenght = -1);

 	// Escribe texto con extras
-	void writeDX(Uint8 flags, int x, int y, std::string text, int kerning = 1, Color textColor = {255, 255, 255}, Uint8 shadow_distance = 1, Color shadow_color = {0, 0, 0}, int lenght = -1);
+	void writeDX(Uint8 flags, int x, int y, const std::string &text, int kerning = 1, Color textColor = Color(), Uint8 shadow_distance = 1, Color shadow_color = Color(), int lenght = -1);

 	// Obtiene la longitud en pixels de una cadena
-	int lenght(std::string text, int kerning = 1) const;
+	int lenght(const std::string &text, int kerning = 1) const;

 	// Devuelve el valor de la variable
 	int getCharacterSize() const;
@@ -79,7 +79,7 @@ public:
 	void setFixedWidth(bool value);

 	// Carga una paleta de colores para el texto
-	void addPalette(std::string path);
+	void addPalette(const std::string &path);

 	// Establece una paleta de colores para el texto
 	void setPalette(int index);
--- a/source/texture.cpp
+++ b/source/texture.cpp
@@ -11,7 +11,7 @@
 #include "stb_image.h" // for stbi_failure_reason, stbi_image_free

 // Constructor
-Texture::Texture(SDL_Renderer *renderer, std::string path)
+Texture::Texture(SDL_Renderer *renderer, const std::string &path)
 	: renderer_(renderer), path_(path)
 {
 	// Inicializa
@@ -23,7 +23,7 @@ Texture::Texture(SDL_Renderer *renderer, std::string path)
 	palettes_.clear();

 	// Carga el fichero en la textura
-	if (path_ != "")
+	if (!path_.empty())
 	{
 		// Obtiene la extensión
 		const std::string extension = path_.substr(path_.find_last_of(".") + 1);
@@ -37,8 +37,8 @@ Texture::Texture(SDL_Renderer *renderer, std::string path)
 		// .gif
 		else if (extension == "gif")
 		{
-			surface_ = loadSurface(path_.c_str());
-			addPalette(path_.c_str());
+			surface_ = loadSurface(path_);
+			addPalette(path_);
 			setPaletteColor(0, 0, 0x00000000);
 			createBlank(width_, height_, SDL_PIXELFORMAT_RGBA8888, SDL_TEXTUREACCESS_STREAMING);
 			SDL_SetTextureBlendMode(texture_, SDL_BLENDMODE_BLEND);
@@ -54,13 +54,12 @@ Texture::~Texture()
 }

 // Carga una imagen desde un fichero
-bool Texture::loadFromFile(std::string path)
+bool Texture::loadFromFile(const std::string &path)
 {
-	const std::string file_name = path.substr(path.find_last_of("\\/") + 1);
 	int req_format = STBI_rgb_alpha;
 	int width, height, orig_format;
 	unsigned char *data = stbi_load(path.c_str(), &width, &height, &orig_format, req_format);
-	if (data == nullptr)
+	if (!data)
 	{
 #ifdef VERBOSE
 		std::cout << "Loading image failed: " << stbi_failure_reason() << std::endl;
@@ -70,19 +69,20 @@ bool Texture::loadFromFile(std::string path)
 	else
 	{
 #ifdef VERBOSE
-		std::cout << "Image loaded: " << file_name.c_str() << std::endl;
+		const std::string file_name = path.substr(path.find_last_of("\\/") + 1);
+		std::cout << "Image loaded: " << file_name << std::endl;
 #endif
 	}

 	int depth, pitch;
 	Uint32 pixel_format;
-	if (req_format == STBI_rgb)
+	/*if (req_format == STBI_rgb)
 	{
 		depth = 24;
 		pitch = 3 * width; // 3 bytes por pixel * pixels por linea
 		pixel_format = SDL_PIXELFORMAT_RGB24;
 	}
-	else
+	else*/
 	{ // STBI_rgb_alpha (RGBA)
 		depth = 32;
 		pitch = 4 * width;
@@ -96,11 +96,11 @@ bool Texture::loadFromFile(std::string path)
 	SDL_Texture *newTexture = nullptr;

 	// Carga la imagen desde una ruta específica
-	SDL_Surface *loadedSurface = SDL_CreateRGBSurfaceWithFormatFrom((void *)data, width, height, depth, pitch, pixel_format);
+	auto loadedSurface = SDL_CreateRGBSurfaceWithFormatFrom(static_cast<void *>(data), width, height, depth, pitch, pixel_format);
 	if (loadedSurface == nullptr)
 	{
 #ifdef VERBOSE
-		std::cout << "Unable to load image " << path.c_str() << std::endl;
+		std::cout << "Unable to load image " << path << std::endl;
 #endif
 	}
 	else
@@ -110,7 +110,7 @@ bool Texture::loadFromFile(std::string path)
 		if (newTexture == nullptr)
 		{
 #ifdef VERBOSE
-			std::cout << "Unable to create texture from " << path.c_str() << "! SDL Error: " << SDL_GetError() << std::endl;
+			std::cout << "Unable to create texture from " << path << "! SDL Error: " << SDL_GetError() << std::endl;
 #endif
 		}
 		else
@@ -137,9 +137,9 @@ bool Texture::createBlank(int width, int height, SDL_PixelFormatEnum format, SDL
 	texture_ = SDL_CreateTexture(renderer_, format, access, width, height);
 	if (!texture_)
 	{
-		#ifdef VERBOSE
+#ifdef VERBOSE
 		std::cout << "Unable to create blank texture! SDL Error: " << SDL_GetError() << std::endl;
-		#endif
+#endif
 	}
 	else
 	{
@@ -239,14 +239,14 @@ SDL_Texture *Texture::getSDLTexture()
 }

 // Crea una nueva surface
-Surface Texture::newSurface(int w, int h)
+/*Surface Texture::newSurface(int w, int h)
 {
-	Surface surf = (Surface)malloc(sizeof(surface_s));
+	Surface surf = static_cast<Surface>(malloc(sizeof(surface_s)));
 	surf->w = w;
 	surf->h = h;
-	surf->data = (Uint8 *)malloc(w * h);
+	surf->data = static_cast<Uint8 *>(malloc(w * h));
 	return surf;
-}
+}*/

 // Elimina una surface
 void Texture::deleteSurface(Surface surface)
@@ -265,29 +265,29 @@ void Texture::deleteSurface(Surface surface)
 }

 // Crea una surface desde un fichero .gif
-Surface Texture::loadSurface(const char *file_name)
+Surface Texture::loadSurface(const std::string &file_name)
 {
-	FILE *f = fopen(file_name, "rb");
+	FILE *f = fopen(file_name.c_str(), "rb");
 	if (!f)
 	{
-		return NULL;
+		return nullptr;
 	}

 	fseek(f, 0, SEEK_END);
 	long size = ftell(f);
 	fseek(f, 0, SEEK_SET);
-	Uint8 *buffer = (Uint8 *)malloc(size);
+	Uint8 *buffer = static_cast<Uint8 *>(malloc(size));
 	fread(buffer, size, 1, f);
 	fclose(f);

 	Uint16 w, h;
 	Uint8 *pixels = LoadGif(buffer, &w, &h);
-	if (pixels == NULL)
+	if (pixels == nullptr)
 	{
-		return NULL;
+		return nullptr;
 	}

-	Surface surface = (Surface)malloc(sizeof(surface_s));
+	Surface surface = static_cast<Surface>(malloc(sizeof(surface_s)));
 	surface->w = w;
 	surface->h = h;
 	surface->data = pixels;
@@ -312,7 +312,7 @@ void Texture::flipSurface()
 	// Vuelca los datos
 	Uint32 *pixels;
 	int pitch;
-	SDL_LockTexture(texture_, nullptr, (void **)&pixels, &pitch);
+	SDL_LockTexture(texture_, nullptr, reinterpret_cast<void **>(&pixels), &pitch);
 	for (int i = 0; i < width_ * height_; ++i)
 	{
 		pixels[i] = palettes_[paletteIndex_][surface_->data[i]];
@@ -327,11 +327,11 @@ void Texture::setPaletteColor(int palette, int index, Uint32 color)
 }

 // Carga una paleta desde un fichero
-std::vector<Uint32> Texture::loadPal(const char *file_name)
+std::vector<Uint32> Texture::loadPal(const std::string &file_name)
 {
 	std::vector<Uint32> palette;

-	FILE *f = fopen(file_name, "rb");
+	FILE *f = fopen(file_name.c_str(), "rb");
 	if (!f)
 	{
 		return palette;
@@ -340,12 +340,12 @@ std::vector<Uint32> Texture::loadPal(const char *file_name)
 	fseek(f, 0, SEEK_END);
 	long size = ftell(f);
 	fseek(f, 0, SEEK_SET);
-	Uint8 *buffer = (Uint8 *)malloc(size);
+	Uint8 *buffer = static_cast<Uint8 *>(malloc(size));
 	fread(buffer, size, 1, f);
 	fclose(f);

-	Uint32 *pal = LoadPalette(buffer);
-	if (pal == nullptr)
+	const auto *pal = LoadPalette(buffer);
+	if (!pal)
 	{
 		return palette;
 	}
@@ -361,9 +361,9 @@ std::vector<Uint32> Texture::loadPal(const char *file_name)
 }

 // Añade una paleta a la lista
-void Texture::addPalette(std::string path)
+void Texture::addPalette(const std::string &path)
 {
-	palettes_.push_back(loadPal(path.c_str()));
+	palettes_.push_back(loadPal(path));
 	setPaletteColor((int)palettes_.size() - 1, 0, 0x00000000);
 }

--- a/source/texture.h
+++ b/source/texture.h
@@ -33,29 +33,29 @@ private:
    int paletteIndex_;                          // Indice de la paleta en uso

    // Crea una nueva surface
-    Surface newSurface(int w, int h);
+    //Surface newSurface(int w, int h);

    // Elimina una surface
    void deleteSurface(Surface surface);

    // Crea una surface desde un fichero .gif
-    Surface loadSurface(const char *file_name);
+    Surface loadSurface(const std::string &file_name);

    // Vuelca la surface en la textura
    void flipSurface(); 

    // Carga una paleta desde un fichero
-    std::vector<Uint32> loadPal(const char *file_name);
+    std::vector<Uint32> loadPal(const std::string &file_name);

 public:
    // Constructor
-    explicit Texture(SDL_Renderer *renderer, std::string path = "");
+    explicit Texture(SDL_Renderer *renderer, const std::string &path = std::string());

    // Destructor
    ~Texture();

    // Carga una imagen desde un fichero
-    bool loadFromFile(std::string path);
+    bool loadFromFile(const std::string &path);

    // Crea una textura en blanco
    bool createBlank(int width, int height, SDL_PixelFormatEnum format = SDL_PIXELFORMAT_RGBA8888, SDL_TextureAccess = SDL_TEXTUREACCESS_STREAMING);
@@ -91,7 +91,7 @@ public:
    SDL_Texture *getSDLTexture();

    // Añade una paleta a la lista
-    void addPalette(std::string path);
+    void addPalette(const std::string &path);

    // Establece un color de la paleta
    void setPaletteColor(int palette, int index, Uint32 color);
--- a/source/tiled_bg.cpp
+++ b/source/tiled_bg.cpp
@@ -1,10 +1,11 @@
 #include "tiled_bg.h"
-#include <SDL2/SDL_pixels.h> // for SDL_PIXELFORMAT_RGBA8888
-#include <SDL2/SDL_stdinc.h> // for SDL_sinf
-#include <stdlib.h>          // for rand
-#include "screen.h"          // for Screen
-#include "sprite.h"          // for Sprite
-#include "texture.h"         // for Texture
+#include <SDL2/SDL_pixels.h>  // for SDL_PIXELFORMAT_RGBA8888
+#include <SDL2/SDL_stdinc.h>  // for SDL_sinf
+#include <stdlib.h>           // for rand
+#include <memory>             // for unique_ptr, make_shared, make_unique
+#include "screen.h"           // for Screen
+#include "sprite.h"           // for Sprite
+#include "texture.h"          // for Texture

 // Constructor
 Tiledbg::Tiledbg(std::string texture_path, SDL_Rect pos, int mode)
--- a/source/title.cpp
+++ b/source/title.cpp
@@ -1,20 +1,24 @@
 #include "title.h"
-#include <SDL2/SDL_keycode.h> // for SDLK_1, SDLK_2, SDLK_3
-#include <SDL2/SDL_render.h>  // for SDL_Renderer
-#include <SDL2/SDL_timer.h>	  // for SDL_GetTicks
-#include <string>			  // for allocator, basic_string, char_traits
-#include <vector>			  // for vector
-#include "asset.h"			  // for Asset
-#include "global_inputs.h"	  // for globalInputs::check
-#include "input.h"			  // for Input, inputs_e, INPUT_DO_NOT_ALLOW_RE...
-#include "jail_audio.h"		  // for JA_GetMusicState, JA_Music_state, JA_P...
-#include "lang.h"			  // for getText
-#include "options.h"		  // for options
-#include "param.h"			  // for param
-#include "screen.h"			  // for Screen
-#include "section.h"		  // for SectionOptions, options, SectionName, name
-
-struct JA_Music_t;
+#include <SDL2/SDL_events.h>   // for SDL_PollEvent, SDL_Event, SDL_KEYDOWN
+#include <SDL2/SDL_keycode.h>  // for SDLK_1, SDLK_2, SDLK_3
+#include <SDL2/SDL_rect.h>     // for SDL_Rect
+#include <SDL2/SDL_render.h>   // for SDL_Renderer
+#include <SDL2/SDL_timer.h>    // for SDL_GetTicks
+#include <string>              // for char_traits, operator+, to_string, bas...
+#include <utility>             // for move
+#include <vector>              // for vector
+#include "asset.h"             // for Asset
+#include "global_inputs.h"     // for check
+#include "input.h"             // for Input, InputType, INPUT_DO_NOT_ALLOW_R...
+#include "jail_audio.h"        // for JA_GetMusicState, JA_Music_state, JA_P...
+#include "lang.h"              // for getText
+#include "options.h"           // for options
+#include "param.h"             // for param
+#include "screen.h"            // for Screen
+#include "section.h"           // for Options, options, Name, name
+#include "texture.h"           // for Texture
+#include "utils.h"             // for Param, OptionsController, Color, Param...
+struct JA_Music_t;  // lines 17-17

 // Constructor
 Title::Title(JA_Music_t *music)
@@ -315,9 +319,9 @@ void Title::resetCounter()
 // Intercambia la asignación de mandos a los jugadores
 void Title::swapControllers()
 {
-	const int num_controllers_ = input_->getNumControllers();
+	const auto num_controllers = input_->getNumControllers();

-	if (num_controllers_ == 0)
+	if (num_controllers == 0)
 	{
 		return;
 	}
@@ -330,7 +334,7 @@ void Title::swapControllers()
 	int playerControllerIndex[MAX_CONTROLLERS];
 	for (int i = 0; i < MAX_CONTROLLERS; ++i)
 	{
-		text[i] = "";
+		text[i].clear();
 		playerControllerIndex[i] = -1;
 	}

--- a/source/title.h
+++ b/source/title.h
@@ -1,24 +1,18 @@
 #pragma once

-#include <SDL2/SDL_events.h> // for SDL_Event
-#include <SDL2/SDL_stdinc.h> // for Uint32
-#include <memory>
-#include "define_buttons.h"
-#include "fade.h"
-#include "game_logo.h"
-#include "text.h"
-#include "tiled_bg.h"
-#include "utils.h" // for Section
-#include "sprite.h"
-#include "texture.h"
-#include "section.h"
-
-class Asset;
-class Input;
-class Screen;
-class Sprite;
-class Texture;
-struct JA_Music_t;
+#include <SDL2/SDL_stdinc.h>  // for Uint32
+#include <memory>             // for unique_ptr, shared_ptr
+#include "define_buttons.h"   // for DefineButtons
+#include "fade.h"             // for Fade
+#include "game_logo.h"        // for GameLogo
+#include "sprite.h"           // for Sprite
+#include "text.h"             // for Text
+#include "tiled_bg.h"         // for Tiledbg
+class Input;  // lines 17-17
+class Screen;  // lines 18-18
+class Texture;  // lines 20-20
+namespace section { enum class Name; }
+struct JA_Music_t;  // lines 21-21

 // Textos
 constexpr const char TEXT_COPYRIGHT[] = "@2020,2024 JailDesigner";
--- a/source/utils.cpp
+++ b/source/utils.cpp
@@ -1,11 +1,10 @@
 #include "utils.h"
-#include <stdlib.h>  // for free, malloc
-#include <algorithm> // for max, min
-#include <cctype>    // for isspace
-#include <iterator>  // for distance
-#include <cmath>
-struct JA_Music_t; // lines 3-3
-struct JA_Sound_t; // lines 4-4
+#include <algorithm> // for min, clamp, find_if_not, transform
+#include <cctype>    // for tolower, isspace
+#include <cmath>     // for cos, pow, M_PI
+#include <compare>   // for operator<
+struct JA_Music_t;   // lines 7-7
+struct JA_Sound_t;   // lines 8-8

 // Colores
 const Color bg_color = {0x27, 0x27, 0x36};
@@ -29,210 +28,112 @@ double distanceSquared(int x1, int y1, int x2, int y2)
 }

 // Detector de colisiones entre dos circulos
-bool checkCollision(Circle &a, Circle &b)
+bool checkCollision(const Circle &a, const Circle &b)
 {
    // Calcula el radio total al cuadrado
-    int total_radius_squared = a.r + b.r;
-    total_radius_squared = total_radius_squared * total_radius_squared;
+    int total_radius_squared = (a.r + b.r) * (a.r + b.r);

-    // Si la distancia entre el centro de los circulos es inferior a la suma de sus radios
-    if (distanceSquared(a.x, a.y, b.x, b.y) < (total_radius_squared))
-    {
-        // Los circulos han colisionado
-        return true;
-    }
-
-    // En caso contrario
-    return false;
+    // Comprueba si la distancia entre los centros de los círculos es inferior a la suma de sus radios
+    return distanceSquared(a.x, a.y, b.x, b.y) < total_radius_squared;
 }

 // Detector de colisiones entre un circulo y un rectangulo
-bool checkCollision(Circle &a, SDL_Rect &b)
+bool checkCollision(const Circle &a, const SDL_Rect &b)
 {
-    // Closest point on collision box
-    int cX, cY;
+    // Encuentra el punto más cercano en el rectángulo
+    int cX = std::clamp(a.x, b.x, b.x + b.w);
+    int cY = std::clamp(a.y, b.y, b.y + b.h);

-    // Find closest x offset
-    if (a.x < b.x)
-    {
-        cX = b.x;
-    }
-    else if (a.x > b.x + b.w)
-    {
-        cX = b.x + b.w;
-    }
-    else
-    {
-        cX = a.x;
-    }
-
-    // Find closest y offset
-    if (a.y < b.y)
-    {
-        cY = b.y;
-    }
-    else if (a.y > b.y + b.h)
-    {
-        cY = b.y + b.h;
-    }
-    else
-    {
-        cY = a.y;
-    }
-
-    // If the closest point is inside the Circle
-    if (distanceSquared(a.x, a.y, cX, cY) < a.r * a.r)
-    {
-        // This box and the Circle have collided
-        return true;
-    }
-
-    // If the shapes have not collided
-    return false;
+    // Si el punto más cercano está dentro del círculo
+    return distanceSquared(a.x, a.y, cX, cY) < a.r * a.r;
 }

 // Detector de colisiones entre dos rectangulos
-bool checkCollision(SDL_Rect &a, SDL_Rect &b)
+bool checkCollision(const SDL_Rect &a, const SDL_Rect &b)
 {
-    // Calcula las caras del rectangulo a
-    const int leftA = a.x;
-    const int rightA = a.x + a.w;
-    const int topA = a.y;
-    const int bottomA = a.y + a.h;
+    const int leftA = a.x, rightA = a.x + a.w, topA = a.y, bottomA = a.y + a.h;
+    const int leftB = b.x, rightB = b.x + b.w, topB = b.y, bottomB = b.y + b.h;

-    // Calcula las caras del rectangulo b
-    const int leftB = b.x;
-    const int rightB = b.x + b.w;
-    const int topB = b.y;
-    const int bottomB = b.y + b.h;
-
-    // Si cualquiera de las caras de a está fuera de b
    if (bottomA <= topB)
-    {
        return false;
-    }
-
    if (topA >= bottomB)
-    {
        return false;
-    }
-
    if (rightA <= leftB)
-    {
        return false;
-    }
-
    if (leftA >= rightB)
-    {
        return false;
-    }

-    // Si ninguna de las caras está fuera de b
    return true;
 }

 // Detector de colisiones entre un punto y un rectangulo
-bool checkCollision(SDL_Point &p, SDL_Rect &r)
+bool checkCollision(const SDL_Point &p, const SDL_Rect &r)
 {
-    // Comprueba si el punto está a la izquierda del rectangulo
-    if (p.x < r.x)
-    {
+    if (p.x < r.x || p.x > r.x + r.w)
        return false;
-    }
-
-    // Comprueba si el punto está a la derecha del rectangulo
-    if (p.x > r.x + r.w)
-    {
+    if (p.y < r.y || p.y > r.y + r.h)
        return false;
-    }
-
-    // Comprueba si el punto está por encima del rectangulo
-    if (p.y < r.y)
-    {
-        return false;
-    }
-
-    // Comprueba si el punto está por debajo del rectangulo
-    if (p.y > r.y + r.h)
-    {
-        return false;
-    }
-
-    // Si no está fuera, es que está dentro
    return true;
 }

 // Convierte una cadena en un valor booleano
-bool stringToBool(std::string str)
+bool stringToBool(const std::string &str)
 {
-    return str == "true" ? true : false;
+    return str == "true";
 }

 // Convierte un valor booleano en una cadena
 std::string boolToString(bool value)
 {
-    return value == true ? "true" : "false";
+    return value ? "true" : "false";
 }

 // Convierte un valor booleano en una cadena "on" o "off"
 std::string boolToOnOff(bool value)
 {
-    return value == true ? "on" : "off";
+    return value ? "on" : "off";
 }

 // Convierte una cadena a minusculas
-std::string toLower(std::string str)
+std::string toLower(const std::string &str)
 {
-    const char *original = str.c_str();
-    char *lower = (char *)malloc(str.size() + 1);
-    for (int i = 0; i < (int)str.size(); ++i)
-    {
-        char c = original[i];
-        lower[i] = (c >= 65 && c <= 90) ? c + 32 : c;
-    }
-    lower[str.size()] = 0;
-    std::string nova(lower);
-    free(lower);
-    return nova;
+    std::string result = str;
+    std::transform(result.begin(), result.end(), result.begin(),
+                   [](unsigned char c)
+                   { return std::tolower(c); });
+    return result;
 }

 // Obtiene el fichero de sonido a partir de un nombre
-JA_Sound_t *getSound(std::vector<SoundFile> sounds, std::string name)
+JA_Sound_t *getSound(const std::vector<SoundFile> &sounds, const std::string &name)
 {
-    for (auto s : sounds)
+    for (const auto &s : sounds)
    {
        if (s.name == name)
        {
            return s.file;
        }
    }
-
    return nullptr;
 }

 // Obtiene el fichero de música a partir de un nombre
-JA_Music_t *getMusic(std::vector<MusicFile> music, std::string name)
+JA_Music_t *getMusic(const std::vector<MusicFile> &music, const std::string &name)
 {
-    for (auto m : music)
+    for (const auto &m : music)
    {
        if (m.name == name)
        {
            return m.file;
        }
    }
-
    return nullptr;
 }

 // Ordena las entradas de la tabla de records
-HiScoreEntry sortHiScoreTable(HiScoreEntry entry1, HiScoreEntry entry2)
+HiScoreEntry sortHiScoreTable(const HiScoreEntry &entry1, const HiScoreEntry &entry2)
 {
-    if (entry1.score > entry2.score)
-    {
-        return entry1;
-    }
-
-    return entry2;
+    return (entry1.score > entry2.score) ? entry1 : entry2;
 }

 // Dibuja un circulo
@@ -275,41 +176,31 @@ void DrawCircle(SDL_Renderer *renderer, int32_t centerX, int32_t centerY, int32_
 }

 // Aclara el color
-Color lightenColor(Color color, int amount)
+Color lightenColor(const Color &color, int amount)
 {
    Color newColor;
-    newColor.r = std::min(255, (int)color.r + amount);
-    newColor.g = std::min(255, (int)color.g + amount);
-    newColor.b = std::min(255, (int)color.b + amount);
+    newColor.r = std::min(255, color.r + amount);
+    newColor.g = std::min(255, color.g + amount);
+    newColor.b = std::min(255, color.b + amount);
    return newColor;
 }

 // Oscurece el color
-Color DarkenColor(Color color, int amount)
+Color DarkenColor(const Color &color, int amount)
 {
    Color newColor;
-    newColor.r = std::max(0, (int)color.r - amount);
-    newColor.g = std::max(0, (int)color.g - amount);
-    newColor.b = std::max(0, (int)color.b - amount);
+    newColor.r = std::min(255, color.r - +amount);
+    newColor.g = std::min(255, color.g - +amount);
+    newColor.b = std::min(255, color.b - +amount);
    return newColor;
 }

 // Quita los espacioes en un string
 std::string trim(const std::string &str)
 {
-    auto start = str.begin();
-    while (start != str.end() && std::isspace(*start))
-    {
-        start++;
-    }
-
-    auto end = str.end();
-    do
-    {
-        end--;
-    } while (std::distance(start, end) > 0 && std::isspace(*end));
-
-    return std::string(start, end + 1);
+    auto start = std::find_if_not(str.begin(), str.end(), ::isspace);
+    auto end = std::find_if_not(str.rbegin(), str.rend(), ::isspace).base();
+    return (start < end ? std::string(start, end) : std::string());
 }

 // Función de suavizado
--- a/source/utils.h
+++ b/source/utils.h
@@ -3,16 +3,19 @@
 #include <SDL2/SDL_gamecontroller.h> // for SDL_GameControllerButton
 #include <SDL2/SDL_rect.h>			 // for SDL_Rect, SDL_Point
 #include <SDL2/SDL_render.h>		 // for SDL_Renderer
-#include <SDL2/SDL_stdinc.h>		 // for Uint8, Uint32
+#include <SDL2/SDL_stdinc.h>		 // for Uint8
 #include <stdint.h>					 // for int32_t
-#include <string>					 // for string, basic_string
+#include <string>					 // for string
 #include <vector>					 // for vector
-#include "input.h"					 // for inputs_e
-#include "lang.h"
-struct JA_Music_t;
-struct JA_Sound_t;
-enum class ScreenFilter;
-enum class ScreenVideoMode;
+enum class InputType : int;
+enum class ScreenFilter;	// lines 14-14
+enum class ScreenVideoMode; // lines 15-15
+namespace lang
+{
+	enum class Code : int;
+}
+struct JA_Music_t; // lines 12-12
+struct JA_Sound_t; // lines 13-13

 // Dificultad del juego
 enum class GameDifficulty
@@ -35,6 +38,7 @@ struct Circle
 struct Color
 {
 	Uint8 r, g, b;
+	constexpr Color(int red = 255, int green = 255, int blue = 255) : r(red), g(green), b(blue) {}
 };

 // Posiciones de las notificaciones
@@ -47,13 +51,6 @@ enum class NotifyPosition
 	RIGHT,
 };

-// Estructura para saber la seccion y subseccion del programa
-/*struct Section
-{
-	section::Name name;
-	section::Options options;
-};*/
-
 // Estructura para las entradas de la tabla de recirds
 struct HiScoreEntry
 {
@@ -114,7 +111,7 @@ struct OptionsAudio
 struct OptionsGame
 {
 	GameDifficulty difficulty;				  // Dificultad del juego
-	lang::Code language;							  // Idioma usado en el juego
+	lang::Code language;					  // Idioma usado en el juego
 	bool autofire;							  // Indica si el jugador ha de pulsar repetidamente para disparar o basta con mantener pulsado
 	std::vector<HiScoreEntry> hi_score_table; // Tabla con las mejores puntuaciones
 };
@@ -235,19 +232,19 @@ struct MusicFile
 double distanceSquared(int x1, int y1, int x2, int y2);

 // Detector de colisiones entre dos circulos
-bool checkCollision(Circle &a, Circle &b);
+bool checkCollision(const Circle &a, const Circle &b);

 // Detector de colisiones entre un circulo y un rectangulo
-bool checkCollision(Circle &a, SDL_Rect &b);
+bool checkCollision(const Circle &a, const SDL_Rect &b);

 // Detector de colisiones entre un dos rectangulos
-bool checkCollision(SDL_Rect &a, SDL_Rect &b);
+bool checkCollision(const SDL_Rect &a, const SDL_Rect &b);

 // Detector de colisiones entre un punto y un rectangulo
-bool checkCollision(SDL_Point &p, SDL_Rect &r);
+bool checkCollision(const SDL_Point &p, const SDL_Rect &r);

 // Convierte una cadena en un valor booleano
-bool stringToBool(std::string str);
+bool stringToBool(const std::string &str);

 // Convierte un valor booleano en una cadena
 std::string boolToString(bool value);
@@ -256,25 +253,25 @@ std::string boolToString(bool value);
 std::string boolToOnOff(bool value);

 // Convierte una cadena a minusculas
-std::string toLower(std::string str);
+std::string toLower(const std::string &str);

 // Obtiene el fichero de sonido a partir de un nombre
-JA_Sound_t *getSound(std::vector<SoundFile> sounds, std::string name);
+JA_Sound_t *getSound(const std::vector<SoundFile> &sounds, const std::string &name);

 // Obtiene el fichero de música a partir de un nombre
-JA_Music_t *getMusic(std::vector<MusicFile> music, std::string name);
+JA_Music_t *getMusic(const std::vector<MusicFile> &music, const std::string &name);

 // Ordena las entradas de la tabla de records
-HiScoreEntry sortHiScoreTable(HiScoreEntry entry1, HiScoreEntry entry2);
+HiScoreEntry sortHiScoreTable(const HiScoreEntry &entry1, const HiScoreEntry &entry2);

 // Dibuja un circulo
 void DrawCircle(SDL_Renderer *renderer, int32_t centerX, int32_t centerY, int32_t radius);

 // Aclara el color
-Color lightenColor(Color color, int amount);
+Color lightenColor(const Color &color, int amount);

 // Oscurece el color
-Color DarkenColor(Color color, int amount);
+Color DarkenColor(const Color &color, int amount);

 // Quita los espacioes en un string
 std::string trim(const std::string &str);
--- a/source/writer.cpp
+++ b/source/writer.cpp
@@ -1,23 +1,21 @@
 #include "writer.h"
+#include "text.h"  // for Text

 // Constructor
 Writer::Writer(std::shared_ptr<Text> text)
-	: text_(text)
-{
-	// Inicializa variables
-	pos_x_ = 0;
-	pos_y_ = 0;
-	kerning_ = 0;
-	caption_ = "";
-	speed_ = 0;
-	writing_counter_ = 0;
-	index_ = 0;
-	lenght_ = 0;
-	completed_ = false;
-	enabled_ = false;
-	enabled_counter_ = 0;
-	finished_ = false;
-}
+	: text_(text),
+	  pos_x_(0),
+	  pos_y_(0),
+	  kerning_(0),
+	  caption_(std::string()),
+	  speed_(0),
+	  writing_counter_(0),
+	  index_(0),
+	  lenght_(0),
+	  completed_(false),
+	  enabled_(false),
+	  enabled_counter_(0),
+	  finished_(false) {}

 // Actualiza el objeto
 void Writer::update()
@@ -45,7 +43,11 @@ void Writer::update()
 		else
 		{
 			// Completado
-			enabled_counter_ > 0 ? enabled_counter_-- : finished_ = true;
+			finished_ = enabled_counter_ <= 0;
+			if (!finished_)
+			{
+				enabled_counter_--;
+			}
 		}
 	}
 }
@@ -78,7 +80,7 @@ void Writer::setKerning(int value)
 }

 // Establece el valor de la variable
-void Writer::setCaption(std::string text)
+void Writer::setCaption(const std::string &text)
 {
 	caption_ = text;
 	lenght_ = text.length();
--- a/source/writer.h
+++ b/source/writer.h
@@ -1,10 +1,10 @@
 #pragma once

-#include <string> // for string, basic_string
-#include <memory>
-#include "text.h"
+#include <memory>  // for shared_ptr
+#include <string>  // for string
+class Text;

-// Clase Writer. Pinta texto en pantalla letra a letra a partir de una cadena y un bitmap
+// Clase Writer. Pinta texto en pantalla letra a letra a partir de una cadena y un objeto Text
 class Writer
 {
 private:
@@ -48,7 +48,7 @@ public:
 	void setKerning(int value);

 	// Establece el valor de la variable
-	void setCaption(std::string text);
+	void setCaption(const std::string &text);

 	// Establece el valor de la variable
 	void setSpeed(int value);
Author	SHA1	Message	Date
Sergio Valor	9825c7fb9b	Pasaeta de include-what-you-use	2024-10-13 21:58:36 +02:00
Sergio Valor	d0a6e4c572	Afegits destructors virtuals en les classes Sprite	2024-10-13 21:23:15 +02:00
Sergio Valor	7c876e1d4d	Acabat amb cppcheck Arreglades les herencies de les classes Sprite	2024-10-13 21:00:33 +02:00
Sergio Valor	809c10048e	Commit pa poder tornar a passar el cppcheck	2024-10-13 19:39:43 +02:00
Sergio Valor	babf02226c	Mes recomanacions de cppcheck	2024-10-13 19:26:27 +02:00
Sergio Valor	46540ad7c3	Optimitzat el tema de comparacions i asignacions de strings buits. Mes que optimitzat, ara està mes mono	2024-10-13 14:25:05 +02:00
Sergio Valor	ba7c44ad06	Actualitzat Makefile	2024-10-13 14:24:15 +02:00