- Criar um padrão desse tipo, para a Activity, seria uma má prática? Deveria ser usado esse padrão apenas quando fosse necessário a "reciclagem" de Views?
A finalidade do Padrão ViewHolder é evitar o uso repetido de findViewById()
para obter as referências às views.
Ele é normalmente referido na utilização da ListView/RecyclerView mas pode e é, nesse sentido/finalidade, utilizado em outras situações.
Quando você declara atributos da Activity para guardar as views e os usa para as aceder em locais diferentes dela, apesar de não ter uma classe para o efeito(ViewHolder), está efectivamente a evitar o uso repetido de findViewById()
.
- Passar uma ViewHolder como parâmetro é uma má prática? (supondo que na classe auxiliar eu use todas as views dentro do ViewHolder)
Não me parece que seja. É comum usar uma classe para agrupar informações que se relacionam para evitar ter de passar, individualmente, muitos parâmetros a um método.
- Usando esse tipo de prática na Activity, e passando como parâmetro (como no exemplo acima), eu deveria configurar o objeto viewHolder = null quando a Activity fosse destruída?
Atribuir null
a uma variável não evita que haja vazamento de memóriaQuanto muito seria anular o objecto VerificarLogin.
O O que importa é que o objecto, neste caso VerificarLogin, tenha um "tempo de vida" inferior ao da Activity.