как исправить эту петлю?

Question

У меня проблема с простым кодом. Что я хочу сделать, это: Например, если пользователь дает только номер negatieve, а не положительным, то я хочу видеть:

"this is not valid" 

Do you want to try again <y/n>? 

Я уже пробовал много комбинации с, если, если еще, но тогда я не вижу этого "do you want to try again?" или я вижу их обоих и результат.

Это не относится и максимальное значение: 0

Это не хорошо.

int invoer; 
int max; 
string repeat; 

Console.WriteLine("Please give a positive number.\nIf you enter a negative number its not going to work"); 

do 
{ 
    invoer = 0; 
    max = 0; 
    repeat = ""; 

    for (int i = 1; invoer >= 0; i++) 
    { 
     Console.Write(i + "> "); 
     invoer = int.Parse(Console.ReadLine()); 

     if (max < invoer) 
      max = invoer; 
    } 

    Console.WriteLine("Maximum value is: " + max); 
    Console.WriteLine("do you want to try again? y/n: "); 
    repeat = Console.ReadLine(); 
} while (repeat == "y" || repeat == "Y"); 

Answer 1

Вы должны использовать uint.TryParse вместо int.Parse. Он не будет принимать отрицательные числа и не будет генерировать исключение из-за неправильного ввода.

Answer 2

Если вы не можете использовать int.TryParse, вы всегда можете убедиться, что ваш вход числовой перед разбором его:

var input = Console.ReadLine(); 
var number = !string.IsNullOrWhiteSpace(input) ? input : "0"; 

var numeralExpression = new System.Text.RegularExpressions.Regex(@"^(\d|-\d)$"); 

if (numeralExpression.IsMatch(number)) 
{ 
    invoer = int.Parse(number); 
} 

Answer 3

Я редактировал этот код для вас. Он задает пользователям неограниченное количество времени, какое положительное значение имеет. Он дает только max, если введенное значение является максимальным. И это не дает max, если значение отрицательное ... Я считаю, что это то, что вы хотите?

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

Если это ответ, пожалуйста, проверьте это.

int invoer; 
    int max; 
    string repeat; 

    Console.WriteLine("Please give a positive number.\nIf you enter a negative number its not going to work"); 

    do 
    { 
     invoer = 0; 
     max = 0; 
     repeat = ""; 

     for (int i = 0; invoer >= 0; i++) 
     { 
      Console.Write(i + "> "); 
      invoer = int.Parse(Console.ReadLine()); 

      if (max < invoer) 
      { 
       max = invoer; 
       Console.WriteLine("Maximum value is: " + max); 
      } 
     } 

     if (invoer < 0) 
     { 
      Console.WriteLine("This is not valid..."); 
     } 


     Console.WriteLine("do you want to try again? y/n: "); 
     repeat = Console.ReadLine(); 
    } 
    while (repeat == "y" || repeat == "Y"); 
} 
} 
}