É sempre complicado dizer qual é a "abordagem certa", ela pode ser melhor num caso e pior noutros.
Os onClickListener
são atribuídos à view. Quem é responsável por facultar a view ao adapter é a classe ViewHolder.
Julgo, assim, que deve ser aí que eles devem ser atribuídos.
Por outro lado, atribuir os "listeners" no onBindViewHolder()
fará com que eles sejam atribuídos sempre que uma linha é apresentada na RecyclerView.
Ao atribuí-los no construtor do ViewHolder eles serão reaproveitados, sendo atribuídos apenas uma vez.
Para obter a posição deve(1) usar o método getAdapterPosition().
public static class MyViewHolder extends RecyclerView.ViewHolder
implements View.OnClickListener{
private final ImageView imageView;
public MyViewHolder(View v) {
super(v);
imageView = (ImageView) v.findViewById(R.id.imageView);
//Atribui o listener ao layout da linha.
v.setOnClickListener(this);
// no entanto ele pode ser aplicado a qualquer uma das views dele.
//imageView.setOnClickListener(this);
}
//Implementa View.OnClickListener
@Override
public void onClick(View v) {
Log.d(TAG, "Elemento " + getAdapterPosition() + " clicado.");
}
}
O onCreateViewHolder()
do adapter seria assim:
@Override
public ViewHolder onCreateViewHolder(ViewGroup viewGroup, int viewType) {
View v = LayoutInflater.from(viewGroup.getContext())
.inflate(R.layout.row_item, viewGroup, false);
return new MyViewHolder(v);
}
O método onBindViewHolder()
deverá apenas atribuir os valores às views.
Esta abordagem pode ser melhorada de forma a que o código a executar, quando for clicado um item, seja externo ao adapter.
Defina uma interface a implementar pela classe que irá executar o código, quando for clicado um item:
public interface ItemClickListener {
void onItemClick(int position);
}
Adicione ao adapter um campo e o respectivo setter para guardar uma instância que implemente essa interface:
private static ItemClickListener itemClickListener;
public void setOnItemClickListener(ItemClickListener itemClickListener){
this.itemClickListener = itemClickListener;
}
Altere a ViewHolder de forma a chamar o método onItemClick()
dessa instância.
public class MyViewHolder extends RecyclerView.ViewHolder
implements View.OnClickListener{
private final ImageView imageView;
public MyViewHolder(View v) {
super(v);
imageView = (ImageView) v.findViewById(R.id.imageView);
//Atribui o listener ao layout da linha.
v.setOnClickListener(this);
// no entanto ele pode ser aplicado a qualquer uma das views dele.
//imageView.setOnClickListener(this);
}
//Implementa View.OnClickListener
@Override
public void onClick(View v) {
if(itemClickListener != null) {
itemClickListener.onItemClick(getAdapterPosition());
}
}
}
Agora é possível declarar o código, a executar quando for clicado um item, no local onde é instanciado o adapter
adapter.setOnItemClickListener(new ItemClickListener() {
@Override
public void onItemClick(int position) {
Log.d(TAG, "Elemento " + position + " clicado.");
}
});
(1) - Justificado por este trecho da documentação:
(...)Sometimes, you may need to get the exact adapter position to do some actions in response to user events. In that case, you should use this method which will calculate the Adapter position of the ViewHolder.
(...)Por vezes pode ser necessário obter a posição exacta do adaptador para fazer algumas acções em resposta a eventos do usuário. Nesse caso, você deve usar este método que irá calcular a posição do ViewHolder no adaptador.