代码之家 › 专栏 › 技术社区 › AJ.

如果一个类有一个公共入口点,将它所需要的一切作为参数,那么它应该是静态的吗?

static .net c#

AJ. · 技术社区 · 14 年前

我刚刚完成了一个类的编码,并意识到它的所有公共功能都封装在一个方法中。它没有属性,没有共享资源,不需要构造函数重载,也不处理它使用的任何内容。看起来像这样:

public class ReportGenerator
{
    public string GenerateReport(List<SomeClass> stuffToReportOn)
    {
        string fileName = String.Empty;
        using(var reportDs = CreateDataSet(stuffToReportOn))
        {
            //do the stuff with the third party tool that 
            //creates the report.
            //Construct the filename.
            //Save the report.
        }
        return fileName;
    }

    private TypedDataSetRequiredByThirdPartyLib CreateDataSet(List<SomeClass> reportItems)
    {
        //create the dataset, calling two other private methods
        //to build the tables/rows
    }
}

完成重构之后,我意识到这个类可以是完全静态的。我的问题是,应该吗?将其所有功能封装在一个公共方法中的类是否应成为静态的?

4 回复 | 直到 14 年前

smirkingman 14 年前

不,预期收益是多少?

更重要的是潜在的错误。如果在类中声明一个静态变量,它将只初始化一次,其值将保持不变,并且每个调用的行为都可能不同。很容易被忽略,这很难调试。

Community CDub 7 年前

不,它将使编写这个类或使用这个类的类的单元测试成为不可能。

用类似的工具 FakeItEasy 您甚至不需要类来实现接口来模拟它(这意味着您可以快速开始模拟一个没有用TDD编写的旧代码基),但它不能绕过静态调用。

更新
假设你需要对一个方法进行单元测试 GenerateWidgetReports 那个电话 ReportGenerator.GenerateReport . 你需要确保 stuffToReportOn 只包含 Widget1 和; Widget2 . 如何编写测试代码?

你最后做的是 生成IDGetReports 使用调用的方法 GetStuffToReportOn 你可以测试一下。 生成IDGetReports 然后就变成了胶水 获取填充器报告 把结果传给 报表生成器.生成器报表 .

但是你仍然没有测试 生成IDGetReports 因为没有它你不能叫它 事实上 生成报告。

最理想的是 生成IDGetReports 获取一个IReportGenerator对象,您模拟 GenerateReport 方法和测试 WIDGET1 和; WIDGET2 .

如果 通用端口 是静态的。

更新2
我坚持纠正,typemock可以截获和重新直接调用静态方法。 See this answer

IAbstract 14 年前

除了单元测试的困难之外,您还应该问自己:

是否更改参数?如果是,多久一次?
对象需要有状态吗?
在这个过程结束后,我是否需要保留这个物体?

如果您确定让它成为一个静态类对您来说是可行的,那么您也可以决定更进一步并创建一个扩展方法:

public static class ReportExtension
{
    public static string GenerateReport(this List<SomeClass> stuffToReportOn)
    {
        string fileName = String.Empty;
        using(var reportDs = CreateDataSet(stuffToReportOn))
        {
            //do the stuff with the third party tool that 
            //creates the report.
            //Construct the filename.
            //Save the report.
        }
        return fileName;
    }

    private static TypedDataSetRequiredByThirdPartyLib CreateDataSet(List<SomeClass> reportItems)
    {
        //create the dataset, calling two other private methods
        //to build the tables/rows
    }
}

我确信我的格式对于扩展方法是正确的。

编辑:

您仍然有单元测试的问题,因为代码依赖于这个静态方法。最终,你必须自己决定利弊。我可能是错的,但是如果您将此作为实例类对依赖代码进行单元测试,您可以获得一些信心,认为它可以作为静态方法或扩展方法来使用。

Pieter van Ginkel 14 年前

是的,这会让你的课更容易使用。