CRUD 3 使用 SQL Server 干净地连接表

Question

我正在使用 Identity (2.2) 开发一个 ASP.NET 项目。我试图放在一起的一件事是数据网格，其中用户和角色将被加载到一个可以运行 CRUD 命令的数据网格上。存储选择语句没有问题：

CREATE PROCEDURE [dbo].SelectUsersRoles
AS
    SET NOCOUNT ON;

    SELECT 
        p.Id, p.Email, p.PhoneNumber, p.LockoutEnabled, p.UserName, 
        p.AspNetUserRole, AspNetRoles.Name 
    FROM 
        AspNetUsers AS p 
    INNER JOIN 
        AspNetUserRoles ON p.Id = AspNetUserRoles.UserId 
    INNER JOIN 
        AspNetRoles ON AspNetUserRoles.RoleId = AspNetRoles.Id
GO

这选择了数据，我可以很好地加载网格。在reviewing this question 和this one 之后，我的想法是正确的更新代码应该是：

[编辑，运行存储过程更新如下，同样的问题]

CREATE PROCEDURE [dbo].UpdateUserRoles
(
    @Email nvarchar(256),
    @UserName nvarchar(256),
    @PhoneNumber nvarchar(MAX),
    @LockoutEnabled bit,
    @original_ID nvarchar(128)
)
AS
    SET NOCOUNT ON;

    UPDATE p
    SET p.Email = @Email, 
        p.Username = @UserName, 
        p.PhoneNumber = @PhoneNumber, 
        p.LockoutEnabled = @LockoutEnabled
    FROM [AspNetUsers] AS p
    INNER JOIN [AspNetUserRoles] AS r ON  p.ID = r.UserID
    INNER JOIN [AspNetRoles] AS b ON r.RoleID = b.ID
    WHERE p.ID = @original_ID
    GO

如果我尝试使用查询生成器，我注意到 VS'13 会自动向其中添加 CROSS JOIN 语句：[并且通常会降低可读性]

UPDATE p
SET p.Email = @Email, p.UserName = @UserName, 
    p.PhoneNumber = @PhoneNumber, p.LockoutEnabled = @LockoutEnabled
FROM            
    AspNetUsers AS p 
INNER JOIN
    AspNetUserRoles AS r ON p.Id = r.UserId 
INNER JOIN
    AspNetRoles AS b ON r.RoleId = b.Id 
CROSS JOIN
    p
WHERE
    (p.Id = @original_ID)

当我尝试运行查询时，这会不断抛出“p”是无效对象的错误。

我是否缺少一些简单的东西，或者将其拆分为连续的更新命令会更好？最终目标是更新用户信息（即电话号码）以及分配给用户的角色。

只是为了设置有问题的数据库，我认为您可以从 Visual Studio 启动任何新的 MVC/Web 表单模板，创建一些用户并设置一些角色。

Answer 1

请删除= 左侧UPDATE P SET 正下方的表别名。这是隐含的，是您的错误的原因。

UPDATE p SET
    Email = @Email, 
    Username = @UserName, 
    PhoneNumber = @PhoneNumber, 
    LockoutEnabled = @LockoutEnabled
FROM [AspNetUsers] AS p
    INNER JOIN [AspNetUserRoles] AS r
        ON  p.ID = r.UserID
    INNER JOIN [AspNetRoles] AS b
        ON r.RoleID = b.ID
WHERE p.ID = @original_ID

注意：连接没有理由 - 假设这些是因为您的查询生成器。你只需要在你的存储过程中。

UPDATE AspNetUsers SET
    Email = @Email, 
    Username = @UserName, 
    PhoneNumber = @PhoneNumber, 
    LockoutEnabled = @LockoutEnabled
WHERE ID = @original_ID