从另一个ActionResult返回ActionResult

时间:2011-02-18 22:36:11

标签: asp.net-mvc controllers actionresult

说我有以下代码,在记事本中嘲笑,所以请原谅任何小错误:)

//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);
    }

我关注的是,为了回发模型状态无效的任何错误,我需要再次生成视图模型,以便可以创建页面上使用这些对象的任何元素(流派,艺术家等)。问题是它需要我将ActionResult中的一些代码复制并粘贴到ActionResult,似乎使我的代码不是很干。

有没有更好的方法来避免这样的重复代码?目前我只是将viewmodel需要的任何默认对象的生成移动到一个单独的方法和/或构造函数中,但是由于我必须生成整个控制器可能需要的所有对象,所以它有点乱。我希望我能做的就是将我的第二个Index Action指向第一个Index Action,并将其作为常规方法使用。虽然我尝试了几种不同的方法,但似乎无法将ActionResult返回到另一个ActionResult中。

有什么想法吗?

2 个答案:

答案 0 :(得分:6)

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

检查此答案以获取代码示例: ModelState.IsValid or Model.IsValid?

答案 1 :(得分: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