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下就可以了。
希望你能认真阅读我给你的回答,并且充分思考,这样才有收获。