funkcja includowania bo nie potrafię symfony :)

pytanie zadane 9 lipca 2015 w PHP przez makoso Mądrala (7,380 p.)

co sądzicie o takim zastosowaniu przełączania stron? taki mój 'kontroler' :)

class funkcje {

    function strona($strona){
        $x=htmlspecialchars($strona);
        $x=htmlentities($x);
        $x=str_replace(' ', "", $x);
        $l=strlen($strona);
        if($l>15)
        {$x='strona';}
        return $x;
    }

}

$funkcja    =NEW funkcje();

@include('/strona/' . $funkcja->strona($_GET['strona']) . '.php');

i mam też pytanie czy mogę złapac jakoś error od include bo walnałbym ifa jeżeli nie znaleziono pliku przekierować na stronę główną

2 odpowiedzi

odpowiedź 9 lipca 2015 przez Comandeer Guru (601,590 p.)
wybrane 9 lipca 2015 przez efiku

Najlepsza

To nie jest klasa, więc nie rób z tego klasy. To jest po prostu funkcja.

Nazwy plików zabezpieczaj basename, bo tutaj jest więcej miejsca do popisu niż przy "XSS".

Co do błędu z nieistniejącym plikiem: sprawdzaj czy istnieje przy pomocy file_exists i dopiero wówczas includuj; w innym wypadku rzucaj błąd.

Ogólnie polecam zajrzeć tutaj: http://www.forumweb.pl/poczatkujacy-webmaster/jak-ulatwic-sobie-zycie-menu-raz-zeby-bylo-caly-czas,271

komentarz 9 lipca 2015 przez makoso Mądrala (7,380 p.)

tak jak Efik i Ty napisaliście file_exists spełniło to co chiałem co do basename nie bardzo rozumiem tutaj jego zastosowanie z tego co patrzyłem w manualu

Dzięki Twoja odpowiedź też rozwiązałaby problem :)

komentarz 9 lipca 2015 przez efiku Szeryf (75,160 p.)

Sorry, ale to Comandeerowi się należy!

odpowiedź 9 lipca 2015 przez efiku Szeryf (75,160 p.)

i mam też pytanie czy mogę złapac jakoś error od include bo walnałbym ifa jeżeli nie znaleziono pliku przekierować na stronę główną

$funkcja    =new funkcje();
 $strona = '/strona/' . $funkcja->strona($_GET['strona']) . '.php');
if(file_exists($strona)){
 include($strona);
} else {
  throw  itd///
}

komentarz 9 lipca 2015 przez makoso Mądrala (7,380 p.)

dzięki działa tak jak chciałem :) szkoda tylko że jak pracuję na localhost to muszę podać twardo scieżkę folderu -.-
ale wszystko miód malina :)

PS. zostawiłeś zamknięcie nawiasu zbędne w $strona :P