我有一个模型类:
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设计模式方面有更多专业知识的人会如何写这个功能。
由于
答案 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
。