为什么我的 C# 代码检查器方法总是返回 true？

如何解决为什么我的 C# 代码检查器方法总是返回 true？

我正在尝试编写一个程序，检查给定的 .txt 文件（包含C# 代码）是否具有有效括号。

当我粘贴这样的简单代码时：

if(true)
{
    Console.WriteLine("Hello");
}

我的程序说它是有效的，这是真的。但是如果我删除一个括号，就像这样：

if(true)
{
    Console.WriteLine("Hello");
 

它仍然说它有效。

这是我检查括号的方法：

static void CheckBrackets()
    {
        try
        {
            Console.Write("You file name (Default path is .....\\netcoreapp2.1\\): ");

            Console.ForegroundColor = ConsoleColor.Blue;
            string fileName = Console.ReadLine();

            Stack openingBrackets = new Stack();
            Queue closingBrackets = new Queue();

            using (StreamReader reader = new StreamReader(fileName))
            {
                bool check = true;
                string str = reader.ReadToEnd();
                char[] code = str.tochararray();

                foreach (char ch in code)
                {
                    if(ch == '(' || ch == '[' || ch == '{')
                    {
                        openingBrackets.Push(ch);
                    }
                    else if(ch == ')' || ch == ']' || ch == '}')
                    {
                        closingBrackets.Enqueue(ch);
                    }
                }

                while (openingBrackets.Count == 0 && closingBrackets.Count == 0)
                {
                    char currentopeningBracket = (char)openingBrackets.Pop();
                    char currentClosingBracket = (char)closingBrackets.Dequeue();

                    if ((currentopeningBracket == '(' && currentClosingBracket != ')')
                     || (currentopeningBracket == '[' && currentClosingBracket != ']')
                     || (currentopeningBracket == '{' && currentClosingBracket != '}'))
                    {
                        check = false;
                    }
                }

                if(check == true)
                {
                    Console.WriteLine("Valid.");
                }
                else
                {
                    Console.WriteLine("Not valid.");
                }
            }
        }
        catch (Exception exp)
        {
            ...
        }
    }

为什么它不起作用的任何想法？

解决方法

您应该使用以下内容修改代码的 while 部分：

if (openingBrackets.Count != closingBrackets.Count)
{
    check = false;
}
else
{
    while (openingBrackets.Count > 0 && closingBrackets.Count > 0)
    {
        char currentOpeningBracket = (char)openingBrackets.Pop();
        char currentClosingBracket = (char)closingBrackets.Dequeue();

        if ((currentOpeningBracket == '(' && currentClosingBracket != ')')
        || (currentOpeningBracket == '[' && currentClosingBracket != ']')
        || (currentOpeningBracket == '{' && currentClosingBracket != '}'))
        {
            check = false;
        }
    }
}