数据库无可用数据时，如何以标准MVC方式设置显示值？

先直接给结论：整体方向是对的，但有一些可以优化的细节，而你问到的Model里的if (!SR.Any())判断是恰当的，完全符合MVC规范。

1. 关于Model层的空数据判断 #

MVC模式里，Model层的核心职责就是处理业务逻辑、数据获取与数据格式化。当数据库查询没返回结果时，提供默认替代值，这属于数据处理的业务规则范畴，完全应该放在Model层来做——这样既能保证数据逻辑的复用性，也贴合“胖Model、瘦Controller、傻View”的最佳实践，不用让视图或者控制器来操心数据的默认值逻辑。所以你这里的实现是正确的。

2. 整体实现的优化建议 #

虽然核心逻辑没问题，但还有几个小地方可以调整，让代码更贴合MVC规范：

控制器层：让代码更“薄” #

你把UpdateMainGrid方法放在控制器里不太合适。控制器的职责应该是处理请求路由、参数传递，调用Model层获取数据，最后返回视图就行。建议把这个方法的逻辑移到Model层，让控制器代码更简洁：

public ActionResult RStatusC(string CurrentStatus) 
{
    var statusModel = new StatusModel();
    statusModel.GetStatus(DB, CurrentStatus);
    return View(statusModel);
}

视图层：避免潜在的空引用问题 #

因为你的Model层已经保证了数据列表至少有一个元素（空数据时会添加默认项），所以视图里的FirstOrDefault()可以改成First()，这样能避免不必要的空引用风险：

<table>
    <tr>
        <td>Status: </td>
        <td>
            <input type="text" name="txtStatus" value="@Model.StatusChoice.First().CurrentStatus" />
        </td>
    </tr>
</table>

Model层：让数据载体更清晰 #

从你的代码看，GetStatus方法直接返回了数据列表，但控制器却把它赋值给了StatusModel实例？建议调整Model的结构，让它作为数据载体，把列表作为属性存储，这样更符合MVC中Model的定位：

public class StatusModel
{
    public List<RequestedStatusC> StatusChoices { get; set; }

    public void GetStatus(DB_Entities DB, string CurrentStatus)
    {
        StatusChoices = DB.Database.SqlQuery<RequestedStatusC>(@"SELECT CurrentStatus FROM MyTable WHERE CurrentStatus = {0} ORDER BY StatusKey", CurrentStatus).ToList();
        if (!StatusChoices.Any())
        {
            StatusChoices.Add(new RequestedStatusC { CurrentStatus = string.IsNullOrEmpty(CurrentStatus) ? "Missing" : CurrentStatus });
        }
    }
}

总结 #

你的核心逻辑（在Model层处理空数据的默认值）完全符合MVC规范，只是在控制器和Model的结构上可以做些小优化，让分层更清晰，代码也更易维护。

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