diasurgical/DevilutionX

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

Ouverte

#2 260 ouverte le 28 juin 2021

Voir sur GitHubGuide contributeur
 (11 commentaires) (2 réactions) (0 personne assignée)C++ (743 forks)batch import
good first issue

Métriques du dépôt

Stars
 (7 255 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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.

Guide contributeur

Direction de recherche
Le problème identifie cinq emplacements dans missiles.cpp (lignes 1634, 3417, 3995, 4017, 4772) où des constantes magiques sont utilisées pour itérer sur les cases adjacentes. La solution consiste à les remplacer par une liste de constantes de l'énumération Direction. Tout d'abord, trouvez la définition de l'énumération Direction (probablement dans un en tête séparé comme source/engine/direction.h ou similaire) et vérifiez le mappage. Ensuite, pour chacun des cinq blocs, remplacez les constantes magiques par les valeurs Direction correspondantes. Le problème comporte 11 commentaires qui peuvent fournir des conseils supplémentaires ou une discussion sur les valeurs d'énumération attendues.
Stack technique
cpp
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75