[PHP]Czy to dobre rozwiązanie Opcje

[baszar]

Siemka, czy ten kod jest ok? Zrobiłem go sam i nie jestem pewien ;d

[PHP] pobierz, plaintext 
include('top.php');
 
$site = $_GET['site'];
if(!file_exists($site.".php")) $site = "news";
if(!isset($site)) $site="news";
 
 
@include(''.$site.'.php');
 
include('stopka.php');
[PHP] pobierz, plaintext 

Chodzi mi o linki np index.php?site=register itd

i jeśli nie ma pliku o nazwie podanej w site to wyświetla news.

Ok jest? Może jest jakieś lepsze rozwiązanie?

Ten post edytował baszar 10.03.2013, 19:23:10

[!*!]

Nie.

- wywal @ z kodu.
- nie wczytuj plików bezpośrednio z GET, użyj switch + ich nazw
- jak już coś wczytujesz z GET, to poza sprawdzeniem czy plik istnieje, użyj basename, aby nie możnabyło wpisać dowolnej ścieżki i pliku
- użyj wyszukiwarki bo już nie jeden taki skrypt tu był.

[markonix]

[PHP] pobierz, plaintext 
@include(''.$site.'.php');
[PHP] pobierz, plaintext 

Niepotrzebne cudzysłowy.

[PHP] pobierz, plaintext 
include($site .'.php');
[PHP] pobierz, plaintext 

[baszar]

A jeśli tego nie zrobie to coś grozi stronie?

Możesz mi pokazać jak to zrobić w switch bo ja dopiero się ucze i nie wiem za bardzo jak to zrobic?

Ten post edytował baszar 10.03.2013, 19:39:45

[!*!]

A jeśli tego nie zrobie to coś grozi stronie?

Poza wczytaniem każdego pliku jaki jest na serwerze? To nie.

[PHP] pobierz, plaintext 
switch($_GET)
{
   case 'stronaA':
    include 'a.php';
break
   case 'stronaB':
    include 'b.php';
break
}
[PHP] pobierz, plaintext 

Ten post edytował !*! 10.03.2013, 19:42:23

[baszar]

Ok

Mój kod teraz wygląda tak:

[PHP] pobierz, plaintext 
switch($_GET['site'])
{
   case 'register':
    include 'rejestracja.php';
    break;
   case 'test':
    include 'test.php';
    break;
    case 'news':
    include 'test.php';
    break;
}
[PHP] pobierz, plaintext 

Ale jak zrobić, że jeżeli nie ma pliku podanego w site to przerzuca na plik news?

Ten post edytował baszar 10.03.2013, 20:00:41

[!*!]

A przeczytałeś manual? switch ma deklaracje default.

[baszar]

Sorka tylko przejrzałem ;/

A teraz powiedź mi jak użyć tego basename bo nie wiem, próbowałem i nic mi nie wychodzi

[!*!]

Tak jak opisali w manualu. Pokaż jak to zastosowałeś.

[baszar]

TAK

[PHP] pobierz, plaintext 
$site = $_GET['site'];
$site1 = basename($site);
 
 
switch($site1)
{
   case 'register':
    include 'rejestracja.php';
    break;
   case 'test':
    include 'test.php';
    break;
    case 'news':
    include 'news.php';
    break;
    default;
    case 'news':
    include 'news.php';
    break;
}
[PHP] pobierz, plaintext 

[fiszol]

Nie tak.

[PHP] pobierz, plaintext 
$site = $_GET['site'];
switch($site1) {
	case 'register':
		$strona = basename('rejestracja.php');
		include $strona;
	break;
	//itd, w basename wskazujesz na plik wiec jesli strony masz w folderze strony
	//to bedzie basename('strony/strona.php');
	//defaulta tez miales zle, tak powinno byc
	default;
		include 'news.php';
	break;
}
[PHP] pobierz, plaintext 

[baszar]

Ok wielkie dzięki jeszcze nie mam takiego pojęcia o tym dopiero zaczynam ;d!

[!*!]

fiszol - po co mu basename do pliku który sam sobie zdeklarował? Tego się używa do odczytywania np. ścieżki z GET, gdy odbieramy coś od użytkownika.

Ten post edytował !*! 10.03.2013, 21:29:35

[fiszol]

Urwana nocka zbiera swoje żniwo.