Android：向Button的onClick传参并避免存储Context实例变量

我懂你的痛点——把Context存为自定义OnClickListener的实例变量不仅不优雅，还暗藏内存泄漏的风险。这里有几个更合理的方案，完全不需要持有Context成员变量就能解决问题：

方案1：利用onClick参数中的View获取Context #

你根本不需要提前传递Context！onClick(View view)方法里的view本身就可以通过view.getContext()拿到关联的Context，而且你的ImageButton本身也是View，同样能调用getContext()。修改你的WhenClicked类如下：

public class WhenClicked implements View.OnClickListener {
    private ImageButton imgbtn;

    // 只需要传入ImageButton，不需要Context参数
    public WhenClicked(ImageButton imgbutton) {
        this.imgbtn = imgbutton;
    }

    @Override
    public void onClick(View view) {
        imgbtn.setColorFilter(Color.argb(150, 255, 255, 255));
        MainActivity.incrementClicked();
        
        // 直接从View获取Context启动新Activity
        Intent intent = new Intent(view.getContext(), TargetActivity.class);
        view.getContext().startActivity(intent);
        // 用imgbtn.getContext()也能达到同样效果
    }
}

然后在MainActivity里初始化时只需传入ImageButton：

btn.setOnClickListener(new WhenClicked(btn));

方案2：用Lambda/匿名内部类简化实现（更推荐） #

如果你的点击逻辑不需要复用，直接用Lambda表达式（Android API 26+支持）或者匿名内部类，就能在Activity内部直接写逻辑，完全不需要自定义类，还能直接用MainActivity.this作为Context，代码更简洁：

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    final ImageButton btn = findViewById(R.id.imageButton2);
    
    btn.setOnClickListener(v -> {
        btn.setColorFilter(Color.argb(150, 255, 255, 255));
        incrementClicked();
        // 直接用当前Activity作为Context启动Intent
        Intent intent = new Intent(MainActivity.this, TargetActivity.class);
        startActivity(intent);
    });
}

额外提醒：避免内存泄漏 #

你之前觉得存储Context“obviously bad”是对的——如果WhenClicked持有Activity的强引用，当Activity被销毁时，这个引用可能导致内存泄漏。而上面的两个方案：

• 方案1里的view.getContext()获取的是与View生命周期绑定的Context，不会额外造成泄漏；
• 方案2的Lambda/匿名内部类只要不持有Activity的静态引用，也不会有泄漏风险（如果有耗时操作，建议用WeakReference包裹Activity）。

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