Witam chcialbym was prosić o szybką ocenę mojego kodu. Co przenieść do modelu jeżeli coś jest do przeniesienia a może któreś rozwiązania są do dupy...

[PHP] pobierz, plaintext 
<?php
 
/**
 * LOGIN CONNTROLER plik wyświetla dane o użytkownikach oraz opcje co można zrobić z użytkownikami.
 */
$Template = new Template();
 
 
@ $swap = $_GET['list_sort'];
@ $swaplvl = $_GET['lvl'];
 
switch ($swap) {
    case 'name':
        $sort = "username";
        break;
    case 'register':
        $sort = "id";
        break;
    case 'online':
        $sort = "logDate";
        break;
 
    default:
        $sort = "id";
        break;
}
 
switch ($swaplvl) {
    case 'all':
        $lvl = "level > 1";
        break;
    case 'admins':
        $lvl = "level > 1 AND level < 4";
        break;
    case 'users':
        $lvl = "level = 4";
        break;
 
    default:
        $lvl = "level > 1";
        break;
}
 
if (isset($options)) {
 
} else {
// PAGINATION
    DataManager::setQuery("SELECT count(*) FROM mod_members WHERE $lvl ");
    $count = DataManager::getArray();
    $pagination = new Pagination($count[0][0], 20, @$_GET['page']);
    $limit = $pagination->getData();
    $sql = "SELECT id, username, level, regDate, editDate, email 
            FROM mod_members WHERE $lvl ORDER BY $sort LIMIT " . $limit['limit'] . "";
}
 
// Odnosniki ktore maja zachowac w pamieci lvl.
include 'model/url.php';
$name = url('list_sort = name');
$register = url('list_sort = register');
$online = url('list_sort = online');
$Template->setArray('list_sort', array('name' => $name, 'register' => $register, 'online' => $online));
 
DataManager::getInstance();
DataManager::setQuery($sql);
$date = DataManager::getArray();
 
include 'lang/PL.php';
$Template->setArray('lang', $lang);
$Template->setArray('result', $date);
$Template->setArray('pagination', $pagination->getData());
$Template->load('view/v_users.php');
 
 
 
 
 
[PHP] pobierz, plaintext 

1. Controler? Ponglish na całego...
2. Oba switche powinny być w warstwie modelu.
3. Odbieranie danych z URL-a ($_GET) w sumie też powinno lecieć przez model.
4. Cały mechanizm stronicowania.
5. Cały DataManager powinien siedzieć w modelu.

Dziękuję za wskazowki jednak nie jestem pewien co do switcha co powinna zwracać więc funkcja i jak to ma działać (wyglądać) mogł bym prosić cie o mały pseudokod?

Tak ponglish jest to fakt staram sie z PL przechodzić na EN ale to nie jest proste i siła przyzwyczajenia bierze górę.

Co do Bazy danych rozumeim że sql w modelu i zwracam tylko to właściwie dane które zapisuje do $TEMPLETE celem dostępności dla vidoku.

Co do switcha

[PHP] pobierz, plaintext 
class Model_X {
 
    protected $_swap = array(
             'name' => 'username',
             'register' => 'id'
             );
 
   public function __construct(array $options = array()) {
 
        if(!empty($options)) {
 
             $this->setOptions($options);
        }
   }
 
   // to mozna wywalic do klasy rodzica
   protected function setOptions($options) {
 
        foreach($options as $key => $value) {
 
               $this->_swap[$key] = $value;
        }
   }
 
 
   public function hasSwap($key) {
 
       if(array_key_exists($key, $this->_swap)) {
 
            return true;
       }
 
       return false;
   }
 
   public function getSwap($key) {
 
        return $this->_swap[$key];
   }
}
 
// dane z jakiegoś configa
$config = array(
      'login' => 'userlogin',
      'id' => 'uniqid'
);
 
$m = new Model_X($config);
 
if($m -> hasSwap('id')) {
 
   echo $m -> getSwap('id');
}
[PHP] pobierz, plaintext 

Jak lecisz w OOP to od początku do końca Dobra rada - zobacz swoje do jakiegoś frameworka jak buduje się aplikacje

Dziękuję za rady jak popoprawiam prześlę ponownie