重新使用控制器内的ActionResult代码

Question

如果我有以下代码（编辑：对不起，如果我不清楚，我想封装以下（忘记它的调用视图），以便我可以做其他东西ActionResult）：

public ActionResult ModelBased(string[] items, PostedItems postedItems) { 
     var model = new ItemsViewModel(); 
     var selectedItems = new List<Item>(); 
     var postedItemIDs = new string[0]; 
     if (postedItems == null) postedItems = new PostedItems(); 

     if (items!= null && items.Any()) { 
      postedCityIDs = items; 
      postedItems.ItemIDs = items; 
     } 

     if (postedItems.ItemIDs != null && postedItems.ItemIDs.Any()) { 
      postedItemIDs = postedIems.ItemIDs; 
      model.WasPosted = true; 
     } 

     if (postedItemIDs.Any()) 
      selectedItems = ItemRepository.GetAll() 
       .Where(x => postedItemIDs.Any(s => x.Id.ToString().Equals(s))).ToList(); 

     model.AvailableItems = ItemRepository.GetAll(); 
     model.SelectedItems = selectedItems; 
     model.PostedItems = postedItems; 

     return View(model); 
    } 

我怎么可能会重用它在我的控制器不同的操作，而无需复制/粘贴。我试着用代码做一个私有方法。但我被困在：

1. 要么叫错了一个操作方法中：private void Item (Item item) {//copied code from above}然后调用在行动Item(item);;或

2. 它与(string[] items, PostedItems postedItems)有关，我做错了;或

3. 一些完全不同的东西我做得不对。

任何示例将不胜感激。

编辑：上面的代码与CheckBoxList一起使用。这是一个特定的CheckBoxList。但我希望能够在其他视图中使用它，而无需将代码复制/粘贴到其他ActionResults。只是拨打ActionResult将无法​​正常工作，因为我打算做其他事情。特别是，我必须在每个ActionResult向导，例如代码：

if ((nextButton != null) && ModelState.IsValid) 
    return RedirectToAction("EMailConfirm"); 
return View("EMail/BasicDetails", myData); 

其返回的具体意见，所以打电话只是ActionResult都不行，除非我失去了一些东西。

Answer 1

return View(model);试图找到原始操作的视图。

指定return View("ModelBased", model);总是呈现命名"ModelBased"

---

public void SomeAction(string[] items, PostedItems postedItems) 
{ 
    // Modify the data as your like 
    return ModelBased(string[] items, PostedItems postedItems); 
} 

public void SomeOtherAction(string[] items, PostedItems postedItems) 
{ 
    // Modify the data as your like 
    return ModelBased(string[] items, PostedItems postedItems); 
} 

private ActionResult ModelBased(string[] items, PostedItems postedItems) { 
    var model = new ItemsViewModel(); 
    var selectedItems = new List<Item>(); 
    var postedItemIDs = new string[0]; 
    if (postedItems == null) postedItems = new PostedItems(); 

    if (items!= null && items.Any()) { 
     postedCityIDs = items; 
     postedItems.ItemIDs = items; 
    } 

    if (postedItems.ItemIDs != null && postedItems.ItemIDs.Any()) { 
     postedItemIDs = postedIems.ItemIDs; 
     model.WasPosted = true; 
    } 

    if (postedItemIDs.Any()) 
     selectedItems = ItemRepository.GetAll() 
      .Where(x => postedItemIDs.Any(s => x.Id.ToString().Equals(s))).ToList(); 

    model.AvailableItems = ItemRepository.GetAll(); 
    model.SelectedItems = selectedItems; 
    model.PostedItems = postedItems; 

    return View(model); 
} 

Answer 2

你举的例子是不清楚的观点，但是，我会正常运行常见的功能集成到一个单独的方法，用[无为]属性标记它。例如。

[NonAction] 
protected UserInfo GetUserInfo(string username) 
{ 
    // Return relevant data 
} 

然后我会在您的操作方法中调用GetUserInfo。

编辑：

你需要考虑的部分意见。您可以将局部视图看作是可以在多个页面上重复使用的控件。例如，我可以在一个局部视图中放置一个登录控件，并将它在多个页面上复制。这将促进代码的重用性。

我不能给你的例子，因为我还没有这样做了一段时间，但你必须做到以下几点：

1. 相反的return View();你必须return PartialView("_NameOfYourPartialView", viewModel);
2. 修改您的视图，因此它不再是视图，而是局部视图。

你需要做一些阅读并为自己尝试一下。

好运

Answer 3

你可以调用从返回ActionResult另一个动作这个动作。

public ActionResult OtherAction() 
{ 
    return ModelBased(items, postedItems); 
} 

另外，为什么private void？你真的想重用哪部分？如果需要Item并返回ItemsViewModel，则应该是private ItemsViewModel - 取决于您想要重复使用的部分。 void不返回任何内容。