JailDesigner
53bf3a1ff2
`renderBackground`, `renderSarcsRow`, `getSarcRowBottomY` i els índexs de paleta `HUD_BG/WALL/FLOOR/SARC` no tenien callers: el Tilemap s'encarrega del render visual i els sarcs passaran a ser actors sprite. `getSarcRect` es queda (la usa `collides`). Includes orfes de `screen.hpp`, `resource_cache.hpp` i `defines.hpp` eliminats. DEAD-04 (ctor Map) ja es va resoldre durant HC-03.
10 KiB
10 KiB
Auditoría source/game/
Rama: audit/game.
Cobertura: source/game/{entities,map,scenes,ui} + source/game/options.{hpp,cpp} (12 fitxers, ~1.486 línies).
Estados posibles:
PENDING— identificat, sense tocarWIP— diff proposat, pendent d'aplicarDONE— aplicat i commiteatWONTFIX— es decideix no tocar (justificació a la nota)DEFERRED— es farà en fase posterior
Els IDs es mantenen estables una vegada assignats. Cada entrada apunta al fitxer i línies afectades en el moment del clone (branca main).
Top 5 prioritari
| ID | Títol | Estat |
|---|---|---|
| HC-03 | Geometria del mapa cablejada a map.hpp (bloqueja "pirámide a pirámide") |
DONE |
| COUP-01 | core/ inclou game/ui/notification_presets.hpp (dependència invertida) |
DONE |
| DUP-01 | PostFX i CrtPi són dos pipelines duplicats a options.cpp |
DONE |
| PATCH-04 | Reescriure silenciosament config.yaml al fallar parseig o versió |
DONE |
| HC-01 | SPEED_PX_PER_S, caixa de col·lisió i rutes d'assets cablejades |
DONE (HC-01); PENDING (HC-02, HC-06) |
1. Duplicació
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| DUP-01 | options.cpp:152-335,443-547 |
PostFX i CrtPi tenen default*Presets/parse*Preset/write*Preset/load*FromFile/save*ToFile/set*File duplicats |
Importante | DONE | Afegits loadPresetsFromFile<T> i savePresetsToFile<T> a l'anonymous namespace; els 4 entry-points passen a ser one-liners que injecten parse/write/defaults per tipus. Els fitxers postfx.yaml/crtpi.yaml regenerats són idèntics a l'anterior. std::cerr manté el comportament — INCONS-02 el corregirà |
| DUP-02 | player.cpp:100-140 |
snapToHorizontalLane i snapToVerticalLane són el mateix algorisme amb eixos permutats |
Importante | PENDING | Unificar en snapToLane(Axis, delta_time) |
| DUP-03 | options.cpp:14-62 |
Tres parelles enumToString/enumFromString amb la mateixa forma |
Menor | PENDING | Taula constexpr + helpers genèrics |
| DUP-04 | options.cpp:79-149 |
Cada parse* repeteix if (node.contains) { ... get_value<T> } |
Menor | PENDING | Helper readField<T>(node, key, dst) |
| DUP-05 | player.cpp:81-88 |
Càlcul MIN_/MAX_ + clamp per eix és la mateixa fórmula que el snap |
Menor | PENDING | Fold en helper |
2. Hardcoding
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| HC-01 | player.cpp:15 |
SPEED_PX_PER_S = 60.0F cablejat |
Importante | DONE | Afegit Defaults::Player::SPEED_PX_PER_S a utils/defaults.hpp (patró usat a l'audit d'àudio); player.cpp fa alias local per reduir soroll. TODO-comment ("si més endavant volem sprint...") eliminat (PATCH-02) |
| HC-02 | player.hpp:21-35 |
SPRITE_WIDTH/HEIGHT, LEGACY_BASE_SIZE, COLLISION_SIZE cablejats |
Importante | DONE | Bloc collision: afegit a player.yaml (size, base_size); Player exposa inline static int sprite_*/base_size/collision_* recalculats al loadFrom(). GameScene crida el load abans del spawn (mateix patró que Map) |
| HC-03 | map.hpp:18-62 |
Tota la geometria del mapa + 4 índexs de paleta en .hpp |
Crítico | DONE | Bloc geometry: afegit a pyramid.yaml; Map exposa inline static int + loadFrom(yaml); derivats (CORRIDOR/BOARD/ORIGIN/PLAYFIELD) recalculats al load. constexpr → const en els locals dependents a player.cpp i game_scene.cpp. Índexs de paleta queden per DEAD-02 |
| HC-04 | notification_presets.hpp:24-38 |
text_color, bg_color, time_ms en constexpr |
Menor | PENDING | Bloc notifications: a config.yaml o tema extern |
| HC-05 | ui_palette.hpp:12-21 |
Índexs de paleta (RENDER_INFO_TEXT=47, SHADOW=34, CANVAS_BG=65) en constexpr |
Menor | PENDING | Bloc ui: a config.yaml |
| HC-06 | game_scene.cpp:24-26,52 |
Rutes data/maps/pyramid.yaml, data/player/player.*, pyramid_1.ogg cablejades |
Importante | DONE | Afegit Defaults::Pyramid::{MAP_YAML,PLAYER_SPRITE,PLAYER_YAML,MUSIC_TRACK}; GameScene::Config struct + defaultConfig() passa les rutes al ctor (default ve de Defaults). Player ctor accepta anim_key en lloc de cablejar "player.yaml". Cache key = basename (mateix conveni que abans) |
| HC-07 | options.cpp:27,55 |
Fallbacks "letterbox"/"off" com a strings màgics que dupliquen els enums |
Menor | PENDING | Passar per toString(Defaults::...) |
| HC-08 | tilemap.cpp:135 |
0xFF000000U com a màscara alfa |
Menor | PENDING | Constant amb nom a defines.hpp |
3. Parxes i improvisacions
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| PATCH-01 | player.cpp:66-77 |
Col·lisió per eix fent AnimatedSprite::update + revertir + reaplicar |
Importante | PENDING | tryMove(dx, dy, collisionFn) a la classe base |
| PATCH-02 | player.cpp:13-15 |
Comentari tipus TODO ("si més endavant volem sprint...") | Menor | DONE | Eliminat en HC-01 (la constant ja no viu al .cpp, sinó a Defaults::Player) |
| PATCH-03 | map.cpp:13-15 |
overlapStrict custom perquè SDL3 considera el toc com intersecció |
Menor | PENDING | Si s'usa més, mourea utils/ |
| PATCH-04 | options.cpp:361-363,371-375,475-480,528-533 |
Reescriure silenciosament config.yaml al fallar parseig o canviar versió |
Crítico | DONE | Helper backupFile() que copia a <path>.bak si existeix. Cridat abans de cada reescriptura forçada (versió mismatch + catch a loadFromFile, presets: absent/invàlid/buit + catch a loadPresetsFromFile). Primera execució no genera .bak. Missatge explícit per motiu de reescriptura |
| PATCH-05 | game_scene.cpp:51 |
if (Audio::get() != nullptr) com a defensa sobre un singleton Meyers |
Menor | PENDING | Assumir invariant, treure |
| PATCH-06 | game_scene.cpp:78-91 |
Ordenació per getFeetY() amb 1 jugador i sense sarcs-sprite: cerimonial |
Menor | PENDING | Comentar "reactivar quan hi haja sarcs-sprite" o treure |
| PATCH-07 | map.hpp:64 + map.cpp:19 |
Map declara ctor però tots els mètodes són static (transició a mitges) |
Importante | PENDING | Decidir: namespace o instanciar |
4. Acoplament innecessari
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| COUP-01 | core/input/global_inputs.cpp:15 |
core/ inclou game/ui/notification_presets.hpp |
Crítico | DONE | notification_presets.hpp mogut a core/rendering/ (al costat del Notifier que consumeix). global_inputs.cpp i el comentari de notifier.hpp actualitzats |
| COUP-02 | player.cpp + Map::PLAYFIELD_*, PITCH_*, GRID_*, collides |
Player coneix la geometria interna del mapa |
Importante | PENDING | Passar const MapInfo& o un World* al ctor del player |
| COUP-03 | options.hpp:96-105 |
Globals inline mutats lliurement des de core/input/global_inputs.cpp |
Importante | PENDING | Setters + signal quan canvien |
| COUP-04 | tilemap.cpp:117-138 |
Clip+blit directe al canvas ARGB que podria viure a Surface |
Menor | PENDING | blitInto(canvas, origin) genèric |
5. Inconsistències internes
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| INCONS-01 | game/* |
3 estils de mòdul: namespace+inline (Options), classe all-static (Map), OOP (Player/Tilemap), namespace+constexpr (UiColor/Notification) | Importante | PENDING | Decidir i uniformar |
| INCONS-02 | options.cpp:372,477,530,579,607 |
std::cerr en comptes de std::cout (convenció del projecte) |
Importante | DONE | Totes les ocurrències (10, incloent les afegides a PATCH-04) substituïdes per std::cout. Test manual: amb config.yaml trencat el missatge de backup va per stdout |
| INCONS-03 | options.cpp vs tilemap.cpp:77-78 |
Diferent política en YAML absent: contains() vs accés directe que llança |
Menor | PENDING | Uniformar (llançar sempre si és un fitxer obligatori) |
| INCONS-04 | options.cpp, player.cpp |
Locals ALL-CAPS (CONTENT, IT, DT, OLD_X) barrejats amb snake_case |
Menor | PENDING | Uniformar estil |
| INCONS-05 | options.cpp:420-430 |
Screen::Config s'omple camp a camp mentre que Audio::Config fa designated initializers |
Menor | PENDING | Uniformar |
6. Codi mort
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| DEAD-01 | map.hpp:69,73,78 |
renderBackground/renderSarcsRow/getSarcRowBottomY no tenen callers |
Importante | DONE | Eliminats; el tilemap fa el render visual i els sarcs seran actors sprite. getSarcRect es queda (la usa collides) |
| DEAD-02 | map.hpp:59-62 |
Índexs HUD_BG/WALL/FLOOR/SARC només usats pel codi mort |
Importante | DONE | Eliminats amb DEAD-01 |
| DEAD-03 | map.cpp:3-5 |
Includes screen.hpp/resource_cache.hpp/defines.hpp orfes si es neteja DEAD-01 |
Menor | DONE | Eliminats amb DEAD-01 |
| DEAD-04 | map.hpp:64 + map.cpp:19 |
Ctor de Map mai instanciat |
Menor | DONE | Ja eliminat durant HC-03 |
| DEAD-05 | game_scene.cpp:78-86 |
Ordenació Z-order cerimonial (1 jugador, sense sarcs-sprite) | Menor | PENDING | Veure PATCH-06 |
7. Violacions de convencions
| ID | Fitxer:línia | Troballa | Severitat | Estat | Notes |
|---|---|---|---|---|---|
| CONV-01 | player.*, map.*, ui/*, game_scene.cpp |
Speed, caixa de col·lisió, geometria, índexs paleta, rutes d'assets, nom del track: tot cablejat | Crítico | PENDING | Tapat parcialment per HC-01/02/03/05/06 |
| CONV-02 | map.*, options.* |
Interfaces+callbacks: Map simulated class, Options globals sense observers |
Importante | PENDING | Cobert per PATCH-07 + COUP-03 |
| CONV-03 | options.cpp |
std::cerr viola la regla "std::cout only" |
Importante | DONE | Resolt amb INCONS-02 |
| CONV-04 | player.cpp:27 |
SDL_GetKeyboardState directe en Player::update (fuga de SDL al gameplay) |
Menor | PENDING | Anar per GlobalInputs/EventBuffer |