Java поток сортирует карту по ключу и значению

Question

Необходимо отсортировать слова в файле по количеству входов по определенному символу (сначала отсортировать по количеству входов, затем по алфавиту).Например, для символа 'e' результат должен быть: avrgspeed = 2;стал = 2;потому что = 2 ... автоматизировано = 1;autowired = 1.

Есть ли лучший способ написать все в стиле потока.

             public class Sorter {

public Map<String, Integer> getDistinctWordsMap(String path, char symbol) throws IOException {
    Pattern delimeter = Pattern.compile("([^a-zA-Z])");
    List<WordParameter> parameterList = new ArrayList<>();
    Files.lines(Paths.get(path))
            .flatMap(delimeter::splitAsStream).map(String::toLowerCase).distinct()
            .forEachOrdered(word -> parameterList.add(new WordParameter(word, symbol)));
    Collections.sort(parameterList);
    return parameterList.stream().filter(w->w.count>0).collect(toMap(n->n.word, n->n.count, (e1, e2) -> e1, LinkedHashMap::new));
}

class WordParameter implements Comparable<WordParameter>{
    String word;
    int count;

    public WordParameter(String word, char symbol) {
        this.word = word;
        this.count = countEntrance(symbol);
    }

    private int countEntrance(char symbol){
        int quantity = 0;
        char[] charArr = word.toCharArray();
        for(int i = 0; i<charArr.length; i++){
            if(charArr[i]==symbol){
                quantity++;
            }
        }
        return quantity;
    }

    @Override
    public int compareTo(WordParameter o) {
        if(count<o.count)
            return 1;
        else if(count>o.count)
            return -1;
        else {
            return word.compareTo(o.word);
        }
    }
}

}

Answer 1

Вы можете определенно уменьшить код шаблона и сделать его более кратким.

Не проверял код ниже, но что-то вроде этого должно быть достаточно:

 Files.lines(Paths.get(path))
      .flatMap(delimeter::splitAsStream)
      .map(String::toLowerCase)
      .filter(s -> s.indexOf(symbol) >= 0)
      .distinct()
      .map(s -> new SimpleEntry<>(s, s.chars().filter(c -> c == symbol).count()))
      .sorted(Map.Entry.<String,Long>comparingByValue(Comparator.reverseOrder())
                    .thenComparing(Map.Entry::getKey))
      .collect(toMap(SimpleEntry::getKey, e -> e.getValue().intValue(), (l, r) -> l, LinkedHashMap::new));

Этоозначает, что вам больше не нужен ваш собственный класс, так как мы используем SimpleEntry в конвейере потока.