Codeigneter. Корректен ли код?

Question

[wowmister]

Форма:

<?php echo validation_errors(); ?>
			<?php echo form_open('master/add_user'); ?>
			<p>username: <input type="text" name="username" value="<?php echo set_value('username'); ?>" /></p>
			<p>password: <input type="text" name="password" value="<?php echo set_value('password'); ?>" /></p>
			<div><input type="submit" value="go" /></div>
			</form>

Контроллер:

<?php
class Master extends CI_Controller {
	
	function dashboard()
	{
		$this->load->view('master/dashboardview');
	}
	
	function add_user()
	{
		$this->form_validation->set_rules('username', 'username', 'required');
		$this->form_validation->set_rules('password', 'password', 'required');
		if ($this->form_validation->run() === FALSE)
		{
			$this->load->view('master/dashboardview');
		}
		else
		{
		$this->load->model('user_model');
		$this->user_model->add_user();
		$this->load->view('master/success');
	    }
	}
}
?>

Модель:

<?php
class User_model extends CI_Model {

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

    function add_user()
    {
		$userInfo = array(
				'username' => $_POST['username'],
				'password' => $_POST['password'],
			);
		$this->db->insert('users', $userInfo);
    }

}
?>

хелперы и библиотеки autoload

upd
Есть ли разница и какая

'username' => $_POST['username'],
или
'username' => $this->input->post('username'),

Answer 1

[Антон Шаманов]

А почему элементы формы реализованы не хелперами? Работа с данными ведется в модели. Слышал про вызов методов цепочкой $obj->method1()->method2()?

Comments

[wowmister]

Не слышал. Начать изучение решил с написания админки для себя, т.е. с дела, разбираюсь по ходу.
почему не в моделях это второй вопрос(( вчера всю ночь залипал на контроллер модель, все перепуталось((

Answer 2

[krypt3r]

Это

$this->load->helper('form');
$this->load->library('form_validation');

я бы вынес в конструктор. $this->form_validation->set_rules () я бы вынес в модель, так же как и описание полей. Да и валидация у вас неполная, имхо. Методу insert() тоже место в модели.

Comments

[wowmister]

Спасибо, как понимаю, то что теперь в первом посте, будет корректнее)

[krypt3r]

wowmister: Я бы передавал элементы $_POST в модель через параметр(ы), естественно, после валидации.

[krypt3r]

Есть ли разница и какая

Есть разница. $this->input->post('username') — это чтобы не городить портянку $var = (isset ($_POST['var']) ? $_POST['var'] : FALSE);