做Winform界面的时候,上面有六个文本框需要判空,之前的思路大致是这样的,代码如下:
1 if (txtUserName.Text.Trim() == "") 2 { 3 MessageBox.Show("用户名不可以为空!", "温馨提示"); 4 txtUserName.Focus(); 5 } 6 else if (txtPassword.Text.Trim() == "") 7 { 8 MessageBox.Show("密码不可以为空!", "温馨提示"); 9 txtPassword.Focus(); 10 } 11 else if (txtConfirmPassword.Text.Trim() == "") 12 { 13 MessageBox.Show("确认密码不可以为空!", "温馨提示"); 14 txtConfirmPassword.Focus(); 15 } 16 else 17 { 18 MessageBox.Show("注册成功!", "温馨提示"); 19 }
代码中写了三个判断,大概就是做一个注册功能时所需要做的判空操作。可是,当我界面有六个文本框需要判空时,就觉得有点麻烦了,还要加三个if else判断。于是,就想能不能将判空操作提取出一个方法来进行呢,这样至少代码看起来不会那么冗余。于是,就有了下面的代码:
1 bool isEmpty = CheckEmpty(txtUserName, txtPassword, txtConfirmPassword); 2 if (isEmpty) 3 { 4 MessageBox.Show("注册成功!", "温馨提示"); 5 }
7 ///8 /// 自定义方法,判断某文本框是否为空 9 /// 10 /// 要进行判断的标签 11 /// 是否全都不为空 12 public bool CheckEmpty(params TextBox[] txt) 13 { 14 bool flag = true; 15 for (int i = 0; i < txt.Length; i++) 16 { 17 if (txt[i].Text.Trim() == "") 18 { 19 string name = txt[i].Name; 20 //将name转换为对应label的值 21 name = Change(name); 22 MessageBox.Show(name + "不可以为空,请重新输入!","温馨提示"); 23 txt[i].Focus(); 24 flag = false; 25 break; 26 } 27 } 28 return flag; 29 } 30 31 /// 32 /// 将文本框Name值转换成对应的Label文本 33 /// 34 /// 35 /// 36 private string Change(string name) 37 { 38 switch (name) 39 { 40 case "txtUserName": 41 name = "用户名"; 42 break; 43 case "txtPassword": 44 name = "密码"; 45 break; 46 case "txtConfirmPassword": 47 name = "确认密码"; 48 break; 49 default: 50 break; 51 } 52 return name; 53 }
这样一来,判空的代码看起来就清晰多了,将需要判空的文本框当做参数传过去即可。不过,又有一个问题,我们可以看到,Change方法中将文本框Name属性值转换为对应的label文本时,每添加一个判断,必须要增加一个case,这样一来,如果有一百个判断,那代码就太长了。那么,有没有一种方法,即可以替代Switch….Case又可以缩短代码行呢?答案是:有的,用Dictionary来代替。代码如下:
1 ///2 /// 将文本框Name值转换成对应的Label文本 3 /// 4 /// 5 /// 6 private string Change(string name) 7 { 8 Dictionary<string, string> myDic = new Dictionary<string, string>(); 9 myDic.Add("txtUserName", "用户名"); 10 myDic.Add("txtPassword", "密码"); 11 myDic.Add("txtConfirmPassword", "确认密码"); 12 return myDic[name]; 13 }
我们可以看到,Dictionary实现了同Switch一样的功能,代码从15行变成了5行,而且Switch每增加一个判断,需要增加三行代码,而Dictionary只需增加一行代码即可。到现在为止,判断这个操作看起来算是不那么麻烦了。我们知道,优秀的代码一般都是可扩展,可复用的。而目前,上面的代码并没有做到可复用。CheckEmpty方法中需要用到Change方法来转换提示信息,而Change方法中的字典是写死的。这个时候,如果想复用CheckEmpty方法就必须将Change方法拿掉,将其当做参数传过来。代码如下:
1 bool isEmpty = CheckEmpty(TopMessage(), txtUserName, txtPassword, txtConfirmPassword); 2 if (isEmpty) 3 { 4 MessageBox.Show("注册成功!", "温馨提示"); 5 } 6 7 ///8 /// 提示信息转换 9 /// 10 /// 11 private Dictionary<string, string> TopMessage() 12 { 13 Dictionary<string, string> myDic = new Dictionary<string, string>(); 14 myDic.Add("txtUserName", "用户名"); 15 myDic.Add("txtPassword", "密码"); 16 myDic.Add("txtConfirmPassword", "确认密码"); 17 return myDic; 18 } 19 20 /// 21 /// 自定义方法,判断某文本框是否为空 22 /// 23 /// 要进行判断的标签 24 /// 是否全都不为空 25 public bool CheckEmpty(Dictionary<string,string> myDic,params TextBox[] txt) 26 { 27 bool flag = true; 28 for (int i = 0; i < txt.Length; i++) 29 { 30 if (txt[i].Text.Trim() == "") 31 { 32 string name = txt[i].Name; 33 //将name转换为对应label的值 34 name = myDic[name]; 35 MessageBox.Show(name + "不能为空,请重新输入","温馨提示"); 36 txt[i].Focus(); 37 flag = false; 38 break; 39 } 40 } 41 return flag; 42 }
这样,CheckEmpty方法就可以复用了。在Web应用中,文本框是有一个属性可以设置是否允许为空的,不过Winform好像还没有,觉得有必要也加上。以上只是个人思考,如有不当或可改进之处,还望不吝赐教。