Android中能否嵌套setOnCheckedChangeListener？子开关点击无响应求助

你的代码 #

GroupView.mSwitchForReminder.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
        for (int i = 0; i < GroupView.GetCount(); i++) {
            ItemView itemView = (ItemView) GroupView.GetItemView(i);
            if (isChecked) {
                itemView.SetReminderStatus(true);
            } else {
                itemView.SetReminderStatus(false);
            }
            itemView.mSwitchForReminder1.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    if (isChecked) {

                    } else {
                        itemView.SetReminderStatus(false);
                    }
                }
            });
        }

    }
});

写法可行性判断与问题原因 #

这种写法完全不可行，点击ItemView的mSwitchForReminder1无反应的核心原因有两个：

• 监听绑定时机错误：只有当GroupView的开关被点击后，ItemView的开关才会绑定监听逻辑。如果用户从未操作过GroupView的开关，ItemView的开关从始至终没有监听，自然点击无任何反应。
• 重复绑定监听：每次GroupView开关状态变化，都会给所有ItemView的开关重新绑定一遍监听，这不仅会覆盖之前的监听逻辑，还会造成不必要的冗余操作，引发逻辑混乱。

解决方案 #

正确的做法是将ItemView开关的监听逻辑提前绑定（比如在ItemView初始化、列表Adapter的绑定阶段），GroupView的开关只负责批量修改所有ItemView的状态，同时避免批量修改时触发ItemView开关的监听。

方案1：优化监听绑定时机与批量修改逻辑

首先在ItemView的初始化/列表绑定阶段（以RecyclerView Adapter为例）绑定监听：

@Override
public void onBindViewHolder(ItemViewHolder holder, int position) {
    ItemView itemView = holder.itemView;
    // 只绑定一次监听，避免重复创建
    itemView.mSwitchForReminder1.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            if (!isChecked) {
                itemView.SetReminderStatus(false);
            }
            // 如需同步GroupView开关状态，可在此处判断所有Item状态后修改GroupView开关
        }
    });
}

然后修改GroupView开关的逻辑，批量修改时暂时移除ItemView的监听，避免触发不必要的逻辑：

GroupView.mSwitchForReminder.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
        for (int i = 0; i < GroupView.GetCount(); i++) {
            ItemView itemView = (ItemView) GroupView.GetItemView(i);
            // 暂时移除监听，避免批量设置时触发ItemView的开关逻辑
            itemView.mSwitchForReminder1.setOnCheckedChangeListener(null);
            // 设置开关状态
            itemView.mSwitchForReminder1.setChecked(isChecked);
            // 恢复监听
            itemView.mSwitchForReminder1.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
                @Override
                public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
                    if (!isChecked) {
                        itemView.SetReminderStatus(false);
                    }
                }
            });
            // 同步提醒状态
            itemView.SetReminderStatus(isChecked);
        }
    }
});

方案2：封装ItemView状态设置逻辑（更优雅）

在ItemView类内部封装状态设置方法，统一处理监听的移除与恢复：

public class ItemView extends View {
    // 提前保存监听实例，避免重复创建
    private CompoundButton.OnCheckedChangeListener mSwitchListener = new CompoundButton.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
            if (!isChecked) {
                SetReminderStatus(false);
            }
        }
    };

    // 初始化时绑定监听
    public ItemView(Context context) {
        super(context);
        init();
    }

    private void init() {
        // 其他初始化逻辑...
        mSwitchForReminder1.setOnCheckedChangeListener(mSwitchListener);
    }

    // 批量设置状态的方法
    public void setBatchReminderStatus(boolean isChecked) {
        // 暂时移除监听
        mSwitchForReminder1.setOnCheckedChangeListener(null);
        // 设置开关状态
        mSwitchForReminder1.setChecked(isChecked);
        // 恢复监听
        mSwitchForReminder1.setOnCheckedChangeListener(mSwitchListener);
        // 同步提醒状态
        SetReminderStatus(isChecked);
    }
}

此时GroupView的逻辑会非常简洁：

GroupView.mSwitchForReminder.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
        for (int i = 0; i < GroupView.GetCount(); i++) {
            ItemView itemView = (ItemView) GroupView.GetItemView(i);
            itemView.setBatchReminderStatus(isChecked);
        }
    }
});

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