FirebaseRecyclerAdapter中setItemClickListener点击事件失效求助

Hey there! Let's dig into why your click events aren't firing even though the list loads successfully. I spotted two key issues in your code that are causing this:

1. 接口方法签名不匹配（核心问题） #

Look at your ItemClickListener interface and how you're calling it in HodViewHolder:

• Your interface defines onClick(View view, int position) (2 parameters)
• But in HodViewHolder's onClick method, you're calling itemClickListener.onClick(view, getAdapterPosition(), false); (3 parameters)

This mismatch means the callback can't be triggered properly—there's no matching method in the interface to execute.

修复方案（二选一） #

选项A：修改接口以匹配调用

Update your ItemClickListener to include the boolean parameter:

public interface ItemClickListener {
    void onClick(View view, int position, boolean isLongClick);
}

选项B：修改ViewHolder的调用以匹配接口

Adjust the onClick method in HodViewHolder to remove the extra parameter:

@Override
public void onClick(View view) {
    itemClickListener.onClick(view, getAdapterPosition());
}

2. Button可能拦截了ItemView的点击事件 #

Your name is a Button, which is clickable by default. This means it might be consuming the click event before it reaches the parent itemView that you set the listener on.

修复方案 #

If you don't need the Button to have its own click behavior, disable its clickability:

• In XML (hod_login_list.xml):

  <Button
      android:id="@+id/name_of_faculty"
      ...
      android:clickable="false"
      android:focusable="false"/>
  

• Or in code (inside HodViewHolder constructor):

  name.setClickable(false);
  name.setFocusable(false);
  

If you want the Button itself to trigger the click event, you can set the listener directly on it instead of the itemView:

public HodViewHolder(View itemView) {
    super(itemView);
    name = (Button) itemView.findViewById(R.id.name_of_faculty);
    // Replace itemView.setOnClickListener(this) with:
    name.setOnClickListener(this);
}

额外优化建议 #

Avoid using the global clickitem variable in populateViewHolder—it could lead to unexpected behavior when scrolling. Instead, access the model directly using getItem(position) in your click listener:

viewHolder.setItemClickListener(new ItemClickListener() {
    @Override
    public void onClick(View view, int position) {
        LeaveReq clickedItem = adapter.getItem(position);
        Intent hodlist = new Intent(LeaveRequests.this, HodReqList.class);
        hodlist.putExtra("hodid", adapter.getRef(position).getKey());
        startActivity(hodlist);
    }
});

After making these changes, your click events should start working as expected!

内容的提问来源于stack exchange，提问作者Vinit Rathi