Помогите мне разок еще по этой темеее. Все работает, но не проходит одно условие.

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

Обсуждается

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

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 Exception { ArrayList<String> list = new ArrayList<>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for(int i = 0; i < 10; i++){ list.add(reader.readLine()); } int maxStr = 0; String strMax = " "; int indexMax = 0; for(int a = 0; a < list.size(); a++){ if(list.get(a).length() > maxStr){ maxStr = list.get(a).length(); strMax = list.get(a); indexMax = a; } } String strMin = " "; int min = list.get(0).length(); int indexMin = 0; for(int i = 0; i < list.size(); i++){ if(list.get(i).length() < min){ min = list.get(i).length(); strMin = list.get(i); indexMin = i; } } if(indexMin < indexMax){ System.out.println(strMin); } else if(indexMax < indexMin){ System.out.println(strMax); } } }

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

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

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

Евгений

Уровень 20

16 июня 2019, 13:21

Попробуй заменить строки с 42 по 46 на другую реализацию. Проверяй все элементы списка на совпадение с минимумом или максимумом. Если раньше встретился минимум - вывод строки, прервать цикл. Если раньше встретился максимум - вывод строки, прервать цикл. И помни что "если, если" и "если, тогда" - разные вещи.

Anonymous #Alekseei

Уровень 15

16 июня 2019, 14:50

Удалил строки с 42 по 46 и сделал так вот:

for(int i = 0; i < list.size(); i++){
            if(list.get(i).length() == min){
                System.out.println(strMin);
                break;
            }
            else if(list.get(i).length() == maxStr){
                System.out.println(strMax);
                break;
            }
        }

Код так же корректно работает как до этого и по прежнему не проходит 3е условие

Евгений

Уровень 20

16 июня 2019, 14:57

у меня проходит без if else. То есть for (...) { if (...) { //код } if (...) { //код } } Не нужно эти условия связывать. Каждое само по себе. что сработает первее.

Anonymous #Alekseei

Уровень 15

16 июня 2019, 16:36

Сделал так, работает но 3е условие по прежнему не проходит

for(int i = 0; i < list.size(); i++){
            if(list.get(i).length() == min){
                System.out.println(strMin);
               break;
            }

            if(list.get(i).length() == maxStr){
                System.out.println(strMax);
             break;
            }

        }

Anonymous #Alekseei

Уровень 15

17 июня 2019, 07:48

Я понял в чем проблема, проблема в формуле нахождения минимального числа.

String strMin = " ";
       int min = list.get(0).length();
       for(int i = 0; i < list.size(); i++){
           if(list.get(i).length() < min){
               min = list.get(i).length();
               strMin = list.get(i);
           }
           }

Значение min установлено как количество символов в строке нулевого элемента, в итоге когда начинается цикл при сравнении элемента с индексом 0 с ним же самим ничего не происходит и он не сравнивается, это не подходит. Вопрос в том каким первоначальным значением я мог бы инициализировать переменную min? Ноль не подходит, длина конкретной строки тоже не подходит.

Евгений

Уровень 20

17 июня 2019, 08:30

public class Solution { public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); ArrayList<String> list = new ArrayList(); String minStr, maxStr; int minLength = 0, maxLength = 0; for(int i = 0; i < 10; i++) { String s = reader.readLine(); list.add(s); } minStr = list.get(0); maxStr = list.get(0); //поиск строки с минимальной длинной for(int i = 0; i < list.size(); i++) { if(list.get(i).length() <= minStr.length()) { minStr = list.get(i); minLength = minStr.length(); } } //поиск строки с максимальной длнной for(int i = 0; i < list.size(); i++) { if(list.get(i).length() >= maxStr.length()) { maxStr = list.get(i); maxLength = maxStr.length(); } } for(int i = 0; i < list.size(); i++) { if (list.get(i).length() == minLength) { System.out.println(list.get(i)); break; } if(list.get(i).length() == maxLength) { System.out.println(list.get(i)); break; } } } } Кароч вот. Сравнивай, ищи свои ошибки, делай выводы.