Recycler View Onclick Android funktioniert nicht

Question

Ich habe eine CustomersAdapter Klasse für meine RecyclerView und ich versuche Onclick-Ereignis auf der RecyclerView-Zeile zu implementieren. Aber leider funktioniert es nicht für mich.

public class CustomersAdapter extends RecyclerView.Adapter<CustomersAdapter.MyViewHolder> { 

    private List<Customers> customerList; 
    Context ctx; 
    @Override 
    public MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType) { 
     View itemView = LayoutInflater.from(parent.getContext()) 
       .inflate(R.layout.customers_layout, parent, false); 

     return new MyViewHolder(itemView); 

    } 

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

     Customers customer = customerList.get(position); 
     holder.customerName.setText(customer.getCustomerName()); 
    } 

    @Override 
    public int getItemCount() { 
     return customerList.size(); 

    } 

    public CustomersAdapter(List<Customers> customerList,Context ctx) { 
     this.customerList = customerList; 
     this.ctx = ctx; 
    } 



    public class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener { 
     public TextView customerName; 

     public MyViewHolder(View view) { 
      super(view); 
      customerName = (TextView) view.findViewById(R.id.cust_name); 

     } 

     @Override 
     public void onClick(View v) { 
      int position = getAdapterPosition(); // gets item position 
      if (position != RecyclerView.NO_POSITION) { 
       Customers customer= customerList.get(position); 
       Toast.makeText(ctx, customer.getCustomerName(), Toast.LENGTH_SHORT).show(); 
      } 
     } 
    } 
} 

XML: customers_layout

<?xml version="1.0" encoding="utf-8"?> 
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" 
    android:layout_width="match_parent" android:layout_height="wrap_content" 
    android:clickable="true" 
    > 

    <android.support.v7.widget.CardView 
     android:layout_width="match_parent" 
     android:layout_height="wrap_content"> 

     <RelativeLayout 
      android:layout_width="match_parent" 
      android:layout_height="match_parent" 
      android:layout_marginTop="10dp" 
      android:layout_marginBottom="10dp" 
      android:layout_marginLeft="5dp" 
      android:layout_marginRight="5dp" 
      > 

      <ImageView 
       android:layout_width="30dp" 
       android:layout_height="30dp" 
       android:id="@+id/cust_img" 
       android:src="@drawable/person" 
       /> 

      <TextView 
       android:layout_width="wrap_content" 
       android:layout_height="wrap_content" 
       android:id="@+id/cust_name" 
       android:textColor="#000" 
       android:text="HELLO" 

       android:layout_marginLeft="13dp" 
       android:layout_marginStart="13dp" 
       android:layout_centerVertical="true" 
       android:layout_toRightOf="@+id/cust_img" 
       android:layout_toEndOf="@+id/cust_img" /> 
     </RelativeLayout> 


    </android.support.v7.widget.CardView> 

</RelativeLayout> 

Answer 1

Versuchen Sie es stattdessen. Implementieren Sie onClick() im Konstruktor.

public MyViewHolder(View view) { 
    super(view); 
    customerName = (TextView) view.findViewById(R.id.cust_name); 
    view.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View v) { 
      int position = getAdapterPosition(); // gets item position 
      if (position != RecyclerView.NO_POSITION) { 
       Customers customer= customerList.get(position); 
       Toast.makeText(ctx, customer.getCustomerName(), Toast.LENGTH_SHORT).show(); 
      } 
     } 
    }); 
} 

Answer 2

Sie vergessen setOnClickListener zu setzen .. hinzufügen setOnClickListener in onCreateViewHolder

private final OnClickListener mOnClickListener = new MyOnClickListener(); 
@Override 
public MyViewHolder onCreateViewHolder(ViewGroup parent, int viewType) { 
    View itemView = LayoutInflater.from(parent.getContext()) 
      .inflate(R.layout.customers_layout, parent, false); 
    itemView.setOnClickListener(mOnClickListener); 
    return new MyViewHolder(itemView); 

} 

Answer 3

Sie für Sie Listenelement-Layout setOnTouchListener(); vergessen. Sie können den Touch-Listener für das Hauptlayout Ihres Listenelements festlegen.

Answer 4

Hörer auf halter.itemView gesetzt.

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

    Customers customer = customerList.get(position); 
    holder.customerName.setText(customer.getCustomerName()); 
    holder.itemView.setOnClickListener(new View.OnClickListener() { 
     @Override 
     public void onClick(View view) { 

     } 
    }); 
} 

in onBindViewHolder-Methode.

Answer 5

Sie haben vergessen, onClickListener zu Ihrer Ansicht hinzuzufügen. Deshalb funktioniert Ihr onclick nicht. Hier müssen Sie Ihren Klick-Listener mit Ihren Ansichten hinzufügen.

public MyViewHolder(View view) { 
     super(view); 
     customerName = (TextView) view.findViewById(R.id.cust_name); 

     //bind the click listener with your view, like 
     customerName.setOnClickListerner(this); // now it will get called when you click this customerName view 
    } 

Answer 6

ViewHolder ist kein View die onClick() implementiert. In diesem Fall könnten Sie es wie folgt emulieren:

public MyViewHolder(View view) { 
    super(view); 
    customerName = (TextView) view.findViewById(R.id.cust_name); 
    //you forgot to call this 
    view.setOnClickListener(this); 
}