Page nav class, er dette smart?
jeg er så småt gået i gang med at programmere igen og skal lige opfriske, men er sådan på indgangs stadiet til oop programmering.Jeg har siddet og bikset denne class sammen til et site jeg er startet på, er meget minimalistisk endnu, men spørgsmålet er egentlig bare..... om det her er god kode???
class pageNav {
protected $action;
public function __construct() {
if(!isset($_GET['a'])) {
$this->action = '';
}
if(isset($_GET['a'])) {
$this->action = $_GET['a'];
}
return $this->action;
}
}
class whichPage extends pageNav {
public function __construct() {} // construct just for view
// static function to load page from ?a=xxx
static function loadPage() {
$page = new pageNav();
if ($page->action == '') { // if no page variable do nothing
}
else
// if page_xxx.php exists load it
if(file_exists('pages/page_' . $page->action . '.php')) {
include_once ('pages/page_' . $page->action . '.php');
}
// else tell user page doesnt exists
else {
echo 'Sorry, that page doesn\'t seem to exist';
}
}
}
ideen er at jeg aldrig skal include og så videre i min kodning af sitet...
jeg har så whichPage::loadPage(); på min index side
kalder derefter alle min (html) incudes for page_xxx.php i en html include mappe.....