自定义Toolbar单按钮切换布局显隐及代码变量未定义问题咨询

你遇到的核心问题是**openLayout变量的作用域不对**——你把它定义在了onOptionsItemSelected方法内部，每次点击菜单选项时，这个方法都会重新执行，变量会被重新初始化为true，导致永远只会执行显示布局的逻辑，根本触发不了隐藏的分支。

修复步骤 #

把openLayout改成类的成员变量，让它的状态能在Activity（或Fragment）的生命周期中持续保存，不会每次点击都被重置。

修改后的完整代码示例 #

// 把openLayout定义为类的成员变量，初始值根据你默认布局状态设置
private boolean openLayout = false; // 默认隐藏布局设为false，默认显示则设为true
private LinearLayout linearLayout; // 建议将linearLayout也改为类成员变量，避免重复findViewById

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.your_activity_layout); // 替换为你的布局文件名
    linearLayout = findViewById(R.id.show); // 在onCreate中仅初始化一次
    // 其他初始化逻辑...
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    int id = item.getItemId();

    if (id == R.id.action_categure) {
        // 直接使用类成员变量的状态判断切换逻辑
        if (openLayout) {
            linearLayout.setVisibility(View.GONE);
            openLayout = false;
        } else {
            linearLayout.setVisibility(View.VISIBLE);
            openLayout = true;
        }
        return true; // 标记事件已处理，无需父类再处理
    } else if (id == R.id.action_card) {
        // 这里可添加action_card的具体处理逻辑
        return true;
    }
    return super.onOptionsItemSelected(item);
}

额外优化提示 #

• 将linearLayout改为类成员变量，仅在onCreate中初始化一次，避免每次点击都执行findViewById，提升性能。
• 处理完对应菜单选项后返回true，明确告知系统该事件已被处理，无需再传递给父类方法。

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