編集時のASP.NET MVCリモート検証ロジック

Question

MVCにモデルとアクションメソッドがあります。

public class employee 
{ 
    [Key] 
    public int id { get; set; } 

    [Required] 
    public string employeeID { get; set; } 

    [Required] 
    [Remote("doesCnicExist", "employee", AdditionalFields = "employeeID", HttpMethod = "POST", ErrorMessage = "A user with this cnic already exists. Please enter a different cnic.")] 
    public string cnic { get; set; } 
} 

[HttpPost] 
    public JsonResult doesCnicExist(string employeeID, string cnic) 
    { 
     var empList = hc.employee.ToList(); 
     bool flag = false; 
     foreach (employee e in empList) 
     { 
      if ((employeeID == e.employeeID) && (cnic == e.cnic)) 
      { 
       flag = true; 
      } 
     } 
     return Json(flag == false); 
    } 

Create()アクションでは、それは素晴らしい動作します。しかし、Edit()アクションでは、プログラムはすでにcnicが存在するとみなします。そして、employeeを同じcnicで更新することはできません。私は追加のemployeeIDフィールドを使用して、編集中に従業員オブジェクトの一意性をどのように得ることができるのか分かりませんか？

Answer 1

idは一意の識別子なので、doesCnicExist()メソッドに渡す必要があります。そして、idが既に存在する行を無視するようにロジックを変更できます。 RemoteAttributeのみクライアント側の検証で、検証が常にサーバー上で実行する必要がありますので、私は別のメソッドにロジックを分離している

[HttpPost] 
public JsonResult doesCnicExist(string cnic, int id) 
{ 
    return Json(IsUnique(cnic, id)); 
} 

private bool IsUnique(string cnic, int id) 
{ 
    if (id == 0) // its a new object 
    { 
    return !hc.employee.Any(x => x.cnic == cnic); 
    } 
    else // its an existing object so exclude existing objects with the id 
    { 
    return !hc.employee.Any(x => x.cnic == cnic && x.id != id); 
    } 
} 

ノートに

public class employee 
{ 
    [Key] 
    public int id { get; set; } 
    .... 
    [Required] 
    [Remote("doesCnicExist", "employee", AdditionalFields = "id", HttpMethod = "POST", ErrorMessage = "A user with this cnic already exists. Please enter a different cnic.")] 
    public string cnic { get; set; } 
} 

とコントローラメソッドにモデルを変更（クライアント側の検証は素晴らしいボーナスだと考えられますが、悪意のあるユーザーは簡単にそれをバイパスできます）。別の方法では、Create()とEdit()のPOSTメソッドでその妥当性を検証し、データベースに保存する際にスローされる可能性のある例外を防ぐこともできます。

RemoteAttributeはクライアント側（UI）属性なので、データモデルには適用しないでください。view modelのビューモデルプロパティに適用されているベストプラクティスに従ってください。データモデルクラスとプロパティ名（つまりPascalCase）には通常の命名規則を使用することをおすすめします。