如何重构 WinForms 应用程序？

Question

我将在不久的将来修复 WinForms 应用程序 (.NET 2.0) 的错误。 查看源代码，我发现很大的代码文件（超过 2000 行），其中大多数是生成的对话框，其中包含大量代码隐藏。

有没有人给我一些建议可以分享？ 有任何错误修复或重构 WinForms 应用程序的战争故事或最佳实践吗？

Answer 1

我首先编写一些单元测试。 如果代码相当密集，您将需要这些来保持理智。

它们会给你一些信心，让你能够在重构中相当积极地进行。

Answer 2

顶部的几件事：

• 将所有生成的代码放入部分类或区域（我使用区域）
• 将所有代码作为函数从操作移动到单独的区域或类
• 为每个函数编写单元测试
• 在这一点之后进行常见的重构模式，将相同/相似的功能合并到一个函数中
• 确保不同的按钮不会使用不同的函数来执行相同的操作（特别是在 VB.NET 中，这是相当常见的设计，因此您可以在应用程序）
• 尽可能正确使用发送者对象，而不是在控件事件处理程序中对控件名称进行硬编码。
• 如果您发现任何此类代码可以转换为一个不错的类，请执行此操作。 从 WinForms 中删除所有非 GUI 代码。 有时需要重构原始类以使其可以与 GUI 一起使用，编写单元测试并重构它们。 （通常即使没有单元测试，你也应该没问题）

Answer 3

简短版本：

购买 Michael Feathers 的书《有效使用遗留代码》。

较长的版本：

维护（更改）此类应用程序时最大的挑战是在不破坏某些内容的情况下进行。 这意味着您需要了解应用程序的行为，而最简单的方法就是围绕它进行测试。 正如您所指出的，一开始这可能会令人畏惧，但这并非不可能。 它需要纪律和耐心。

您不必从单元测试开始。 通常更容易获得自动化框架（例如 NUnitForms 或 White）来首先将应用程序作为黑盒驱动。 围绕您需要更改的区域建立一套测试，以便您有足够的信心在不破坏某些内容的情况下进行更改。 然后开始针对单元可测试性进行重构。

如果它与我正在开发的应用程序类似，它主要涉及：

• 删除未使用的代码（这是一种干扰）。
• 删除公共静态方法调用并将其替换为接口（依赖注入是这里的关键技术）。
• 将重复的代码提取到类中。
• 将文件处理、网络 IO 和用户界面代码隐藏在适配器后面（这些最初可能是非常薄的包装器，足以让您在测试时用存根替换它们）。
• 寻找机会将行为提取到小班中。

有时重写代码的各个部分比重构它们更容易，但这需要通过测试或引用规范（如果存在）来理解行为。

除了任何实用建议之外，如果您是从事此工作的团队的一员，请确保你们一起工作。 这将使工作变得更轻松、更愉快，并确保您今天所做的更改明天不会被不了解您所做工作的人撤销。

我可以就这个话题写一整天，但羽毛先生更擅长，而且我很饿。 祝你好运！

Answer 4

• 首先尝试理解代码及其背后的逻辑。
• 一旦您理解了一些代码，请尝试为业务对象建模必要的类
• 在您的新设计中使用某种层方法（3层是经典）：接口层，业务层，数据访问层（或其他一些服务层） ）。
• 要有足够的耐心并记录一切！

我认为这是基本的东西。 这里很多人一定有很多建议。 祝你好运！