使用 String.ToUpper() 时程序退出；在包含空格的字符串上

Question

首先让我说我是 C# 新手。

我目前正在制作我的第一个命令行应用程序，它在当前状态下可以做两件事。其中一个是计算器，我需要更多的学习才能真正让它工作，另一个是字符串大写字母。

我有一个string nameCapInput = Console.Readline() 接受用户输入，然后对其进行分析以确保不允许使用数字：

using System;
using System.Linq;

namespace First_Console_Project
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("My first ever console application - 2020/2/26\n\n\n");
        programSel:
            Console.WriteLine("What do you want to do?\n");
            Console.WriteLine("1. Calculate Numbers \n2. Capitalize Letters/Strings");
            Console.WriteLine("Input your desired action:");
            var inputVar = Console.ReadLine();
            switch (inputVar)
            {
                case "1":
                    //Calculator code goes here
                    Console.WriteLine("Number 1 succeeded, opening calculator... Stand by");
                    Console.WriteLine("Calulator Loaded.");
                    Console.WriteLine("Doesn't work right now. Type \"exit\" to get back to the \"what do you want to do\" page.");
                    //Code goes here when I have learned the proper methods
                calcInput:
                    var calcInput = Console.ReadLine();
                    if (calcInput == "exit")
                    {
                        goto programSel;
                    } else
                    {
                        Console.WriteLine("Unknown command. Type \"exit\" to get back to the \"what do you want to do\" page.");
                        goto calcInput;
                    }
                case "2":
                    Console.WriteLine("Loading string capitalizer...");
                    Console.WriteLine("Type any string made of letters only without spaces, because if you use spaces, the program will exit. The output will make them all uppercase. Type \"exit\" to get back to the \"what do you want to do\" page.");
                inputCap:
                    string nameCapInput = Console.ReadLine();
                    bool containsInt = nameCapInput.Any(char.IsDigit);
                    bool isMadeOfLettersOnly = nameCapInput.All(char.IsLetter);
                    if (nameCapInput == "exit")
                    {
                        goto programSel;
                    }
                    else if (containsInt)
                    {
                        Console.WriteLine("You can't capitalize numbers. Use letters only. Try again.");
                        goto inputCap;
                    }
                    else if (isMadeOfLettersOnly)
                    {
                        string upper = nameCapInput.ToUpper();
                        Console.WriteLine($"The uppercase version of your entered text is: {upper}");
                        goto inputCap;
                    }
                    break;
                    }
            }
        }
}

现在，一切正常，它将我放入其中的所有内容都大写除了字符串，其中包含空格。当我输入一个带有空格的字符串时，程序会以代码 0 退出。我还不太擅长 C#，所以我真的不知道从这里往哪里走。任何帮助表示赞赏。

每次我在 C# 中学习新东西时，我都会尝试将其实施到我的项目中，这样我就可以真正学习如何实施它，以了解何时以及如何使用我所学的内容。这是一个例子。

编辑：添加了其余代码。 非常感谢大家。我在这里学到了两件事：

1. goto 是个坏习惯
2. 我绝对需要开始学习调试我自己的代码。

Answer 1

问题的症结在于您仅检查输入是否包含字母（而不是空格）。一个简单的解决方法是稍微更改您的 LINQ。

bool isMadeOfLettersOnly = nameCapInput.All(c => char.IsLetter(c) || char.IsWhiteSpace(c));

所以现在输入带有字母或的空格将被认为是有效的。

此外，您使用goto 是一个非常糟糕的主意。一般来说，永远不应该有任何理由使用goto。

要解决此问题，请使用 while 循环和方法：

public static void Main()
{
    bool exit = false;
    do {
        exit = ProcessInput();
    }
    while(!exit);
}

private static bool ProcessInput()
{
    string nameCapInput = Console.ReadLine();

    bool containsInt = nameCapInput.Any(char.IsDigit);
    bool isMadeOfLettersOnly = nameCapInput.All(c => char.IsLetter(c) || char.IsWhiteSpace(c));

    if (nameCapInput.Equals("exit", StringComparison.CurrentCultureIgnoreCase))
    {
        return true; //exiting so return true
    }
    else if (containsInt)
    {
        Console.WriteLine("You can't capitalize numbers. Use letters only. Try again.");
    }
    else if (isMadeOfLettersOnly)
    {
        string upper = nameCapInput.ToUpper();
        Console.WriteLine("The uppercase version of your entered text is: {0}", upper);
    }   
    return false; //no exit, so return false
}

这只是一个快速的重构，你可以做得更好。

小提琴here

Answer 2

查看文档：https://docs.microsoft.com/en-us/dotnet/api/system.char.isletter?view=netframework-4.8

根据IsLetter函数的文档，空格不包含在return true case中。

我建议您为此使用正则表达式或将您的最后一个案例更改为

else if (!containsInt)
{
    var upper = nameCapInput.ToUpper();
    Console.WriteLine($"The uppercase version of your entered text is: {upper}");
    goto inputCap;
}

同时查看 goto 的文档：https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/goto

goto 语句将程序控制直接传递给带标签的语句。

goto 的一个常见用途是将控制转移到特定的 switch-case 标签或 switch 语句中的默认标签。

goto 语句对于跳出深度嵌套的循环也很有用。

你不是这种情况，所以你不应该使用它。