где-то закралась ошибка

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

package com.javarush.task.task07.task0712; import java.util.ArrayList; import java.io.*; public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> strings = new ArrayList<String>(); BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); // strings = new ArrayList<String>(); int minStr = 20; int maxStr = 0; int indexMin = 0; int indexMax = 0; for(int i = 0;i < 10;i++) { String s = reader.readLine(); strings.add(s); if(minStr >= s.length()){ minStr = s.length(); indexMin = i; } if(maxStr < s.length()){ maxStr = s.length(); indexMax = i; } //strings.add(s); } //for(int i = 0; i < 10;i++) //{ // String s1; //s1 = strings.get(i); if(indexMin < indexMax){ //if(s1.length() == minStr) // { System.out.println(strings.get(indexMin)); // break; //} }else{ //if(s1.length() == maxStr) //{ System.out.println(strings.get(indexMax)); //break; } //System.out.println(indexMin); // System.out.println(indexMax); } } переделал, та же проблема. Если вначале короткие, потом длинная, после длинной строки идёт короткая, то на выходе будет длинная

"20 это условно. больше этой строчки ориентировочно не будет." - это совершенно недопустимое допущение. Обязательно будет. Подумай, каким значением надо инициализировать эту переменную, что бы это работало всегда (подскажу - это не число). И соверешнно невозможно твой код прочитать, какая-то каша из закомментирорванных и не закомментированных строк.

import java.util.ArrayList;
import java.io.*;

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList<String> strings = new ArrayList<String>();
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        int minStr;
        int maxStr = 0;
        int indexMin = 0;
        int indexMax = 0;

        for(int i = 0;i < 10;i++){
            String s = reader.readLine();
            strings.add(s);
            	if(maxStr < s.length()){
            		maxStr = s.length();
            		indexMax = i;
            	}
            	minStr = maxStr;
            	if(minStr > s.length()){
            		minStr = s.length();
            		indexMin = i;
            	}

        }
        if(indexMin < indexMax){
        	System.out.println(strings.get(indexMin));
            }else{
            	System.out.println(strings.get(indexMax));
            	}
    }
}

теперь задал минимальную длину строки равную максимальной, после того как такова найдена.

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

minStr = maxStr;

Проверь, например

a
aaaaa
aaa
aaa
aaa
aaa
aaa
aaa
aaa
aaa

Рабочая идея, хорошая. Реализация, как Ксения заметила, не очень, надо сначала одним циклом максимальный найти, а потом вторым циклом мимнимальный. Но есть идея ещё получше: попробуй проинициализировать мин и макс длиной первой строки. Сначала считать одну строку, инициализировать мин и макс по ней, потом только запустить цикл по остальным (счётчик цикла начать с единицы). Это один из стандартных приёмов при поиске минимального/максимального значения. И мин, и макс ты тогда найдёшь за один проход цикла.

import java.util.ArrayList;
import java.io.*;

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList<String> strings = new ArrayList<String>();
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        int minStr;
        int maxStr = 0;
        int indexMin = 0;
        int indexMax = 0;


        for(int i = 0;i < 10;i++){


            strings.add(reader.readLine());
        }

		for(int i = 0;i < 10;i++){
            if(maxStr < strings.get(i).length()){
            		maxStr = strings.get(i).length();
            		indexMax = i;
            	}
        }
        minStr = maxStr;
        for(int i = 0;i < 10;i++){
            	minStr = maxStr;
            	if(minStr > strings.get(i).length()){
            		minStr = strings.get(i).length();
            		indexMin = i;
            	}

        }
        if(indexMin < indexMax){
        	System.out.println(strings.get(indexMin));
            }else{
            	System.out.println(strings.get(indexMax));
            	}
    }
}

дошёл до такого, я на верном пути?

29-я строка зачем?

это невнивательность😁 спасибо, решено)