Android中getBindingAdapterPosition部分Item返回-1的问题排查与解决

问题场景 #

在Android中使用RecyclerView展示Task对象列表，适配器的ViewHolder内实现了重命名功能。执行重命名时通过getBindingAdapterPosition()获取选中项位置，但部分Item（集中在索引6、7附近，列表变长后类似位置也会出现）会返回RecyclerView.NO_POSITION（即-1），其余Item正常。

已尝试的无效方案：

• 判断getBindingAdapterPosition()是否为-1以跳过重命名，仅能规避错误无法解决根本问题
• 调用adapter.notifyItemChanged(adapterPosition)通知更新，问题仍存在

相关代码 #

public static class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener
{
    TextView textView;
    OnItemListener onItemListener;
    ArrayList<Task> list;
    private final RecyclerViewAdapter adapter;

    MyViewHolder(@NonNull View itemView, OnItemListener onItemListener, ArrayList<Task> list, RecyclerViewAdapter adapter) {
        super(itemView);

        textView = itemView.findViewById(R.id.singleView);
        this.onItemListener = onItemListener;
        this.list = list;
        this.adapter = adapter;

        itemView.setOnClickListener(this);
        itemView.setOnLongClickListener(view -> {
            onLongClick(); // 此处未传递position参数，但方法定义需要int参数
            return true;
        });
    }

    // 方法定义需要int position，但调用时未传参
    private void onLongClick(int position)
    {
        String text = textView.getText().toString();
        final EditText input = new EditText(itemView.getContext());
        input.setBackground(ContextCompat.getDrawable(itemView.getContext(), R.drawable.edit_text_dialog));
        input.setTextColor(ContextCompat.getColor(itemView.getContext(), R.color.colorText));
        input.setText(text);
        input.setSingleLine();
        input.setImeOptions(EditorInfo.IME_ACTION_SEND);
        input.setPadding(28, 28, 28, 28);

        AlertDialog.Builder builder = new AlertDialog.Builder(itemView.getContext(), R.style.DialogTheme);
        builder.setTitle(R.string.dialog_edit_title)
                .setMessage(R.string.dialog_edit_message)
                .setView(input)
                .setPositiveButton(R.string.dialog_edit_positive_button, (dialogInterface, i) ->
                {
                    String editedText = input.getText().toString();
                    if (!editedText.isEmpty())
                    {
                        // 调用saveEditedTaskName时传递了3个参数，但方法定义仅接受2个
                        saveEditedTaskName(itemView.getContext(), editedText, position);
                    }
                })
                .setNegativeButton(R.string.dialog_edit_negative_button, (dialogInterface, i) -> dialogInterface.cancel());
        AlertDialog dialog = builder.show();

        input.setOnEditorActionListener((textView, actionId, keyEvent) ->
        {
            if (actionId == EditorInfo.IME_ACTION_SEND)
            {
                String editedText = input.getText().toString();
                if (!editedText.isEmpty())
                {
                    // 同样传递了3个参数，与方法定义不符
                    saveEditedTaskName(itemView.getContext(), editedText, position);
                }
                dialog.dismiss();
                return true;
            }
            return false;
        });
    }

    // 方法定义仅接受2个参数，但调用时传入了3个
    private void saveEditedTaskName(Context mContext, String editedText)
    {
        SharedPreferences sharedPrefs = mContext.getSharedPreferences(MainActivity.SHARED_PREFS, 0);
        String categoryId = sharedPrefs.getString(MainActivity.CATEGORY_ID_CHOICE, "");

        FirebaseHelper firebaseHelper = new FirebaseHelper();

        int adapterPosition = getBindingAdapterPosition();
        if (adapterPosition != RecyclerView.NO_POSITION)
        {
            Task task = list.get(getBindingAdapterPosition());
            String taskId = task.getTaskId();

            DatabaseReference databaseReference = firebaseHelper.getDatabaseReference();
            databaseReference.child(categoryId).child(DatabaseNodes.TASKS)
                    .child(taskId).child(DatabaseNodes.TASK_NAME).setValue(editedText);

            if (adapter != null)
            {
                adapter.notifyItemChanged(adapterPosition);
            }
        }
        else
        {
            Toast.makeText(mContext, "An error occurred. Please try again.", Toast.LENGTH_SHORT).show();
        }
    }

    // Other code
}

问题原因 #

1. 参数传递错误：代码中onLongClick方法定义需要int position参数，但调用时未传递；saveEditedTaskName方法定义仅接受2个参数，但调用时传入了3个。这会导致编译错误或变量未初始化，直接引发位置获取异常。
2. 视图复用导致位置失效：RecyclerView会回收滑出屏幕的Item视图复用，而弹窗是异步操作——用户打开弹窗后可能滑动RecyclerView，导致原ViewHolder对应的Item被回收，此时getBindingAdapterPosition()会返回-1。
3. ViewHolder持有冗余引用：ViewHolder直接持有ArrayList<Task>和RecyclerViewAdapter的引用，当Adapter数据源更新时，ViewHolder内的列表引用可能与当前数据源不一致，导致位置匹配错误。

解决方法 #

1. 修复参数传递错误 #

修正onLongClick和saveEditedTaskName的参数定义与调用逻辑，确保参数一致：

// 移除onLongClick的position参数，直接在方法内获取或使用绑定的Task对象
private void onLongClick() {
    // ... 弹窗逻辑不变，调用saveEditedTaskName时无需传递position
}

// 调整saveEditedTaskName参数，移除多余的position参数
private void saveEditedTaskName(Context mContext, String editedText) {
    // ... 后续逻辑修改为使用绑定的Task对象
}

2. 直接绑定Task对象到ViewHolder #

避免依赖位置信息，在ViewHolder中持有当前Item对应的Task实例，彻底摆脱位置依赖：

public static class MyViewHolder extends RecyclerView.ViewHolder implements View.OnClickListener {
    TextView textView;
    OnItemListener onItemListener;
    Task currentTask; // 直接持有当前Item的Task对象

    MyViewHolder(@NonNull View itemView, OnItemListener onItemListener) {
        super(itemView);
        textView = itemView.findViewById(R.id.singleView);
        this.onItemListener = onItemListener;
        itemView.setOnClickListener(this);
        itemView.setOnLongClickListener(view -> {
            onLongClick();
            return true;
        });
    }

    // 绑定数据时设置当前Task
    void bind(Task task) {
        this.currentTask = task;
        textView.setText(task.getTaskName());
    }

    private void onLongClick() {
        String text = currentTask.getTaskName();
        final EditText input = new EditText(itemView.getContext());
        // ... 弹窗创建逻辑不变
        builder.setPositiveButton(R.string.dialog_edit_positive_button, (dialogInterface, i) -> {
            String editedText = input.getText().toString();
            if (!editedText.isEmpty()) {
                saveEditedTaskName(itemView.getContext(), editedText);
            }
        });
        // ... 其他弹窗逻辑不变
    }

    private void saveEditedTaskName(Context mContext, String editedText) {
        SharedPreferences sharedPrefs = mContext.getSharedPreferences(MainActivity.SHARED_PREFS, 0);
        String categoryId = sharedPrefs.getString(MainActivity.CATEGORY_ID_CHOICE, "");

        FirebaseHelper firebaseHelper = new FirebaseHelper();
        // 直接使用绑定的Task对象获取ID，无需依赖位置
        String taskId = currentTask.getTaskId();

        DatabaseReference databaseReference = firebaseHelper.getDatabaseReference();
        databaseReference.child(categoryId).child(DatabaseNodes.TASKS)
                .child(taskId).child(DatabaseNodes.TASK_NAME).setValue(editedText)
                .addOnSuccessListener(aVoid -> {
                    // 更新本地Task对象名称
                    currentTask.setTaskName(editedText);
                    // 通知Adapter更新当前Item
                    if (onItemListener != null) {
                        onItemListener.onTaskUpdated(getBindingAdapterPosition());
                    }
                })
                .addOnFailureListener(e -> {
                    Toast.makeText(mContext, "更新失败，请重试", Toast.LENGTH_SHORT).show();
                });
    }

    public interface OnItemListener {
        void onClick(int position);
        void onTaskUpdated(int position);
    }
}

3. 优化Adapter数据绑定 #

在Adapter的onBindViewHolder中调用ViewHolder的bind方法传递Task对象：

@Override
public void onBindViewHolder(@NonNull MyViewHolder holder, int position) {
    Task task = taskList.get(position);
    holder.bind(task);
}

4. 移除ViewHolder的冗余引用 #

ViewHolder只负责展示数据和触发事件，不要持有ArrayList<Task>和RecyclerViewAdapter的引用，通过OnItemListener回调将事件传递给上层处理。

替代方案 #

• 使用getAbsoluteAdapterPosition()：如果是单个Adapter场景，getAbsoluteAdapterPosition()与getBindingAdapterPosition()效果一致，但在ConcatAdapter场景下更可靠，不过仍无法解决视图复用导致的异步操作位置失效问题。
• 使用DiffUtil更新列表：通过DiffUtil.Callback计算数据差异，仅更新变化的Item，避免全量刷新导致的视图状态混乱。
• 将业务逻辑移至ViewModel/Adapter：ViewHolder仅负责UI交互，重命名、数据库操作等逻辑放在ViewModel或Adapter中，降低ViewHolder的职责耦合。

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