C# MVC数据库更新后Alert提示不显示问题排查

你写的js代码无法弹窗核心有2个问题：

• 客户端js无法直接识别服务端的ViewBag对象：ViewBag是ASP.NET MVC服务端的动态对象，只能在Razor渲染阶段通过@前缀输出到页面，你直接在js代码里写ViewBag.cmp，浏览器端js上下文根本不存在这个对象，会直接抛出ViewBag is not defined的未捕获错误，阻断后续alert执行，按F12打开浏览器控制台就能看到这个报错。
• 未做空值判断：页面首次加载、或者更新流程没走完的场景下，ViewBag.cmp是null值，就算语法写对了也可能因为空值导致js报错。

方案1：直接修正现有视图代码（最小改动） #

把视图里的js代码改成下面的写法，用Razor语法把服务端值渲染为js可识别的字符串，同时加空值判断：

<script type="text/javascript">
$(function () {
    // 加@前缀标记服务端变量，空值兜底，Html.Raw避免特殊字符转义
    var successTip = '@Html.Raw(ViewBag.cmp ?? "")';
    if (successTip) {
        alert(successTip);
    }
});
</script>

方案2：用TempData实现更稳定的提示传递（推荐） #

ViewBag的值仅在当前视图渲染请求内有效，如果你后续调整逻辑为操作完成后跳转页面，ViewBag的值会直接丢失。改用TempData存临时提示更适配Post-Redirect-Get的流程，也能避免刷新页面重复弹出提示：

1. 控制器里的赋值逻辑替换为：

if(Cnt_element==AutoID.Length)
{ 
    TempData["TaskCompleted"] = 1;
    TempData["cmp"] = "数据库更新成功，行排序已完成";
}

2. 视图里的js对应调整为读取TempData：

<script type="text/javascript">
$(function () {
    var successTip = '@Html.Raw(TempData["cmp"] ?? "")';
    if (successTip) {
        alert(successTip);
    }
});
</script>

你当前的EF更新逻辑把SaveChanges()放在了循环内，每更新一条就提交一次数据库，数据量稍大时性能很差，可以把提交操作挪到循环外，所有修改完成后一次性提交：

foreach (int id in AutoID)
{
    if (id == 0)
    {
        Cnt_element++;
        continue;
    }
    var value = entities.TBL_MASTER_INTAKE_ASSIGNED_TO.Find(id);
    value.Preference = orderby;
    orderby += 1;
    Cnt_element++;
}
// 所有字段修改完成后一次性提交
entities.SaveChanges();

内容的提问来源于stack exchange，提问作者electro_guy