diasurgical/DevilutionX

Refactor devilution.DefaultClass::CheckCursMove()

Ouverte

#1 995 ouverte le 17 mai 2021

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)C++ (743 forks)batch import
good first issuehelp wanted

Métriques du dépôt

Stars
 (7 255 étoiles)
Métriques de merge PR
 (Merge moyen 2j 12h) (15 PRs mergées en 30 j)

Description

I've selected devilution.DefaultClass::CheckCursMove() for refactoring, which is a unit of 421 lines of code. Addressing this will make our codebase more maintainable and improve Better Code Hub's Write Short Units of Code guideline rating! 👍

Here's the gist of this guideline:

  • Definition 📖 Limit the length of code units to 15 lines of code.
  • Why❓ Small units are easier to analyse, test and reuse.
  • How 🔧 When writing new units, don't let them grow above 15 lines of code. When a unit grows beyond this, split it in smaller units of no longer than 15 lines.

You can find more info about this guideline in Building Maintainable Software. 📖

Good luck and happy coding! :shipit: :sparkles: :100:

Guide contributeur

Direction de recherche
Examinez la fonction CheckCursMove() dans Source/cursor.cpp de la ligne 244 à 692 et refactorisez la en unités plus petites de 15 lignes maximum chacune, en veillant à ce que la logique soit preservée.
Stack technique
cpp
Domaine
game dev
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.
Une demi journée
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
GitC++
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70