c#MVC3需要帮助使Controller方法更干燥

时间:2013-07-17 09:54:25

标签: c# asp.net-mvc-3 dry

我有一个模型类:

public class Service
{

    public int ID { get; set; }

    public string description { get; set; }

    public decimal price { get; set; }
    public decimal cost { get; set; }
    public decimal manDay { get; set; }
    public decimal FTE{ get; set; }

}

我还有一个简单的控制器,它将循环遍历整个Services数据库表,并根据输入参数返回一个Service属性的JSON表:

public JsonResult _columns(int attributeCode)
{
    IQueryable<Service> services = db.Services.AsQueryable();

    MyJSONContainer jcontainer = new MyJSONContainer();

    switch(attributeCode)
    {
        case 0: 
            foreach(Service s in services) 
                jcontainer.addEntry(s.description, s.price);
            return Json(jcontainer);
        case 1: 
            foreach(Service s in services) 
                jcontainer.addEntry(s.description, s.cost);
            return Json(jcontainer);
        case 2: 
            foreach(Service s in services) 
                jcontainer.addEntry(s.description, s.manDay);
            return Json(jcontainer);
        default:
            foreach(Service s in services) 
                jcontainer.addEntry(s.description, s.FTE);
            return Json(jcontainer);   
    }
}

现在这段代码运行良好,但不是很干,特别是因为我将来可能会在foreach循环中为代码添加更多复杂性。

我很想知道在DRY设计模式方面有更多专业知识的人会如何写这个功能。

由于

2 个答案:

答案 0 :(得分:2)

您可以在类中添加一个方法(或者作为辅助类中的扩展方法,如果您不希望在模型类中使用“attributeCode”逻辑,则进行微小更改)

public decimal GetValueFor(int attributeCode) {
  switch(attributeCode) {
    case 0 : return price;
    case 1 : return cost;
    case 2 : return manDay;
    default: return FTE;
  }
}

然后在你的行动中

public JsonResult _columns(int attributeCode)
{
    var services = db.Services.AsQueryable();
    var jcontainer = new MyJSONContainer();

    foreach(var service in services) 
        jcontainer.addEntry(service.description, service.GetValueFor(attributeCode));

    return Json(jcontainer);

}

答案 1 :(得分:1)

如果您希望将来扩展它,那么我会将任务移到他们自己的班级。

首先是一般建筑师:

static class JsonResultBuilder
{   
    public static JsonResult Build(Database db, int attributeCode)
    {
        var propertyRetriever = JsonServicePropertyRetrievers.Get(attributeCode);

        IQueryable<Service> services = db.Services.AsQueryable();
        MyJSONContainer jcontainer = new MyJSONContainer();

        foreach(Service s in services)
            jcontainer.addEntry(s.Description, propertyRetriever(s));

        return Json(jcontainer);
    }
}

然后是一个单独的类,您可以使用以下命令轻松添加更多属性检索器:

static class JsonServicePropertyRetrievers
{
    private static readonly Dictionary<int, Func<Service, object>> ServicePropertyRetrievers = new Dictionary<int, Func<Service, object>>();
    private static readonly Func<Service, object> DefaultServicePropertyRetriever;

    static JsonResultBuilder()
    {
        DefaultServicePropertyRetriever = s => s.FTE;

        Register(0, s => s.price);
        Register(1, s => s.cost);
        Register(2, s => s.manDay);
    }

    public static void Register(int attributeCode, Func<Service, object> propertyRetriever)
    {
        ServicePropertyRetrievers[attributeCode] = propertyRetriever;
    }

    public static Func<Service, object> Get(int attributeCode)
    {
        Func<Service, object> propertyRetriever;
        if (!ServicePropertyRetrievers.TryGetValue(attributeCode, out propertyRetriever))
            propertyRetriever = DefaultServicePropertyRetriever;

        return propertyRetriever;
    }
}

这种方式没有越来越大的switch语句。如果您希望动态添加对代码的支持,则可以轻松使用JsonServicePropertyRetrievers.Register方法。如果您不想公开对此的支持,只需将该方法设为私有。如果要删除检索方法,可以添加Remove方法。

如果可能,我还建议您为enum实施attributeCode