jordimeije / uilightbaker Goto Github PK

Op meerdere plekken heb je de kans om tegen een out-of-range exception aan te lopen.

Hier wordt een array gevuld met waardes.

Hier gaan we er vanuit dat sceneCountInBuildSettings ondertussen niet veranderd is, in de praktijk zal dit niet snel gebeuren maar in theorie kan je hier redelijk makkelijk tegenaan lopen.

Wat je hier gaat doen is elke scene die in de scenebuild window zit bakken

Dit hangt samen met Issue #14

https://docs.microsoft.com/en-us/dotnet/api/system.string.format?view=netcore-3.1

Probeer code styling (indenting, brace positions, spacing) consistent te houden, maakt de code vele malen makkelijker te lezen en onderhouden

Dit hele script maakt erg veel aannames en gaat er vanuit dat de gebruiker alles altijd perfect opgezet heeft. Hoe zou je deze gegevens op een andere, misschien vriendelijkere, manier kunnen opvragen/krijgen zonder dat de gebruiker moet weten hoe het hele systeem precies werkt?

Aangezien het grootste deel van de informatie ook al opgevraagd/gegenereerd wordt door de code zou je kunnen overwegen om dit script argumenten te laten accepteren en daar alle informatie in te stoppen.

Voorbeeld:

Dit verwacht dat de gebruiker een bestand (genaamd param.txt) in dezelfde folder heeft staan met de juiste waardes en formaat (comma separated).

Zelfs voor comments is het belangrijk om consistentie te bewaren, of elke regel met // beginnen of wrappen met /* ... */

Als het idee is om de code 1 seconde te laten slapen kan je hier beter Thread.Sleep() gebruiken. Thread.Sleep() is een blokkende call terwijl Task.Delay() een sleep is zonder de thread vast te houden. Wat je hier gedaan hebt (await ...) maakt van Task.Delay() eigenlijk een Thread.Sleep()

Dit is een super zeurderige maar probeer er voor te zorgen dat de variabel naam zelf al een hit geeft waar het voor gebruikt wordt.

Wat ik van toggleSD krijg is:

• Het zal iets aan en uit zetten.
• SD staat voor 2 woorden, eerste woord begint met S en de 2e met D.

Stel je renamed het naar should_shutdown:

• Het bepaalt of iets zal worden afgesloten, wat dat 'iets' is kom je achter door verder te lezen of omdat je toevallig weet dat dit script de PC kan afsluiten.

Wat is het idee achter dit stuk code?
Waarom wordt alleen guids[0] geopened?

Wat zou er gebeuren als deze git calls falen?

Probeer code styling (indenting, brace positions, spacing) consistent te houden, maakt de code vele malen makkelijker te lezen en onderhouden

Ik heb EditorGUILayout.TextField nog niet eerder gebruikt maar ik kan me moeilijk voorstellen dat een lege null returned i.p.v. een empty string ("")

Word dit nog ergens gebruikt? Zo ja, kan het dan niet beter zijn eigen bestand krijgen?

Zie ik het verkeerd of is deze waarde nergens gebruikt? Ongebruikte variabelen kan je het beste weggooien, ze nemen geheugen en cpu ticks op die uiteindelijk nergens voor nodig zijn.

Probeer 1 class per bestand aan te houden als standaard regel. De enige uitzondering die echt voor die regel bestaat zijn interne classes.

In plaats van 2 arrays maken die aan elkaar gerelateerd zijn kan je ook één array maken die beide waardes voor je vast houdt.

Zou je hier ooit scenes in willen hebben zitten die niet mee gebuild zullen worden? Zo nee, overweeg dan om over EditorBuildSettings.scenes heen te lopen ipv sceneCountInBuildSettings te gebruiken. Het belangrijke verschil tussen deze 2 is dat je van EditorBuildSettings.scenes een EditorBuildingSettingsScene terugkrijgt waar extra informatie (zoals EditorBuildingSettingsScene.enabled) in zit.

Als er meerdere scenes worden toegevoegd nadat dit geïnitialiseerd is zal er meer in de array toegevoegd worden dan mag.

Boosdoener:

Al deze variabelen zitten in de globale class scope, dit betekent dat ze geïnitialiseerd worden zodra een instantie van de class wordt aangemaakt en opgeruimd worden zodra de class out-of-scope gaat. Normaal gesproken wil je proberen variabel scoping zo precies mogelijk te houden.