Здравей, разгледах ти набързо кода и имам няколко въпроса и препоръки към теб:

Относно камъните (Gems) - подходът, който си избрал с това да имаш още едно поле, което да носи типа на камъка, вместо отделник конкретни класове за тези типове виждаш, че води до доста излишни switch-case конструкции на няколко места. Бих те посъветвал да ги изнесеш в класове, точно както си направил с оръжията. Ситуацията е аналогична.

Относно оръжията - имаш едни изнесени константи, не е лошо нещо...обаче трябват ли ти наистина някъде другаде освен при създаването на конкретното оръжие? Но това не е кой знае какво. Хубаво си ги разделил в класове, но в абстрактния клас виждам едни автоматични пропъртите и заедно с тях, едни методи, които заместват get-a, отново с доста switch-case конструкции. Точно затова ползваме енумерация, за да можеш да кажеш на съответното пропърти да върне резултат модифициран спрямо тази енумерация, каквато и да е стойността ѝ в конкретната инстанция на класа. За да се възползваш от това удобство разбира се, трябва да презапишеш стойността на членовете на съответните енумерации по този начин -> Common = 1, Regular = 2 и т.н., защото в случая не са последователни и дефолтните стойности отдолу не ти вършат работа.

И други неща мога да коментирам, но мисля, че ако поработиш над това, сам ще се сетиш как можеш да подобриш кода още повече от към четимост и ефективност.