誤った値を入力しました。スタートに行く。

Question

編集。 受け取った答えに基づいて、私は学習の初期段階で私が知っている以上に多くのことを尋ねていることに気づいたので、もっと学び、理解するまでこの質問を忘れてしまいました。私は自分自身を過ぎて混乱させたくありません。

ガイダンスを探している初心者。

私はここで質問した前の質問から、私はgoto Startに紹介されました。 switch文で

私はそれを使い始めました。ユーザーが無効な入力を入力したときにプログラムを開始したいと思っていました。例の下の私の電卓で、ユーザが数字以外の何かを入力した場合、入力を無視してユーザに数字を入力するように促すにはどうすればよいですか？ 「無効な入力です。再試行してください」と表示されることもあります。

数字だけが入力された場合、コンソールがクラッシュしてエラーが表示されます。

援助未テスト

 int num1, num2, output; 
     string op; 

     Console.Write("\n\n"); 
     Console.WriteLine("Calculator\n"); 
     Console.WriteLine("============="); 
     Console.Write("\n\n"); 

     Start: 

     Console.Write("Please enter first number:"); 
     num1 = Convert.ToInt32(Console.ReadLine()); 


     Console.Write("Please enter second number: "); 
     num2 = Convert.ToInt32(Console.ReadLine()); 

     Operator: 

     Console.WriteLine("Please select operator: "); 
     Console.WriteLine("\nAddition : +"); 
     Console.WriteLine("Multiplication: *"); 
     Console.WriteLine("Division: /"); 
     Console.WriteLine("Subtraction: -"); 
     Console.Write("Enter Operator: "); 
     op = Console.ReadLine(); 




     switch (op) 
     { 
      case "+": 
       output = num1 + num2; 
       Console.WriteLine("{0} added to {1} = {2}", num1, num2, output); 
       break; 

      case "*": 
       output = num1 * num2; 
       Console.WriteLine("{0} multiplied by {1} = {2}", num1, num2, output); 
       break; 

      case "/": 
       if (num2 == 0) 
       { 
        Console.WriteLine("Cannot divide by zero. Please try again"); 
        goto Start; 

       } 
       else 
       { 
        output = num1/num2; 
        Console.WriteLine("{0} divided by {1} = {2}", num1, num2, output); 
        break; 
       } 

      case "-": 
       output = num1 - num2; 
       Console.WriteLine("{0} minus{1} = {2}", num1, num2, output); 
       break; 

      default: 
       Console.WriteLine("You entered an invalid operator. Please try again\n"); 
       goto Operator;      

     } 

     Console.WriteLine("\nPress enter to continue...."); 
     Console.ReadLine(); 

Answer 1

、何をするにしても、ませ使用gotoを行ってください。これまでたぶんを除いて他のcaseに落ちるかもしれませんが、それは非常にまれです。それはあなたのコードを読むことができず、混乱させ、維持しがたいものにします。 spaghetti codeを参照してください。

int num1; 
do 
{ 
    Console.Write("Please enter first number:"); 
} while (!Int32.TryParse(Console.ReadLine(), out num1)); 

をと（gotoを避けるために）このような何かにあなたのswitchを変更します：

bool inputOk = false; 
while (!inputOk) 
{ 
    Console.WriteLine("Please select operator: "); 
    //... 
    op = Console.ReadLine(); 

    inputOk = true; 
    switch (op) 
    { 
     //... 
     default: 
      Console.WriteLine("You entered an invalid operator. Please try again\n"); 
      inputOk = false; 
      break; 
    } 
} 

あなたが入力が有効であるかどうかわからない場合は代わりに Convertの Int32.TryParseを行う必要があり、言っ

Answer 2

を事前に感謝しますが、これはあなたが望むことがあります

Start: 

Console.Write("Please enter first number:"); 
if (!int.TryParse(Console.ReadLine(), out num1)) 
    goto Start; 
//num1 = Convert.ToInt32(Console.ReadLine()); 


Console.Write("Please enter second number: "); 
num2 = Convert.ToInt32(Console.ReadLine()); 

Operator: 

... 

そして、あなたが必要な場合は、NUM2のためおそらく同じ。

Answer 3

これを達成する最良の方法は、EXCEPTIONを使用することです。あなたが学んでいるので、私はそれを行う方法のサンプルコードを提供しませんが、それについて読んでほしいでしょうhereとhere

Answer 4

gotoスイッチの中に定義しようとするとしようとします。あなたの例ではスタートケースはないので、どこに行くのか分からない。

詳細情報here。

私は方法を使用することをお勧めします。

Answer 5

この状況では、goto文が適切ではないと思います。あなたのコードがたくさんある場合、あなたのコードは本当に乱雑になることがあります。 FormatExceptionを使用することは、例外をスローするのに時間がかかるため、お勧めしません。

代わりにInt32.TryParseを使用してください。

ここにサンプルコードがあります。これをプログラムに適用できるかどうかを確認してください。

int input; 
while(true) { 
    Console.WriteLine("Please enter a number"); 
    bool succeeds = Int32.TryParse(Console.ReadLine(), out input); 
    if (!succeeds) { 
     Console.WriteLine("Input Invalid! Try again!"); 
    } else { 
     break; 
    } 
} 
// make use of input here 

Answer 6

 int num1, num2, output; 
    string op; 

    Console.Write("\n\n"); 
    Console.WriteLine("Calculator\n"); 
    Console.WriteLine("============="); 
    Console.Write("\n\n"); 

Start: 

    do 
    { 
     Console.Write("Please enter valid first number:"); 
    } while (!int.TryParse(Console.ReadLine(), out num1)); 


    do 
    { 
     Console.Write("Please enter valid second number:"); 
    } while (!int.TryParse(Console.ReadLine(), out num2)); 

    Operator: 

    Console.WriteLine("Please select operator: "); 
    Console.WriteLine("\nAddition : +"); 
    Console.WriteLine("Multiplication: *"); 
    Console.WriteLine("Division: /"); 
    Console.WriteLine("Subtraction: -"); 
    Console.Write("Enter Operator: "); 
    op = Console.ReadLine(); 




    switch (op) 
    { 
     case "+": 
      output = num1 + num2; 
      Console.WriteLine("{0} added to {1} = {2}", num1, num2, output); 
      break; 

     case "*": 
      output = num1 * num2; 
      Console.WriteLine("{0} multiplied by {1} = {2}", num1, num2, output); 
      break; 

     case "/": 
      if (num2 == 0) 
      { 
       Console.WriteLine("Cannot divide by zero. Please try again"); 
       goto Start; 

      } 
      else 
      { 
       output = num1/num2; 
       Console.WriteLine("{0} divided by {1} = {2}", num1, num2, output); 
       break; 
      } 

     case "-": 
      output = num1 - num2; 
      Console.WriteLine("{0} minus{1} = {2}", num1, num2, output); 
      break; 

     default: 
      Console.WriteLine("You entered an invalid operator. Please try again\n"); 
      goto Operator;      

    } 

    Console.WriteLine("\nPress enter to continue...."); 
    Console.ReadLine();