Калькулятор Application Crashes

Question

Я пытаюсь создать простое приложение калькулятора на C#, и я не знаю, почему он сбой, когда я делаю следующие шаги.

1. Введите 0,2
2. Нажмите Вычитание
3. Введите 0

сбой приложения сразу. Я предполагаю, что это как-то связано с функцией Zero(), поскольку это то, что выполняется при нажатии кнопки Zero. Условные утверждения предназначены для того, чтобы заботиться о случаях, когда они не должны возникать, как последовательные шифры и т. Д. Вот исходный код. Кстати, функции для других цифр идентичны.

public partial class MainWindow : Window 
    { 
     protected double firstNumber, secondNumber; 
     protected string textBoxContents; 
     protected int selectedFunction; 
     public MainWindow() 
     { 
      InitializeComponent(); 
      firstNumber = 0; 
      secondNumber = 0; 
      selectedFunction = 0; 
      textBoxContents = "0"; 
     } 
     private void Zero(object sender, RoutedEventArgs e) 
     { 
      if (Convert.ToDouble(textBoxContents) > 0 || textBoxContents[textBoxContents.Length - 1] == '.') 
      { 
       if(selectedFunction != 0) 
        textBoxContents = textBoxContents + "0"; 
      } 
      else if (textBoxContents == null) 
      { 
       textBoxContents = textBoxContents + "0"; 
      } 
      ResultBox.Content = textBoxContents; 
     } 
     private void One(object sender, RoutedEventArgs e) 
     { 
      textBoxContents = textBoxContents + "1"; 
      ResultBox.Content = textBoxContents; 
     } 
     private void Decimal(object sender, RoutedEventArgs e) 
     { 
      textBoxContents = textBoxContents + "."; 
      ResultBox.Content = textBoxContents; 
     } 
     private void Addition(object sender, RoutedEventArgs e) 
     { 
      firstNumber = Convert.ToDouble(textBoxContents); 
      textBoxContents = null; 
      selectedFunction = 1; 
     } 
     private void Subtraction(object sender, RoutedEventArgs e) 
     { 
      firstNumber = Convert.ToDouble(textBoxContents); 
      textBoxContents = null; 
      selectedFunction = 2; 
     } 
     private void Multiplication(object sender, RoutedEventArgs e) 
     { 
      firstNumber = Convert.ToDouble(textBoxContents); 
      textBoxContents = null; 
      selectedFunction = 3; 
     } 
     private void Division(object sender, RoutedEventArgs e) 
     { 
      firstNumber = Convert.ToDouble(textBoxContents); 
      textBoxContents = null; 
      selectedFunction = 4; 
     } 
     private void Result(object sender, RoutedEventArgs e) 
     { 
      secondNumber = Convert.ToDouble(textBoxContents); 
      double thirdNumber = 0; 
      switch (selectedFunction) 
      { 
       case 1: 
        thirdNumber = firstNumber + secondNumber; 
        break; 
       case 2: 
        thirdNumber = firstNumber - secondNumber; 
        break; 
       case 3: 
        thirdNumber = firstNumber * secondNumber; 
        break; 
       case 4: 
        thirdNumber = firstNumber/secondNumber; 
        break; 
       default: 
        break; 
      } 
      textBoxContents = Convert.ToString(thirdNumber); 
      ResultBox.Content = textBoxContents; 
     } 
     private void ClearEverything(object sender, RoutedEventArgs e) 
     { 
      textBoxContents = null; 
      firstNumber = 0; 
      secondNumber = 0; 
      selectedFunction = 1; 
      ResultBox.Content = Convert.ToString(0); 
     } 
     private void ToggleNegative(object sender, RoutedEventArgs e) 
     { 
      if (Convert.ToDouble(textBoxContents) != 0) 
      { 
       textBoxContents = Convert.ToString(Convert.ToDouble(textBoxContents) * -1); 
       ResultBox.Content = textBoxContents; 
      } 
      else 
       ResultBox.Content = Convert.ToString(0); 
     } 
    } 

Answer 1

private void Zero(object sender, RoutedEventArgs e) 
{ 
    if (Convert.ToDouble(textBoxContents) > 0 || 
     textBoxContents[textBoxContents.Length - 1] == '.') 
    { 
     if(selectedFunction != 0) 
      textBoxContents = textBoxContents + "0"; 
    } 
    else if (textBoxContents == null) 
    { 
     textBoxContents = textBoxContents + "0"; 
    } 
    ResultBox.Content = textBoxContents; 
} 

Эта логика кажется хребтом. Если значение текстового поля пуст, тогда он будет взорваться из-за индексатора с другой стороны ||. Я думаю, что это может быть переписан, чтобы сказать:

private void Zero(object sender, RoutedEventArgs e) 
{ 
    var dblVal = Convert.ToDouble(textBoxContents.Text); 
    textBoxContents.Text = dblVal.ToString(); 
    ResultBox.Content = textBoxContents.Text; 
} 

Другими словами, если текстовое поле пустое преобразование даст 0.0; если он заканчивается на 1., он даст 1.0; если это .5, он даст 0.5. Просто используйте Convert.

Answer 2

Десятичный разделитель локализован, вы уверены, что вы используете правильную культуру («» вместо «»)?

Если это вопрос, проверить это Stack Question

Answer 3

в функции вычитателыюм вы делаете

textBoxContents = NULL;

, а затем в нуле у вас есть

textBoxContents [textBoxContents.Length - 1]

вот почему он выходит из строя

вы должны проверить нуль перед любой операцией по textBoxContents

Answer 4

textBoxContents имеет значение null после нажатия кнопки вычета. Вместо textBoxContents = null;textBoxContents = "0"; или textBoxContents = string.Empty;. Почему вы все равно устанавливаете его на нуль?

Вызов textBoxContents.Length в вашем методе Zero вызывает NullReferenceException.

Как уже упоминалось, перед вашей логикой в ​​Zero() швы немного обходны и, конечно, могут быть меньше.