假设我有一个复杂的 View 模型，其中包含大量数据，例如国家/地区列表、产品、类别等，每次我创建 ViewModel 时都需要从数据库中获取这些数据。

我要解决的主要问题是，当我处理 POST 操作时，一些 TestModel 被发布了不正确的值，这导致 ModelState.IsValid 为 false，那么我必须返回与当前发布的模型相同的 View 。这迫使我再次获取我的类别列表，因为我是在 GET 操作中这样做的。这在 Controller 中添加了很多重复的代码，我想删除它。目前我正在做以下事情:

我的模型和 View 模型:

数据库中存储的模型、实体:

public class Category
{
    public int Id { get; set; }
    public string Name { get; set; }

    public IEnumerable<Category> SubCategories { get; set; }
}

查看模型:

public class CategoryModel
{
    public int Id { get; set; }
    public string Name { get; set; }
}

public class TestModel
{
    [Required]
    [MaxLength(5)]
    public string Text { get; set; }

    public int SelectedCategory { get; set; }
    public IEnumerable<CategoryModel> Categories { get; set; }
    public SelectList CategoriesList
    {
        get
        {
            var items = Categories == null || !Categories.Any() 
                ? Enumerable.Empty<SelectListItem>()
                : Categories.Select(c => new SelectListItem
                {
                    Value = c.Id.ToString(),
                    Text = c.Name
                });

            return new SelectList(items, "Value", "Text");
        }
    }
}

我的 Controller :

public class HomeController : Controller
{
    private readonly Repository _repository = ObjectFactory.GetRepositoryInstance();

    public ActionResult Index()
    {
        var model = new TestModel
        {
            Categories = _repository.Categories.Select(c => new CategoryModel
            {
                Id = c.Id,
                Name = c.Name
            })
        };
        return View(model);
    }

    [HttpPost]
    public ActionResult Index(TestModel model)
    {
        if (ModelState.IsValid)
        {
            return RedirectToAction("Succes");
        }

        model.Categories = _repository.Categories.Select(c => new CategoryModel
        {
            Id = c.Id,
            Name = c.Name
        });
        return View(model);
    }

    public ActionResult Succes()
    {
        return View();
    }
}

我想删除重复的类别获取和映射，基本上是这段代码:

.Categories = _repository.Categories.Select(c => new CategoryModel
            {
                Id = c.Id,
                Name = c.Name
            })

来自 Controller 。此外，我想删除 ModelState 有效性检查，我只想在 ModelState.IsValid 时执行操作，以保持 Controller 代码尽可能干净。到目前为止，我有以下用于删除 ModelState 有效性检查的解决方案:

创建自定义 ValidateModelAttribute

public class ValidateModelAttribute : ActionFilterAttribute
{
    public override void OnActionExecuting(ActionExecutingContext filterContext)
    {
        var viewData = filterContext.Controller.ViewData;

        if(viewData.ModelState.IsValid) return;

        viewData.Model = filterContext.ActionParameters["model"];
        filterContext.Result = new ViewResult
        {
            ViewData = viewData,
        };
     }
 }

现在模型在 Action 执行之前得到验证。如果出现验证错误，我们将使用相同的 View 和最近发布的相同模型。因此， Controller POST 操作如下所示:

[HttpPost]
[ValidateModelAttribute]
public ActionResult Index(TestModel model)
{
    // Do some important stuff with posted data
    return RedirectToAction("Success");
}

这很好，但是现在我的 TestModel 的 Categories 属性是空的，因为我必须从数据库中获取类别，并相应地映射它们。 那么可以将我的 View 模型修改成这样吗:

public class TestModel
{
    private readonly Repository _repository = ObjectFactory.GetRepositoryInstance();

    ...

    public int SelectedCategory { get; set; }
    public IEnumerable<CategoryModel> Categories {
        get
        {
            return _repository.Categories.Select(c => new CategoryModel
            {
                Id = c.Id,
                Name = c.Name
            });
        }
    }

    ...
}

这将使我们拥有非常干净的 Controller ，但它不会导致某种性能或架构问题吗？它不会破坏 View 模型的单一职责原则吗？ ViewModel 是否应该负责获取它需要的数据？

最佳答案

这不对。 View 模型应该主要是由服务/查询甚至 Controller 填充的 DTO。以前的版本没有问题，你的 Controller 只是几行代码。

但是您的存储库并不是真正的存储库，它是一个 ORM。一个适当的存储库(这里它只是一些查询对象)将直接返回 View 模型的类别列表。

关于您的自动验证属性，不要重新发明轮子，其他人(在本例中是我)做到了 before .

关于c# - 在 View 模型中使用存储库可以吗？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/31516228/