Недоступный код обнаружен C# beginner

Question

Привет, я новичок в C# и мог бы с кем-нибудь помочь узнать, что проблема здесь. Я продолжаю ошибаться, говоря, что недостижимый код был обнаружен в строке «_parcelService» внизу.

using System; 
using System.Collections.Generic; 
using System.Linq; 
using System.Web; 
using System.Web.Mvc; 
using ABC.Data; 
using ABC.Services; 
using ABC.Services.Service; 
namespace ABC.Controllers 
{ 
    public class AdminController : Controller 
    { 
     private ParcelService _parcelService; 
     public AdminController() 
     { 
      _parcelService = new ParcelService(); 
     } 

     // GET: Admin 
     public ActionResult Index() 
     { 
      return View(); 
     } 

     [HttpGet] 
     public ActionResult AddParcel() 
     { 
      return View(); 
     } 
     [HttpPost] 
     public ActionResult AddParcel(ParcelDetail parcel) 
     { 
      return View(); 

      { 
       _parcelService.AddParcel(parcel); 

       return RedirectToAction("Parcels", 
        new { TrackingID = parcel.TrackingID, controller = "Parcel" }); 
      } 
     } 

    } 
} 

Answer 1

На этой линии вы уже возвращал результат действия:

return View(); 

Там не может быть ничего после того, как заявление, что возвращается из функции. Так просто избавиться от этой линии:

[HttpPost] 
public ActionResult AddParcel(ParcelDetail parcel) 
{ 
    _parcelService.AddParcel(parcel); 

    return RedirectToAction("Parcels", 
     new { TrackingID = parcel.TrackingID, controller = "Parcel" }); 
} 

Или может быть, вы хотите применить некоторую логику, которая бы вновь вынести тот же вид, если проверка модели не удается, и в противном случае могли бы назвать к службе бэкэнда и перенаправления:

[HttpPost] 
public ActionResult AddParcel(ParcelDetail parcel) 
{ 
    if (!ModelState.IsValid) 
    { 
     // The model that was passed to this action was not valid 
     // => Redisplay the same view so that the user can correct 
     // the errors 
     return View(); 
    } 

    // At this stage we know that the model is valid and we can submit it 
    // for processing 
    _parcelService.AddParcel(parcel); 

    // redirect to a different action by returning the corresponding result 
    return RedirectToAction(
     "Parcels", 
     new { TrackingID = parcel.TrackingID, controller = "Parcel" } 
    ); 
} 

Answer 2

Ошибка, которую вы получаете, является самостоятельной пояснительной записью. Ниже

return View() 

Вы возвращаете представление. Поэтому каждый код после этого недоступен. Я считаю, что правильная версия кода является следующее:

[HttpPost] 
public ActionResult AddParcel(ParcelDetail parcel) 
{ 
    // Here you add the parcel 
    _parcelService.AddParcel(parcel); 

    // Then you make a redirect to a view that are visible all the parcels. 
    return RedirectToAction("Parcels", 
    new 
    { 
     TrackingID = parcel.TrackingID, 
     controller = "Parcel" 
    }); 
} 

Было бы хорошо, прежде чем позвонить в AddParcel для обеспечения некоторых валидаций, как если бы вы возражаете правомочно (проверьте Darin'n ответ). При условии, что parcel вы отправите эти проверки, затем вы должны позвонить AddParcel. В противном случае вы должны вернуть клиенту неправильный объект, который она разместила, и объяснить, почему это неправильно.

Answer 3

return View(); // problem here 

{ 
_parcelService.AddParcel(parcel); 

return RedirectToAction("Parcels", 
       new { TrackingID = parcel.TrackingID, controller = "Parcel" }); 
} 

Вот где ваша проблема. вы возвращаетесь, прежде чем ваша логика (код) когда-либо попадет. Кроме того, у вас есть 2 возвращается ...

Так, так как у вас есть return RedirectToAction вы можете просто удалить return View()

так решение:

[HttpPost] 
    public ActionResult AddParcel(ParcelDetail parcel) 
    { 
     // return View(); or just delete it.. since there is really no point to keep it commented out. 

     { 
      _parcelService.AddParcel(parcel); 

      return RedirectToAction("Parcels", 
       new { TrackingID = parcel.TrackingID, controller = "Parcel" }); 
     } 
    }