我怎样才能更好地设计这个？ （避免使用面向对象设计的 switch 语句）

Question

我对面向对象设计了解一点，但我不确定如何使用我的代码中的这些原则。这是我正在做的事情：（

    public void Query(Agency agency, Citation queryCitation) {
        queryCitation.AgencyCode = agency.AgencyCode;

        switch (agency.ClientDb.Type) {
            case "SQL":
                QueryOracle(agency, queryCitation);
                break;
            case "PIC":
                QueryPick(agency, queryCitation);
                break;
        }
    }

其中大部分是来自 NHibernate 的对象。我正在使用遗留数据库系统，并将其部分重构为代码库。）显然，我可以在这里做一些不同的事情，这样我就不会具有相同输入的不同数据库查询不需要重复的函数。它应该只根据代理对象知道是使用 Oracle 数据库还是 Pick 数据库连接。 （如果您从未听说过 Pick 数据库，那么在我开始在这里工作之前我也没有听说过。我们通过 HTTP 请求对其进行查询，因此它不是 SQL。）

我应该创建一个接口，例如称为“ClientDbConnection” ”然后创建两个实现该接口的类，将查询数据库的代码移至这些类，然后用“agency.clientDb.Query(queryCitation)”之类的内容替换整个函数？我想我在这里大声思考，但对此的任何意见将不胜感激。

Answer 1

Agency 是你控制的类吗？如果是这样，请执行以下操作：

public abstract class GenericDb
{
    public abstract void Query(parms);
}

在您的 Agency 类中，您可以有

public GenericDb ClientDb {get; set;}

然后有一个 SqlDb 类，例如：

public class SqlDb : GenericDb
{
    public void Query(parms);
}

public class PicDb : GenericDb
{
    public void Query(parms);
}

然后此代码：

public void Query(Agency agency, Citation queryCitation) {
        queryCitation.AgencyCode = agency.AgencyCode;

        switch (agency.ClientDb.Type) {
            case "SQL":
                QueryOracle(agency, queryCitation);
                break;
            case "PIC":
                QueryPick(agency, queryCitation);
                break;
        }
    }

变为

public void Query(Agency agency, Citation queryCitation) {
        queryCitation.AgencyCode = agency.AgencyCode;
        agency.ClientDb.Query(queryCitation);
    }

因为继承，它将知道 ClientDb 有一个 GenericDb 基类。它会通过ClientDb参数的类型知道是否应该运行SqlDb或PicDb或Oracle等。

Answer 2

您可能希望在此处实施策略模式。基本上， switch 语句中的每个可能的“类型”都将成为实现相同接口的自己的类。

应用策略模式

然后，您可以使用以“type”值作为参数的工厂方法。该方法将返回正确的类（其返回类型是上面提到的接口）。

Answer 3

我会重构以利用接口。我可能会这样做：

public interface IQuery
{
    void Execute(Agency agency, Citation query);
}

public class OracleQuery : IQuery
{
    // Implementation
}

public class PickQuery : IQuery
{
    // Implementation
}

然后，您可以更改 Agency 类来存储 IQuery 对象的实例，而不是（或除了）ClientDb 对象：

public class Agency
{
    public IQuery Query { get; set; }
}

并且然后在初始化代码中（通常设置 ClientDb 属性），您可以将实例设置为适当的 IQuery 实现：

agency.Query = new PickQuery();

Answer 4

要编写更少的代码但提高声明性代码级别的可读性，您可以切换到带有每个数据库委托的字典。它可以很容易地扩展并且非常可读。考虑到更实用的方法，你会得到类似的东西

void Query(Agency agency, Citation queryCitation)
{
    Dictionary<string, Action<Agency, Citation>> QueryMap = new Dictionary<string, Action<Agency, Citation>>
    {
        { "SQL", QueryOracle},
        { "PIC", QueryPic}
    };


    queryCitation.AgencyCode = agency.AgencyCode;

    QueryMap[agency.ClientDb.Type](agency, queryCitation);
}

Answer 5

ADO.NET 有一组通用类：DbCommand、DbConnection 等...还实现了另一组通用接口：IDbCommand、IDbConnection 等...

因此您可以使用它们，但最终可能会变得相当复杂。您的解决方案的优点是它非常具有可读性。另外，也许选择数据库没有任何 ADO.NET 提供程序...

PS：不过，我会替换 Type 属性类型，并使用枚举代替。

Answer 6

有两种OOP解决方案多态性和访客模式。