Arreglado el color del marcador al cambiar de paleta

source/common/utils.cpp

@@ -575,4 +575,14 @@ std::string boolToString(bool value)
     {
         return "false";
     }
+}
+
+// Compara dos colores
+bool colorAreEqual(color_t color1, color_t color2)
+{
+    const bool r = color1.r == color2.r;
+    const bool g = color1.g == color2.g;
+    const bool b = color1.b == color2.b;
+
+    return (r && g && b);
 }

source/common/utils.h

@@ -125,4 +125,7 @@ bool stringToBool(std::string str);
 // Convierte un valor booleano en una cadena
 std::string boolToString(bool value);
 
+// Compara dos colores
+bool colorAreEqual(color_t color1, color_t color2);
+
 #endif

source/game.cpp

@@ -52,8 +52,8 @@ Game::Game(SDL_Renderer *renderer, Screen *screen, Resource *resource, Asset *as
 	blackScreen = false;
 	blackScreenCounter = 0;
 
-	this->player->setInvincible(debug->getEnabled());
-	board.music = !debug->getEnabled();
+	// this->player->setInvincible(debug->getEnabled());
+	// board.music = !debug->getEnabled();
 
 	section.name = SECTION_PROG_GAME;
 	section.subsection = 0;
@@ -100,7 +100,7 @@ void Game::checkEventHandler()
 
 			case SDL_SCANCODE_D:
 				debug->switchEnabled();
-				player->setInvincible(debug->getEnabled());
+				options->invincible = debug->getEnabled();
 				board.music = !debug->getEnabled();
 				board.music ? JA_ResumeMusic() : JA_PauseMusic();
 				break;
@@ -312,11 +312,9 @@ bool Game::changeRoom(std::string file)
 		// Crea un objeto habitación nuevo a partir del fichero
 		room = new Room(resource->getRoom(file), renderer, screen, asset, options, itemTracker, &board.items, debug);
 
-		// Actualiza el marcador
-		const color_t c = room->getBorderColor(); // Obtiene el color del vorde
-		const color_t cBlack = stringToColor(options->palette, "black");
-		board.color = (c.r == cBlack.r && c.g == cBlack.g && c.b == cBlack.b) ? stringToColor(options->palette, "white") : c; // Si el color es negro lo cambia a blanco
-		// board.color = (c.r + c.g + c.b == 0) ? stringToColor(options->palette, "white") : c; // Si el color es negro lo cambia a blanco
+		// Pone el color del marcador en función del color del borde de la habitación
+		setScoreBoardColor();
+
 		if (roomTracker->addRoom(file))
 		{ // Incrementa el contador de habitaciones visitadas
 			board.rooms++;
@@ -384,7 +382,7 @@ void Game::checkEndGame()
 // Mata al jugador
 void Game::killPlayer()
 {
-	if (this->player->getInvincible() || options->invincible)
+	if (options->invincible)
 	{
 		return;
 	}
@@ -437,12 +435,13 @@ void Game::switchPalette()
 	// Modifica la variable
 	options->palette = (options->palette == p_zxspectrum) ? p_zxarne : p_zxspectrum;
 
+	// Recarga las paletas
 	room->reLoadPalette();
 	player->reLoadPalette();
 	scoreboard->reLoadPalette();
 
-	const color_t c = room->getBorderColor();
-	board.color = (c.r + c.g + c.b == 0) ? stringToColor(options->palette, "white") : c; // Si el color es negro lo cambia a blanco
+	// Pone el color del marcador en función del color del borde de la habitación
+	setScoreBoardColor();
 }
 
 // Establece la pantalla en negro
@@ -477,4 +476,19 @@ void Game::renderBlackScreen()
 		screen->clean();
 		screen->setBorderColor(stringToColor(options->palette, "black"));
 	}
+}
+
+// Pone el color del marcador en función del color del borde de la habitación
+void Game::setScoreBoardColor()
+{
+	// Obtiene el color del borde
+	const color_t c = room->getBorderColor();
+
+	// Si el color es negro lo cambia a blanco
+	const color_t cBlack = stringToColor(options->palette, "black");
+	board.color = colorAreEqual(c, cBlack) ? stringToColor(options->palette, "white") : c;
+
+	// Si el color es negro brillante lo cambia a blanco
+	const color_t cBrightBlack = stringToColor(options->palette, "bright_black");
+	board.color = colorAreEqual(c, cBrightBlack) ? stringToColor(options->palette, "white") : c;
 }

source/game.h

@@ -43,17 +43,17 @@ private:
 	Test *test;
 
 	// Variables
-	JA_Music music;			 // Musica que suena durante el juego
-	Uint32 ticks;			 // Contador de ticks para ajustar la velocidad del programa
-	Uint32 ticksSpeed;		 // Velocidad a la que se repiten los bucles del programa
-	section_t section;		 // Seccion actual dentro del juego
-	std::string currentRoom; // Fichero de la habitación actual
-	playerSpawn_t spawnPoint;	 // Lugar de la habitación donde aparece el jugador
-	JA_Sound deathSound;	 // Sonido a reproducir cuando muere el jugador
-	board_t board;			 // Estructura con los datos del marcador
-	bool paused;			 // Indica si el juego se encuentra en pausa
-	bool blackScreen;		 // Indica si la pantalla está en negro. Se utiliza para la muerte del jugador
-	int blackScreenCounter;	 // Contador para temporizar la pantalla en negro
+	JA_Music music;			  // Musica que suena durante el juego
+	Uint32 ticks;			  // Contador de ticks para ajustar la velocidad del programa
+	Uint32 ticksSpeed;		  // Velocidad a la que se repiten los bucles del programa
+	section_t section;		  // Seccion actual dentro del juego
+	std::string currentRoom;  // Fichero de la habitación actual
+	playerSpawn_t spawnPoint; // Lugar de la habitación donde aparece el jugador
+	JA_Sound deathSound;	  // Sonido a reproducir cuando muere el jugador
+	board_t board;			  // Estructura con los datos del marcador
+	bool paused;			  // Indica si el juego se encuentra en pausa
+	bool blackScreen;		  // Indica si la pantalla está en negro. Se utiliza para la muerte del jugador
+	int blackScreenCounter;	  // Contador para temporizar la pantalla en negro
 
 	// Actualiza el juego, las variables, comprueba la entrada, etc.
 	void update();
@@ -109,6 +109,9 @@ private:
 	// Dibuja la pantalla negra
 	void renderBlackScreen();
 
+	// Pone el color del marcador en función del color del borde de la habitación
+	void setScoreBoardColor();
+
 public:
 	// Constructor
 	Game(SDL_Renderer *renderer, Screen *screen, Resource *resource, Asset *asset, options_t *options, Input *input, Debug *debug);

source/player.cpp

@@ -21,7 +21,6 @@ Player::Player(player_t player)
     color = stringToColor(options->palette, "white");
     onBorder = false;
     border = BORDER_TOP;
-    invincible = false;
     autoMovement = false;
     alive = true;
     maxFallHeight = BLOCK * 4;
@@ -433,6 +432,13 @@ void Player::move()
         autoMovement = false;
     }
 
+    // Si ha salido de una superficie automatica, detiene el movimiento automatico
+    if (state == s_standing && isOnFloor() && !isOnAutoSurface())
+    {
+        // Deja de estar enganchado a la superficie automatica
+        autoMovement = false;
+    }
+
     // Se mueve hacia arriba
     if (vy < 0.0f)
     {
@@ -688,7 +694,7 @@ playerSpawn_t Player::getSpawnParams()
 void Player::reLoadTexture()
 {
     sprite->getTexture()->reLoad();
-    //texture->reLoad();
+    // texture->reLoad();
 }
 
 // Recarga la paleta
@@ -729,18 +735,6 @@ void Player::updateFeet()
     feet[1] = {p.x + 7, p.y + h - 1};
 }
 
-// Obtiene el valor de la variable
-bool Player::getInvincible()
-{
-    return invincible;
-}
-
-// Establece el valor de la variable
-void Player::setInvincible(bool value)
-{
-    invincible = value;
-}
-
 // Cambia el estado del jugador
 void Player::setState(state_e value)
 {

source/player.h

@@ -76,7 +76,6 @@ public:
 	state_e prevState;					   // Estado previo en el que se encontraba el jugador
 	bool onBorder;						   // Indica si el jugador esta en uno de los cuatro bordes de la pantalla
 	int border;							   // Indica en cual de los cuatro bordes se encuentra
-	bool invincible;					   // Si es invencible, no puede morir
 	bool autoMovement;					   // Indica si esta siendo arrastrado por una superficie automatica
 	bool paused;						   // Indica si el jugador esta en modo pausa
 	SDL_Rect lastPosition;				   // Contiene la ultima posición del jugador, por si hay que deshacer algun movimiento
@@ -181,12 +180,6 @@ public:
 	// Establece el valor de la variable
 	void setRoom(Room *room);
 
-	// Obtiene el valor de la variable
-	bool getInvincible();
-
-	// Establece el valor de la variable
-	void setInvincible(bool value);
-
 	// Comprueba si el jugador esta vivo
 	bool isAlive();