Voorkom inline strings

[RichardRam87]

Je hebt feedback gekregen van RichardRam87
op:

private void OnTriggerStay(Collider other)
    {
        if(other.gameObject.tag == "Pan") {
            SoupFill sf = other.GetComponentInChildren<SoupFill>();
            sf.fillAmount -= 0.001f;
        }
    } 

URL: https://github.com/AlexJanson/Mythe/blob/master/Mythe-Nisse/Assets/Scripts/WaterTap.cs

Feedback: Probeer niet 'inline' strings te gebruiken zoals: switch (type) { case preloader". Het gaat hier om het stuk ""preloader"". 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. Deze variabele maak je het liefste helemaal bovenin je script aan zodat andere developers ze ook eenvoudig kunnen vinden. Vaak zijn deze variabelen 'constants' zodat je er ook vanuit andere Classes naar kunt verwijzen.

Een voorbeeld van zo'n Class kun je hier vinden: http://unity3d.com/learn/tutorials/projects/stealth/tag-management . In dit artikel laten ze de Class Monobehaviour extenden. Maar dit is niet nodig.