1

Всем привет! Есть такой конструктор который считывает информацию о квартире:

public Flat(Address flatAddress, double flatArea, boolean furnishing, int bedroomsNum, double flatPrice, List<InfrastructureObject> infrustructure)

Также есть конструкторы Адреса:

public Address(String country,String town,String street,int houseNumber)

И конструктор инфраструктуры:

 public InfrastructureObject(InfrastructuresType infrastructuresType,String infrastructureObjectName,double distance)

в итоге , когда я создаю квартиру(объект) получается огромный конструктор вроде этого:

Flat flat1 = new Flat(new Address("Great Britain", "London", "Linkoln St.", 25), 290,
                true, 4, 750, new LinkedList<>(Arrays.asList(
                new InfrastructureObject(InfrastructuresType.SCHOOL, "School №19", 900),
                new InfrastructureObject(InfrastructuresType.HYPERMARKET, "Victoria Gardens hypermarket", 800))));

Как можно сократить такой конструктор, дабы он занимал меньше места по скольку в будущем мне надо будет создавать список квартир? Спасибо за предложения!!!

3
  • Все параметры в конструкторе обязательны ? И эти данные должны быть уже инициализированы при создании объекта ?
    – GinTasan
    12 апр 2019 в 22:51
  • Для сокращения прототипа, можно использовать передачу коллекции. Иначе это сделать нельзя.
    – And
    12 апр 2019 в 23:20
  • Если какой либо из ответов помог вам, то примите его и поставьте плюс, нажав на галочку и стрелку рядом с ответом. 13 апр 2019 в 7:21

3 ответа 3

3

В зависимости от того объект у вас должен ли быть неизменяемый и все ли параметры должны быть проиницилизированны.

  1. Можно сделать конструктор без параметров и использовать set-ры
    • ваш объект будет изменяемы - что не всегда хорошо
  2. Можно применить билдер паттерн
  3. Для списков можно использовать add методы
    • например для Flat можно добавить метод что-то типа addInfrastructureObject(InfrastructureObject infrustructure)
1
  • Нет, объект квартиры должен быть неизмяем и определен на стадии пре-компиляции, по сколько в рантайме мы должны будем указывать парметры по которым будет определятся список рекоммендованных квартир.
    – Wage
    12 апр 2019 в 23:42
1

Во-первых, хардкодить конкретные названия прямо в Java - это значит самому себе проблемы в будущем создать. Надо объекты создавать из БД - ну на крайний случай с текстового файла. Тогда и код будет выглядеть более компактным автоматически.

Во-вторых, InfrastructuresType.SCHOOL - не очень хорошее решение с позиции ООП - потому что в методах Вам тогда придется писать "если тип такой-то то делай это, иначе если тип такой-то ..." - во что потом превращаются такие методы видел неоднократно. Намного лучше и короче просто new School(), причем в нем не надо будет тип проверять - у каждого объекта свой переопределенный метод.

В-третьих, new LinkedList<> - это тоже не очень хорошо. Например, есть документ, а есть журнал документов - это разные сущности со своим поведением и свойствами. Думаю то же самое есть и в Вашем случае.

Если хотя бы эти три рекомендации используете - код значительно сократится и будет намного более читаемым.

1
  • В данном случае речь идёт о простом консольном приложении и насчёт типов инфраструктуры всё так и задумывалось по скольку недвижимость также будет харатеризоваться расстояние до определённных объектов социальной инфраструктуры.
    – Wage
    13 апр 2019 в 7:39
0

Могу предложить использовать паттерн Fluent Interface. Он похож на Builder. Суть такова - в классе, для которого у вас пишется такой большой конструктор, вы должны объявить пустой публичный конструктор по умолчанию + set методы для каждой переменной, которые возвращают this. Пример:

class ClassA {
  ...
  public ClassA setVar1(... var1) {
    this.var1 = var1;
    return this;
  }
  ...
}

В итоге, при создании объекта вам нужно будет сделать так:

ClassA a = new ClassA()
             .setVar1(...)
             .setVar2(...)
             .setVar3(...);
8
  • А теперь представьте на сколько раздуется код внутри метода, если будет 100+ объектов.
    – And
    14 апр 2019 в 23:22
  • @And внутри какого метода? Там не один метод, и ничего в методах не раздуется из-за присутствия объектов. Да, количество методов увеличиться. Но я не говорю, что стоит отказываться от его предыдущей формы. 15 апр 2019 в 5:09
  • Как вы узнаете какой объект или тип прилетел, без его жесткой проверки через классификацию или инстанс, к тому-же вы, не укоротили прототип, он по прежнему будет множественной передачей, через запятую. Самым верным решением будет коллекция, она может быть собрана по разному и унаследованна также с множественной передачей данных, а прилетит только лишь одна, лишь разбор будет разным, если это потребуется.
    – And
    15 апр 2019 в 9:39
  • @And про что вы говорите? Вы знакомы с паттерном Fluent Interface? И что за прототип? Если вы говорите про конструктор - то я предполагаю, что там будет 2 вида конструктора - пустой публичный(как в коде выше) и публичный конструктор как у автора(чтобы при желании он мог использовать свой вариант). 15 апр 2019 в 9:47
  • Пф... Там где перечисляются аргументы. Зачем лепить велик, когда есть для этого коллекция, решающая задачу ТС'а моментально одной коллекцией и уже имеющей функцией добавки разных объектов. Причем за место тучи методов и классов, можно сделать всего один и юзать коллекцию, причем там и сортировки есть и все, все, все, нежели вы будете лепить велик.
    – And
    15 апр 2019 в 11:31

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.