ドメインモデルからビューモデルへの変換時にヌル参照の問題が発生する

Question

ストアドプロシージャから返されるデータがない場合、以下のコードはnull参照例外をスローします。データが存在する場合、メソッドは正常に実行されます。

私は以下のコードで何か問題がありますか？モデルからオブジェクトを作成する必要はありますか？

public PersonVM GetStaff() 
{ 
    PersonDM personDM = _Repo.GetStaff(); 
    PersonVM personVM = PersonVM.ToViewModel(personDM); 
    return personVM; 
} 

public class PersonDM 
{ 
    public int RoleID { get; set; } 
    public string Name { get; set; } 
} 

public class PersonVM 
{ 
    public int RoleID { get; set; } 
    public string Name { get; set; } 

    public static PersonVM ToViewModel(PersonDM model) 
    { 
     return new PersonVM 
     { 
      RoleID = model.RoleID, 
      Name = model.Name 
     }; 
    } 

    public PersonDM ToEntityModel() 
    { 
     return new PersonDM 
     { 
      RoleID=this.=RoleID, 
      Name = this.Name, 
     } 
    } 
} 

SP personDMから返されるデータがない場合、NULLになります。 NULLを返さずにヌル値で埋める必要があります。達成することは可能ですか？

私は以下のコードでList<PersonVM>を返している方法と同じことをしました。データが存在しない場合は、VMにNULL値が設定されます。 _Repo.GetStaff()がnullを返すされているのでpersonDMがnullであると仮定すると、タイプPersonVM

public List<PersonVM> GetPartyByPartyRelationship(int partyRoleId, int partyRelationshipTypeId) 
    { 
     List<PersonDM> personDMList = _partyManagerRepo.GetPartyByPartyRelationship(partyRoleId, partyRelationshipTypeId); 
     List<PersonVM> personVMList = new List<PersonVM>(); 
     foreach (PersonDM personDM in personDMList) 
     { 
      personVMList.Add(PersonVM.ToViewModel(personDM)); 
     } 

     return personVMList; 
    } 

Answer 1

を返すメソッドに以下のコードを適用することができますどのように、あなたがしているようNullReferenceExceptionがスローされていることを驚くべきことではありませんnull参照でToViewModel()のオブジェクトプロパティにアクセスしようとしています。

ヌルチェックを追加すると、GetStaff()またはToViewModel()のいずれかが適切に処理されます。あるいは、ヌル・progagating演算子を利用するためにあなたのToViewModel()方法を変更 - 更新

public static PersonVM ToViewModel(PersonDM model) 
{ 
    if (model == null) 
     return new PersonVM(); 

    return new PersonVM 
    { 
     RoleID = model.RoleID, 
     Name = model.Name 
    }; 
} 

：あなたの更新に基づいて、あなたはあなたがnullのチェックを行うことができ、ヌルのプロパティでのviewmodelを返すようにしたいと言います：

public static PersonVM ToViewModel(PersonDM model) 
{ 
    return new PersonVM 
    { 
     RoleID = model?.RoleID, 
     Name = model?.Name 
    }; 
}