Почему игнорит пустую строку, как самую короткую.

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

Решен

Почему выполняет работу, игнорируя пустую строку. Подскажите в чем проблема, всю голову сломал. Если не вводить пустой строки, то все условия выполняются.

1. Создай список строк.
2. Добавь в него 10 строчек с клавиатуры.
3. Узнай, какая строка в списке встретится раньше: самая короткая или самая длинная.
Если таких строк несколько, то должны быть учтены самые первые из них.
4. Выведи на экран строку из п.3. Должна быть выведена одна строка.

Требования:

Объяви переменную типа список строк и сразу проинициализируй ee.
Программа должна считывать 10 строк с клавиатуры и добавлять их в список.
Программа должна выводить на экран самую короткую строку, если она была раньше самой длинной.
Программа должна выводить на экран самую длинную строку, если она была раньше самой короткой.
Должна быть выведена только одна строка.

package com.javarush.task.task07.task0712; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import java.util.ArrayList; /* Самые-самые */ public class Solution { public static void main(String[] args) throws IOException { ArrayList<String> strings = new ArrayList<>(); String k = new String(); int i = 0; int l = 0; int max = 0; int min = 0; BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for (i = 0; i < 10; i++) { String s = reader.readLine(); strings.add(s); } for (i = 0; i < 10 && strings.get(i).length() != k.length(); i++) { if (strings.get(i).length() >= k.length()) { k = strings.get(i); max = i; } } String m = strings.get(l); for (int j = 0; j < 10 && strings.get(j).length() != m.length(); j++) { if (strings.get(j).length() <= m.length()) { m = strings.get(j); min = j; } } if (max < min) { System.out.println(k); } else { System.out.println(m); } } }

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

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

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

hidden #598481

Уровень 27

7 октября 2019, 16:10

Ты все чересчур усложнил обилием ненужных переменных. Я в целом задумку понял, но ты сам себя перехитрил. Например, в качестве самой длинной строки ты готов запоминать любую, которая длиннее, чем пустая:

if (strings.get(i).length() >= k.length()) {
    k = strings.get(i);
    max = i;
}

Но здесь не стоит исправлять отдельные строчки: тебе вообще не нужна пустая строка k, не нужно заранее инициализировать индекс цикла(!), вместо этого я предлагаю тебе воспользоваться принципом kiss, удалить нафиг ВСЕ созданные переменные е-к-л-м-н и подумать о том, что тебе действительно нужно. А нужно сделать три простых шага: 1) найти индекс самой короткой строки в списке 2) найти индекс самой длинной строки в списке 3) вывести на экран элемент по тому значению (из найденных), которое меньше. UPD Да, я ошибся в прочтении, и ты переопределяешь "k" как найденную самую длинную строку в цикле, но во-первых я ошибся из-за нейминга, а во-вторых это концептуально не отменяет всего выше сказанного: твои переменные путают тебя самого: например, ты выходишь из цикла каждый раз, когда считал не пустую строку, то есть на первой же не пустой введенной строке:

for (i = 0; i < 10 && strings.get(i).length() != k.length(); i++) {

потому что ты решил заморочиться и при нахождении пустой строки выйти из цикла, если пустая строка была введена. А на практике получил ненужное усложнение, которое привело к ошибке. Потому что тебе в решении вообще не нужна пустая строка.

Сергей

Уровень 20

7 октября 2019, 20:37

Спасибо за совет, познакомился с принципом kiss, хороший принцип, только не всегда ему удается следовать. Я честно пытался упростить задачу: стер ее полностью и написал заново,

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList <String> list = new ArrayList<>();
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        for (int i=0; i<10; i++) {
            String s = reader.readLine();
            list.add(0,s);
        }
        String max = "";
        int i;
        for (i=0; i<10; i++) {
            if (list.get(i).length() > max.length()) {
                max = list.get(i);
            }
        }
            String min = list.get(0);
        int j;
            for (j=0; j<10; j++) {
                if (list.get(j).length() < min.length()) {
                    min = list.get(j);
                }
            }
                if (i<j) {
                    System.out.println(max);
                } else {
                    System.out.println(min);
                }
            }
        }

и у меня вроде даже немного сократилось решение, но проблема осталась та же и еще добавилась новая: теперь он реагирует только на самый короткий вариант строки, не замечая самые длинные. Я совсем закипел (точнее мой мозг). Можно ли разжевать разжеванное вами ещё конкретнее?

hidden #598481

Уровень 27

7 октября 2019, 20:58

А сейчас ты пытаешься использовать свои индексы i и j и для перебора, и для сохранения предельной строки. Только вот проблема в том, что при переборе и i и j все равно станут равны 9, и в итоге всегда будет выполняться else. Заведи отдельные переменные с нормальными названиями, например indexOfLongest, indexOfShortest, оставь переменным под названиям i и j служебную функцию - списки перебирать. Такие названия только для этого и годятся. PS самую короткую строку можно инициализировать, как и самую длинную - первым элементом. И перебирать элементы можно один раз, производя обе проверки: длиннее макс? короче мин?

Артем Загоруйко1с программист в ип

7 октября 2019, 15:19

нужно ведь вывести и самую короткую и самую не короткую, если я понял условия правильно if (max < min) { System.out.println(k); } else { System.out.println(m); }

Сергей

Уровень 20

7 октября 2019, 15:39

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

Артем Загоруйко1с программист в ип

11 октября 2019, 05:32

узрел, истину глаголишь! имхо: зачем тащить везде string в циклах? не достаточно ли индексов выборки? после цикла получаешь наименьший индекс из max и min индексов, он и будет самый первый, а какой он, самый длинный string или самый короткий вот и узнаем, ибо самим уже тоже стало интересно. и наверно достаточно одного цикла, в нём ты и проверяй самый короткий и длинный

Сергей

Уровень 20

12 октября 2019, 16:46

решил проблему. Всё оказалось до безобразия просто, я добавлял введенные с клавиатуры строки не в конец массива "list.add(s)", а в начало "list.add(0,s)". Даже стыдно стало, спасибо всем за помощь )

Сергей

Уровень 20

12 октября 2019, 16:48

итог оказался таким.

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList <String> list = new ArrayList<>();
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        for (int i=0; i<10; i++) {
            String s = reader.readLine();
            list.add(s);
        }
        String max = list.get(0);
        String min = list.get(0);
        int i = 0;
        int j = 0;
        int indmax = 0;
        int indmin = 0;
        for (i=0; i<10; i++) {
            if (list.get(i).length() > max.length()) {
                max = list.get(i);
                indmax = i;
            }
        }
            for (j=0; j<10;j++) {
                if (list.get(j).length() < min.length()) {
                    min = list.get(j);
                    indmin = j;
                }
            }
        if (indmax<indmin) {
                    System.out.println(max);
                } else {
                    System.out.println(min);
                }
            }
        }