Android AppBar Switch仅生效一次问题求助

你的问题核心在于Service直接操作Activity的Menu元素，并且重复设置ActionView导致监听失效，同时这种跨组件直接操作UI的方式本身就不安全，容易引发状态不一致和内存泄漏问题。下面一步步帮你解决：

1. 移除Service中直接操作Activity Menu的代码 #

Service不应该直接持有Activity的引用（比如MainActivity.getThis()这种单例模式很危险，容易导致内存泄漏），更不能直接修改Activity的UI元素。我们改用广播来通知MainActivity更新开关状态。

修改后的Service代码：

@Override
public int onStartCommand(Intent intent, int flags, int startId) {
    if (intent.getAction() != null && intent.getAction().equals("SWITCH OFF")) {
        stopForeground(true);
        stopSelf();
        // 发送广播通知Activity更新开关状态
        Intent updateSwitchIntent = new Intent("UPDATE_SWITCH_STATE");
        updateSwitchIntent.putExtra("IS_CHECKED", false);
        sendBroadcast(updateSwitchIntent);
    } else {
        Toast.makeText(this, "Service Started", Toast.LENGTH_SHORT).show();
    }
    return START_NOT_STICKY;
}

2. 优化Activity中开关监听的设置逻辑 #

你在onCreateOptionsMenu和onPrepareOptionsMenu中都调用了switchlistener，每次都会重新设置ActionView，这会导致原来的Switch实例被替换，监听也会失效。我们只需要在onCreateOptionsMenu中初始化一次ActionView，在onPrepareOptionsMenu中只更新状态，不要重复创建视图。

同时，在Activity中注册广播接收器，接收Service发来的状态更新通知：

修改后的MainActivity代码：

private Switch mSwitch;
private BroadcastReceiver switchUpdateReceiver;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    
    // 注册广播接收器
    switchUpdateReceiver = new BroadcastReceiver() {
        @Override
        public void onReceive(Context context, Intent intent) {
            if ("UPDATE_SWITCH_STATE".equals(intent.getAction()) && mSwitch != null) {
                boolean isChecked = intent.getBooleanExtra("IS_CHECKED", false);
                // 更新开关状态时先移除监听，避免触发不必要的回调
                mSwitch.setOnCheckedChangeListener(null);
                mSwitch.setChecked(isChecked);
                mSwitch.setOnCheckedChangeListener(switchChangeListener);
            }
        }
    };
    IntentFilter filter = new IntentFilter("UPDATE_SWITCH_STATE");
    registerReceiver(switchUpdateReceiver, filter);
}

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    getMenuInflater().inflate(R.menu.menu, menu);
    // 只在这里初始化一次ActionView和监听
    MenuItem appBarSwitch = menu.findItem(R.id.app_bar_switch);
    appBarSwitch.setActionView(R.layout.switch_item);
    mSwitch = appBarSwitch.getActionView().findViewById(R.id.action_switch);
    mSwitch.setOnCheckedChangeListener(switchChangeListener);
    return true;
}

@Override
public boolean onPrepareOptionsMenu(Menu menu) {
    // 这里仅确保开关状态与服务实际运行状态一致，无需重复设置ActionView和监听
    return super.onPrepareOptionsMenu(menu);
}

// 把监听抽成单独变量，方便复用和临时移除
private CompoundButton.OnCheckedChangeListener switchChangeListener = new CompoundButton.OnCheckedChangeListener() {
    @Override
    public void onCheckedChanged(CompoundButton compoundButton, boolean isChecked) {
        if (isChecked) {
            Toast.makeText(MainActivity.this, "Fired up!", Toast.LENGTH_SHORT).show();
            Intent intent = new Intent(getApplicationContext(), BroadcastReceiverService.class);
            startService(intent);
        } else {
            Intent intent = new Intent(getApplicationContext(), BroadcastReceiverService.class);
            intent.setAction("SWITCH OFF");
            startService(intent);
        }
    }
};

@Override
protected void onDestroy() {
    super.onDestroy();
    // 注销广播接收器，避免内存泄漏
    if (switchUpdateReceiver != null) {
        unregisterReceiver(switchUpdateReceiver);
    }
}

3. 额外注意事项 #

• 不要使用MainActivity.getThis()这类方式获取Activity实例，会导致Activity无法被GC回收，引发内存泄漏。
• 更新开关状态时先移除监听，避免触发不必要的回调（比如Service通知关闭开关时，不会再次调用stopService）。
• 如果是Jetpack组件环境，也可以用LiveData或ViewModel替代广播，实现更优雅的状态同步。

这样修改后，Service和Activity通过广播解耦，开关监听不会因重复创建视图失效，状态同步也更可靠。

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