2 u011475211 u011475211 于 2016.03.22 12:53 提问

重构自己的代码,减去重复的部分

在实际的项目中,有一个程序包含三个窗体,这三个窗体包含同一个模块(执行Sql插入的模块,由三个textBox和一个执行插入操作的按钮),
sql语句如下:
sqlStr="insert into task(headline,keyword,content) values(' "
+textBox_headline.Text + " ','"
+ textBox_keyword.Text + "','"
+textBox_content.Text + "')";
现在的问题是,三个窗体都出现相同的代码,显然违反了Don't Repeat Yourself的
原则,我的想法是将这部分提取到一个类中,减少重复,但是textBox_headline,
textBox_keyword,text_content不知道如何处理,因为新的类根本就没有textBox,
请问如何解决这个问题?

2个回答

danielinbiti
danielinbiti   Ds   Rxr 2016.03.22 13:46
已采纳
TextBox作为参数传入
 sub insertSQL(textBox_headline as TextBox,textBox_keyword as TextBox,textBox_content as TextBox)
   sqlStr="insert into task(headline,keyword,content) values(' " 
+textBox_headline.Text + " ','" 
+ textBox_keyword.Text + "','"
+textBox_content.Text + "')";
 end sub
 建议传入的参数是TextBox的值,不建议把控件名称传入到类或者模块里面
danielinbiti
danielinbiti 回复caoxwei2: public void setForm(Form frm) { (frm.Controls["TextBox2"] as TextBox).Text = (frm.Controls["TextBox1"] as TextBox).Text; }
一年多之前 回复
u011475211
u011475211 不能传入窗体,传入窗体已经假定作为形式参数的窗体有txtbox,但是实际情况是出现编译错误
一年多之前 回复
danielinbiti
danielinbiti 回复caoxwei2: 嗯,根据实际情况吧,如果窗体都一模一样,可以传入窗体对象
一年多之前 回复
u011475211
u011475211 不过您的方法倒是启发了我,应该可以传入form窗体,然后使用form.textbox,我使用的是c#语言
一年多之前 回复
u011475211
u011475211 我只是实例而已,实例情况有十几个textbox,这么多参数是不合理的
一年多之前 回复
herozhangbz
herozhangbz   2016.03.22 12:58

你就用三个全局的string代替其中的textbox.text呗,给全局变量赋值就行了

Csdn user default icon
上传中...
上传图片
插入图片
准确详细的回答,更有利于被提问者采纳,从而获得C币。复制、灌水、广告等回答会被删除,是时候展现真正的技术了!