-4

デフォルトはチェックボックスなしです

プログラムを実行して [はい] チェックボックスをクリックすると、プログラムがオーバーフローしました

private void checkEdit1_Click(object sender, EventArgs e)
        {
            checkEdit2.Checked = false;
            textEdit1.Enabled = true;
            answered = true;
            optional = textEdit1.Text;

            if (!checkEdit1.Checked)
            {
                checkEdit1.Checked = true;
                checkEdit2.Checked = false;
                textEdit1.Enabled = true;
                optional = textEdit1.Text;
            }

        }

        private void checkEdit2_Click(object sender, EventArgs e)
        {
            checkEdit1.Checked = false;
            textEdit1.Enabled = false;
            answered = false;

            if (!checkEdit2.Checked)
            {
                checkEdit2.Checked = true;
                checkEdit1.Checked = false;
                textEdit1.Enabled = false;
                answered = false;
            }

        }

エラーは何だと思いますか？

4

2 に答える 2

1

イベントの代わりに、次のClickようにイベントを使用する必要がありますCheckedChanged。 

checkEdit1.CheckedChenged += new EventHandler(checkEdit1_CheckedChanged);
checkEdit2.CheckedChenged += new EventHandler(checkEdit2_CheckedChanged);

private void checkEdit1_CheckedChanged(object sender, EventArgs e)
{
    if(checkEdit1.Checked == checkEdit2.Checked)
      checkEdit2.Checked = !checkEdit.Checked;
}

private void checkEdit2_CheckedChanged(object sender, EventArgs e)
{
    if(checkEdit1.Checked == checkEdit2.Checked)
      checkEdit2.Checked = !checkEdit.Checked;
}

しかし、この場合の最善の方法は、ラジオ ボタンのグループを使用することです。

于 2012-11-29T18:11:36.487 に答える
0

これらのメソッドがに接続されていると仮定すると、checkEdit1すでに変更されているcheckEdit2ため、に変更を加えないことをお勧めします。代替の状態を変更するだけです。checkEdit1checkEdit1_Click

ただし、もう一方の状態を変更すると、注意しない限り、コールバックされます。最終的に、コンピュータはあきらめます-オーバーフロー！

@ Cyborgx37のコメントで述べたように、ここではラジオボタンの方がUXの選択肢として適しています。

考えられる解決策は、単一のメソッドをOnClicktoBOTHチェックボックスにバインドすることです。

private bool internallyUpdating = false;

private void CheckboxClick(object sender, EventArgs e)
{
    if ( !internallyUpdating )
    {
        // Prevent subsequent changes
        internallyUpdating = true;

        // Exchange 'checked' state
        if ( sender == checkEdit1 )
        {
            checkEdit2.Checked = !checkEdit2.Checked;
        } 
        else // if (sender == checkEdit2)
        {
            checkEdit1.Checked = !checkEdit1.Checked;
        }

        // other logic here..

        // restore 'on change' functionality.
        internallyUpdating = false;
    }
于 2012-11-29T18:04:27.940 に答える