romanevg001 / helloworld_java Goto Github PK

У нас есть уже класс Member, зачем мы добавляем метод getArbitraryMember сюда? Не вижу логики, это главный класс приложения, вся логика, связанная с бизнес объектами должна быть в соответствующих классах, как минимум

У нас уже есть класс, отвечающий за конфиг приложения. Стоит считывание текстового файла вынести туда, сделать класс бином или просто статическим и в конструкторе инициализировать конфиг всего приложения. Класс ModernFamily у тебя за логику отвечает, при чём здесь конфиги?

Класс Family по-прежнему содержит список членов семьи, детализируя фразу, корой отвечать. По постановке задачи мы должны иметь возможность добавлять члена семьи не меняя код. В текущей реализации член семьи добавляется через создание отношений между ними, а фразу, которой он отвечает, задать произвольно нельзя. Может стоит добавить в конфиг отдельный файл с членами семиьи и фразами, которыми они общаются? Только с инициализацией конфига нужно будет аккуратней быть, у нас в файле с отношенияями между членами семьи может появиться несуществующий "пассажир", которого нет в списке членовй семьи, нам целостность данных никто не гарантирует. Можем создать "пассажир" с предупреждением, что такого нет в списке, а можем отвергнуть эту троку конфига, мол таких не знаем, в списках не значатся, игнорируем и "пассажира" и его отношения.

Зачем нам getResource в 57 строке? К Классу приложения не имеет отношения, да и вообще мы вынесли все ресурсы в текстовый файл.