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; 
};