Не работает метод length()

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

Решен

Приветствую, не совсем понятно почему не работает метод length, поля jr1 и jr2 являются String и поэтому они должны иметь возможность вызывать этот метод. Заметил что если убрать < String> у StringComparator то код работает. Почему?

Перед тобой программа, которая сортирует список строк по их длине по возрастанию.
Это делает метод sortStringsByLength(ArrayList<String>). Для сортировки строк используется метод Collections.sort(ArrayList<String>, Comparator<String>), который принимает список строк и компаратор в виде лямбда-выражения.

Твоя задача — переписать реализацию метода sortStringsByLength(ArrayList<String>), чтобы вместо лямбда-выражения использовался класс StringComparator, который реализует интерфейс Comparator<String>, не меняя логику работы метода sortStringsByLength(ArrayList<String>).

То есть нужно, чтобы в классе StringComparator метод int compare(String s1, String s2) возвращал числовое значение согласно этим условиям:

отрицательное число, если длина строки s1 меньше длины строки s2;
положительное число, если длина строки s1 больше длины строки s2;
0, если длины строк s1 и s2 одинаковы.

Метод main() не принимает участие в тестировании.

Требования:

В отдельном файле нужно создать публичный класс StringComparator, который реализует интерфейс Comparator<String>.
В классе StringComparator нужно создать публичный метод int compare(String s1, String s2).
Метод compare(String s1, String s2) нужно реализовать согласно условию.
В методе sortStringsByLength(ArrayList<String>) нужно сортировать список строк по возрастанию их длины.
В методе sortStringsByLength(ArrayList<String>) для сортировки списка строк по возрастанию их длины нужно использовать собственный компаратор (объект типа StringComparator).

package com.javarush.task.pro.task18.task1805; import java.util.ArrayList; import java.util.Collections; /* Прощание с лямбда-выражением */ public class Solution { public static void main(String[] args) { var strings = new ArrayList<String>(); Collections.addAll(strings, "JavaRush", "Amigo", "Java Developer", "CodeGym"); sortStringsByLength(strings); for (String string : strings) { System.out.println(string); } } public static void sortStringsByLength(ArrayList<String> strings) { StringComparator<String> comp = new StringComparator<String>(); Collections.sort(strings, comp); } }

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

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

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

Павел Безумный учёный

Expert

4 февраля 2023, 23:08

решение

Класс StringComparator нет необходимости параметризовать типом String. Поскольку класс реализует интерфейс Comparator<String>, то это автоматически подразумевает, что аргументы реализуемого метода compare() также ограничиваются типом String. Если в объявлении класса сразу после его имени использовать оператор <>, то компилятор распознает такую запись как объявление класса-дженерика, то есть подразумевается, что класс StringComparator параметризуется некоторым типом String, причём String в данном случае - это не конкретный класс, а просто абстрактное имя типа-параметра. Говоря иначе, код

class StringComparator<String> implements Comparator<String> {
    public int compare(String jr1, String jr2) {
        return jr2.length() - jr1.length();
    }
}

равноценен коду

class StringComparator<T> implements Comparator<T> {
    public int compare(T jr1, T jr2) {
        return jr2.length() - jr1.length();
    }
}

Здесь T не является каким-то конкретным типом, поэтому по ссылкам jr1 и jr2 нельзя вызвать метод length().

Alexander Komlichenko

Уровень 51

4 февраля 2023, 22:42

В компараторе поменяйте переменные местами. Уберите дженерик в 25 строке, т.к. класс уже ограничен строками.