Y/N或y/n在循环中

Question

嗨，你们在执行循环中的Y/N或y/n时会遇到一些麻烦。我设计的方式是用户可以使用Y和N的大写字母和小写字母作为循环中的答案。顺便说一下，这里是我的代码，但似乎无法使其工作：

do 
     { 
      Console.WriteLine("\nSelect additional topping/s\n"); 

      Console.WriteLine("1 - Extra meat: 200"); 
      Console.WriteLine("2 - Extra cheese: 100"); 
      Console.WriteLine("3 - Extra veggies: 80\n"); 

      int selectedTopping = Convert.ToInt32(Console.ReadLine()); 

      switch (selectedTopping) 
      { 
       case 1: 
        pizza = new MeatToppings(pizza); 
        break; 

       case 2: 
        pizza = new CheeseToppings(pizza); 
        break; 

       case 3: 
        pizza = new VeggieToppings(pizza); 
        break; 

       default: 
        break; 
      } 

      Console.WriteLine("\nAdd more toppings? Y/N"); 


     } 

     while ((Console.ReadLine() == "Y") || (Console.ReadLine() == "y")); 

Answer 1

while ((Console.ReadLine() == "Y") || (Console.ReadLine() == "y")); 

这是要读2条不同的线路，因为你打电话ReadLine()两次。您需要调用一次并保存值。

Answer 2

您可以使用ToUpper

while ((Console.ReadLine().ToUpper() == "Y")); 

Answer 3

尝试使用String.Equals和StringComparison：

String.Equals(Console.ReadLine(), "y", StringComparison.CurrentCultureIgnoreCase); 

从MSDN：

CurrentCultureIgnoreCase：使用文化敏感的排序规则比较字符串，当前的文化，并忽略被比较的字符串的情况。

OrdinalIgnoreCase：使用序号排序规则并忽略字符串的情况下，比较字符串进行比较。

Answer 4

要检查Y或y忽略的情况下，你应该使用string.Equals(string,StringComparison)超载。

while (Console.ReadLine().Equals("Y", StringComparison.InvariantCultureIgnoreCase)); 

请参阅的The Turkish İ Problem and Why You Should Care使用ToUpper或ToLower与忽略大小写字符串比较之前。

您当前的代码正在从控制台读取两行代码，这就是为什么您的代码保持第二个值的原因。

Answer 5

正如Austin刚刚指出的，您在while循环语句中使用了两次ReadLine。

有一点值得一提的是试图遵循模块化的规则，这将有助于加快实施和调试我们的代码。

自从我做了任何C＃编程，所以sudo编码这种Java风格已经有一段时间了。

由于它是命令行编程，您可能必须多次验证用户输入。我要做的一件事是让工具类包含常见的用户输入任务。

public class TerminalUtil { 
    private TerminalUtil() {} 

    public static boolean isYes(String msg){ return (msg.ToUpper() == "Y" || msg.ToUpper() == "YES"); } 
    public static boolean isNo(String msg){ return (msg.ToUpper() == "N" || msg.ToUpper() == "NO"); } 
    // You also might want basic conditionals to check if string is numeric or contains letters. 

    // I like using recursion for command line utilities so having a method that can re-print messages is handy 
    public static void display(String[] messages){ 
     for(String msg : messages){ 
      Console.WriteLine(msg); 
     } 
    } 

    public static boolean enterYesOrNo(String[] messages, String[] errorMessages){ 
     display(messages) 
     String input = Console.ReadLine(); 
     if(isYes(input)){ 
      return true; 
     } else if(isNo(input)){ 
      return false; 
     } else { 
      display(errorMessages); // Maybe something like, you didn't enter a yes or no value. 
      enterYesOrNo(messages, errorMessages); // Recursive loop to try again. 
     } 

    } 
} 

下面是代码来订购比萨饼可能是什么样子

public class OrderPizza{ 
    public static int selectToppings(){ 
     String[] message = new String[4]; 
     message[0] = ("\nSelect additional topping/s\n"); 
     message[1] = ("1 - Extra meat: 200"); 
     message[2] = ("2 - Extra cheese: 100"); 
     message[3] = ("3 - Extra veggies: 80\n"); 

     int option = TerminalUtils.entryNumeric(message, {"You entered an non-numeric character, try again"}); 
     if(option > 0 && option <= 3){ 
      return option; 
     } else { 
      Console.WriteLine("Number must be between 1 - 3, try again."); 
      return selectToppings(); 
     } 
    } 

    public static Pizza order(){ 
     Pizza pizza = new Pizza(); 

     while(true){ 
      int toppingCode = selectTopping(); 
      pizza.addTopping(toppingCode); 
      if(!TerminalUtil.enterYesOrNo({"\nAdd more toppings? Y/N"}, {"Please enter a 'Y'es or 'N'o"})){ 
       break; 
      } 
     } 
    } 
} 

这样做的主要好处是，while循环的业务逻辑已经减少，那么你可以在重用代码TerminalUtils。这并不意味着一个优雅的解决方案，我很懒，这是凌晨3点的IRL，但它应该足以让球滚动。

你应该重新考虑做的一件事是使用整数代码来表示浇头。使用枚举可能会使事情更容易实现。

我还注意到，您添加了三种不同类型的比萨，我假设三个不同的对象。

由于您正在循环添加配料到披萨，因此需要制作一个抽象类的披萨。通过这种方式，您可以扩展通用的预制比萨饼，如意大利辣香肠或奶酪，如果您想要客户定制他们的订单，则可以使用抽象比萨类。

Answer 6

我没有发现比更好的办法：

while (str!="N") 
{ 
    str = Console.ReadLine(); 
    str = str.ToUpper(); 
    if (str == "Y"); 
     break; 
};