忽略回文中的空白

Question

我正在编写一个程序来检查一个单词是否是回文。程序本身有效 但是当我输入这行特定的文本“太糟糕了 - 我隐藏了一个靴子”时，它不会拿起它 并忽略空格和连字符。

我查看了以前的问题，但找不到答案。

我的Regex.Replace 不正确吗？

 string s, revs = "";
        Console.WriteLine("Please Enter Word");
        s = Console.ReadLine();
        string r = Regex.Replace(s , @"[^-\s]", "");

        for (int i = s.Length-1; i >= 0; i--)
        {
            revs += s[i].ToString();
        }

        if (revs == s)
        {
            Console.WriteLine("Entered string is palindrome \n String was {0} and reverse string was {1}", s, revs);
        }
        else
        {
            Console.WriteLine("String entered was not palindrome.");
        }
        Console.ReadKey();

Answer 1

您的代码包含两个错误：

• 这是正确的正则表达式条件 @"(\s|-)" 你的替换所有内容，但要删除的字符除外
• 正如 jeffdot 指出的那样，您正在替换（错误地）但随后针对原始字符串进行测试

这样考虑，正确的代码应该是：

string s, revs = "";
        Console.WriteLine("Please Enter Word");
        s = Console.ReadLine();
        string r = Regex.Replace(s , @"(\s|-)", "");

        for (int i = r.Length-1; i >= 0; i--)
        {
            revs += r[i].ToString();
        }

        if (revs == r)
        {
            Console.WriteLine("Entered string is palindrome \n String was {0} and reverse string was {1}", s, revs);
        }
        else
        {
            Console.WriteLine("String entered was not palindrome.");
        }
        Console.ReadKey();

除了第一个 WriteLine 之外，您应该决定打印什么（因为 revs 不再包含空格和连字符）。

或者如果你只想使用String.Replace，你可以使用它两次：

s=s.Replace(' ','');
s=s.Replace('-','');

Answer 2

您将 Regex.Replace 调用的结果分配给 string r 并且您从不引用它。这就是您的比较失败的原因。

为了更清楚地说明这一点：您正在删除空格并将该产品引用为 string r，但您正在反转并仅针对 string s 进行测试。

Saverio 还指出您的正则表达式不起作用（我自己没有测试过）。

你有：

    string r = Regex.Replace(s , @"[^-\s]", "");

    for (int i = s.Length-1; i >= 0; i--)
    {
        revs += s[i].ToString();
    }

在那里，您应该迭代和反转r 而不是s——因为s 仍然有它的所有空白。

此外，正如 Saverio 所说，为什么不直接替换？

string r = s.Replace(" ", string.Empty).Reverse();

我没有尝试编译它，但只要您不使用应该可以工作的非常旧版本的 .NET 框架。可能需要您是 using System.Linq 才能使用 Reverse() 扩展名。