Что делаю не так?

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

Архивный

...

Спасибо

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

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

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

Artem Morozov

Уровень 31

8 апреля 2018, 04:10

or just try for each instead of for with a <10

HashMap<String, Cat> mapCat = new HashMap<>();
        for (String cote : cats) {
            mapCat.put(cote, new Cat(cote));
        }
        return mapCat;

--------

Уровень 40

Master

25 февраля 2018, 19:54

решение

Попробуй добавлять в словарь всех котов из получаемого массива, а не только первые 10 (которых может и не быть)

Morzes

Уровень 8

25 февраля 2018, 20:22

спасибо

Арман Матаков

Уровень 39

22 апреля 2018, 19:58

А почему нужно добавлять всех котов? Ведь мы в данном случае перебираем массив а не коллекцию, какое это имеет значение?

--------

Уровень 40

Master

22 апреля 2018, 20:12

В двух словах - потому, что метод называется addCatsToMap, а не add10CatsToMap. С самого начала стоит приучаться не писать неполноценные одноразовые методы, способные правильно работать только в одном-единственном случае (как здесь - массив или коллекция, неважно, только на 10 элементов), и падающие с ошибкой или выдающие неверные результаты в остальных 99,9% ситуаций.

Арман Матаков

Уровень 39

22 апреля 2018, 20:22

Понял, дело в валидаторе) Я думал что что-то не так с кодом,тема новая, пока не понятая до конца. Спасибо за пояснение.