Question

我试图强迫用户在允许下一步之前选择一个rabiobutton。我使“下一步”按钮不可见，但我有10个radiobutton，如果检查其中任何一个，必须进行验证。根据定义，只能检查1个radiobutton。我的代码看起来像这样：

b1.Text = "Next";
b1.Parent = fpn1;
fpn1.Controls.Add(b1);
b1.Dock = DockStyle.Bottom;
b1.BackColor = Color.LightGray;
b1.Visible = false;
RadioButton rb;
while (b1.Visible == false)
        {
            MessageBox.Show("LOOOL");
            //Thread.Sleep(5000);
            rb = fpn1.Controls.OfType<RadioButton>()
                             .FirstOrDefault(r => r.Checked);
            if (rb != null)
            {
                b1.Visible = true;
            }
        }

因此，虽然没有点击我的单选按钮，但b1是不可见的。问题是......这会进入无限循环。用户甚至不能再选择任何按钮，因为页面将无法加载。有什么改变的想法吗？

我还能做些什么来获得想要的结果？

Answer 1

无限循环非常擅长阻止应用程序执行任何操作。从本质上讲，你正在考虑倒退。你在想：

继续使按钮不可见，直到发生某些事情。

为什么呢？一旦你使按钮不可见，它将保持，直到你改变它。所以，相反，想一想：

当发生某事时，让按钮可见。

在这种情况下，“发生的事情”是用户更改单选按钮的值。所以你想要一个该事件的处理程序：

private void radioButton_CheckedChanged(Object sender, EventArgs e) { // your logic here }

您可以使用表单设计器将此功能分配给各种单选按钮的所有CheckedChanged事件，因此它们都使用相同的处理程序。

那么“你的逻辑”是什么？嗯，真的，这取决于你。声音就像你想要等到选择了几个不同的单选按钮分组一样？所以你要根据它建立一些条件。从高层次来看，这在语义上看起来像这样：

if (allRadioButtonsSelected()) b1.Visible = true;

如果您所拥有的代码行符合您的要求：

rb = fpn1.Controls.OfType<RadioButton>().FirstOrDefault(r => r.Checked)

然后你甚至可以使用它：

if (fpn1.Controls.OfType<RadioButton>().FirstOrDefault(r => r.Checked) != null) b1.Visible = true;

虽然它完全听起来不像你正在寻找的那样，因为它会告诉你是否只选中一个单选按钮。但我可能误解了你，所以这取决于你。

关键是，在检查是否发生了某些事情时，不要保持循环并阻止线程。因为当你阻止线程时，它永远不会发生。相反，使用事件处理程序在事件发生时对事件做出响应。

Answer 2

您正在阻止线程，这就是它没有加载的原因。我没有看到你需要循环的原因，因为用户单击单选按钮可以引发事件。然后，当您处理事件时，将可见性设置为true。

保持按钮不可见，直到单击RadioButton c＃

2 个答案: