Panowie z racji tego, że nie ogarnąłem MVC to na chwilę obecną sobie odpuściłem i postanowiłem pisać stronę opartą o smarty.
Zresztą jest to mały serwis logowanie, rejestracja więc zrobię to po staremu - smarty.

[PHP] pobierz, plaintext 
 
<?php
error_reporting(E_ALL);
ob_start();
define( 'dostep', 1 );
header('Content-Type: text/html; charset=utf-8');
include ("tempp.php");
require_once 'libs/Smarty.class.php';
if (!file_exists('templates/'.$template_name.'/index.tpl')){
	echo 'Brak pliku index.tpl lub folderu. Główny plik pownien znajdować się w folderze templates/nazwa_szablonu';
	exit;
 
}else{
 
	$page=basename($_GET['w']);
	if (!empty($_GET['w']) and file_exists("controller/".$_GET['w'].".php")){
		switch($page) {
			//case ''.$page.'': $file = "controller/".$page.".php"; 
			case ''.$page.'' : include('controller/'.$page.'.php');
			break;
			default:  $file = "controller/home.php";}
	}else{
 
		$file='controller/home.php';
	}
 
	$smarty = new Smarty;
	$smarty -> assign('page', $page);
	$smarty -> assign('file', $file);
	$smarty -> display(''.$template_name.'/index.tpl');
 
}
ob_end_flush(); 
?>
 
[PHP] pobierz, plaintext 

Czy powyższy kod jest optymalnie napisany? Nie zawiera błędów?
Druga sprawa to nie wiem czemu nie wczytuje mi podstrony test.

index.php?w=test

plik test.tpl znajduje się w templates
plik test.php znajduje się w controller

1) Uzywanie smarty nie wyklucza posiadania MVC.
2) Uzywaj basename() do w
3) A niby czemu ma ci wczytac strone test? W kodzie co tu pokazales nigdzie tego nie robisz

@nospor zedytowałem:

$page=basename($_GET['w']);

czy teraz dobrze ?

rozumiem że to :

[PHP] pobierz, plaintext 
if (!empty($_GET['w']) and file_exists("controller/".$_GET['w'].".php"))
[PHP] pobierz, plaintext 

powinno wyglądać teraz tak:

[PHP] pobierz, plaintext 
if (!empty($page) and file_exists("controller/".$page.".php")){
[PHP] pobierz, plaintext 

Powiedzmy.

Po co ten case w tym IF?? Przeciez jak IF jest spelniony to page istnieje i po grzyba jeszcze to caseowac?

@nospor czyli twoim zdaniem to:

[PHP] pobierz, plaintext 
if (!empty($_GET['w']) and file_exists("controller/".$_GET['w'].".php")){
		switch($page) {
			//case ''.$page.'': $file = "controller/".$page.".php"; 
			case ''.$page.'' : include('controller/'.$page.'.php');
			break;
			default:  $file = "controller/home.php";
                 }
}else{
 
		$file='controller/home.php';
}
[PHP] pobierz, plaintext 

powinno wyglądać tak:

[PHP] pobierz, plaintext 
if (!empty($page) and file_exists("controller/".$page.".php")){
include('controller/'.$page.'.php');
}else{
 
		$file='controller/home.php';
}
[PHP] pobierz, plaintext 

może i faktycznie jest zoptymalizowany kod, ale czy bezpieczniejszy?

Tak, tak powinno wygladac.

A co niby twoim zdaniem dodaniem tam zbednego case w jaki sposob wplywa na bezpieczenstwo?

Czy bezpieczniej nie wiem, ale kiedyś uczono robić na case'ach :-)

Tak, ale na caseah miales porownywac zmienna do jakiejs wartosci... A ty tutaj porownywale czy zmienna $page rowna sie zmiennej $page... Ot mi bezpieczenstwo zmienna $page jest ZAWSZE rowna zmiennej $page.

@nospor postanowiłem powrócić do swojego "szkieletu" strony:-) i oprzeć go o smarty (na razie MVC na bok).

[PHP] pobierz, plaintext 
		$smarty = new Smarty;
		$smarty->template_dir = 'templates/';
		$smarty->compile_dir  = 'templates_c/';
 
		$page=basename($_GET['w']);
		if (!empty($page) and file_exists("controller/".$page.".php")){
			define($page, true);
			$file='controller/'.$page.'.php';
		}else{
 
			$page='controller/start.php';
		}
 
		$smarty -> assign('page', $page);
		$smarty -> assign('file', $file);
		$smarty -> display(''.$template_name.'/index.tpl');
[PHP] pobierz, plaintext 

zauważyłem jak dla mnie poważny błąd w tej składni - mianowicie, dołączam do index.tpl owy plik:

[PHP] pobierz, plaintext 
{include_php file="controller/test.php"}
[PHP] pobierz, plaintext 

który to zostaje przekazany do

[PHP] pobierz, plaintext 
$smarty -> display(''.$template_name.'/test.tpl');
[PHP] pobierz, plaintext 

a następnie wczytany w index. OK wszystko działa. Ale np jak wywołam sobie index.php?w=test <-- to mi ten test wczyta w index tak jakby 2 x.
Domyślam się , że pliki includowane mogę sobie w inny katalog wstawić, tak aby wykraczał poza "controller" ale czy i w jaki inny sposób zapobiegać takim "jajom"


Dodatkowo chciałbym się dowiedzieć od ekspertów po co się stosuje taki kod na początku pliku:

[PHP] pobierz, plaintext 
if (eregi("start.php", $_SERVER['PHP_SELF'])) {
    Header("Location: ./index.php");
    die();
}
[PHP] pobierz, plaintext 

Oczywiście wiem, że eregi już jest przestarzałe i można zastąpić to strstr();

nikt nie zna odpowiedzi?

Poprostu ktos przekierowuje start.php na index.php . A po grzyba to robi to sie jego spytac. Moze kiedys to start.php byl glownym plikiem a teraz juz tego nie chce