[CI] MVC Opcje

[Exodus]

Zacząłem swoją przygodę z CI. Czy dobrze myślę?

[PHP] pobierz, plaintext 
<?php
class Index extends Controller
{
 
/*
Konstruktor
*/
 
function __construct()
		{
			 parent::Controller();
 
			 $this->load->model('New_items'); 
		}
 
/*
Strona glowna
*/
 
function index()
		{
 
$this->response['new_movies'] = _getNewMovies();
 
$this->load->view('index', $this->response);
		}
 
}
 
/*
Najnowsze filmiki
*/
 
function _getNewMovies()
		{
 
$query = $this->get_new_movies('4');
 
 
 
 
		}
?>
[PHP] pobierz, plaintext 

chodzi mi o te funkcje _getNewMovies.

Np mam pobrac z bazy 3, najnowsze filmy, 4 cytaty itp. Czy dobrym pomyslem jest robienie nowych funkcji nie widocznych dla URL'a?

[Exodus]

Zacząłem już trochę robić (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) .

Czy ten model jest napisany, dobrze i optymalnie ?

[PHP] pobierz, plaintext 
<?php
class New_items extends Model
{
 
	/*
	Konstruktor
	*/
 
	function __construct(){
 
		parent::Model();
		$this->load->database();
 
	}
 
	/*
	Najnowsze filmiki
	*/
 
	function getNewMovies($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('add_date', 'DESC');
	$this->db->where('accepted', '1'); 
 
	return $this->db->get('video');
 
	}
 
	/*
	Najnowsze gry On-Line
	*/
 
	function getNewGames($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('add_date', 'DESC');
	$this->db->where('accepted', '1'); 
 
	return $this->db->get('games');	
 
	}
 
	/*
	Najnowsze zdjęcia z galerii
	*/
 
	function getNewPictures($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('add_date', 'DESC');
	$this->db->where('accepted', '1'); 
 
	return $this->db->get('gallery');	
 
	}
 
	/*
	Najnowszy cytat
	*/
 
	function getNewCytat(){
 
	$this->db->limit(1,0);
	$this->db->orderby('name',RAND());
	$this->db->where('accepted', '1'); 
 
	return $this->db->get('cytaty');	
 
	}
 
	/*
	Banner prawa strona
	*/
 
	function getReklama(){
 
	$this->db->limit(1,0);
	$this->db->orderby('id',RAND());
	$this->db->where('page', 'index'); 
 
	return $this->db->get('banners');	
 
	}
 
	/*
	Najaktywniejsi użytkownicy
	*/
 
	function getMostActiveUsers($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('points','DESC');
	 
	return $this->db->get('users');	
 
	}
 
	/*
	Najnowsi użytkownicy
	*/
 
	function getNewUsers($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('register_date','DESC');
	$this->db->where('activation', '0')	 
 
	return $this->db->get('users');	
 
	}
 
 
}
?>
[PHP] pobierz, plaintext 

[grzesiek_g]

function getNewUsers($how){

$this->db->limit($how,0);
$this->db->orderby('register_date','DESC');
$this->db->where('activation', '0')

return $this->db->get('users');

}

Można też tak:

[PHP] pobierz, plaintext 
<?php
function getNewUsers($how){
 
	$this->db->limit($how,0);
	$this->db->orderby('register_date','DESC');
	$this->db->where('activation', '0')	 
 
	if ($res = $this->db->get('users')->result()) 
		return $res;
	else
		return FALSE;	
	}
?>
[PHP] pobierz, plaintext 

Od razu mamy pobrany w modelu wynik zapytania poprzez funkcje result(), oraz sprawdzamy czy zapytanie się wykonało.

Ale może normanos jeszcze się wypowie na ten temat.