如果使用 LINQ，效果会更好吗？

Question

下面的方法选择管理员权限并从缓存的 DataTable 返回 bool，使用 Linq 会表现更好吗？

你可能会问为什么不测试一下。好吧，由于缺乏知识，我无法用 Linq 编写它

 DataRow[] result = PrivilegeMap.Select("privilegeActionId=" + (int)actionId);
            bool moderatorHasIt = Convert.ToBoolean(result[0]["moderatorHasIt"]);
            bool adminHasIt = Convert.ToBoolean(result[0]["adminHasIt"]);                                        
            if (myRole == User.Role.Admin)
            {
                return Convert.ToBoolean(adminHasIt);
            }
            if (myRole == User.Role.Moderator)
            {
                return Convert.ToBoolean(moderatorHasIt);
            }
            else
            {
                return false;
            }

Answer 1

可能是这样。

让我们假设以下情况：

1. 基础表的起始大小、读取频率和更新频率的平衡使得将整个表加载到内存中并只是重复查看来替换整个表是没有意义的。

2. 每个 ID 仅有一个匹配行。

3. 该行中有很多其他有趣的字段，我们在这里不关心。

然后，如果我们用 Linq2SQL Table 替换 PrivilegeMap，则等效的 LINQ 代码将类似于：

var result = PrivilegeMap.Where(p => p.PrivilegeActionId == actionID).Select(p => new{p.ModeratorHasIt, p.AdminHasIt}).First()
if (myRole == User.Role.Admin)
{
  return result.AdminHasIt;
}
if (myRole == User.Role.Moderator)
{
  return result.ModeratorHasIt;
}
else
{
  return false;
}

(就此而言 var result = PrivilegeMap.Where(p => p .PrivilegeActionId == actionID).First(p => new{p.ModeratorHasIt, p.AdminHasIt}) 也可以写成var result = (from p in PrivilegeMap where p.PrivilegeActionId == actionID select new{p.ModeratorHasIt, p.AdminHasIt}).First() 这只是相同 LINQ 操作的不同语法）。

假设 actionID 是 2
您的代码将按照以下方式转换为 SQL：

SELECT * FROM Privileges WHEREprivilegeActionId=2

上面的 Linq 将转换为：

SELECT TOP 1 adminHasIt, moderatorHasIt FROM Privileges WHEREprivilegeActionId

您可以看到，如果这是一个包含很多内容的表列和/或如果有多个匹配行，这可能会更有效。

（如果 PrivilegeMap 是可枚举的但不可查询，它将变成一个加载和扫描整个内容的操作，因此效率很低）。

但另一方面，生成该 SQL 的代码可能更复杂，并且需要在设置权限实体对象方面进行一些工作。如果这是一次性操作，那么从开发人员或运行时效率方面来看肯定不值得，但否则它可能对双方都有利。

但是，请注意，在我们的两种情况下，我们都进行了不必要的查询。我实际上会将我的替换为：

if (myRole == User.Role.Admin)
{
  return PrivilegeMap.Where(p => p.PrivilegeActionId == actionID).Select(p => p.AdminHasIt).First();
}
if (myRole == User.Role.Moderator)
{
  return PrivilegeMap.Where(p => p.PrivilegeActionId == actionID).Select(p => p.ModeratorHasIt);
}
else
{
  return false;
}

它将要么只查询 adminHasIt，只查询 moderatorHasIt，要么根本不查询，因为我们保证 myRole 的任何其他值都为 false，无论什么数据库状态。

同样，您可以通过以下方式获得更简单的改进：

if(myRole != User.Role.Admin && myRole != User.Role.Moderator)
  return false;
DataRow[] result = PrivilegeMap.Select("privilegeActionId=" + (int)actionId);
if (myRole == User.Role.Admin)
{
  return Convert.ToBoolean(result[0]["adminHasIt"]);
}
if (myRole == User.Role.Moderator)
{
  return Convert.ToBoolean(result[0]["moderatorHasIt"]);
}

这里，如果我们不可能使用它，我们首先完全避免数据库查询，只转换我们关心的字段，并且随后不将 bool 转换为 bool。这种关于实际使用哪些数据的局部思考更容易做到，虽然大多数此类节省都很小，但有些节省很大（这个可能是），而且它们是一个习惯问题，而不是复杂的权衡问题。

Answer 2

使用 LINQ 可能会更快，因为数据表不必解析查询字符串，但通过更改数据表示形式以避免首先查询数据表，您将获得最大的优势。

创建一个 IDictionary 来表示您感兴趣的权限，以 actionId 为键。然后当你需要查找时，你可以返回dict[actionId。

无论如何，我怀疑这可能是一种“过早优化”的情况：您是否测试过您的程序并发现这段代码占了您处理时间的很大一部分？

Answer 3

您列出的代码不完整，因为 myRole 变量未定义。

考虑到这一点，我建议将其写在 switch 语句中，然后继续，直到它被确定为问题为止。另外，它会更容易阅读（在我看来）。