diasurgical/DevilutionX

Replace "int x, int y" arguments with "Point point"

Offen

#1.684 geöffnet am 25.04.2021

Auf GitHub ansehenContributor Guide
 (2 Kommentare) (1 Reaktion) (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

A lot of functions take two ints as the coordinates for their operations, this is both verbose and error-prone. Instead we should use the Point structure, see an example of this type of cleanup done for the GetDirection() https://github.com/diasurgical/devilutionX/commit/f77c52941afd73d4e734c0696d082218cd04c3a7

Try to split the changes up in one function per commit to making it easy to review and less prone to typos.

Contributor Guide

Research-Richtung
Studiere den Beispiel Commit (f77c529), um das Muster zum Ersetzen von `int x, int y` durch `Point` zu verstehen. Durchsuche die Codebasis nach Funktionen mit zwei int Koordinatenparametern, wie z. B. in Geometrie oder Rendering Modulen. Ersetze jedes Parameterpaar durch eine `Point` Referenz, aktualisiere alle Aufrufstellen und stelle die Konsistenz mit der bestehenden `Point` Verwendung sicher. Committe Änderungen pro Funktion wie angefordert, um die Überprüfung zu erleichtern.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
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.
60