Question

嗨，我有一个类，它有一个带有一些条件检查的方法，并根据条件附加一个字符串。我尝试用抽象工厂模式重构这段代码。

我的问题与默认表达式类有关。如果其他规则都不适用于相关过滤器，那么我想应用默认表达式规则。但是在我的申请上;它取决于_expressions列表中StandartExpression实例顺序的顺序。如果先添加，则不会检查其他规则。

我想我错过了那种模式。你能帮帮我吗？

这是我的应用程序代码：

public class Filter
{
    private string _operator = "="; // =, !=, <,> ...

    public string Name { get; set; }
    public object Value { get; set; }
    public string Operator { get { return _operator; } set { _operator = value; } }

}

Filter filter = null;
StringBuilder whereClause = new StringBuilder();

for (int i = 0; i < array.Length; i++)
{
    filter = array[i];

    if (filter.Value.ToString().ToLower().Equals("null"))
    {
        whereClause.Append(filter.Name + " " + filter.Operator + " null ");
    }
    else if (filter.Operator.ToLower().Equals("like"))
    {
        whereClause.Append(filter.Name + ".Contains(@" + i + ")");
    }
    else
    {
        whereClause.Append(filter.Name + " " + filter.Operator + " @" + i);
    }
    whereClause.Append(" AND ");
}

现在，这是抽象工厂模式代码：

public interface ICustomExpression
{
    bool ExpressionIsValid(Filter filter);
    string GetExpressionStr(Filter filter, int index);
}


public class StandardExpression : ICustomExpression
{
    public bool ExpressionIsValid(Filter filter)
    {
        return true;
    }

    public string GetExpressionStr(Filter filter, int index)
    {
        return filter.Name + " " + filter.Operator + " @" + index;
    }
}

public class LikeExpression : ICustomExpression
{
    public bool ExpressionIsValid(Filter filter)
    {
        return filter.Operator.ToLower().Equals("like");
    }

    public string GetExpressionStr(Filter filter, int index)
    {
        return filter.Name + ".Contains(@" + index + ")";
    }
}
public class NullExpression : ICustomExpression
{
    public bool ExpressionIsValid(Filter filter)
    {
        return filter.Value.ToString().ToLower().Equals("null");
    }

    public string GetExpressionStr(Filter filter, int index)
    {
        return filter.Name + " " + filter.Operator + " null ";
    }
}

public static class ExpressionFactory
{
    private static List<ICustomExpression> _expressions = null;
    public static List<ICustomExpression> Expressions
    {
        get
        {
            if (_expressions == null)
            {
                Build();
            }
            return _expressions;
        }
    }

    private static void Build()
    {
        _expressions = new List<ICustomExpression>();
        _expressions.Add(new NullExpression());
        _expressions.Add(new LikeExpression());
        _expressions.Add(new StandardExpression());
    }
}

这就是我在应用程序代码上使用该结构的方式：

StringBuilder whereClause = new StringBuilder();
Filter filter = null;
var array = filterList.ToArray();
for (int i = 0; i < array.Length; i++)
{
    filter = array[i];

    foreach (ICustomExpression exp in ExpressionFactory.Expressions)
    {
        if (exp.ExpressionIsValid(filter))
            whereClause.Append(exp.GetExpressionStr(filter, i));
    }


    whereClause.Append(" AND ");
}

Answer 1

你的感觉是对的：“特殊”和“标准”表达之间存在隐藏的区别，为了清晰起见，应明确表达。

然而，在我看来，更大的问题是你的工厂实际上并没有单独处理类型选择和实例化，而是将部分责任委托给客户。每个客户都必须知道如何使用集合属性Expressions，因此还必须知道我上面提到的这种隐含的区别。

理想情况下，ExpressionFactory的客户端应该只提供一个过滤器以获取正确的ICustomExpression实例。这是这样的：

public static class ExpressionFactory
{
    private static StandardExpression _standardExpression = new StandardExpression();

    private static ICustomExpression[] _specialExpressions = new []
    {
        (ICustomExpression)new NullExpression(),
        (ICustomExpression)new LikeExpression()
    };

    public static ICustomExpression GetBy(Filter filter)
    {
        var match = _specialExpressions.SingleOrDefault(e => e.ExpressionIsValid(filter));

        if (match == null)
            return _standardExpression;

        return match;
    }
}

如何创建表达式（或者更确切地说是在您的情况下选择）的规则现在只有工厂知道，因此在一个地方实现。

我还明确区分了“特殊”和“标准”表达式，因此实现不再依赖于添加ICustomExpression的顺序。

请注意，现在工厂的使用变得更加直截了当：

for (int i = 0; i < array.Length; i++)
{
    filter = array[i];  
    var exp = ExpressionFactory.GetBy(filter);          
    whereClause.Append(exp.GetExpressionStr(filter, i));        
    whereClause.Append(" AND ");
}

以下是重构代码的工作示例：https://dotnetfiddle.net/uzVJhM

顺便说一下：您的实现看起来更像是factory method pattern的实例。

抽象工厂模式默认类问题

1 个答案: