diasurgical/DevilutionX

Reduce use of globals

Aperta

#2435 aperta il 20 lug 2021

Vedi su GitHubGuida contributor
 (21 commenti) (0 reazioni) (0 assegnatari)C++ (743 fork)batch import
good first issuehelp wanted

Metriche repository

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

Descrizione

The following globals are being referenced a lot where it could instead have been done by reference, changing this will make functions more dependable/testable.

  • Items
  • Lights
  • Missiles
  • Objects
  • QuestData
  • Quests
  • VisionList
  • Players
  • Monsters
  • Towners

Guida contributor

Direzione di ricerca
Esamina ciascuna variabile globale elencata, determina dove viene utilizzata e riorganizza le funzioni per accettare la variabile come parametro (per riferimento) invece di fare affidamento sulla variabile globale. Inizia con quelle già contrassegnate come completate (Lights, Missiles, VisionList, Monsters, Towners) per comprendere il modello, quindi applica agli elementi rimanenti (Items, Objects, QuestData, Quests, Players). Assicurati che tutti i punti di chiamata siano aggiornati e che il codice compili.
Tech stack
cpp
Dominio
game dev
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
C++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60