JavaRush/Помощь по Java задачам/Плохое приданное! :)

14 уровень

16.04.2019
575просмотров
15комментариев

Плохое приданное! :)

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

Решен

В задаче придается метод toString, с помощью которого в условиях рекомендуется выводить наши объекты на экран в виде строки. У меня он не работал! О_о Пришлось его корректировать. У кого-то он работал без вмешательства? Мне интересно - это намеренная ошибка? Типа, что бы мы ее выявили и исправили? Просто об этом в задании ни слова. И если это не "задумка ментора", то может "что-то где-то слетело"? Сегодня как раз обновление прошло, может повлияло? (я задачу скачал как раз в момент, когда сайт "лег" на обновление)

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

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

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

Justinian Judge в Mega City One

Master

16 апреля 2019, 15:54

Знать бы какой был. А какой у тебя в результате? Я ничего не менял в этом методе в этой задаче.

hidden #2054204

Уровень 14

16 апреля 2019, 15:58

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 = 0; i < childCount; i++) { // тут откорректирова цикл, что бы в текст добавлялось то, что удалил в строке выше
                    Human child = (Human) this.children.get(i); // тут тоже корректировал ругалось уже и не помню на что, помню что вся строка красным подчеркнута была, через альт-энтер исправил.
                    if(i > 0)
                        text += ",";
                    text += " " + child.name; //тут только внутри кавычек поправил, что бы последствия работы выше не сказывались на выводе
                }
            }
            return text;

IceBerg

Уровень 41

Master

16 апреля 2019, 16:07

this.children.get(0).name; - тут убрал, а то ругалось на нейм

Причина? Должно норм работать.

Justinian Judge в Mega City One

Master

16 апреля 2019, 16:08

вероятней всего ты поломал логику программы в другом месте, и у тебя слетел метод. Что пришлось подправлять его

hidden #2054204

Уровень 14

16 апреля 2019, 16:29

2 IceBerg Нейм выделено красным было (не подчеркнуто, а именно красное, т.е. строка this.children.get(0).name, в ней this.children.get(0). - это норм и даже не подчеркнуто, а name красное) и просило заменить уже и не помню почему. Помню, что не мог понять и решил кусок метода переписать. А строка this.children.size() - все норм. Т.е. список видит, а параметр первого объекта из списка не может взять. Так я и его в цикл запихнул (в оригинале цикл стартовал с int i = 1. 2 Justinian Разве логика программы могла повлиять? Ведь этому методу и так и так приходит на вход объект, внутри которого есть список с объектами, откуда он и должен дергать данные. Я же, по сути, эти же данные ему и скормил, но иначе, чем он пытался вытащить. (см. выше) Могу и весь код скинуть, если поможет.

IceBerg

Уровень 41

Master

16 апреля 2019, 16:35

Конечно может, если у тебя name красным, значит логика с этой переменной и нарушена, бросай весь код.

hidden #2054204

Уровень 14

16 апреля 2019, 16:43

Вот код с изначальным методом.

import java.util.ArrayList;

public class Solution {
    public static void main(String[] args) {
        Human son = new Human("Petr", 7, true);
        Human daughter = new Human("Polly", 6, false);
        Human son2 = new Human("Alex", 3, true);
        Human papa = new Human("Kolia", 36, true, son, daughter, son2);
        Human mama = new Human("Toma", 34, false, son, daughter, son2);
        Human ded1 = new Human("Phil", 67, true, papa);
        Human ded2 = new Human("Dino", 60, true, mama);
        Human baba1 = new Human("Liza", 62, false, papa);
        Human baba2 = new Human("Valia", 58, false, mama);
        System.out.println(ded1.toString());
        System.out.println(ded2.toString());
        System.out.println(baba1.toString());
        System.out.println(baba2.toString());
        System.out.println(papa.toString());
        System.out.println(mama.toString());
        System.out.println(son.toString());
        System.out.println(daughter.toString());
        System.out.println(son2.toString());
    }
    public static class Human {
        //напишите тут ваш код
         String name;
         int age;
         boolean sex;
         ArrayList children;

hidden #2054204

Уровень 14

16 апреля 2019, 16:44

public  Human(String name, int age, boolean sex, Object  kid){
            this.name = name;
            this.age = age;
            this.sex = sex;
            ArrayList<Object> deti = new ArrayList<>();
            deti.add(kid);
            this.children = deti;
        }
        public  Human(String name, int age, boolean sex, Object  kid1, Object kid2, Object kid3){
            this.name = name;
            this.age = age;
            this.sex = sex;
            ArrayList<Object> deti = new ArrayList<>();
            deti.add(kid1);
            deti.add(kid2);
            deti.add(kid3);
            this.children = deti;
        }
        public Human(String name, int age, boolean sex){
            this.name = name;
            this.age = age;
            this.sex = sex;
            this.children = new ArrayList(0);
        }
        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;
        }
    }
}

IceBerg

Уровень 41

Master

16 апреля 2019, 16:49

ArrayList<Object>

Вот и Троянский конь - ArrayList<Human> Там вообще в конструкторе дичь твориться с его инициацией, объяви

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

В конструкторе

this.children = children;

Параметр должен приходить соответственный, а не Object kid

hidden #2054204

Уровень 14

16 апреля 2019, 17:24

Ну да, в конструкторе пришлось извращаться, а то ругалось :) Я сначала так сделал ArrayList<Object> children = new ArrayList<>();
и в конструкторе

this.children = children;

Тогда вопрос на мое понимание. (вариант для деда, как можно было сделать) поле класса

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

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

this.children = children;
children.add(papa);

В параметрах конструктора подаем объект Human рара. И получаем список детей деда, состоящий из одного объекта класса Human - рара. Так?

IceBerg

Уровень 41

Master

16 апреля 2019, 17:43

решение

Если в параметрах приходит Human papa, то просто

children.add(papa);

без

this.children = children;

В идеале использовать конструкцию (Human... children) принимает произвольное количество аргументов в массив children, а дальше просто можно в список конвертнуть.

public static class Human {
        //напишите тут ваш код
        public String name;
        public boolean sex;
        public int age;
        public ArrayList<Human> children;

        public Human (String name,  boolean sex, int age, Human... children){
            this.name = name;
            this.sex = sex;
            this.age = age;
            this.children = (ArrayList<Human>) Arrays.asList(children);
            // или this.children.addAll(Arrays.asList(children));
        }

hidden #2054204

Уровень 14

16 апреля 2019, 17:43

Все, разобрался. Исправил конструкторы и метод toString() работает. Привел поле класса Хуман к виду

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

а конструкторы к виду:

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

hidden #2054204

Уровень 14

16 апреля 2019, 17:48

Ясно! Спасибо! Кстати, у моего изначального способа есть преимущество! В "дети" можно добавлять объекты разных классов, а не только Хуман. Вдруг кошечку или собачку решат удочерить. В истории прецеденты имелись. :))))

IceBerg

Уровень 41

Master

16 апреля 2019, 17:53

полезный

Вот только если мы держим в руках <Object>, то мы не знаем кошечка это, человек или ещё что-то)) соответственно мы не знаем если в "объекта" имя, умеет ли "объект" что-то делать и т.д. Поэтому у тебя и была ошибка с name.

hidden #2054204

Уровень 14

16 апреля 2019, 18:00

Спасибо!