11

Qual é a abordagem certa para pegar o click no RecyclerView?

1 - Dentro do onBindViewHolder usar a position, mesmo que dentro dos métodos setOnClickListener, o que transforma a variavel position em FINAL;

2 - Dentro do onBindViewHolder usar o holder.getAdapterPosition(), mesmo que dentro dos métodos setOnClickListener, o que transforma a variavel holder em FINAL;

3 - Algum outro jeito que envolva interface/outros jeitos, como neste exemplos. Outro.

EXEMPLO:

@Override
    public void onBindViewHolder(final AdapterViewHolder holder, final int position) {


        holder.imagem.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Toast.makeText(context, "posicao "+position,Toast.LENGTH_SHORT).show();
                Toast.makeText(context, "posicao "+holder.getAdapterPosition(),Toast.LENGTH_SHORT).show();
            }
        });


    }
0

2 Respostas 2

12

É 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.

9
  • E se eu quiser que uma imageview tenha um setOnClick e nesse onClick ela precise abrir uma nova tela? É só passar a lista no construtor do viewHolder e setar o onClick no imageView normalmente? Commented 19/10/2016 às 17:51
  • por exemplo, o notifyDataSetChanged(); funciona normal? Commented 19/10/2016 às 17:55
  • Eu adicionei o listener ao layout, mas pode ser adicionado a qualquer uma das views desse layout. Se necessita de outros dados para construir o listener passe-os no construtor. Em relação ao notifyDataSetChanged() poderá haver situações em que o método getAdapterPosition() pode retornar NO_POSITION mas julgo que não será o caso durante uma chamada a onCreateViewHolder().
    – ramaral
    Commented 19/10/2016 às 18:06
  • O 'notifyDataSetChanged();' só atualiza a lista que está na classe adapter, mas não atualiza a lista que está no viewHolder. Vou ter que manter duas listas e atualizá-las ao mesmo tempo? Commented 21/10/2016 às 14:17
  • vou ter que manter uma instancia global do holder no adapter e sempre que chamar notifyDataSetChanged eu atualizo também a lista do holder... isso não é gambiarra? Commented 21/10/2016 às 14:20
1

Eu costumo setar uma TAG para pegar a posição correta.

Por exemplo:

@Override
    public void onBindViewHolder(final AdapterViewHolder holder, final int position) {

        holder.imagem.setTag(position);
        holder.imagem.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                int clickPosition = (int) view.getTag();

                Toast.makeText(context, "posicao "+clickPosition,Toast.LENGTH_SHORT).show();

            }
        });


    }
5
  • Estou usando o Glide e ele crashou o app. Olha que engraçado. "You must not call setTag() on a view Glide is targeting" Commented 19/10/2016 às 13:37
  • Da uma olhada na sua versão do Glide, para ser aceito o setTag, deve ser a versão 3.6.0 Commented 19/10/2016 às 14:07
  • Estou na 3.7.0 ... Pelo que entendi eles removeram essa opção. É isso? Commented 19/10/2016 às 14:15
  • Encontrei uma pergunta em que cita exatamente essa questão, da uma olhada na solução, ve se te ajuda: stackoverflow.com/questions/34833627/… Commented 19/10/2016 às 16:21
  • esse final int position tem algum problema pelo fato de ser final? Commented 21/10/2016 às 14:47

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .