nlferdinl / generations-of-me Goto Github PK

Je hebt feedback gekregen van BerendWeij
op:

inline 

URL: https://github.com/NLferdiNL/Generations-Of-Me/blob/master/Assets/_Scripts/Audio/AudioController.cs

Feedback: Probeer niet 'inline' strings te gebruiken zoals inline. Je maakt snel typfouten. Voor andere developers is het ook moeilijk om de 'strings' te vinden in je code. Probeer daarom altijd met variabelen te werken waar de 'strings' in zijn opgeslagen en plaats deze variabelen bovenin je Class.
In het geval van de namen van je Scenes en tags is het erg handig om een 'constants' Class te maken met alleen maar constant variables. Hierin zet je alle namen. Vervolgens kun je verwijzen naar deze variabelen door bijvoorbeeld gebruik te maken van Application.LoadLevel(Scenes.GAME). Een voorbeeld van zo'n Class kun je hier vinden: http://unity3d.com/learn/tutorials/projects/stealth/tag-management .

Je hebt feedback gekregen van BerendWeij
op:

private int coinCount;

    public GameObject coinCountText;

    Text _coinCountTextField; 

URL: https://github.com/NLferdiNL/Generations-Of-Me/blob/master/Assets/_Scripts/Player/Pickup.cs

Feedback: Let op de single responsibility principle.

Wat is de verantwoordelijkheid van deze Class?

Data bijhouden?
Data weergeven?
Collision checken?
enzovoorts...

Single responsibility betekent 1 verantwoordelijkheid uit het lijstje hierboven. Dus niet meerdere.

Je hebt feedback gekregen van BerendWeij
op:

dictionary 

URL: https://github.com/NLferdiNL/Generations-Of-Me/blob/master/Assets/_Scripts/Data/KeyboardConfig.cs

Feedback: Probeer meer gebruik te maken van dictionary jongens. Dat maakt je code een stuk overzichtelijker.

Bijna altijd als je een switch case gebruikt, kun je ook een dictionary gebruiken.