JavaRush/Помощь по Java задачам/Как упростить? слишком большой код

19 уровень

13.12.2019
682просмотров
11комментариев

Как упростить? слишком большой код

Вопрос по задаче

Обсуждается

public class Solution {
    public static void main(String[] args) {
        ArrayList<Human> list = new ArrayList<>(10);
        ArrayList<Human> list1 = new ArrayList<>(10);
        ArrayList<Human> list2 = new ArrayList<>(10);
        Human child1 = new Human("A", true, 2019,list);
        Human child2 = new Human("D", false, 2019,list);
        Human child3 = new Human("E", true, 2019,list);
        System.out.println(child1.toString());
        System.out.println(child2.toString());
        System.out.println(child3.toString());
        list.add(child1);
        list.add(child2);
        list.add(child3);
        Human father = new Human("V", true,2019 , list);//напишите тут ваш код
        list1.add(father);
        Human mother = new Human("G", false, 2019, list);//напишите тут ваш код
        list2.add(mother);
        Human ded1 = new Human("S",true,2019, list1);
        Human ded2 = new Human("Sl",true,2019, list2);
        Human babo1 = new Human("N",false,2019, list1);
        Human babo2 = new Human("Em",false,2019, list2);

        System.out.println(ded1.toString());
        System.out.println(ded2.toString());
        System.out.println(babo1.toString());
        System.out.println(babo2.toString());
        System.out.println(father.toString());
        System.out.println(mother.toString());
    }

    public static class Human {
        String name;
        boolean sex;
        int age;
        ArrayList<Human> children;
        public Human(String name, boolean sex, int age, ArrayList<Human> children){
            this.name = name;
            this.sex = sex;
            this.age = age;
            this.children = children;
        }

            public String toString() {
            String text = "";
            text += "Имя: " + this.name;
            text += ", пол: " + (this.sex ? "мужской" : "женский");
            text += ", возраст: " + this.age;

            int childCount = this.children.size();
            if (childCount > 0) {
                text += ", дети: " + this.children.get(0).name;

                for (int i = 1; i < childCount; i++) {
                    Human child = this.children.get(i);
                    text += ", " + child.name;
                }
            }
            return text;
        }
    }
}

Комментарии (11)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Инсаф Бакиров

Уровень 23

19 декабря 2019, 08:34

А разве с конструкторами задача принимается? В условиях же нет такого.

hidden #598481

Уровень 27

18 марта 2020, 08:35

Конечно примется. Мы же не изменяем логику исходного конструктора. Что мешает нам пользоваться своими?

hidden #598481

Уровень 27

13 декабря 2019, 09:55

Чтобы не собирать каждый раз вручную список детей, можно сделать второй конструктор, который будет сам собирать список из переданных объектов. В этом случае можно будет просто перечислять объектов детей в аргументах при создании объекта родителя. Еще можно создать конструктор с тремя аргументами, чтобы не передавать каждый раз пустой список детей, если детей нет, но это потребует еще одно из двух действий: 1) либо каждый раз при обращении к списку children в классе Human проверять, что children не null 2) либо (и этот вариант мне нравится больше) по умолчанию инициализировать поле children как пустой список, при этом добавить этому полю модификатор final, а в конструкторе вместо присваивания

this.children = children;

сделать заполнение списка значениями из аргумента. Если что-то из написанного кажется непонятным, могу объяснить. Вообще мой совет - на этапе создания класса продумывать разные варианты конструкторов, которые тебе могут пригодиться. В этом случае пользование этим классом снаружи становится гораздо проще, вместо того, чтобы каждый раз писать несколько оберток аргумента, необходимых для создания объекта, один раз описываешь их в конструкторе, а снаружи передаешь аргумент в удобном тебе виде.

Avetis Grigoryan

Уровень 19

13 декабря 2019, 18:40

Вопросы есть! Сначала объявляю

final ArrayList<Human> children=new ArrayList<>();

Затем в конструкторе

public Human(String name, boolean sex, int age, Human children){
...
this.children.add(children);
        }

И уже в main

Human child1 = new Human("A", true, 2019,null);
Human father = new Human("V", true,2019 , child1);

Вот только null в качестве аргумента не принимает( И не понятно, если несколько детей, то как их вписать, ведь место для аргумента всего одно(

hidden #598481

Уровень 27

13 декабря 2019, 21:04

Не, тут надо действовать тоньше. Как в медицине, главное - не навредить. Поэтому имеющийся конструктор ломать мы не будем, аргументы оставим как были - с листом. К его реализации вернёмся чуть позже Чтобы можно было создавать людей без детей, добавим конструктор с 3 аргументами:

public Human(String name, boolean sex, int age) {
    //...
}

условно назовем его базовым и обозначим в уме цифрой 1. При пользовании 1 конструктором человек будет создаваться с тремя заполненными полями и инициализированным, но пустым списком детей. Вторым будет конструктор со списком детей. Чтобы нам не повторять одно и то же, в его реализации можно вызвать конструктор 1 и дополнить его заполнением списка:

public Human(String name, boolean sex, int age, List<Human> children) {
    this(name, sex, age);
    this.children.addAll(children);
}

так конструктор 2 сначала выполнит код конструктора 1, а потом заполнит список children содержимым 4го аргумента. Ну и наконец самое интересное. Как сделать так, чтобы список собирался сам. Для этого мы воспользуемся хитростью, которая называется varargs, или переменное число аргументов. Особенности ее использования ты можешь погуглить самостоятельно, а я напишу тебе реализацию на нашем примере:

public Human(String name, boolean sex, int age, Human ... children) {
    this(name, sex, age, Arrays.asList(children));
}

Human ... children это и есть переменное число аргументов Human, то есть с 4 и далее аргумента можно использовать аргументы класса Human, и они будут собраны в единый массив Human[] children. Методом Arrays.asList мы преобразуем этот массив в список и вызовем второй конструктор с полученным набором аргументов. С таким конструктором ты можешь создавать объекты так: Human mother = new Human("Ann", false, 30, child1, child2, child3, child999);

Avetis Grigoryan

Уровень 19

15 декабря 2019, 09:00

не работает(

public static class Human {
        String name;
        boolean sex;
        int age;
        final ArrayList<Human> children=new ArrayList<>();
        public Human(String name, boolean sex, int age){
            this.name = name;
            this.sex = sex;
            this.age = age;
        }
        public Human(String name, boolean sex, int age, Human...children){
            this(name, sex, age, new ArrayList<>(Arrays.asList(children)));
        }

hidden #598481

Уровень 27

15 декабря 2019, 09:33

А второй конструктор где?

Avetis Grigoryan

Уровень 19

15 декабря 2019, 12:40

вот же

public Human(String name, boolean sex, int age, Human...children){
            this(name, sex, age, new ArrayList<>(Arrays.asList(children)));

hidden #598481

Уровень 27

15 декабря 2019, 14:14

Это третий. Он обращается ко второму. А второй должен быть этот:

public Human(String name, boolean sex, int age, ArrayList<Human> children) {
    this(name, sex, age);
    this.children.addAll(children);
}

Avetis Grigoryan

Уровень 19

16 декабря 2019, 05:06

Все равно ничего не понял) Как один конструктор к другому обращается) ладно... может со временем придет

Алексей Backend Developer

13 декабря 2019, 09:51

Например избавиться от ArrayList list, list1, list2 в main