Não seria melhor usar um parse ao invés de

Olá, Rafael!

Você está absolutamente correto em ambas as suas observações.

Em relação à sua primeira dúvida, sim, seria mais adequado realizar um parse antes da comparação. Isso ocorre porque, como você mencionou, o campo de avaliação está sendo tratado como uma string. Portanto, a comparação está sendo feita de maneira alfabética e não numérica, o que pode levar a resultados inesperados, como o episódio com nota 10 sendo considerado como se fosse nota 1.

Para resolver isso, você poderia fazer algo assim:

.sorted(Comparator.comparing(episodio -> Double.parseDouble(episodio.getAvaliacao())).reversed())

Desta forma, você estaria convertendo a avaliação para um número double antes de fazer a comparação, o que deve resolver o problema.

Sobre a sua segunda dúvida, você também está correto. O Comparator.comparing(DadosEpisodio::avaliacao) está usando o método avaliacao da classe DadosEpisodio para realizar a comparação. Isso é um exemplo do uso de Method References em Java, que é uma forma concisa de expressar uma lambda que chama um método específico. Nesse caso, para cada DadosEpisodio na sua lista, o método avaliacao é chamado e o resultado é usado para a comparação.

Espero ter ajudado e bons estudos!