diasurgical/DevilutionX

Refactor devilution.DefaultClass::CheckCursMove()

Offen

#1.995 geöffnet am 17.05.2021

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (743 Forks)batch import
good first issuehelp wanted

Repository-Metriken

Stars
 (7.255 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 2T 12h) (15 gemergte PRs in 30 T)

Beschreibung

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:

Contributor Guide

Research-Richtung
Untersuchen Sie die Funktion CheckCursMove() in Source/cursor.cpp von Zeile 244 bis 692 und refaktorisieren Sie sie in kleinere Einheiten von maximal 15 Zeilen, wobei die Logik erhalten bleibt.
Tech Stack
cpp
Domain
game dev
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70