这是一个非常模糊/主观的问题。我想知道这是否是使用 ajax 调用向/从浏览器发送/检索数据的最佳方式。在后端webservice上,我想使用 Entity Framework 。下面是两个示例函数。
“最佳”的标准是编写代码的速度、可读的代码和健壮的架构。
感谢您的任何反馈、建议和意见。
获取函数
[WebMethod]
public AjaxEmployee EmployeeGetById(int employeeID, bool getTimeOff)
{
using (Time_TrackerEntities ctx = new Time_TrackerEntities())
{
var results = from item in ctx.Employees
where item.ID == employeeID
orderby item.Last_Name
select new AjaxEmployee
{
ID = item.ID,
Employee_ID = item.Employee_ID,
First_Name = item.First_Name,
Middle_Name = item.Middle_Name,
Last_Name = item.Last_Name,
Supervisor_ID = item.Supervisor_ID,
Active = item.Active,
Is_Supervisor = item.Is_Supervisor
};
var emp = results.FirstOrDefault();
if (getTimeOff)
{
var results2 = from item2 in ctx.Time_Off
where item2.Employee_ID == emp.Employee_ID
select new AjaxTime_Off
{
ID = item2.ID,
Employee_ID = item2.Employee_ID,
Date_Off = item2.Date_Off,
Hours = item2.Hours
};
emp.Time_Off = results2.ToList<AjaxTime_Off>();
}
return emp;
}
}
保存功能
[WebMethod]
public bool EmployeeSave(AjaxEmployee emp)
{
using (Time_TrackerEntities ctx = new Time_TrackerEntities())
{
var results = from item in ctx.Employees
where item.ID == emp.ID
select item;
var myEmp = results.FirstOrDefault();
if (myEmp == null)
{
myEmp = new Employee();
ctx.Employees.AddObject(myEmp);
}
myEmp.Employee_ID = emp.Employee_ID;
myEmp.First_Name = emp.First_Name;
myEmp.Middle_Name = emp.Middle_Name;
myEmp.Last_Name = emp.Last_Name;
myEmp.Supervisor_ID = emp.Supervisor_ID;
myEmp.Active = emp.Active;
myEmp.Is_Supervisor = emp.Is_Supervisor;
return ctx.SaveChanges() > 0;
}
}
最佳答案
还有一些需要改进的地方。
Save() 方法 - 不要从左到右复制,使用 EF 内置逻辑
取而代之的是:
myEmp.Employee_ID = emp.Employee_ID;
myEmp.First_Name = emp.First_Name;
myEmp.Middle_Name = emp.Middle_Name;
myEmp.Last_Name = emp.Last_Name;
myEmp.Supervisor_ID = emp.Supervisor_ID;
myEmp.Active = emp.Active;
myEmp.Is_Supervisor = emp.Is_Supervisor;
你可以这样做:
ctx.Employees.ApplyCurrentValues(emp)
.
它的作用是在图中查找具有相同键的实体(存在,因为您刚刚使用 FirstOrDefault()
检索了它),并使用您传入的实体覆盖标量值 - 这是正是你在做什么。
所以你的 7 行变成了 1 行,而且如果你添加任何额外的标量属性 - 你将不必重构你的代码。请记住 - 仅适用于标量属性,不适用于导航属性。
为什么要为主键检索构建查询?只需对 SingleOrDefault() 使用谓词
取而代之的是:
var results = from item in ctx.Employees
where item.ID == emp.ID
select item;
var myEmp = results.FirstOrDefault();
这样做:
var myEmp = ctx.Employees.SingleOrDefault(x => x.ID == emp.Id);
或者更好的是,使用管道/过滤器技术:
var myEmp = ctx.Employees.WithId(emp.Id).SingleOrDefault();
在哪里WithId
是一个 IQueryable<Employee>
扩展方法,它根据提供的员工 ID 过滤查询。这允许从您的存储库/DAL 中分离过滤/业务逻辑。它应该放在您的领域模型中,这样您就可以拥有一个非常流畅的 API,通过您的 ORM 查询您的领域实体。
当您通过主键检索实体时,您应该始终使用SingleOrDefault()
或 Single()
, 从不 FirstOrDefault()
或 First()
.如果它是一个主键 - 应该只有一个,所以如果存在多个主键,你应该抛出一个异常,这就是 SingleOrDefault()
。做。正如@Shiraz 提到的那样 - 你的 FirstOrDefault()
将使下面的查询崩溃。使用 <First/Single>OrDefault()
时始终需要进行空值检查.
可以对您的 Get 方法进行相同的改进。
总的来说,您的代码在功能上没有任何问题 - 它只需要细微的改进、空值检查和异常处理。
我强烈推荐的唯一功能改进是将您的网络服务代码重构为通用存储库。由于代码非常简单,可以在任何实体中重复使用。 Web 服务不应该关心事务、主键或 EF 逻辑。它甚至不应该引用 EF DLL。将此逻辑封装在存储库后面并将持久性逻辑委托(delegate)给那里(当然是通过接口(interface))。
进行我上面提到的更改后,您的 Web 服务方法每个代码不应超过 5-7 行。
您的 web 服务中有太多的智能 - 它应该是愚蠢和持续的无知。
关于c# - 如何改进 Entity Framework 和 Javascript 交互,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/4988349/