diasurgical/devilutionX

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

Open

#1684 aperta il 25 apr 2021

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)C++ (743 fork)batch import
good first issuehelp wanted

Metriche repository

Star
 (7255 star)
Metriche merge PR
 (Merge medio 2g 12h) (15 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Studia il commit di esempio (f77c529) per comprendere il modello di sostituzione di `int x, int y` con `Point`. Cerca nel codebase le funzioni con due parametri di coordinate int, come quelle nei moduli di geometria o rendering. Sostituisci ogni coppia di parametri con un riferimento `Point`, aggiorna tutti i siti di chiamata e assicura la coerenza con l'uso esistente di `Point`. Committa le modifiche per funzione come richiesto per facilitare la revisione.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60