Блог

Я пишу следующие 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;
}
  

Вы устанавливаете recoDoseSize резервное поле, а не RecoDoseSize свойство, в котором содержится ваш код. Таким образом, ваш код не выполняется. Вам нужно изменить вторую строку тела вашего метода на

 RecoDoseSize = double.Parse(textBox8.Text);
  

(обратите внимание на заглавную R ).

Другие дали правильный ответ на вопрос, как указано. А именно, что вы должны вызывать верхний регистр RecoDoseSize , если вы хотите использовать средство получения / установки.

Однако показывать окно сообщения внутри установщика — крайне плохая практика, потому что это нарушает принцип наименьшего удивления.

Когда кто-то смотрит на строку RecoDoseSize = double.Parse(textBox8.Text); , совсем не очевидно, что эта операция может привести к появлению окна сообщения.

Иногда бывают исключения, когда имеет смысл использовать установщик, вызывающий изменения пользовательского интерфейса (например, 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, потому что при синтаксическом анализе вы получите ошибку, если текст не является допустимым double . Что делает TryParse, так это возвращает true или false в зависимости от того, удалось ли это, и он заполняет параметр out результатом, если это успешно.

Итак, что это делает, так это если либо не удалось проанализировать результат, либо в результате <= 0 отображается окно сообщения. В этом случае это также return из метода, поэтому остальная его часть не выполняется. В качестве альтернативы остальная часть метода может находиться в else блоке, и в этом случае return он не нужен. Это зависит от стиля, какой способ предпочтительнее.

На самом деле вы никогда не используете средство получения / установки. Вы используете фактическое имя поля: recoDoseSize напрямую. Измените его на RecoDoseSize .

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

Вы не должны обрабатывать фокус в своем операторе set.

Кроме того, вам нужно убедиться, что это значение не равно null, иначе вы не сможете сравнить его ни с чем (больше, чем и т.д.).