в чем проблема?

package com.javarush.task.task07.task0712;

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> l = new ArrayList<String>();
        BufferedReader r = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < 10; i++) {
            String k = r.readLine(); //это надо упростить, лишняя переменная-посредник
            l.add(k);
        }
        String k = " ", smax = " ", smin = " "; //пробел - тоже символ. а если будет строка короче - нулевой длинны? как на нее отреагирует ваш код?
        k = l.get(0); //зачем опять посредники?
        int min = k.length(); //сразу берите длинну строки из нулевой ячейки
        k = " "; //а это зачем?

        int cmin = 0;
        int max = 0;
        int cmax = 0;
        for (int i = l.size(); i > 0; i--) { //неправильное условие. длинна массива (списка) всегда на единицу больше индекса его последней ячейки
            k = l.get(i); //излишнее обилие переменных, но логика верная
            if (k.length() > max) {
                max = k.length();
                cmax = i;
                smax = k;
            }
            if (k.length() < min) {
                min = k.length();
                cmin = i;
                smin = k;
            }
        }
        if (cmin < cmax) {
            System.out.println(smin);  // почему не печатать из списка нужную строку с нужным индексом? зачем вводить переменную, которая дублирует данные списка?
        } else
            System.out.println(smax);
    }
}

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList<String> list = new ArrayList<String>();
        ArrayList<Integer> listOfLength = new ArrayList<>();
        BufferedReader r = new BufferedReader(new InputStreamReader(System.in));
        for (int i = 0; i < 10; i++) {
            list.add(r.readLine());
            listOfLength.add(list.get(i).length());
        }
        Collections.sort(listOfLength);
        for (int i = 0; i < list.size(); i++) {
            if (list.get(i).length() == listOfLength.get(0) || list.get(i).length() == listOfLength.get(listOfLength.size() - 1)) {
                System.out.println(list.get(i));
                break;
            }
        }
    }
}