避免在以下遗留代码中进行大量代码更改的优雅方法

Question

在某些遗留代码中，以下代码片段出现数百次：

myObj.ReportGenerator.Preview = reportingObj.PreviewDocument;

...而“ReportGenerator”和“ReportingObj”都是第三方库的实例，因此不可修改。

这段代码在 Windows XP 下运行良好，但在 Windows 7 中运行该程序确实需要以下额外的代码行：

reportingObj.Render();
myObj.ReportGenerator.Preview = reportingObj.PreviewDocument;

不幸的是，这段代码在所有代码库中出现了数百次，手动搜索它们听起来就像一个非常容易出错的过程。

由于“ReportGenerator”和“reportingObj”是第三方，我无法更改它们的 getter / setter。

解决此类问题的优雅方法是什么？

Answer 1

您可以将 ReportingObj 包装在您自己的类中，在该类中您只需委托给原始 ReportingObj，但对于 PreviewDocument 属性检查是否 Render()< /code> 被调用，如果没有调用它 - 像这样：

public Foo PreviewDocument
{
    get
    {
        if (!_rendered)
        {
            _originalreportingObj.Render();
            _rendered = true;
        }
        return _originalreportingObj.PreviewDocument;
    }
}

Answer 2

您可以更改 myObj 的类（我认为该类在您的控制之下），并让 ReportGenerator 属性返回一个包装类，该包装类可以调用 的原始设置器预览或在调用Render()后调用它：

public class ReportGeneratorWrapper
{
     private ReportGenerator m_InnerReportGenerator;

     public PreviewDocument Preview
     {
         get
         {
             return m_InnerReportGenerator;
         }
         set
         {
             if (IsNT6OrAbove)
                 value.Render();

             m_InnerReportGenerator = value;
         }
     }
}

Answer 3

您可能会发现最少的返工量是创建一个静态类，如下所示：

public class Previewer
{
     public static PreviewDocumentType PreviewDocument(ReportingObject reportingObj) {
       reportingObj.Render();
       return reportingObj.PreviewDocument;
     }
}

其中 PreviewDocumentType 是从 PreviewDocument 返回的类型，ReportingObject 是报告对象的类型。

然后您可以替换

reportingObj.PreviewDocument;

为

Previewer.PreviewDocument(reportingObj);