infinitespace-studios/VectorRumble

Clean up usage of Linq and `.ToArray()`.

Open

#59 aperta il 27 ago 2025

Vedi su GitHub
 (0 commenti) (1 reazione) (1 assegnatario)C# (5 fork)auto 404
bughelp wanted

Metriche repository

Star
 (15 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Both Linq and ToArray produce allot of Garbage, and should not be used at all unless absolutely necessary. Using it for loading and saving is ok, but in a game loop no. This is because things like ToArray make copies of the data. If these are only used for a short period of time then the garbage collector will need to clean them up. It would be better to use the lists directly. Also .Any() should be removed in favour of .Length > 0 for exactly the same reason.

We should review all the code and remove usage.

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.