do-while loop freezing program

Question

Я пытаюсь использовать цикл do-while, но он ничего не делает после ввода ввода и инициирования события нажатия кнопки. Предполагается рассчитать сумму и список за все последующие годы, пока не будет < = 40 000. Я могу заставить программу работать без цикла, но не с ней.

private double InterestEarned(double AMT, double AIR = 0.07) 
    {       
     return AMT * AIR; 
    } 

    private double InheritanceAmount(double BAL, double IR, double AIR = 0.07) 
    {    
     return (BAL * IR * AIR) - 40000; 
    } 

    private void button1_Click(object sender, EventArgs e) 
    { 
     try 
     { 
      double AMT; 
      AMT = (double.Parse(textBox1.Text)); 

      if (radioButton1.Checked==true) 
      { 
       do 
       { 
       const double IR3 = 0.03; 
       double BAL, IR, earn; 
       int year = 2014; 

       AMT = (double.Parse(textBox1.Text)); 
       IR = IR3; 
       year++; 
       BAL = InheritanceAmount(AMT, IR); 
       earn = InterestEarned(AMT); 

       listBox1.Items.Add("You have chosen a 3% inflation rate. Your investment starts at" + AMT.ToString("C") + " and earn 7% a year. You withdraw $40,000 a year."); 
       listBox1.Items.Add("Year" + "\t" + "Interest Earned" + "\t" + "Balance"); 
       listBox1.Items.Add(year++ + "\t" + earn.ToString("C") + "\t" + BAL.ToString("C")); 
       } while (AMT > 40000); 
      } 
       else if (radioButton2.Checked==true) 
       { 
        do 
        { 
        const double IR4 = 0.04; 
        double BAL, IR, earn; 
        int year = 2014; 

        AMT = (double.Parse(textBox1.Text)); 
        IR = IR4; 
        year++; 
        BAL = InheritanceAmount(AMT, IR); 
        earn = InterestEarned(AMT); 

        listBox1.Items.Add("You have chosen a 4% inflation rate. Your investment starts at" + AMT.ToString("C") + " and earn 7% a year. You withdraw $40,000 a year."); 
        listBox1.Items.Add("Year" + "\t" + "Interest Earned" + "\t" + "Balance"); 
        listBox1.Items.Add(year++ + "\t" + earn.ToString("C") + "\t" + BAL.ToString("C")); 
        } while (AMT > 40000); 
       } 
      else 
      {     
       MessageBox.Show("Please select an inflation rate.");      
      } 

     } 
     catch (Exception ex) 
     { 
      MessageBox.Show(ex.Message); 
     } 

Answer 1

Д.О. ... в то время как цикл условно от значения AMT быть больше, чем 40000. Но ценность АМТ приходит только от пользователя (с помощью текстового поля) и никогда не меняется снова. Таким образом, цикл просто происходит вечно (который, поскольку он выполняется на вашем потоке пользовательского интерфейса, заблокирует пользовательский интерфейс). Либо ваше условие неверно, либо вам нужно изменить значение AMT в цикле.

Answer 2

Ваше условие основано на переменной AMT (> 40000). Вы правильно инициализируете эту переменную, прежде чем делать это. Тем не менее, вы также повторно инициализируете эту переменную внутри цикла do while, поэтому переменная никогда не достигает условия, которое заставит ее выйти из цикла do while.

Первое, что нужно сделать здесь комментировать строку, где AMT переменная устанавливается обратно в свое первоначальное значение:

do 
{ 
    const double IR3 = 0.03; 
    double BAL, IR, earn; 
    int year = 2014; 

    //Comment the line below 
    //AMT = (double.Parse(textBox1.Text)); 
    IR = IR3; 
    year++; 
    BAL = InheritanceAmount(AMT, IR); 
    earn = InterestEarned(AMT); 

    //... 

} while (AMT > 40000); 

Следующая вещь, которую вы должны сделать, это увеличить AMT на основе значения earn :

AMT += earn; 

последняя вещь, которую вы должны рассмотреть способ избежать бесконечного цикла, когда AMT исходное значение < = 0 или когда проценты устанавливаются в 0.