Wygląda na to, że dobrze byłoby tu zastosować enum zamiast klasy. Łatwo będzie wtedy zamieniać stringi w obiekty tej klasy i odwrotnie. Dałoby to też możliwość nie używania stringów jako paramatrów getName() tylko silnego typu tego enuma. To jest tylko drobna uwaga i możesz ją też zignorować - daj tylko znać czy będziesz to zmieniać czy nie