Последние два условия не работают. Видимо, не понимаю, как работает цикл for по коллекции

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

Решен

Из-за того, что не понимаю, как работает цикл for по коллекции последние два условия не выполняются. Прошу помочь разобраться и в конкретной задаче, и в целом. P.S. Извиняюсь за длинный код: когда не понимаю, где ошибка, начинаю расписывать детально и System.out.println втыкать для наглядности.

Создай список строк.
Считай с клавиатуры 5 строк и добавь в список.
Используя цикл, найди самую короткую строку в списке.
Выведи найденную строку на экран.
Если таких строк несколько, выведи каждую с новой строки.

Требования:

Создай объект типа список строк.
Программа должна считывать 5 строк с клавиатуры и записывать их в список.
Программа должна выводить самую короткую строку на экран.
Если есть несколько строк с длиной равной минимальной, то нужно вывести каждую из них с новой строки.

package com.javarush.task.jdk13.task07.task0709; import java.io.BufferedReader; import java.io.InputStreamReader; import java.util.ArrayList; /* Самая короткая строка */ public class Solution { public static void main(String[] args) throws Exception { //напишите тут ваш код ArrayList<String> list = new ArrayList<String>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for (int i = 0; i < 5; i++) { list.add(reader.readLine()); } ArrayList<Integer> dliny = new ArrayList<Integer>(); for (int i = 0; i < 5; i++) { dliny.add(list.get(i).length()); } // System.out.println(dliny.get(1)); = 7 // System.out.println(dliny.get(4)); = 6 int minNum = dliny.get(0); // System.out.println(minNum); = 7 for (int j : dliny) { if (j < minNum) minNum = j; } // System.out.println(minNum); = 4 for (int k : dliny) { if (k == minNum) System.out.println(list.get(k)); // выводит машину, а не дочь, видимо, потому что в "к" уже 4, а не 2 } } }

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

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

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

solid

Уровень 33

4 декабря 2022, 22:26

решение

Для начала это не "for по коллекции" и не "улучшенный цикл for", как написали ниже. Это foreach и применяется он не только к коллекциям. Синтаксис foreach для списка: for (тип элемента из списка переменная : список, содержимое которого перебираем) Пример foreach:

List<String> list = new ArrayList<>();
//...
for (String line : list) {
    System.out.println(line);
}

Тоже, только для for:

List<String> list = new ArrayList<>();
//...
for (int i = 0; i < list.size(); i++) {
    String line = list.get(i);
    System.out.println(line);
}

Разбор кода. Четыре цикла в данном коде, говорит о непродуманности подхода к таске. 14 строка: правило хорошего тона - создавать список к абстракции, в данном случае к List. 15 строка: если ничего особого не требуется, то проще использовать Scanner. Получается короче запись и минус эксепшн к методу. Далее идут три цикла: на чтение с консоли, на заполнение ещё одного списка длинами строк и на поиск минимальной длины. Последний цикл - попытка вывода в консоль результата из одного списка через другой. Более простое и понятное решение: считываем 5 строк и сразу ищем минимальную длину; выводим в консоль строки равные минимальной длине.

List<String> list = new ArrayList<>();
Scanner scanner = new Scanner(System.in);
int min = Integer.MAX_VALUE;
for (int i = 0; i < 5; i++) {
    String line = scanner.nextLine();
    list.add(line);
    if (min > line.length()) {
        min = line.length();
    }
}
for (String line : list) {
    if (min == line.length()) {
        System.out.println(line);
    }
}

Надеюсь разбор данной таски поможет в будущем находить оптимальный подход для написания более короткого кода. Чем меньше строк - тем легче ориентироваться в коде и меньше ошибок, в большинстве случаев.)

Aлександр 52

Уровень 21

5 декабря 2022, 07:52

Спасибо за разъяснения. Если не затруднит ответь еще на два вопроса, пожалуйста. 1) В первой строке последнего кода

List<String> list = new ArrayList<>();

почему нет префикса Array? Разве без него скомпилируется? 2) Твой код работает. Но вернусь к моему исходному коду: почему это

for (int k : dliny) {
            if (k == minNum)
                System.out.println(list.get(k)); // выводит машину, а не дочь, видимо, потому что в "к" уже 4, а не 2
        }

не работает

solid

Уровень 33

5 декабря 2022, 10:28

1. Я выше уже писал. Конкретных реализаций может быть много и правилом хорошего тона считается приведение их к общей абстракции:

List<String> arrayList = new ArrayList<>();
List<String> linkedList = new LinkedList<>();
List<String> vector = new Vector<>();

2. Тоже уже ниже писали. Список dliny (лучше было назвать listOfLengths) содержит 5 длин строк, допустим 5, 4, 6, 4, 7. minNum, соответственно, будет 4. В первой итерации for (int k : dliny) k будет равно 5. Условие if (k == minNum) не срабатывает 5 == 4 - false. Вторая итерация for (int k : dliny) k будет равно 4. Условие if (k == minNum) выполняется 4 == 4 - true. Переходим к System.out.println(list.get(k)); Разбираем эту строку: вывести в консоль содержимое списка по индексу 4. Значение по индексу 4 в списке равно 7. А если минимальная длина строки была бы больше, то ушли бы за пределы списка. В чём ошибка: list.get(k) вместо индекса подставляется значение элемента списка dliny, т.е. длина. Правильное решение:

int i = 0;
for (int k : dliny) {
    if (minNum == k) {
        System.out.println(list.get(i));
    }
    i++;
}

Где i и есть предложенный счётчик. Либо проще через for:

for (int i = 0; i < list.size(); i++) {
    if (minNum == dliny.get(i)) {
        System.out.println(list.get(i));
    }
}

Aлександр 52

Уровень 21

5 декабря 2022, 11:00

Спасибо за разъяснения. Предпоследний код - красота, эврика. Это как раз то, до чего я пытался додуматься. Чисто азарт поиска ответа был))) Так-то, конечно, последний код оптимальнее.

Алексей Погонин

Уровень 47

4 декабря 2022, 20:56

полезный

for (int k : dliny) {
            if (k == minNum)
                System.out.println(list.get(k)); // выводит машину, а не дочь, видимо, потому что в "к" уже 4, а не 2
        }

Вот здесь проблема, улучшенный цикл for идет по коллекции, используя синтаксис (элемент коллекции : коллекция). То есть переменная k, это не счетчик и не номер элемента в листе, это сам элемент. Допустим самое короткое слово "дочь", оно лежит во 2 ячейке. Мы проверили, "дочь" подходит и слово нужно вывести, но на вывод мы подаем не номер элемента, а его длину, то есть 4, это нужно исправить. В последнем цикле нужно использовать счетчик, что бы всегда иметь под рукой номер элемента.

Aлександр 52

Уровень 21

5 декабря 2022, 07:55

А как реализовать этот счетчик? Речь о switch?