ASP.NET MVC提交用户资料表单后出现NullReferenceException求助

让我直接帮你定位几个导致空引用的关键问题：

1. 提交成功后返回的ViewModel未正确初始化 #

看你的HttpPost方法里这段代码：

HomeUserViewmodel newviewmodel = new HomeUserViewmodel(); 
viewmodel.user = currentUser; 
// ...
return View(newviewmodel);

你创建了newviewmodel，但却把currentUser赋值给了参数里的viewmodel，而不是newviewmodel！这就导致返回给视图的newviewmodel.user是null，视图里访问@Model.user.UserName自然就抛出空引用异常了。

应该改成：

HomeUserViewmodel newviewmodel = new HomeUserViewmodel(); 
newviewmodel.user = currentUser; // 给新的viewmodel赋值
Session.Clear(); 
Session["User"] = currentUser; 
return View(newviewmodel);

2. 编辑失败时未传递ViewModel #

当EditProfile返回false时，你直接return View();，没有传入任何ViewModel对象。这时候视图里的Model就是null，访问@Model.user肯定会报错。

解决办法是返回包含当前用户数据的ViewModel，比如：

else { 
    // 重新构造ViewModel并赋值当前用户数据
    HomeUserViewmodel errorViewModel = new HomeUserViewmodel();
    errorViewModel.user = currentUser;
    // 还可以添加错误提示，方便用户感知
    ViewBag.Error = "编辑失败，请重试";
    return View(errorViewModel); 
}

3. 视图中的潜在风险：Session存在但Model可能为null #

你的视图里先判断Session["User"]不为空，就直接访问@Model.user，但如果是Post请求失败返回的情况，Session虽然有值，但Model可能是null（比如之前的错误写法），这也会触发异常。

可以在视图里增加对Model和Model.user的判断，比如：

@if(Session["User"] != null && !string.IsNullOrEmpty(Convert.ToString(Session["User"])) && Model != null && Model.user != null) { 
    <h2>Profile of @Session["User"].ToString()</h2> 
    <!-- 表单内容 -->
}

额外优化建议 #

• 把HomeUserViewmodel里的public User user;改成自动属性：public User user { get; set; }，符合C#编码规范，也能避免一些潜在的绑定问题。
• 考虑使用[Bind]特性限制Post方法的参数绑定字段，避免过度绑定带来的安全问题，比如：

public ActionResult Index([Bind(Include = "Address,City,Mail,Phonenumber,Description")] HomeUserViewmodel viewmodel)

内容的提问来源于stack exchange，提问作者Thomas van Schijndel