Sob o ponto de vista do encapsulamento, normalmente considero que a melhor abordagem é a seguinte:
public class SuaClasse {
// Embora AlgumaCoisa seja imutável, a lista em si é mutável.
private List<AlgumaCoisa> algumaLista;
public SuaClasse() {
// ...
}
public List<AlgumaCoisa> getAlgumaLista() {
return List.copyOf(algumaLista);
}
public void adicionarItem(AlgumaCoisa adicionando) {
this.algumaLista.add(adicionando);
}
}
O método List.copyOf(Collection<E>)
foi adicionado no Java 10 e cria uma cópia imutável de uma dada lista. Se a lista dada for modificada futuramente, a cópia não será alterada.
Em versões mais antigas do Java, eu usava isso (neste caso, Java 7):
public List<AlgumaCoisa> getAlgumaLista() {
return Collections.unmodifiableList(new ArrayList<>(algumaLista));
}
De toda forma, a lista interna nunca é exposta diretamente para qualquer outra classe que quiser acessá-la. Isso garante o encapsulamento. Se alguma classe externa tiver interesse em modificar o conteúdo dessa lista de uma instância x
, deverá utilizar algum dos métodos expostos por x
, o que justifica o fato da lista retornada ser imutável.
Também justifica-se ser uma cópia porque alterações subsequentes por meio do método adicionarItem
não devem misteriosamente se refletir em listas que embora tenham sido obtidas anteriormente a partir da instância x
, estejam em códigos que não mais tem interesse em x
propriamente dito.
E mesmo que uma cópia seja retornada, qualquer tentativa de modificar essa cópia diretamente é no mínimo um code smell ou um chamariz para bugs. Logo, mesmo sendo uma cópia, ela ainda assim tem que ser imutável.
Usar Iterator
diretamente é difícil. O Iterator
tem limitações de não poder ser iterado mais de uma vez (a menos que copiem-se todos os itens numa lista, mas aí seria mais fácil já dar essa lista previamente construída) e também tem limitação de não poder determinar o tamanho da lista. Além disso, o Iterator
é algo bastante ultrapassado, pois agora existe o Stream
.
Existem alguns cenários onde faz sentido retornar-se um Stream
ao invés de uma lista. Um exemplo, é quando está se percorrendo os registros de um banco de dados ou de um arquivo linha a linha. Nesse caso, faz-se isso:
public Stream<AlgumaCoisa> {
// Faz alguma coisa.
}
Mas isso só se justifica quando não é desejado que todos os itens estejam pré-preparados em memória por questão de desempenho ou de economia de memória. Caso não seja esse o caso, retorna-se a lista.
Por fim, ainda pode se ter a preocupação de que o getter pode ficar pesado de ser executado, pois sempre que é chamado, uma nova cópia da lista será feita mesmo se for idêntica a cópia produzida na chamada anterior ao método. Então, pode-se fazer isso:
public class SuaClasse {
// Embora AlgumaCoisa seja imutável, a lista em si é mutável.
private List<AlgumaCoisa> algumaLista;
private List<AlgumaCoisa> algumaListaImutavel;
public SuaClasse() {
// ...
}
public List<AlgumaCoisa> getAlgumaLista() {
if (algumaListaImutavel == null) algumaListaImutavel = List.copyOf(algumaLista);
return algumaListaImutavel;
}
public void adicionarItem(AlgumaCoisa adicionando) {
this.algumaListaImutavel = null;
this.algumaLista.add(adicionando);
}
}