隐藏静态方法可以吗？

Question

我有一个抽象 Catalog 类，如下所示。它有一个静态方法 OpenCatalog()，用于根据提供的位置类型返回特定的具体目录。一旦确定了目录类型，它就会调用正确的具体目录类型的特定 OpenCatalog() 方法。例如，我可能有一个存储在 SQL 数据库中的 Catalog 实现，或者另一个存储在文件系统中的 Catalog 实现。请参阅下面的代码。

public abstract class Catalog
{
    public static ICatalog OpenCatalog(string location, bool openReadOnly)
    {

        if(location is filePath)
        {
            return FileSystemCatalog.OpenCatalog(string location, bool openReadOnly);
        }
        else if(location is SQL server)
        {
            return SqlCatalog.OpenCatalog(string location, bool openReadOnly);
        }
        else
        {
            throw new ArgumentException("Unknown catalog type","location");
        }
    }

    ...

}

public abstract class FileSystemCatalog:Catalog
{

    public static new ICatalog OpenCatalog(string location, bool openReadOnly)
    {
        //Deserializes and returns a catalog from the file system at the specified location
    }

    ...

}



public abstract class SqlCatalog:Catalog
{

    public static new ICatalog OpenCatalog(string location, bool openReadOnly)
    {
        //creates an returns an instances of a SqlCatalog linked to a database
        //at the provided location          
    }

    ...

}

首先，隐藏静态方法可以吗？我知道这是可能的，但这似乎也是人们不应该经常做的事情。这也是一个有效的例子，可以隐藏静态方法，还是有更好的方法来完成我想做的事情？

Answer 1

看起来您正试图以一种非常尴尬的方式创建一个抽象工厂。实际发生的情况是您违反了单一职责原则并将目录创建问题与目录问题混合在一起。您需要做的就是使 CatalogFactory 成为非静态类。这使您可以灵活地进行以后的任何操作（例如依赖注入）。

public class CatalogFactory {
  public ICatalog CreateCatalog(string location, bool openReadOnly)
  {

    if(location is filePath)
    {
        return OpenFileCatalog(string location, bool openReadOnly);
    }
    else if(location is SQL server)
    {
        return OpenSqlCatalog(string location, bool openReadOnly);
    }
    else
    {
        throw new ArgumentException("Unknown catalog type","location");
    }
  }
  FileSystemCatalog OpenFileCatalog(string location, bool openReadOnly) {
    return new FileSystemCatalog{/*init*/};
  }
  SqlCatalog OpenSqlCatalog(string location, bool openReadOnly) {
    return new SqlCatalog{/*init*/};
  }

}

Answer 2

您并没有真正隐藏它，因为

Catalog.OpenCatalog(...);

如果您想要基类版本，您总是可以这样做。事实上，静态方法与特定的类相关联，并且不是虚拟的。您可以在派生类上调用基类中定义的静态方法，这非常方便。

Answer 3

我从未在“代码气味”方面找到反对使用私有/内部静态方法的论点。

一个很好的实际示例可能是某种服务/实用程序库中的扩展方法，您只想在该库中扩展。

internal static ShippingRateType ToShippingRateType(this ProviderShippingRateType rateType) { }