C# · 12月 24, 2021

c#重构两个几乎相同的方法

重构是好的,但有时候解决如何重构并确实是否可以实际重构某些内容并不容易!

我有许多几乎相同的方法 – 我可以重构它们,但重构的一部分超出了我的逻辑.

这是两个未重构的方法:

private void projectToolStripMenuItem_Click(object sender,EventArgs e) { if (projectToolStripMenuItem.Checked) { projectToolStripMenuItem.Checked = false; if (!projectForm.IsDisposed) projectForm.Hide(); } else { if (projectForm.IsDisposed) projectForm = new frmProject(); projectForm.Show(dockPanel,DockState.DockRight); projectToolStripMenuItem.Checked = true; } } private void logginToolStripMenuItem_Click(object sender,EventArgs e) { if (logginToolStripMenuItem.Checked) { logginToolStripMenuItem.Checked = false; if (!outputForm.IsDisposed) outputForm.Hide(); } else { if (outputForm.IsDisposed) outputForm = new frmOutput(); outputForm.Show(dockPanel,DockState.DockBottom); logginToolStripMenuItem.Checked = true; } }

使用Refactoring,我会得到一个这样的方法,以前的未重构方法会调用

private void refactoredMethod(TooStripMenuItem menuItem,DockContent frmName){ if (menuItem.Checked) { menuItem.Checked = false; if (!frmName.IsDisposed) frmName.Hide(); } else { if (frmName.IsDisposed) frmName= new frmProject(); // Still Problematic frmName.Show(dockPanel,DockState.DockRight); menuItem.Checked = true; } }

那么我们有一个几乎完全重构的方法 – 有一个问题,我怎么能告诉我希望从frmName变量实例化哪个表单?

解决方法 您可以使方法通用并利用new()泛型约束. private TForm refactoredMethod<TForm>(TooStripMenuItem menuItem,TForm frmName) where TForm : Form,new(){ if (menuItem.Checked) { menuItem.Checked = false; if (!frmName.IsDisposed) frmName.Hide(); } else { if (frmName.IsDisposed) frmName= new TForm(); frmName.Show(dockPanel,DockState.DockRight); menuItem.Checked = true; } return frmName;}

所以你可以称之为

projectForm = refactoredMethod<frmProject>(projectToolStripMenuItem,projectForm);

一个限制是您的表单应该有一个公共无参数构造函数.如果你有一个带参数化构造函数的Form,你可以传递Func< TForm>作为工厂方法的方法.