如何在Dialog中调用finish()正确关闭Activity？

崩溃原因分析 #

从代码来看，Dialog中调用finish()崩溃大概率是上下文引用问题或者数据库操作未捕获异常：

• 匿名内部类中直接调用finish()，可能因上下文指向不明确导致Activity实例引用异常；
• 使用getApplicationContext()显示Toast，其生命周期与Activity不一致，可能引发上下文冲突；
• DbHelper的deleteNote方法如果存在未捕获的数据库异常，也会导致崩溃（表面看起来是finish()引发）。

修复方案 #

修改Dialog的点击事件代码，明确指定Activity上下文，并添加异常捕获：

@Override
public boolean onOptionsItemSelected(@NonNull MenuItem item) {
    Intent intent = new Intent();
    String content = etContentDetail.getText().toString();
    int id = getIntent().getIntExtra("id",0);
    DbHelper dbHelper = new DbHelper(this);

    switch (item.getItemId()) {

        case R.id.update:
            dbHelper.noteUpdate(id, content, DateTime.date(), DateTime.time(), System.currentTimeMillis());
            setResult(RESULT_OK, intent);
            finish(); // 补充分号，修复原代码语法错误
            break;

        case R.id.delete:
            AlertDialog.Builder builder = new AlertDialog.Builder(SecondActivity.this); // 明确绑定Activity上下文
            builder.setTitle("Delete Note");
            builder.setMessage("Are you sure you want to delete this note?");
            builder.setPositiveButton("Continue ", new DialogInterface.OnClickListener() {
                @Override
                public void onClick(DialogInterface dialogInterface, int i) {
                    try {
                        dbHelper.deleteNote(id);
                        // 使用Activity上下文显示Toast，避免生命周期冲突
                        Toast.makeText(SecondActivity.this, "Note Deleted", Toast.LENGTH_SHORT).show();
                        setResult(RESULT_OK, intent);
                        // 明确调用当前Activity的finish()方法
                        SecondActivity.this.finish();
                    } catch (Exception e) {
                        // 捕获数据库操作异常，方便排查问题
                        e.printStackTrace();
                        Toast.makeText(SecondActivity.this, "Delete failed: " + e.getMessage(), Toast.LENGTH_SHORT).show();
                    }
                }
            });
            builder.setNegativeButton("Cancel ", new DialogInterface.OnClickListener() {
                @Override
                public void onClick(DialogInterface dialogInterface, int i) {
                    dialogInterface.dismiss(); // 主动关闭Dialog，避免内存泄漏
                }
            }).create().show();
            break;
    }
    return true;
}

关键修改点 #

• 用SecondActivity.this替代默认上下文，确保Dialog和Toast都绑定当前Activity实例；
• 在数据库操作外层添加try-catch，捕获可能的异常，避免应用崩溃；
• 补全update分支中finish()的分号（原代码的语法错误）；
• 在Cancel按钮中主动调用dialogInterface.dismiss()关闭Dialog，减少内存泄漏风险。

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