4

假设我有以下代码，在记事本中模拟，所以请原谅任何小错误:)

//Default page
public ActionResult Index()
    {
        var musicViewModel
        {
         Albums = GetTopSellingAlbums(5),
         Genres = GetTopGenres(5),
         Artists = GetTopArtists(5)
         };

        return View(musicViewModel);
    }

[HttpPost]
public ActionResult Index(MusicViewModel musicViewModel)
    {

        //For the example, pretend I have a class called musicStoreSubmission in my
        //viewmodel which holds a few different fields the user fills out.

        if(ModelState.IsValid)
        {
            //Do some actions based on the user submitting a form
        }

        //Else, refresh page with errors in Modelstate.
        var musicViewModel
        {
         Albums = GetTopSellingAlbums(5),
         Genres = GetTopGenres(5),
         Artists = GetTopArtists(5)
         };

        return View(musicViewModel);
    }

我担心的是，为了回发任何导致 ModelState 无效的错误，我需要再次生成视图模型，以便可以创建页面上使用这些对象的任何元素（流派、艺术家等）。问题是它需要我将一些代码从 ActionResult 复制并粘贴到 ActionResult，这似乎使我的代码不是很干燥。

有没有更好的方法来避免这样的重复代码？目前，我只是将视图模型所需的任何默认对象的生成移至单独的方法和/或构造函数中，但这有点混乱，因为我必须生成整个控制器可能需要的所有对象。我希望我能做的是将我的第二个索引操作指向第一个索引操作，并将其用作常规方法。不过，我尝试了几种不同的方法，但似乎无法将 ActionResult 返回到另一个 ActionResult。

有什么想法吗？

4

2 回答 2

6

我建议应用Post/Redirect/Get模式。它非常适合 MVC Web 应用程序。

检查代码示例的此答案： ModelState.IsValid 或 Model.IsValid？

于 2011-02-18T23:01:50.027 回答
4

您可以像这样返回另一个 ActionResult 方法：

[HttpPost]
public ActionResult Index(MusicViewModel musicViewModel)
{
    if(ModelState.IsValid)
    {
        //Do some actions based on the user submitting a form
    }
    return MyAction();
}

或者您可以将发布的模型传递回 ViewResult

[HttpPost]
public ActionResult Index(MusicViewModel musicViewModel)
{
    if(ModelState.IsValid)
    {
        //Do some actions based on the user submitting a form
    }
    return View(musicViewModel);
}

第二种方法更好，因为您不重建 ViewModel

于 2011-02-18T22:48:07.477 回答