Поправете ме ако греша, но toString() методите не трябва да са private. Доколкото разбирам, @Override toString() се прави за да дефинира как се отпечатва даден обект. Примерно: System.out.print(soldier)

Също, ако не се лъжа, toString() няма нужда да се декларират в интерфейса, защото се наследяват от Object.

Поздрави,

Ивелин Тенев

Да прав си, така е - toString си е публичен метод на Object. Няма как да бъде private. Става дума за публичните гетъри от класовете на войниците. Понеже ги пишем към интерфейси, интерфейса ни задължава те да са публични, но иначе за целите на задачата можеше да си останат private защото единственото място където се ползват е в toString() метода на войника. В случая се упражняваме да пишем класовете към интерфейси.

Да, не бях разбрал напълно предишния ти пост. Снощи до 3 си играх с това и още не съм се преборил :)

Доколкото разбирам правилата са следните:

• Ако поле се ползва само от неговия клас, няма нужда от getter към него.
• Ако имаме getter, няма идея той да не е public. В абстрактните класове property-тата могат да са protected, въпреки че няма голяма идея, тъй като абстрактните класове не могат да се инстанциират.
• Private пропертита нямат смисъл.

Към ОП: Ако не се лъжа, field-вете на Soldier не трябва да се достъпват от наследниците му, а да се преизползва super.toString() метода.

Поздрави,

Ивелин Тенев

Привет,

Ето моето решение:

https://github.com/kosio197/InterfacesAndAbstraction/tree/master/src/bg/softuni/militaryelite

http://pastebin.com/4DmX09KY

Това е моето решение - работната версия която качих в джъджа(всички класове и интерфейси са в 1 файл). Иначе дава 100/100.

Github , като цяло задачката е добра и има доста за писане.

Иначе единствено може би ти дава NullPointerException . Не знам как ти е имплементирана логиката , провери различните кейсове.