(PHP) Ez jó módszer oldalakat behívni egy div-be? Szóval biztonságos?

Paraméterben kapott értéket így SOHA nem teszünk be sehova.

Nem biztonságos!

[link]

Nem, mivel bármit behívhat.

[link]

1000+1 megoldás van rá. Én template megoldással csináltam és MVC felépítéssel.

Azaz kap egy ...hu/A/B Ami az 'A' a controller és a 'B' az action. Ha vagy-vagy:

- controller nem létezik

- action nem létezik

- View réteg nem létezik

Akkor a View a 404-es algoritmuson fusson végig. 1x elég megírni, és bármire müködik, ha a feltételek nem teljesülnek.

A te általad írt kódban van 1-2 problem.

1) $_GET['page'] az nem fog tartalmazni ".php" stringet, míg pedig az isset hamis esetén ott még is van else. Ergó: ha ...&page=alma lenne akkor az include "alma"-t kapna, php tag nélkül, amire max pislog 2tőt a fordító.

2) if-et lezárod pontosvesszővel, holott nem kéne, mivel az else is abba az ifbe tartozik.

+1) Include fgv-nek nem kötelező zárójel. Kb. úgy is használható, mint az echo.

Általad írt és általam módosított:

<?php

$page = isset($_GET['page']) ? $_GET['page'] : "main";

if(file_exists($page)) include "$page.php"

else include "404.html";

?>