我有一些设置用户属性的代码:
us = new UserSession();
us.EmailAddr = emailAddr;
us.FullName = fullName;
us.UserROB = GetUserROB(uprUserName);
us.UserID = GetUserID(uprUserName);
us.UserActive = GetUserActive(uprUserName);
其中GetUserROB
,GetUserID
和GetUserActive
看起来都像这样:
private static string GetUserActive(string userName)
{
using (Entities ctx = CommonSERT.GetContext())
{
var result = (from ur in ctx.datUserRoles
where ur.AccountName.Equals(userName, StringComparison.CurrentCultureIgnoreCase)
select new
{
Active = ur.active
}).FirstOrDefault();
if (result != null)
return result.Active;
else
return "N";
}
}
它有效,但我不认为这是正确的方法。如何在一个LINQ调用中分配userROB,ID和Active属性?没有必要有3个独立的功能来执行此操作?
答案 0 :(得分:2)
您可以创建一个接受UserSession
对象作为参数的方法,然后在其中设置所有三个属性。我在这里改变了你的GetUserActive
:
private static void GetUserData(string userName, UserSession user)
{
using (Entities ctx = CommonSERT.GetContext())
{
var result = (from ur in ctx.datUserRoles
where ur.AccountName.Equals(userName, StringComparison.CurrentCultureIgnoreCase)
select new
{
Active = ur.active,
ID = ur.ID,
//...select all properties from the DB
}).FirstOrDefault();
if (result != null)
user.UserActive = result.Active;
user.UserID = result.ID;
//..set all properties of "user" object
}
}
答案 1 :(得分:2)
如果我理解正确,我相信你可以这样做:
private static void GetUserData(string userName, UserSession userSession)
{
using (Entities ctx = CommonSERT.GetContext())
{
var result = (from ur in ctx.datUserRoles
where ur.AccountName.Equals(userName, StringComparison.CurrentCultureIgnoreCase)
select new
{
UserActive = ur.active,
UserROB = ur.ROB,
UserID = ur.ID
}).FirstOrDefault();
}
if (result != null) {
userSession.UserActive = result.UserActive;
userSession.UserROB = result.UserROB;
userSession.UserID = result.UserID;
}
}
在select new
中,您可以放置任意数量的属性,这样您就可以在一次往返中从数据库中获取几个属性,并在以后处理它。
在我给出的示例中,我将UserSession作为参数传递,在任何情况下,您已经有其他方法从其他方法填充。
答案 2 :(得分:0)
您可以考虑规范化您的域模型并使用类型为User
的{{1}}属性,然后您的方法将返回所有相关的用户数据。