数据更新 1.外键 2.对象

[wrost]

请问这2中写法有区别吗？




这是我以前用的方法
oldUser.RoleId = user.RoleId;

        public static void EditUser(RegisteredUsers user)
        {
            using (var db = new WEBVODEntities())
            {
                var oldUser = db.RegisteredUsers.SingleOrDefault(a => a.Id == user.Id);
                oldUser.RealName = user.RealName;
                oldUser.RoleId = user.RoleId;
                db.SaveChanges();
            }
        }

然后别人说有点问题，该下边这样写
oldUser.Role = db.Role.Single(r => r.Id == user.RoleId);

        public static void EditUser(RegisteredUsers user)
        {
            using (var db = new WEBVODEntities())
            {
                var oldUser = db.RegisteredUsers.SingleOrDefault(a => a.Id == user.Id);
                oldUser.RealName = user.RealName;
                oldUser.Role = db.Role.Single(r => r.Id == user.RoleId);
                db.SaveChanges();
            }
        }

[wrost]

引用 3 楼 yeness 的回复:

第二种明显验证了这个user.RoleId是否存在。

user.RoleId 不存在的话，就会引发异常，因为外键要么为null，要么已存在 oldUser.Role = db.Role.Single(r => r.Id == user.RoleId); 这样的话，返回null，赋值为null，也不会出错。 请问是这个意思吗？ 谢谢

[xuan.ye]

引用 4 楼 wrost 的回复:

[quote=引用 3 楼 yeness 的回复:] 第二种明显验证了这个user.RoleId是否存在。

user.RoleId 不存在的话，就会引发异常，因为外键要么为null，要么已存在 oldUser.Role = db.Role.Single(r => r.Id == user.RoleId); 这样的话，返回null，赋值为null，也不会出错。 请问是这个意思吗？ 谢谢[/quote]不是不会出错吧？应该是不会出现脏数据，如果这个用户对应的角色没有了，那么就不插入角色

[wrost]

引用 5 楼 yeness 的回复:

[quote=引用 4 楼 wrost 的回复:] [quote=引用 3 楼 yeness 的回复:] 第二种明显验证了这个user.RoleId是否存在。

user.RoleId 不存在的话，就会引发异常，因为外键要么为null，要么已存在 oldUser.Role = db.Role.Single(r => r.Id == user.RoleId); 这样的话，返回null，赋值为null，也不会出错。 请问是这个意思吗？ 谢谢[/quote]不是不会出错吧？应该是不会出现脏数据，如果这个用户对应的角色没有了，那么就不插入角色[/quote] 非常谢谢，我大概理解了你的意思。 我还有一个问题，另外开一个帖子吧

[xuan.ye]

别人说第二种是对的，也是有道理的，可以保证数据的完整性，这是在一个事务当中完成。 第二种更严谨

[wrost]

引用 1 楼 yeness 的回复:

别人说第二种是对的，也是有道理的，可以保证数据的完整性，这是在一个事务当中完成。 第二种更严谨

这里的事务是什么意思？

[xuan.ye]

第二种明显验证了这个user.RoleId是否存在。