Android CheckBox OnClickListener 邏輯錯誤：if-else 缺少大括號導致 Toast 重複彈出

經典 Java 語法陷阱：if/else 後若不加大括號 {}，僅控制緊跟的單條語句。原代碼中 else 只管了 setVisibility，導致後續 Toast.show() 無條件執行。

錯誤代碼分析

on.setOnClickListener(v -> {
    if (((CheckBox) v).isChecked()) 
        receiverEmail.setVisibility(View.GONE);           // 屬於 if
    if (((CheckBox) v).isChecked()) 
        Toast.makeText(..., "Saved", ...).show();         // 屬於 if
    else 
        receiverEmail.setVisibility(View.VISIBLE);        // 屬於 else
    Toast.makeText(..., "Enter Email", ...).show();       // ❌ 無條件執行！
});

正確代碼：加上大括號明確作用域

on.setOnClickListener(v -> {
    CheckBox cb = (CheckBox) v;
    if (cb.isChecked()) {
        receiverEmail.setVisibility(View.GONE);
        Toast.makeText(getContext(), "Your Information is Saved", Toast.LENGTH_LONG).show();
    } else {
        receiverEmail.setVisibility(View.VISIBLE);
        Toast.makeText(getContext(), "Enter Your Email Address", Toast.LENGTH_LONG).show();
    }
});

// 第二個 CheckBox 同理
on2.setOnClickListener(v -> {
    CheckBox cb = (CheckBox) v;
    if (cb.isChecked()) {
        clientID.setVisibility(View.GONE);
        Toast.makeText(getContext(), "Your Information is Saved", Toast.LENGTH_LONG).show();
    } else {
        clientID.setVisibility(View.VISIBLE);
        Toast.makeText(getContext(), "Enter Your Email Address", Toast.LENGTH_LONG).show();
    }
});

最佳實踐：永遠使用大括號

進階：使用 OnCheckedChangeListener 替代 OnClickListener

CheckBox 狀態改變更語義化用 setOnCheckedChangeListener，避免點擊與狀態不同步：

checkBox.setOnCheckedChangeListener((buttonView, isChecked) -> {
    if (isChecked) {
        editText.setVisibility(View.GONE);
        Toast.makeText(context, "Saved", Toast.LENGTH_SHORT).show();
    } else {
        editText.setVisibility(View.VISIBLE);
        Toast.makeText(context, "Enter Email", Toast.LENGTH_SHORT).show();
    }
});

Kotlin 簡化寫法

checkBox.setOnCheckedChangeListener { _, isChecked ->
    editText.visibility = if (isChecked) View.GONE else View.VISIBLE
    Toast.makeText(context, if (isChecked) "Saved" else "Enter Email", Toast.LENGTH_SHORT).show()
}

總結

1. if/else 必須加大括號，無論語句多寡。
2. CheckBox 狀態監聽首選 OnCheckedChangeListener。
3. Toast 文案提取為常量/字符串資源，避免硬編碼重複。
4. 代碼審查時重點檢查無大括號的流程控制語句。

這個錯誤看似低級，但極其常見，甚至出現在資深代碼中。養成「永遠加大括號」習慣，能省去無數調試時間。