Witam


Tworzę obecnie strone klanową, wraz z systemem logowania, własnym forum itp. jednocześnie ucząc się samemu PHP, MySQL itd.
Na stronie index.php mam kod, który wyświetla/ładuje (include) linki z menu lub z tych właśnie podstron.

Obecnie załadowany link wygląda tak:

www.strona.pl/index.php?inc=podstrona

To ładuje mi plik podstrona z folderu inc. Jeżeli chce zrobić odwołanie do innego folderu, a często chce aby nie robić sobie kilkanastu+ plików w jednym miejscu (odzielnie pliki logowania, odzielnie forum, odzielnie działy) muszę robić nową funkcje if(isset){}.


Kod który mam na stronie index.php wygląda tak:

[PHP] pobierz, plaintext 
<?PHP
 
if (isset($_GET['dia'])) {
    if (file_exists('dia/diablo.php'))
        include('dia/diablo.php');
 
}
 
		else {
 
if (isset($_GET['log'])) {
$strona=$_GET['log'];
    if (file_exists('log/'.$strona.'.php'))
        include('log/'.$strona.'.php');
    else
        include('inc/error.php');
}
		else {
 
 
if (isset($_GET['inc'])) {
$strona=$_GET['inc'];
    if (file_exists('inc/'.$strona.'.php'))
        include('inc/'.$strona.'.php');
    else
        include('inc/error.php');
}
 
		else {
    //załaduj stronę główną
    $strona="news";
    include('inc/'.$strona.'.php');
}}}
 
 
?>
[PHP] pobierz, plaintext 

Wydaje mi się, że stworzony powyżej kod można skrócić i zoptymalizować.

Można skrócić zapis, ale z optymalizacją nie ma to wiele wspólnego. Wielkich różnic nie zauważysz.

Użyj switch, a w opcji default dołącz stronę która będzie wyświetlana w przypadku nie podania podstrony w inc

Tyle że switch będzie działać podobnie, prawda? Chodzi mi o to, że gdy klikne link index.php?nowy_folder=jakiś_plik abym nie musiał dodawać kolejnej instrukcji dla nowy_folder.

Czy zrobisz to na IF czy SWITCH, nie ma różnicy, ważna jest logika skryptu. Przeanalizuj poniższy kod:

[PHP] pobierz, plaintext 
<?php
$pages = array(
	'dia'=>'dia/diablo.php',
	'log'=>'log/{log}.php',
	'inc'=>'inc/{inc}.php',
);
 
$errorPage = 'inc/error.php';
$page = 'inc/news.php';
 
foreach($pages as $k=>$v){
	if(isset($_GET[$k])){
		$page = realpath(str_replace('{'.$k.'}',$_GET[$k],$v));
		if(!file_exists($page)){
			$page = $errorPage;
		}
		break;
	}
}
include($page);
[PHP] pobierz, plaintext 

Zrób sobie tablicę kluczy (folderów) które są dozwolone (potem tylko dodasz do tej tablicy kolejny klucz)i uogólnij całość do sprawdzenia czy podany parametr znajduje się w tablicy. Jeżeli jest to sprawdzaj czy taki folder istnieje i czy istnieje plik w tym folderze i w końcu dołączaj go.