정지 ASP.NET MVC 모델 클래스의 모든 게터를 호출

Question

당신은 바인딩 모델 중에이이 NullReferenceException와 충돌합니다 모델 클래스에 다음과 같은 두 가지 속성을 정의하는 경우 :

 public Customer Customer { get; private set; } //set in the action method 
     public bool Name => Customer.Name; 

이것은 Customer이 여전히 있기 때문에 모델 바인딩 중에는 null이고 ASP.NET MVC에서는 Name에 대한 getter를 호출합니다.

스택은 다음과 같습니다이 모델 검증과 같은 스택에서

System.ComponentModel.ReflectPropertyDescriptor.GetValue(Object component) +525 
    System.Web.Mvc.ModelMetadata.get_Model() +34 
    System.Web.Mvc.DataAnnotationsModelValidator.Validate(Object container) +151 
    System.Web.Mvc.<Validate>d__1.MoveNext() +387 
    System.Web.Mvc.DefaultModelBinder.OnModelUpdated(ControllerContext controllerContext, ModelBindingContext bindingContext) +163 
    System.Web.Mvc.DefaultModelBinder.BindComplexElementalModel(ControllerContext controllerContext, ModelBindingContext bindingContext, Object model) +83 
    System.Web.Mvc.DefaultModelBinder.BindComplexModel(ControllerContext controllerContext, ModelBindingContext bindingContext) +1754 
    System.Web.Mvc.ControllerActionInvoker.GetParameterValue(ControllerContext controllerContext, ParameterDescriptor parameterDescriptor) +460 
    System.Web.Mvc.ControllerActionInvoker.GetParameterValues(ControllerContext controllerContext, ActionDescriptor actionDescriptor) +137 
    System.Web.Mvc.ControllerActionInvoker.InvokeAction(ControllerContext controllerContext, String actionName) +982 
    System.Web.Mvc.<>c__DisplayClass22.<BeginExecuteCore>b__1e() +39 
    System.Web.Mvc.Async.AsyncResultWrapper.<.cctor>b__0(IAsyncResult asyncResult, Action action) +21 
    System.Web.Mvc.Controller.EndExecuteCore(IAsyncResult asyncResult) +53 
    System.Web.Mvc.Async.WrappedAsyncVoid`1.CallEndDelegate(IAsyncResult asyncResult) +36 
    System.Web.Mvc.Controller.EndExecute(IAsyncResult asyncResult) +38 
    System.Web.Mvc.MvcHandler.<BeginProcessRequest>b__5(IAsyncResult asyncResult, ProcessRequestState innerState) +44 
    System.Web.Mvc.Async.WrappedAsyncVoid`1.CallEndDelegate(IAsyncResult asyncResult) +65 
    System.Web.Mvc.MvcHandler.EndProcessRequest(IAsyncResult asyncResult) +38 
    System.Web.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() +399 
    System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +137 

이 모든 게터를 조회한다. 모델 유효성 검사를 사용하지 않습니다.

이 상황을 어떻게 처리 할 수 ​​있습니까? ASP.NET MVC가 (명백한) 이유없이 모든 getter를 호출하지 않도록 할 수 있습니까?

Answer 1

모델 바인더 new가 모델의 인스턴스를 모델링 한 경우 모델의 속성을 반영하여 이름이 일치하는 항목을 FormCollection의 값으로 검색하는 것이 좋습니다. 무슨 일이 일어나고있는 건 위험한 Name 소품이 불려지면 Customer 소품이 널려서 NullRef가 null이라는 것입니다.

.NET에서 이러한 속성을 확인하는 순서는 실제로 무작위 일 수는 없지만 코드를 그대로 처리하여 코드가 크게 향상됩니다. null로 검사하지 않는 한 기본적으로 nullable 인 클래스에서 메서드/소품을 직접 호출하는 것은 끔찍한 생각입니다. 여기에는 두 가지 옵션이 있습니다. (1) 생성자가 "Customer"속성을 초기화하도록 Model 클래스를 다시 디자인하거나, (2) "Name"메서드에서 null 체크를 추가하십시오.

여기에 가장 쉬운 방법입니다 그냥 널 (null) 검사 당신이 그것을 잡을 때 그것을 :

public bool Name => Customer?.Name ?? false; 

이 당신이 서로 연결 널 (NULL) 소품이있는 모델을 가지고있는 근본적인 문제를 해결하지 않습니다. 모델의 생성자가 모델 바인딩을 망칠 까봐 걱정하지 마십시오. Model Binder는 (1) 모델 클래스를 초기화 한 다음 (2) 수화를 시도합니다. 따라서 Model 클래스의 생성자에서 Customer 클래스/prop를 초기화해도 고객 필드에 대한 UI 필드 매핑에 영향을주지 않습니다.