减少程式的if和else,重构你的程式

今天审核自己以前写的程式,发现下面面这段代码,写得非常不理想,if层次太多,而且else也多。看过《重构》的人,都会认为那是有着很浓的坏味道。

上面代码,它的意思是文本框的值变化而作出另一文本框TextBoxBankName显示。如果文本框的值不为空,去搜索数据记录,如果有记录,把BankName显示于TextBoxBankName内。其它一切条件,TextBoxBankName均为空。

上面的代码是没有问题,是能正常运行,但它有很多坏味道,if与else层资多,现Insus.NET把它重构,让其更简洁明了。

 重构过之后,它的程式,先给TextBoxBankName赋值为string.Empty,第二步是获取数据,参考不管是空与否,均作为参数传入对象,如果文本框是空或不正确,得到的记录都为空。最后是判断数据行是否大于零,如大于零,就给TextBoxBankName赋值。

 

posted @ 2013-04-09 20:06  Insus.NET  阅读(1207)  评论(4编辑  收藏  举报