为什么DialogResult = DialogResult.OK不起作用

[waiyuyi]

我调用了一个函数，然后再调用这个，点击OK没反应，代码如下
else if(2 == nType)
{
if (OnCheck())
{
return;
}
// else
// {
InitAdmin();
SQLServer.ModifyOperator(m_Admin);
DialogResult = DialogResult.OK;
// }

}
private bool OnCheck()
{
if ("" == textName.Text.Trim())
{
MessageBox.Show("用户名不能为空!", "提示");
}
else if ("" == textPsw.Text.Trim())
{
MessageBox.Show("密码不能为空!", "提示");
}
else if ("" == textPsw2.Text.Trim())
{
MessageBox.Show("重复密码不能为空!", "提示");
}
else if (textPsw.Text != textPsw2.Text)
{
MessageBox.Show("密码和重复密码不相同，请重新输入", "提示");
textPsw.Text = "";
textPsw2.Text = "";
}
return true;
}

[gxingmin]

OnCheck函数一直都是返回true，也就是说代码执行到
if (OnCheck())
{
return;
}
就返回了，下面的else代码不可能会执行到

[orochiheart]

this.DialogResult = DialogResult.ok;

[gxingmin]

OnCheck函数应该改造一下，MessageBox后面加上return false;

private bool OnCheck()
  {
  if ("" == textName.Text.Trim())
  {
  MessageBox.Show("用户名不能为空!", "提示");
return false;

  }
  else if ("" == textPsw.Text.Trim())
  {
  MessageBox.Show("密码不能为空!", "提示");   
return false;

  }
  else if ("" == textPsw2.Text.Trim())
  {
  MessageBox.Show("重复密码不能为空!", "提示");
return false;

  }
  else if (textPsw.Text != textPsw2.Text)
  {
  MessageBox.Show("密码和重复密码不相同，请重新输入", "提示");
  textPsw.Text = "";
  textPsw2.Text = "";   
return false;

  }
  return true;
  }

[孟子E章]

DialogResult dr = new DialogResult ();
dr = frm2.ShowDialog();
if ( dr == DialogResult.OK )

[waiyuyi]

还是不可以，改了之后还是同样的效果

[waiyuyi]

我把OnCheck()中的代码直接放进去是可以实现的，这是为什么呢？

[gxingmin]

你跟踪一下，因为你的OnCheck函数返回true了，if条件成立了，所以执行return;了，下面的代码不执行了...

[waiyuyi]

我把Oncheck（）函数最后的return true,改成return false,结果还是一样
else if(2 == nType)
{
if (!OnCheck())
{
return ;
}
else
{
InitAdmin();
SQLServer.ModifyOperator(m_Admin);
this.DialogResult = DialogResult.OK;
}
private bool OnCheck()
{
if ("" == textName.Text.Trim())
{
MessageBox.Show("用户名不能为空!", "提示");
}
else if ("" == textPsw.Text.Trim())
{
MessageBox.Show("密码不能为空!", "提示");
}
else if ("" == textPsw2.Text.Trim())
{
MessageBox.Show("重复密码不能为空!", "提示");
}
else if (textPsw.Text != textPsw2.Text)
{
MessageBox.Show("密码和重复密码不相同，请重新输入", "提示");
textPsw.Text = "";
textPsw2.Text = "";
}
return false;
}
那该如何改啊

[深海之蓝]

[Quote=引用 7 楼 的回复:]
你跟踪一下，因为你的OnCheck函数返回true了，if条件成立了，所以执行return;了，下面的代码不执行了...
[/Quote]
人家都告诉你加断点跟踪了，你就跟踪一下，别人家说的答案，你改了不成功再来反馈一遍，这样学不到东西的。要学会自己思考。