JailDesigner
424d0d2b89
Resuelve la categoría de findings de tidy que son bugs reales o cambios
de tipo concretos, no transforms automáticos:
Bugs reales (bugprone-* y clang-diagnostic-*):
- bugprone-empty-catch en postfx_config.cpp: el catch silencioso del
parser RGB era intencional (fallback a defaults si el array no parsea
como int). Marcado con @INTENTIONAL (keyword ya configurado en
.clang-tidy via bugprone-empty-catch.IgnoreCatchWithKeywords) y
comentario ampliado explicando la decisión.
- clang-diagnostic-unused-private-field en Starfield: el campo
'densitat_' se asignaba en el constructor pero nunca se leía. El
parámetro 'densitat' se reparte directamente en las CapaConfig al
construir, así que el field era código muerto. Eliminado.
- bugprone-branch-clone en vector_text.cpp: el switch de
get_shape_filename tenía dos grupos consecutivos (dígitos 0-9 y
mayúsculas A-Z) con cuerpo idéntico. Fusionados en un único case con
comentario explicando que comparten path porque la shape se llama
igual que el caracter.
- bugprone-switch-missing-default-case en Input::handleEvent: el switch
manejaba solo SDL_EVENT_GAMEPAD_ADDED/REMOVED y caía por fall-through
a un return {} fuera del switch. Añadido default: explícito con
comentario sobre qué hace Input vs el resto del sistema.
- bugprone-implicit-widening-of-multiplication-result en
GameScene::bullets_ y Collision::Context::bullets: 'MAX_BALES * 2'
es int*int y se widening implícitamente a std::size_t para el
template arg de std::array. Cast explícito a size_t en ambos sitios.
Otros mecánicos:
- performance-enum-size: 10 enums sin tipo subyacente pasaron a
': std::uint8_t' (PrimitiveType, InputAction, Mode, SceneType,
Option, AnimationState, TitleState, ModeSpawn, EstatStage,
ShipState). #include <cstdint> añadido donde faltaba.
- modernize-use-equals-default en SpawnController: el ctor por
defecto tenía cuerpo vacío ({}). Pasado a '= default;'.
Cero supresiones. La única "marca" es @INTENTIONAL en el empty-catch,
que es el mecanismo configurado en el .clang-tidy del proyecto para
distinguir intencionales de accidentales.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
84 lines
2.8 KiB
C++
84 lines
2.8 KiB
C++
// scene_context.hpp - Sistema de gestió de escenes i context de transiciones
|
|
// © 2026 JailDesigner
|
|
|
|
#pragma once
|
|
|
|
#include <cstdint>
|
|
|
|
#include "core/system/game_config.hpp"
|
|
|
|
namespace SceneManager {
|
|
|
|
// Context de transición entre escenes
|
|
// Conté l'escena destinació i opciones específiques per aquella escena
|
|
class SceneContext {
|
|
public:
|
|
// Tipo de escena del juego
|
|
enum class SceneType : std::uint8_t {
|
|
LOGO, // Pantalla de start (logo JAILGAMES)
|
|
TITLE, // Pantalla de título con menú
|
|
GAME, // Juego principal (Asteroids)
|
|
EXIT // Salir del programa
|
|
};
|
|
|
|
// Opciones específiques para cada escena
|
|
enum class Option : std::uint8_t {
|
|
NONE, // Sin opciones especials (comportament per defecte)
|
|
JUMP_TO_TITLE_MAIN, // TITLE: Saltar directament a MAIN (starfield instantani)
|
|
// MODE_DEMO, // GAME: Mode demostració con IA (futur)
|
|
};
|
|
|
|
// Constructor inicial con escena LOGO i sin opciones
|
|
SceneContext() = default;
|
|
|
|
// Canviar escena con opción específica
|
|
void setNextScene(SceneType next_scene, Option option = Option::NONE) {
|
|
next_scene_ = next_scene;
|
|
option_ = option;
|
|
}
|
|
|
|
// Consultar escena destinació
|
|
[[nodiscard]] auto nextScene() const -> SceneType {
|
|
return next_scene_;
|
|
}
|
|
|
|
// Consultar opción actual
|
|
[[nodiscard]] auto option() const -> Option {
|
|
return option_;
|
|
}
|
|
|
|
// Consumir opción (retorna value i reseteja a NONE)
|
|
// Utilitzar cuando l'escena processa l'opción
|
|
[[nodiscard]] auto consumeOption() -> Option {
|
|
Option value = option_;
|
|
option_ = Option::NONE;
|
|
return value;
|
|
}
|
|
|
|
// Reset opción a NONE (sin retornar value)
|
|
void resetOption() {
|
|
option_ = Option::NONE;
|
|
}
|
|
|
|
// Configurar match antes de transicionar a GAME
|
|
void setMatchConfig(const GameConfig::MatchConfig& config) {
|
|
match_config_ = config;
|
|
}
|
|
|
|
// Obtenir configuración de match (consumit per GameScene)
|
|
[[nodiscard]] auto getMatchConfig() const -> const GameConfig::MatchConfig& {
|
|
return match_config_;
|
|
}
|
|
|
|
private:
|
|
SceneType next_scene_{SceneType::LOGO}; // SceneType a la qual transicionar
|
|
Option option_{Option::NONE}; // Opción específica per l'escena
|
|
GameConfig::MatchConfig match_config_; // Configuración de match (jugadors active, mode)
|
|
};
|
|
|
|
// Variable global inline per gestionar l'escena actual (backward compatibility)
|
|
// Sincronitzada con context.nextScene() por el Director
|
|
inline SceneContext::SceneType actual = SceneContext::SceneType::LOGO;
|
|
|
|
} // namespace SceneManager
|