Nesse desafio pude finalmente criar a classe "Filme" e remover as listas individuais com os valores extraídos da API. Também decidi melhorar um pouco o meu código e estou mais satisfeito com ele agora.

Tive a oportunidade de aprender um pouco sobre Records, apesar de não ter usado, e quebrei a cabeça mudando as minhas expressões regulares.

Reflexões feitas no desafio:

Algumas reflexões: você acha que faz sentido ter setters ou um construtor padrão? Um filme deve ser interfaceado? Deve ser imutável? Justifique a sua decisão no seu repositório.

Respostas:

1. Não vi sentido em colocar setters na classe "Filmes", pois os filmes não são algo em que a gente vai ficar mexendo, os filmes têm as informações deles e é isso. Único caso em que faria sentido uma mudança seria se o filme tivesse sido registrado com informações erradas.

2. Não acho que um filme deve ser interfaceado, pois ele não precisa ter comportamentos específicos além dos getters e nem precisa ter uma ordem natural nesse caso, pois os filmes já são adicionados à lista por rank.

3. O filme deve ser imutável, pois tanto título quanto imagem, ano e rank não vão mudar (não no meu projeto, pelo menos, se a ordem dos rankings mudarem ou algo do tipo, é dever da API atualizar).

import io.github.raphaelrighetti.filmes.modelos.Catalogo;

public class TestaTudo {

Catalogo catalogo = new Catalogo(response.body());

catalogo.getFilmes().forEach(System.out::println);

System.out.println(catalogo.getFilmes().size());

package io.github.raphaelrighetti.filmes.modelos;

import java.util.ArrayList;

import java.util.Collections;

import java.util.List;

import java.util.regex.Matcher;

import java.util.regex.Pattern;

public class Catalogo {

private List<Filme> filmes = new ArrayList<>(250);

public Catalogo(String json) {

checaStringNull(json);

json = formataJson(json);

preencheFilmes(json);

}

public List<Filme> getFilmes() {

checaListaVazia(filmes);

return Collections.unmodifiableList(filmes);

}

private void preencheFilmes(String json) {

Matcher m = Pattern.compile("\\{[^{}]*}").matcher(json);

while (m.find()) {

String[] information = m.group().split(",\"");

String nome = getValorIsolado(information[2]);

String imagem = getValorIsolado(information[5]);

String ano = getValorIsolado(information[4]);

String rank = getValorIsolado(information[1]);

filmes.add(new Filme(nome, imagem, ano, rank));

}

}

private void checaStringNull(String json) {

if (json == null) {

throw new NullPointerException("É necessário passar os dados!");

}

}

private void checaListaVazia(List<Filme> lista) {

if (lista.isEmpty()) {

throw new NullPointerException("A lista está vazia!");

}

}

private String formataJson(String json) {

Matcher mStart = Pattern.compile("^\\{\"items\":(\\[(.*)]),\"errorMessage\":\"\"}$").matcher(json);

if (mStart.find()) {

return mStart.group(2);

}

return json;

}

private String getValorIsolado(String s) {

Matcher m1 = Pattern.compile(":\"([^\"]*)\"").matcher(s);

if (m1.find()) {

return m1.group(1);

}

return s;

}

}

package io.github.raphaelrighetti.filmes.modelos;

import java.time.Year;

public class Filme {

private final String nome;

private final String urlImage;

private final Year year;

private final int rank;

public Filme(String nome, String urlImage, String year, String rank) {

this.nome = nome;

this.urlImage = urlImage;

this.year = Year.parse(year);

this.rank = Integer.parseInt(rank);

}

public String getNome() {

return nome;

}

public String getUrlImage() {

return urlImage;

}

public Year getYear() {

return year;

}

public int getRank() {

return rank;

}

@Override

public String toString() {

return "[Nome: " + nome +

", Imagem URL: " + urlImage +

", Ano: " + year +

", Rank: " + rank + "]";

}

}