基于数据库查询的静态选择列表

在重构一个ASP。. NET MVC 5 web项目，我看到了一个机会，将一些选择列表移动到另一个类中，多个控制器可以访问它们。这将允许我删除重复的代码。

在本例中，选择列表需要访问数据库。手工编写列表是不可行的，因为列表可能会随着时间的推移而改变(因此需要数据库查询)。

虽然我没有编译器错误，页面显示按预期工作，但我不确定采用这种方法是否会产生其他问题。下面所示的代码方法是实现此结果的"坏"方法吗?有更好的方法吗?

总之，这就是我正在做的:

• 类和方法是静态的
• 一个私有静态只读数据库上下文定义在顶部
• 显示的两个函数查询数据库并产生期望的结果。
因为这是一个静态类，所以没有dispose方法。

类:

public static class ElpLookupLists
{
    private static readonly EllAssessmentContext Db = new EllAssessmentContext();
    // code...
    internal static IEnumerable<SelectListItem> StandardSelectList(string selectedDomain)
    {
        return Db.ElpStandardLists.Where(m => m.Domain == selectedDomain)
            .Select(m => m.Standard).Distinct()
            .Select(z => new SelectListItem { Text = z.ToString(), Value = z.ToString() }).OrderBy(z => z.Value)
            .ToList();
    }
    internal static IEnumerable<SelectListItem> PerformanceIndicatorSelectList(string selectedDomain, int? selectedStandard,
        string selectedSubConcept)
    {
        var query =
        Db.ElpStandardLists.Where(m => m.Domain == selectedDomain).Where(m => m.Standard == selectedStandard);
        if (!string.IsNullOrEmpty(selectedSubConcept)) query = query.Where(m => m.SubConcept == selectedSubConcept);
        var list =
            query.Select(m => m.PerformanceIndicator)
                .Distinct().OrderBy(m => m)
                .Select(z => new SelectListItem { Text = z.ToString(), Value = z.ToString() })
                .OrderBy(z => z.Text).ToList();
        return list;
    }
}