コントローラまたはモデルで処理されるフォームデータ：どちらが優れていますか？

Question

どちらが優れているか、フォームデータを処理するために推奨されているのか不思議です。

// Controller 
// validation etc 
$data = array('name'=>$this->input->post('name'), '... etc'); 
$this->user_model->insert($data); 

// Model 
function insert($data){ 
    $this->db->insert('users',$data); 
} 

または

// Controller 
// validation etc 
$this->user_model->insert(); 

// Model 
function insert($data){ 
    $data = array('name'=>$this->input->post('name'), '... etc'); 
    $this->db->insert('users',$data); 
} 

私は現時点では第二の方法を使用しています。挿入に問題はありませんが、更新にはそれほど適していないと言わなければなりません。テーブルの特定のフィールドだけを更新する新しいメソッドを作成する必要があります。私は最初の方法に変更することを考えています。私は誰もが各方法の賛否両論を明確にするのを助けることができるかどうか疑問に思っています。

ありがとうございます！

Answer 1

は、私は同意する最小限のロジックで私が唯一の間のパスとして、あなたのコントローラを使用して作成しようとしていたポイントは、そのモデルとライブラリは重い処理の大部分を実行するはずですが、モデルとライブラリのメソッドはできるだけ再利用可能でなければならず、コントローラはトラフィックディレクターとして機能し、コールするモデルとそれらに送信するデータを決定する必要があります。

あなたの例では、前者が理にかなっています。これをこのように考えてみましょう。ユーザーをフォームの投稿以外の方法で追加することはできないのでしょうか？

別のオプションあなたは両方の長所をしたい場合は、モデル内のラッパー関数を作成します。もちろん

// Controller 
// validation etc 
$this->user_model->insert_from_post(); 

// Model 
function insert_from_post(){ 
    $data = array('name'=>$this->input->post('name'), '... etc'); 
    $this->insert('users',$data); 
} 

function insert($data){ 
    $this->db->insert('users',$data); 
} 

この例では、それが過剰に思えます。しかし、より複雑なアプリケーションでは、結果を確立し、予測し、テストした機能を再利用できるというメリットがすぐに得られます。

Answer 2

コントローラを最小限に抑えてください。ビュー/モデルとの間でデータをやりとりするためにコントローラを使用することを検討してください。また、私はあなたのモデル、HTMLエンコーディング、日付解析などの作業を重視することをお勧めします。ビューで行うべき唯一の作業は、変数を吐き出すかループすることです。

Codeigniter 2.1ではすべてのform_validation設定アプリケーション/設定/ form_validation.php

内部の一つのファイルにだから、本当にすべてのコントローラが行う必要がある

if ($this->form_validation->run('auth/register')) {/** interact with your models or libraries (keep the heavy stuff outside **/)}else{//show form} 

が編集言っている：@catfish

コントローラを最小限に保つことは、「コントローラに何も入れない」と同じではありません。上で気がついたら、コントローラにフォームの検証を含めましたが、コントローラからform_validationの設定を除外することができます。

最小コントローラーはそうそうです。

class someclass extends CI_Controller 
{ 
//The form validation rules go inside the application/config/form_validation.php 
//and is called automatically once the class/method is init 

public function __construct() 
{ 
    parent::__construct(); 
} 

public function form() 
{ 
    $this->load->view('templates/public', array('content'=>'form_view.php')); 
} 

public function validate() 
{ 
     if($this->form_validation->run('someclass/validate'))// optionally pass the config key 
     { 
      if(Model::method($this->input->post())) 
      { 
       //do something positive 
      }else 
      { 
      $this->session->set_flashdata('error', $this->lang->line('some_error')); 
      redirect('/'); 
      } 
     }else 
     { 
     $this->form();//show form again 
     } 
} 

public function work_with_model() 
{ 
    $this->load->view('_index', array(
     'model_data'=> model::method()// do all looping, html encoding, time/date parsing etc and only send back a string or an array/object ready for output 
    )); 
} 

} 

アプリケーション/設定/ form_validation.php

$config = array('someclass/validate'=>array(
    array('field'=>'Field', 'label'=>'Label', 'rules'=>'trim|required|xss_clean|serialize') 
));