Skrypt includowania PHP Opcje

[Mlodycompany]

Witam. Wpadłem znów na genialny pomyśł opracowania skryptu includującego strone. Oto jego Skrypt:

[PHP] pobierz, plaintext 
<?php
$get_name = addslashes($_GET['name']);
$dir = "./";
if(!($fd = opendir($dir)))
exit("Bład");
 
if($get_name != "" && $get_name == $get_name){
while($file = readdir($fd)){
if($get_name == $file){
include($file);
}
}
}
?>
[PHP] pobierz, plaintext 

I teraz zaczoł się problem. A mianowicie. URL wygląda tak :index.php?name=home
zmienna $get_name = home
zmienna $file nie jest równa zmiennej $get_name ponieważ zmienna $file powinna przybrac wartosc home.php a home != home.php

I wpadłem na kolejny genialny pomysł. Oto on: dodac do $get_name koncówke .php i wtedy $get_name bedzie rowna $file. Problem jest w tym, że nie wiem jak można dodac tą końcówkę i czy wogóle się da to zrobic. Czy macie jakieś propozycje?(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

[marcio]

Heh nie znam sie na OOP wiec nie dam rady tego napisc ale idea jest obra moze pokazalbys jak ty to robisz bo do tej pory ja tak to robilem jak napisalem wczesniej i wszystko dobrze i bezpiecznie dzialalo ale moze bylo lamersko napisane (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

[Sedziwoj]

Heh nie znam sie na OOP wiec nie dam rady tego napisc ale idea jest obra moze pokazalbys jak ty to robisz bo do tej pory ja tak to robilem jak napisalem wczesniej i wszystko dobrze i bezpiecznie dzialalo ale moze bylo lamersko napisane (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Wiesz, skoro działa i wystarcza, to znaczy że dobrze, nie wiem dlaczego miało by być "lamersko".

Ale możesz coś w tym stylu:

[PHP] pobierz, plaintext 
<?php
interface IAction {
   public function run();
}
?>
[PHP] pobierz, plaintext 

a w głównym kontlorerze:

[PHP] pobierz, plaintext 
<?php
if( !ctype_alfnum( $_GET['action'] ){
   //Not found
}
$strClassName = 'prefix' . $_GET['action'];
if( class_exists( $strClassName ) ){
   $objAction = new $strClassName();
   if( $objAction instanceof IAction ){
	  $objAction->run();
   }
}
?>
[PHP] pobierz, plaintext 

W tym przypadku przydaje się autoloader dobrze napisany, aby ładował akcje sam.
Oczywiście co przekazujesz do akcji, czy akcja jest tylko interfejsem (lepiej nie), czy też jest abstrakcyjna, to potem ułatwia, masz pewne rzeczy z głowy. Np. jak chcesz autoryzacje, taką prostszą to może wystarczyć
UserFactory::gerAuth()->mustLevel( iUserAuth::ADMIN );

Lub po prostu odpowiednie dziedziczenie, po abstrakcji z autoryzacją i zaimplementowanie odpowiedniej metody...
Sposobów jest wiele, ale obiektowe podejście wiele rzeczy ułatwia jak się dobrze zrobi.