Dada a descrição do problema, salvo eu não ter entendido alguma coisa, não há nada melhor que possa ser feito.
Se tiver algum detalhe não explicado, eventualmente pode-se achar um otimização mas não existe milagre. Só se houver um critério que facilite tomar algum atalho.
Uma coisa que pode-se fazer é ter uma única lista que tenha todos os dados, mas dificilmente será uma boa ideia, facilita de um lado, dificulta de outra.
Mesmo com esta solução, você poderá ter um for
. Mas e daí? Por que isto é um código melhor? Só porque tem menos laços? Isso não significa que o código é mais bem escrito. De qualquer forma provavelmente teria que ter três if
para identificar a quebra do agrupamento, então talvez seria até pior. E outra, muda a estrutura de dados por causa do algoritmo. Aí sim, seria uma bela de uma má prática na maioria das situações.
E entenda que boa ou má prática se refere aos casos comuns, a um grande número de casos. Elas não podem ser seguidas em todos os casos.
O que você pode fazer é separar isto em métodos para cada interação, assim você só teria um for
em cada método. Mas ainda teria 4 for
. Isto deixa o código melhor? Em alguns caos, sim. Mas não necessariamente, depende do objetivo. Separar demais pode trazer tantos problemas quanto amontoar demais. Seria algo assim (a grosso modo):
void getGeneros(ArrayList biblioteca) {
for (Genero genero : biblioteca.getLista()) {
getArtistas(genero);
}
}
void getArtistas(ArrayList genero) {
for (Artista artistas : genero.getLista()) {
getAlbuns(artistas);
}
}
void getAlbuns(ArrayList artista) {
for (Album album : artista.getLista()) {
getMusicas(album);
}
}
void getMusicas(ArrayList album) {
for (Musica musica : album.getLista()) {
GravaMusica(musica);
}
}
Coloquei no GitHub para referência futura.
É só um exemplo. Tenho dúvidas se é melhor, eu acho que ficou pior, ficou repetitivo dependendo que cada GetLista()
faz. De qualquer forma acho que os nomes das variáveis e tipos estão melhores definidos aí. Já que você gosta de boas práticas, procure nomear bem tudo em seu programa, isto ajuda bastante. Poderia ser um exemplo melhor mas eu não sei de todo o contexto.
Se o problema for performance talvez seja possível quebrar a operação e colocar os processadores para trabalhar em paralelo, mas aí o problema já seria diferente do que foi descrito. E pode nem valer o esforço.
for
é o de menos.