diasurgical/DevilutionX

Refactor missiles.cpp to use Direction enum instead of magic Point/Displacement constants

Aperta

#2260 aperta il 28 giu 2021

Vedi su GitHubGuida contributor
 (11 commenti) (2 reazioni) (0 assegnatari)C++ (743 fork)batch import
good first issue

Metriche repository

Star
 (7255 stelle)
Metriche merge PR
 (Merge medio 2g 12h) (15 PR mergiate in 30 g)

Descrizione

There are multiple places in missiles.cpp which iterate through tiles adjacent to a given location.

https://github.com/diasurgical/devilutionX/blob/5537fe748e63b06ed2e474f7dea6ebd1216fa3d3/Source/missiles.cpp#L1634 https://github.com/diasurgical/devilutionX/blob/5537fe748e63b06ed2e474f7dea6ebd1216fa3d3/Source/missiles.cpp#L3417 https://github.com/diasurgical/devilutionX/blob/5537fe748e63b06ed2e474f7dea6ebd1216fa3d3/Source/missiles.cpp#L3995 https://github.com/diasurgical/devilutionX/blob/5537fe748e63b06ed2e474f7dea6ebd1216fa3d3/Source/missiles.cpp#L4017 https://github.com/diasurgical/devilutionX/blob/5537fe748e63b06ed2e474f7dea6ebd1216fa3d3/Source/missiles.cpp#L4772

These lists could be directly replaced with a list of the appropriate Direction enum constants. This would retain the same behaviour as existing code while making the intent clearer.

Guida contributor

Direzione di ricerca
Il problema identifica cinque posizioni in missiles.cpp (righe 1634, 3417, 3995, 4017, 4772) in cui vengono utilizzate costanti magiche per iterare sulle celle adiacenti. La soluzione consiste nel sostituirle con un elenco di costanti dell'enum Direction. Innanzitutto, trova la definizione dell'enum Direction (probabilmente in un header separato come source/engine/direction.h o simile) e verifica la mappatura. Quindi, per ciascuno dei cinque blocchi, sostituisci le costanti magiche con i valori Direction corrispondenti. Il problema ha 11 commenti che potrebbero fornire ulteriori indicazioni o discussioni sui valori enum previsti.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75