RecyclerView.Adapter通知方法调用后TableLayout重复及视图点击失效问题

Hey there! Let's break down and fix those two annoying issues you're having with your CardView list and editable TableLayout. I've run into similar problems before, so let's tackle them one by one.

1. Fixing Duplicate TableLayout Rows After Edit/Delete #

The root cause here is almost always RecyclerView view reuse—when you call notifyItemChanged() or related methods, the old TableLayout in the reused ViewHolder still has its original rows, and your code is adding new ones on top instead of replacing them. Here's how to fix it:

Step 1: Clear TableLayout Before Refilling #

In your Adapter's onBindViewHolder() method, always clear all existing child views from the TableLayout before adding new rows. This ensures you start with a clean slate every time the ViewHolder is reused.

@Override
public void onBindViewHolder(MyCardViewHolder holder, int position) {
    // Get the data for the current CardView
    CardItem currentItem = mCardList.get(position);
    
    // Critical: Clear existing rows first to avoid duplicates
    holder.tableLayout.removeAllViews();
    
    // Repopulate the TableLayout with updated data
    for (TableRowData row : currentItem.getTableRows()) {
        TableRow newRow = createTableRow(holder.itemView.getContext(), row, position);
        holder.tableLayout.addView(newRow);
    }
}

Step 2: Ensure Your Data Model Is Updated Correctly #

Before calling any notify methods, double-check that the underlying data for the CardView is actually modified. For example, when deleting a TableRow, make sure you remove it from the list stored in your CardItem object—if the data stays the same, the TableLayout will just re-add the old rows.

Step 3: Use Precise Notify Methods #

Avoid overusing notifyDataSetChanged() (which refreshes the entire list) unless necessary. Stick to notifyItemChanged(position) for edits, or notifyItemRemoved(position) for deletions—these are more efficient and reduce the chance of unexpected view reuse side effects.

2. Fixing Unresponsive View Clicks #

Click issues usually happen because child views in the TableLayout (like EditTexts or Buttons) are stealing focus from the parent CardView. Here's how to resolve this:

Step 1: Block Child View Focus #

Add android:descendantFocusability="blocksDescendants" to your CardView's XML layout. This prevents child views from intercepting touch events meant for the parent CardView:

<androidx.cardview.widget.CardView
    android:id="@+id/card_view"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:clickable="true"
    android:focusable="true"
    android:descendantFocusability="blocksDescendants">

    <TableLayout
        android:id="@+id/table_layout"
        android:layout_width="match_parent"
        android:layout_height="wrap_content" />

</androidx.cardview.widget.CardView>

Step 2: Set Click Listeners in ViewHolder Constructor #

Don't set click listeners in onBindViewHolder()—this can lead to multiple listeners being attached as ViewHolders are reused. Instead, set them once in the ViewHolder's constructor, and use an interface to pass clicks back to your activity/fragment:

public class MyCardViewHolder extends RecyclerView.ViewHolder {
    CardView cardView;
    TableLayout tableLayout;

    public MyCardViewHolder(View itemView, OnCardClickListener listener) {
        super(itemView);
        cardView = itemView.findViewById(R.id.card_view);
        tableLayout = itemView.findViewById(R.id.table_layout);

        // Set click listener once here
        cardView.setOnClickListener(v -> {
            if (listener != null) {
                listener.onCardClicked(getAdapterPosition());
            }
        });
    }

    // Interface for click callbacks
    public interface OnCardClickListener {
        void onCardClicked(int position);
    }
}

Step 3: Disable Focus on Editable Child Views #

If you have EditTexts in your TableRows, add these attributes to prevent them from stealing focus (you can toggle these if you need the EditText to become editable on user tap):

<EditText
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:focusable="false"
    android:focusableInTouchMode="false" />

Quick Recap #

• For duplicate TableLayout rows: Always clear the TableLayout before refilling, ensure your data model is updated, and use precise notify methods.
• For unresponsive clicks: Block child view focus with descendantFocusability, set click listeners in the ViewHolder constructor, and disable focus on editable child views when needed.

内容的提问来源于stack exchange，提问作者Iago Brayham