Мнения по задача Capitalism

krisitown 45 Точки

Мнения по задача Capitalism

Важно: Игнорирайте професионалното име, натиснах Enter 1 път повече.

Бих искал да чуя мнение по това което съм написал, не е цялата задача остава ми да измисля как да направя заплатите и да направя проверките. Искам да разбера какво мога да направя по-добре, за да съм добре подготвен за изпита. Също така мислите ли, че това ще е достатъчно за минаване и моля поставете оценка ако имате време.

Линк: https://github.com/krisitown/Capitalism

Другата примерна задача: https://github.com/krisitown/Empires

Тагове:

12/12/2015 17:52:04 C# OOP Basics 15/12/2015 17:14:52

Filkolev 4482 Точки

Database бих го кръстил CapitalismData. Може да се получи колизия в даден момент, има други Database класове в различни билиботеки, които може някога в бъдещето да се ползват в приложението.
Факторито бих го разделил не две - едно за отделите и едно за служителите.
Служителите ги сложи в отделен неймспейс. Не е особено подредено да имаш 10 вида служители и класа за отделите на едно и също място.
Командите в еднжина най-малкото ги извади в методи, за да съкратиш големия суич.
В момента енджина ти в обвързан с някои конкретни класове - конзолата и факторито. Помисли как да избегнеш това.
Имаш интерфейси само за ентититата, хубаво е да извадиш още (доста свързано с предната точка).
Помисли кои методи в кои класове може да се направят виртуални - в момента приложението ти не е open for extension, понеже няма методи, които могат да се override-нат. Дай свобода мнимум за добавяне на нови видове служители.

Това са неща, които отгоре-отгоре видях, не съм чел задълбочено кода, може да има и други неща, които да могат да се подобрят.

12/12/2015 18:56:56

krisitown 45 Точки

Благодаря за отговора, много ми помагаш, защото не бях сигурен над какво да работя за да се подготвя за изпита и да си подобря ООП уменията като цяло.

12/12/2015 21:35:37

karlite 20 Точки

Добър ден бих искал да попитам какво мислиш за Factory да е не - статичен клас както в случая... Има ли значение..? Какви са плюсовете и минусите...? Благодаря предварително?

12/12/2015 22:25:22

Filkolev 4482 Точки

Статичен клас не може да се наследи, което значи, че не може да се разшири. Т.е. по-добре да не е статично факторито, особено когато вътре има суич.

12/12/2015 22:29:03

Покажи всички коментари