Преработка на кода на задачата Military Elite
Здравейте,
за първи път качвам кода на задача, която минава 100/100 в Judge, но за всичко си има първи път. Днес събрах кураж и седнах да пиша задачата Military Elite. Старах се през цялото време да следвам конвенциите и принципите, които сме учили, но при крайните класове и интерфейси, може би си личи, че съм изпушил, а да не говорим за Engine класа. Ще съм много благодарен, ако някой колега може да ми помогне с рефакторирането на кода. Все още не съм сигурен кога да запазвам в интерфейс, и кога в клас. Затова на някои места е каша. Качвам кода и условието.
https://github.com/kaloyantihomirov/CSharp-OOP/tree/master/MilitaryElite
https://softuni.bg/downloads/svn/csharp-fundamentals/2019-May/CSharp-OOP-June-2019/04.%20CSharp-OOP-Interfaces-and-Abstraction/04.%20CSharp-OOP-Interfaces-And-Abstraction-Exercises.docx
Благодаря за отговора! Има ли на места, където мога да вдигна абстракцията и да запазя обекта в интерфейс? Говоря за Add методите в интерфейсите (AddMission и другите) и листовете.
обект в интерфейс айде не....
това в командо интерфейса е безумие... тея неща не се държат там.. Сега теоритично може да се слагат структури и properties в интерфейсите но не се прави не е добра практика. А колко повече да вдигаш абстракцията няма смисъл.То идеята и е да вдигнеш общите неща. Сега това че командото има addmission ами ок да си стой само при него.
Дали ти трябва отделен интерфейс icommando не можеше всичките да са просто soldier или isoldier. Абстракцията е хубаво нещо ама човек не трябва да се увлича в нея.
Така ни го показаха на упражнението, така го правим (ICommando интерфейса с Add метода в него). Условието на задачата гласи да направим за всеки клас интерфейс.
Извинявай за фрапантната грешка! Мисля, че се разбра какво исках да кажа:
Можеше просто да ме поправиш, за да знам как да го кажа следващия път... Просто исках да напиша тази задача като хората, съобразено, разбира се, с това, което сме учили. Все още не съм гледал лекцията за полиморфизъм, а мисля, че точно това се разглежда там.