Добрый день
У меня тут концептуальный вопрос возник - а почему в spring для бинов их политика создания называется областью видимости? Как бы причем тут видимость - или это такой перевод англоязычного термина?

хотя если переводить гугл переводтчиком то такое вполне может быть

ну пишут много где про то, что scope это область видимости я вот тоже как-то с этого орнул

сделал вот так:
оставил public class Army<T extends Unit & Race>

public T getRandomUnit() {
       ArrayList<Unit> units = new ArrayList<>();
       units.addAll(cavalry);
       units.addAll(infantry);
       return units.isEmpty() ? null : (T) units.get((int) (Math.random() * units.size()));
   }

   public T getRandomUnit(T unit) {
       if (unit instanceof Cavalry) return (T) cavalry.get((int) (Math.random() * cavalry.size()));
       if (unit instanceof Infantry) return (T) infantry.get((int) (Math.random() * infantry.size()));
       return null;
   }

Вы не правильно приводите к типу, у вас идея должна подсвечивать жёлтым

+ да, подсвечивает
но работает и задания приняли =(

мне кажется по идее вообще в данной программе не должно быть public T getRandomUnit()
мы там и так все ограничили, мне кажется, что хватило бы:

public Unit getRandomUnit() {}
и
public Unit getRandomUnit(Unit unit) {

Я бы не принял)

+ да, подсвечивает
но работает и задания приняли =(

мне кажется по идее вообще в данной программе не должно быть public T getRandomUnit()
мы там и так все ограничили, мне кажется, что хватило бы:

public Unit getRandomUnit() {}
и
public Unit getRandomUnit(Unit unit) {

я тоже не любою когда в IDEA что то подсвечивает)
задам вопрос кстати, зачем там Т надо использовать

хотят идеальную по их мнению реализацию,  иные лесом

А в итоге больше на говнокод похоже)

есть ли смысл для получения списка в методе getList() создавать новый список копировать туда все значения и возвращать уже новый?