C# 使用了未赋值的变量

namespace WindowsFormsApplication1
{
class parenthese
{
public string correct(ref char[] s)
{
Stack h = new Stack();
string m;
for (int i = 0; i < s.Length; i++)
{
if (s[i] == '(' || s[i] == '{' || s[i] == '[')//遇到(,{,[则将其入栈
{
h.Push(s[i]);

            }
            else if (s[i] == ')')
            {
                if (Convert.ToChar(h.Peek()) == '(')
                {
                    h.Pop();
                   m= "括号配对!";
                }
                else { m= "括号不配对!"; }
            }
            else if (s[i] == '}')
            {
                if (Convert.ToChar(h.Peek()) == '{')
                {
                    h.Pop();
                  m= "括号配对!";
                }
                else { m= "括号不配对!"; }
            }
            else if (s[i] == ']')
            {
                if (Convert.ToChar(h.Peek()) == '[')
                {
                    h.Pop();
                   m= "括号配对!";
                }
                else { m= "括号不配对!"; }
            }
            else if (h.Count != 0) {m= "括号不配对!"; }
        }

        return m;
    }
}

}
图片说明

1.考虑s.Length等于0的情况,for循环不会执行,自然m就没有被赋值;
2.考虑s.Length不等于0的情况,for循环会执行,但并非每个判断分支都对m进行了赋值;
所以编译器提醒你使用了未赋值的变量m。

string m;
->
string m = string.Empty;

另外帮你简化的代码,怎么不用

让你在出栈前也要判断,你也没有修改。如果用户输入()))),那么你的程序就出错了。
你光加上
else if (h.Count != 0) {m= "括号不配对!"; }
其实这个倒是不用加的,直接在最后return下就可以了。

希望你能认真阅读我给你的回答,并且充分思考,这样才有收获。