diasurgical/DevilutionX

Refactor 31 lines occurring 2 times in 2 files: drlg_l2.cpp, drlg_l4.cpp

Offen

#1.997 geöffnet am 17.05.2021

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

Repository-Metriken

Stars
 (7.255 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

I've selected for refactoring 31 lines of code which are duplicated in 2 file(s) (1, 2). Addressing this will make our codebase more maintainable and improve Better Code Hub's Write Code Once guideline rating! 👍

Here's the gist of this guideline:

  • Definition 📖 Do not copy code.
  • Why❓ When code is copied, bugs need to be fixed in multiple places. This is both inefficient and a source of regression bugs.
  • How 🔧 Avoid duplication by never copy/pasting blocks of code and reduce duplication by extracting shared code, either to a new unit or introduce a superclass if the language permits.

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

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

Contributor Guide

Research-Richtung
Vergleichen Sie die duplizierten Codeblöcke in drlg l2.cpp und drlg l4.cpp, identifizieren Sie die gemeinsame Logik und extrahieren Sie sie in eine neue gemeinsam genutzte Funktion, um Redundanzen zu vermeiden.
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
C++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75