获取和设置不起作用

Question

我正在编写以下 get 和 set 来验证文本框的输入。基本上它应该检查用户是否输入了所有值。 当我将文本框留空时，它不会执行任何操作，并在使用该变量的输出中显示“0”。然而，它确实显示系统生成的异常并停止执行，但我想知道为什么它不通过属性验证输入？

这是我的代码：

public double RecoDoseSize
{
    get
    {
        return recoDoseSize;
    }
    set
    {
        if (!(value>0))
        {
            MessageBox.Show("Please Enter the recommended dose size for this product");
            textBox8.Focus();
        }
        recoDoseSize = value;
    }
}

---

private void Submit2_Click(object sender, RoutedEventArgs e)
{
    TotalContentProduct = double.Parse(textBox7.Text);
    recoDoseSize = double.Parse(textBox8.Text);
    NoOfDosespUnit = TotalContentProduct/recoDoseSize;
}

Answer 1

您正在设置 recoDoseSize（支持字段），而不是 RecoDoseSize（其中包含您的代码的属性）。因此，您的代码不会被执行。您需要将方法主体的第二行更改为

RecoDoseSize = double.Parse(textBox8.Text);

（注意大写的R）。

Answer 2

其他人已经给出了上述问题的正确答案。也就是说，如果您想使用 getter/setter，则应该调用大写的 RecoDoseSize。

然而，在设置器内显示消息框是非常糟糕的做法，因为它违反了最小意外原则。

当有人查看 RecoDoseSize = double.Parse(textBox8.Text); 行时，这一操作可能会导致出现消息框，这一点并不明显。

偶尔会有例外，让 setter 触发 UI 更改确实有意义（例如控件上的 Visible 属性），但默认情况下应始终不执行此操作，除非您确定它会更有效。不这样做会让人感到困惑（例如，如果您设置 Visible = false 但它仍然可见，那会令人惊讶）。

关于您对如何实现它的评论，检查应该在点击处理程序中完成，并且该属性可以只是一个自动属性，如下所示：

public double RecoDoseSize { get; set; }

private void Submit2_Click(object sender, RoutedEventArgs e)
{
    TotalContentProduct = double.Parse(textBox7.Text);

    double enteredSize;
    if (!double.TryParse(textBox8.Text, out enteredSize) || enteredSize <= 0)
    {
        MessageBox.Show("Please Enter the recommended dose size for this product");
        textBox8.Focus();
        return;
    }
    RecoDoseSize = enteredSize;
    NoOfDosespUnit = TotalContentProduct / recoDoseSize;
}

您将需要使用 TryParse 因为使用 Parse 如果出现以下情况，您将收到错误该文本不是有效的double。 TryParse 的作用是根据是否成功返回 true 或 false，如果成功，它会使用结果填充 out 参数。

因此，如果它无法解析结果，或者结果是 <= 0，它会显示消息框。在这种情况下，它也会从该方法返回，因此其余部分不会被执行。或者，该方法的其余部分可以位于 else 块中，在这种情况下不需要 return。这只是一种风格的问题，更喜欢哪种方式。

Answer 3

您实际上从未使用过 getter/setter。您直接使用实际字段名称：recoDoseSize。将其更改为 RecoDoseSize。

Answer 4

private void Submit2_Click(object sender, RoutedEventArgs e)
{
    TotalContentProduct = double.Parse(textBox7.Text);
    RecoDoseSize= double.Parse(textBox8.Text);
    NoOfDosespUnit = TotalContentProduct/recoDoseSize;
}

Answer 5

您不应该在 set 语句中处理焦点。

另外，您需要确保该值不为空，否则您无法将其与任何内容进行比较（大于等）。