MVC Webサイトでフォーム提出を検証して処理する場所

Question

私はPHPベースのModel-View-Controller構造化Webサイトで作業しています。私は、モデルがビジネスロジックを処理し、HTML（または何でも）をユーザに提示し、コントローラがこれを容易にすることを理解する。私が立ち往生している場所は形である。 コントローラーにどのくらいの処理を加え、モデルをどのくらい入れますか？

ユーザーの最初の&の姓を更新しようとしているとします。私がしたいのは、AJAXを使って私のコントローラの1つにフォームを送信することです。私は、データを（もう一度）サーバ側で検証し、有効な場合はデータベースに保存してから、JSON応答を成功またはエラーとしてビューに返します。

コントローラに自分のユーザーモデルのインスタンスを作成するか、モデルの静的メソッドにコントローラをリレーするだけでよいですか？

オプション＃1：モデルのProcess POST

<form action="/user/edit-user-form-submit/" method="post"> 
    <input type="text" name="firstname"> 
    <input type="text" name="lastname"> 
    <button type="submit">Save</button> 
</form> 

<?php 
    class user 
    { 
     public function __construct($id){} // load user from database 
     public function set_firstname(){} // validate and set first name 
     public function set_lastname(){} // validate and set last name 
     public function save_to_database(){} // save object fields to database 

     public static function save_data_from_post() 
     { 
      // Load the user 
      $user = new user($_POST['id']); 

      // Was the record found in the db? 
      if($user->exists) 
      { 
       // Try to set these fields 
       if(
        $user->set_firstname($_POST['firstname']) 
        and 
        $user->set_lastname($_POST['lastname']) 
       ) 
       { 
        // No errors, save to the dabase 
        $user->save_to_database(); 

        // Return success to view 
        echo json_encode(array('success' => true)); 
       } 
       else 
       { 
        // Error, data not valid! 
        echo json_encode(array('success' => false)); 
       } 
      } 
      else 
      { 
       // Error, user not found! 
       echo json_encode(array('success' => false)); 
      } 
     } 
    } 

    class user_controller extends controller 
    { 
     public function edit_user_form() 
     { 
      $view = new view('edit_user_form.php'); 
     } 
     public function edit_user_form_submit() 
     { 
      user::save_data_from_post(); 
     } 
    } 
?> 

オプション＃1：モデルのProcess POST

<form action="/user/edit-user-form-submit/" method="post"> 
    <input type="text" name="firstname"> 
    <input type="text" name="lastname"> 
    <button type="submit">Save</button> 
</form> 

<?php 
    class user 
    { 
     public function __construct($id){} // load user from database 
     public function set_firstname(){} // validate and set first name 
     public function set_lastname(){} // validate and set last name 
     public function save_to_database(){} // save object fields to database 
    } 

    class user_controller extends controller 
    { 
     public function edit_user_form() 
     { 
      $view = new view('edit_user_form.php'); 
     } 
     public function edit_user_form_submit() 
     { 
      // Load the user 
      $user = new user($_POST['id']); 

      // Was the record found in the db? 
      if($user->exists) 
      { 
       // Try to set these fields 
       if(
        $user->set_firstname($_POST['firstname']) 
        and 
        $user->set_lastname($_POST['lastname']) 
       ) 
       { 
        // No errors, save to the dabase 
        $user->save_to_database(); 

        // Return success to view 
        echo json_encode(array('success' => true)); 
       } 
       else 
       { 
        // Error, data not valid! 
        echo json_encode(array('success' => false)); 
       } 
      } 
      else 
      { 
       // Error, user not found! 
       echo json_encode(array('success' => false)); 
      } 
     } 
    } 
?> 

2ここでは、これは仕事ができるかの2つの例であります例はまったく同じことをします、私はそれを実現します。しかし、これを行う正しい方法と間違った方法がありますか？私はスキニーコントローラと脂肪モデルについて多くのことを読んできましたが、オプション＃1はどこから来ましたか。どのようにこれを扱っていますか？ありがとう、そして長い質問のために申し訳ありません！

Answer 1

すぐに言えば、ではなくのいずれかの方法を使用することができます。少し変更する必要があります。

これを考慮してください：モデルは実際にポスト、ゲット、その他のものについて「認識」していません。彼らは、あなたのビジネスに関係する事柄を、あなたの場合にはユーザーとして知る必要があります。

アプローチ＃1を使用することができますが、ではなく、モデルから直接ポスト変数にアクセスする必要があります。代わりに、関数をパラメータの配列にして、ユーザーを作成するために使用します。

このようにすれば、シェルスクリプトなどでコードを簡単に再利用することができます。$_POSTなどはありません。

コントローラで2番目のアプローチはより冗長ですが、これもできることです。しかし、おそらくスタイルのより良いアプローチは、 "サービスクラス"を使用することです。このサービスにはメソッドがあります。たとえば、 "createUserFromArray"と言うと、配列を受け取り、ユーザーを返します。再度、このメソッドをパラメータとして$_POSTに渡します。変更した＃1の関数に渡す方法と似ています。

コントローラだけが入力を直接処理する必要があります。これは、コントローラがリクエストを処理し、ポストについて知ることができるからです。

tl; drあなたのモデルでは、直接$_POSTのような超球体を使用しないでください。

Answer 2

私の視点から見ると、仕事でやっているところでは、モデルは渡されるデータの検証とフィルタリングを処理しますが、コントローラを使用してモデル内のデータをプッシュします。

上記のコメントに記載されているモデルは、$ _POSTまたは$ _GETについてはコントローラが処理しなければならないユーザー入力であることを知る必要はありません。一方、モデルは、プロジェクトの異なるコード部分でデータの検証を何度も何度も繰り返したくないため、モデルに渡されたデータの検証をすべて処理する必要があります。

Answer 3

MVCデザインパターンの大きな理由は、それが分離の懸念を維持する良い方法だということです。ビューはモデルを知らないで、その逆もあります。コントローラーは、モデルとビューの間を仲介する一種の交通警察官としてのみ存在します。したがって、コントローラはビ​​ューからデータを取得し、ビューがどのように実装されているかを（HTMLフォームを介して）知らなくてもモデルがデータを理解し、モデルに渡すことができるように最小限の処理を行い、データを保持することができます。

これにより、複数のコントローラ間でアイテムを保存するコードを複製せずに、アイテムをHTMLフォーム以外の方法で作成/保存/永続化する必要がある場合に、モデルを再利用できます。

更新：私は妥当性検査について言及しなかった。コントローラはデータを保持するのと同じ行に沿って、必要なデータの正確なフォーマットを知っているModelであるため、コントローラはデータを取得して検証のためにModelに渡す必要があります。モデルが無効な場合、モデルが無効な場合にモデルが例外をスローすることで、検証と永続性を組み合わせることができます。コントローラは必要に応じてキャッチして対処できます（JSONエラー応答のレンダリングなど）。