ActionBar点击首个菜单项触发异常，点击第二项正常的问题求助

这种问题我之前也碰到过几次，大概率是点击事件处理或者参数传递/解析的环节出了疏漏，咱们一步步拆解排查：

1. 先检查ActionBar菜单项的点击事件逻辑 #

最常见的坑是代码穿透——第一个菜单项的case分支没有正确标记事件已处理，导致代码继续执行后续分支的逻辑，Intent被重复修改甚至重复启动Activity，引发异常。

错误示例： #

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    Intent intent = new Intent(this, TargetActivity.class);
    switch (item.getItemId()) {
        case R.id.menu_first:
            intent.putExtra("msg", 123);
            startActivity(intent);
            // 这里漏掉了return true，代码会继续往下走！
        case R.id.menu_second:
            intent.putExtra("msg", 456);
            startActivity(intent);
            return true;
        default:
            return super.onOptionsItemSelected(item);
    }
}

修复方案： #

每个菜单项的处理分支结束后，一定要用return true标记事件已处理，避免代码穿透：

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    Intent intent = new Intent(this, TargetActivity.class);
    int itemId = item.getItemId();
    
    if (itemId == R.id.menu_first) {
        intent.putExtra("msg", 123);
        startActivity(intent);
        return true; // 告诉系统：这个事件我处理完了
    } else if (itemId == R.id.menu_second) {
        intent.putExtra("msg", 456);
        startActivity(intent);
        return true;
    }
    
    return super.onOptionsItemSelected(item);
}

2. 检查目标Activity的参数解析逻辑 #

如果点击第一个菜单项时目标Activity崩溃，很可能是参数类型不匹配或者未做非空/异常判断。比如你传递的是int类型的123，但在目标Activity里错误用了getStringExtra获取，就会拿到null，后续操作触发空指针异常。

错误示例： #

// 目标Activity中错误的解析方式
String msgStr = getIntent().getStringExtra("msg");
if (msgStr.equals("123")) { // msgStr是null，直接抛NullPointerException
    // 执行123对应的操作
}

修复方案： #

根据传递的参数类型用对应的方法获取，同时设置默认值避免异常：

// 目标Activity正确的解析逻辑
int msg = getIntent().getIntExtra("msg", -1); // -1作为默认值，表示未获取到有效参数
if (msg == 123) {
    // 执行123对应的操作
} else if (msg != -1) {
    // 其他菜单项的操作
}

3. 最后检查菜单项ID是否匹配 #

确认menu.xml中第一个菜单项的id，和代码中onOptionsItemSelected里判断的id完全一致——比如xml里是@+id/menu_first_item，但代码里写的是R.id.menu_first，就会导致第一个菜单项的点击事件走default分支，Intent没有传参，目标Activity处理时出错。

示例menu.xml： #

<menu xmlns:android="http://schemas.android.com/apk/res/android">
    <item
        android:id="@+id/menu_first" <!-- 和代码中的id保持一致 -->
        android:title="第一个菜单项"/>
    <item
        android:id="@+id/menu_second"
        android:title="第二个菜单项"/>
</menu>

按照这几步排查，应该就能解决第一个菜单项的异常问题啦。

内容的提问来源于stack exchange，提问作者Hatay Berkay Işıkoğlu