Да не би да е заради импортите, които са javax? Иначе на нулев тест се вижда, че хвърля NullPointerException.

Може да пробваш да сглобиш името на класа; основната трудност при рефлекшъна в Java е, че няма готов метод, който да ти върне тип по простото му име или колекция от типовете, която да филтрираш след това. Но, имайки неймспейса, където искаш да търсиш даден клас, както и името му, може директно да поискаш да ти бъде върнат този тип чрез пълното му име. Оттам нататък трябва да се провери какво се случва ако няма такъв тип, дали метода връща null или хвърля грешка.

При този подход задължаваме програмистите, които пишат приложението, да спазват структурата му стриктно, т.е. като напишат нов вид единица, да я сложат в неймспейса на единиците, което си има предимства.

Javax не би трябвало да  е проблем, но за другото съм съгласен. Направих го с този "задължаващ" подход и мина в джъджа без проблеми. Работи и доста по-бързо, защото няма цикли (поне не явни).

 

public class BuildingFactoryImpl implements BuildingFactory {
    private static final String BUILDINGS_PACKAGE_NAME = "OOPSampleExam14Dec2015Empires.models.buildings.";

    @Override
    public Building createBuilding(String buildingType, UnitFactory unitFactory, ResourceFactory resourceFactory) {

        char first = Character.toUpperCase(buildingType.charAt(0));
        buildingType = first + buildingType.substring(1);
        Building building = null;
        try {
            Class buildingClass = Class.forName(BUILDINGS_PACKAGE_NAME + buildingType);
            Constructor[] ctors = buildingClass.getConstructors();
            building = (Building) ctors[0].newInstance(unitFactory, resourceFactory);
        } catch (ClassNotFoundException e) {
            e.printStackTrace();
        } catch (InvocationTargetException e) {
            e.printStackTrace();
        } catch (InstantiationException e) {
            e.printStackTrace();
        } catch (IllegalAccessException e) {
            e.printStackTrace();
        }
        return building;
    }
}