このWHILEループを動作させるにはどうすればよいですか？

Question

現在、IF ELSEステートメントを使用して正しく実行したと思った演習を行っていますが、一度提出すれば、WHILEループまたはDO-WHILEループを使用するよう指示されました。あなたはWHILEループをどこで使用したのかわかりますが、それは私に割り当てられたエラーメッセージの無限ループを与えています。私はそれを止める方法を知らないのです！

static void Main(string[] args) 
    { 
     decimal hours; 
     const decimal HOURLY_RATE = 2.5m; 
     const decimal MAX_FEE = 20.00m; 
     Console.WriteLine("Enter the amount of hours parked:"); 
     hours = decimal.Parse(Console.ReadLine()); 
     decimal parkingCost = hours * HOURLY_RATE; 

     while (hours < 1 || hours > 24) 
     { 
      Console.Write("Enter correct amount of hours - 1 to 24. "); 
     } 

     if (parkingCost > MAX_FEE) 
     { 
      Console.Write("Total fee is $" + MAX_FEE); 
      Console.WriteLine(" Time parked in hours is " + hours); 
     } 
     else 
     { 
      parkingCost = hours * HOURLY_RATE; 
      Console.WriteLine("The cost of parking is " + parkingCost.ToString("C")); 
      Console.WriteLine("Time parked in hours is " + hours); 
     } 
     Console.WriteLine("Press any key to continue..."); 
     Console.ReadKey(); 
    } 
} 

だから、アイデアは、「時間の正確な量を入力 - 24に1を」するために私が欲しいということであるユーザーはプログラムが行きそうでない場合のために、24未満1以上の数値を入力した場合に表示しますIFとELSEステートメントを先に進めてください。

Answer 1

ちょうど

static void Main(string[] args) 
{ 
    decimal hours; 
    const decimal HOURLY_RATE = 2.5m; 
    const decimal MAX_FEE = 20.00m; 
    Console.WriteLine("Enter the amount of hours parked:"); 
    hours = decimal.Parse(Console.ReadLine()); 
    decimal parkingCost = hours * HOURLY_RATE; 

    while (hours < 1 || hours > 24) 
    { 
     Console.Write("Enter correct amount of hours - 1 to 24. "); 
     hours = decimal.Parse(Console.ReadLine()); 
    } 


    parkingCost = hours * HOURLY_RATE; // to recalculate 

    if (parkingCost > MAX_FEE) 
    { 
     Console.Write("Total fee is $" + MAX_FEE); 
     Console.WriteLine(" Time parked in hours is " + hours); 
    } 
    else 
    { 
     parkingCost = hours * HOURLY_RATE; 
     Console.WriteLine("The cost of parking is " + parkingCost.ToString("C")); 
     Console.WriteLine("Time parked in hours is " + hours); 
    } 
    Console.WriteLine("Press any key to continue..."); 
    Console.ReadKey(); 
} 

}ループ内

Answer 2

あなたは、新しい値を取得するために、再度、whileループ内の値を読み込むのを忘れて：あなたはエラー

do { 
    Console.WriteLine("Enter the amount of hours parked:"); 
    try{ 
    hours = decimal.Parse(Console.ReadLine()); 
    if(hours <1 || hours > 24) 
     Console.Write("Enter correct amount of hours - 1 to 24. "); 
    } 
    catch { Console.WriteLine("Please enter a valid numeric value"); } 
} while(hours < 1 || hours > 24); 

上の最初の読み取りおよびループのみを取り除くことができますが

while (hours < 1 || hours > 24) 
    { 
     Console.Write("Enter correct amount of hours - 1 to 24. "); 
     Console.WriteLine("Enter the amount of hours parked:"); 
     hours = decimal.Parse(Console.ReadLine()); 
    } 

または実行などを

try catchを追加しました。数値ではない値を入力するとエラーが発生する可能性があるためです。