Include fil alt efter hvad der står i adresselinien

<?
        if ($_GET['inc'] == "") {
            include 'start.php';
        } else {
            include $inc.".php";
        }
?>

<?
        if ($_GET['inc'] == "") {
            include 'start.php';
        } else {
            include $_GET['inc'].".php";
        }
?>

Stammer du ;-)


Følgernde kommer:

Warning: main(.php): failed to open stream: No such file or directory in /customers/troels-ravn.dk/troels-ravn.dk/httpd.www/NyWeb/index.php on line 46

Warning: main(): Failed opening '.php' for inclusion (include_path='.:/usr/lib/php') in /customers/troels-ravn.dk/troels-ravn.dk/httpd.www/NyWeb/index.php on line 46

ja så stammer jeg....men jeg ville have troet det virkede...hmm :o/
linje 46 er?

include $inc.".php";

<?
        if (!empty($_GET['inc']) {
            include("start.php");
        } else {
            include($_GET['inc'].".php");
        }
?>

ups

<?
        if (empty($_GET['inc']) {
            include("start.php");
        } else {
            include($_GET['inc'].".php");
        }
?>

havde lige overset at den første skulle være en default.

Og en enkelt ) i første linie, men ellers virker det.

donslund... så har du jo ikke brugt min "stammen" de er jo forskellige de to indlæg

Lad os håbe på at folk så ikke skriver: index.php?inc=http://minwebserver.dk/sletalt.txt

hvor sletalt.txt er php kode der sletter alt.

Eller hvad man nu kan finde på.

ah, bare index.php?inc=http://minwebserver.dk/sletalt

og så sætte minwebserver.dk til at returnerer en textfil når sletalt.php kaldes

(For at være lidt mere præcis :) )

Hva med:

        <?php
        $openinc = $_REQUEST['inc']; //Finder $inc og får den lagt over i $opencom
        if ($openinc) //Hvis $opencom er udfyldt skal den gå videre
            {
                if ( file_exists( "$openinc.php" )) //Tjekker om filen findes
                {
                    include( "$openinc.php" ); //Hvis filen findes bliver den includeret
                }
                else
                {
                    echo "Der er desvære ikke noget fil der hedder dette."; //Hvis filen ikke finde udskriver den dette
                }
            }else //Forsiden udskrives
            {
                include ("start.php"); //Includere forsiden
            }
        ?>

Samme problem, så vidt jeg kan se virker file_exists() vist også på http:// streams. (er ikke 100% sikker)

Omvendt, hvis du bruger koden på et webhotel, så kan man jo selv bestemme hvilken man ønsker at include (alt efter opsætning).

Mit svar er, enten lav det om, eller i det mindste lav en liste over hvilke "inc"'er der er lovlige og så tjekke op imod dem. Dermed bestemmer og kontrollerer du helt præcist hvilke filer der bliver kaldt.

ja, dennismp har (som altid) ret, det er en meget usikker løsning.
dog virker include altså ikke hen over andre domæner, det skal være lokalt.
og man kan altid droppe en masser sikkerhed ned i det.

b_ball $_REQUEST er rimeligt dumt at bruge

$_REQUEST = $_GET $_POST og $_COOKIE (mener at $_SESSION er undtaget)

Tjaa man kan vel også bare sige:

  <?php
        if ($inc) //Hvis $opencom er udfyldt skal den gå videre
            {
                if ( file_exists( "$inc.php" )) //Tjekker om filen findes
                {
                    include( "$inc.php" ); //Hvis filen findes bliver den includeret
                }
                else
                {
                    echo "Der er desvære ikke noget fil der hedder dette."; //Hvis filen ikke finde udskriver den dette
                }
            }else //Forsiden udskrives
            {
                include ("start.php"); //Includere forsiden
            }
        ?>

eller skrive GET...

Mht. det med at man kan includere filer der ikke var meningen man ville includere så kan man jo bare søre for at alle filer der skal includeres ligger i en mappe og at den kun kan includere fra den mappe.

file_exists() kan kun kontrollere lokale filer
include() kan også inklude fra anden webserver, afhængig af php.ini

--- En Lidt mere sikker løsning:

// tilladte filer
$incfiles = array(
    'start' => 'start.php',
    'biografi' => 'biografi.php',
    'hej' => 'etheltandetfilnavn.php'
);


$getinc = isset($_GET['inc']) ? $_GET['inc'] : 'start';

if(array_key_exists($getinc, $incfiles)){
    $incfile = $incfiles[$getinc];
}else{
    $incfile = $incfiles['start'];
}

include($incfile);

hyber: Hvorfor virker det ikke på andre domæner? PHP kan sagents (mener endda at det er default) forstå include("http://www.google.com");

Jeg har desværre set at flere af eksperterne herinde ikke har taget højde for den slags. Jeg har en anden måde at gøre det på, men det er lidt omstændigt at forklare.

Hvis jeg må komme med et forslag:

$allowed = array('side1' => 'side1.php','sideti' => 'sub3/side10.php','om os' => 'about.php');

if( isset($_GET['inc']) )
  $inc = $_GET['inc'];
else
  $inc = 'side1'; // default

if( isset($allowed[$inc]) )
  include($allowed[$inc]);
else
  include('securityerror.php');

så kan siden kaldes med
index.php?inc=side1  - vil loade side1.php
index.php?inc=sideti - vil loade sub3/side10.php
index.php?inc=om+os  - vil loade about.php
index.php            - vil loade side1.php
index.php?inc=http://www.google.com - vil loade securityerror.php

ah, ranglen var hurtigere. Men samme løsning :)

"Tip: As of PHP 5.0.0 this function[file_exists] can also be used with some URL wrappers."

herunder ftp (men ikke http)

synes eller jeg havde testet include på andre domæner...

Jeg bruger selv denne omtalte include() arkitektur, men med mod_rewrite og en bunke sikkerhed ovenpå :) Det hjælper en del.

testede lige, file_exists() blokker for andre urls på min side, php 4.3.x

Tillykke :) Jeg har også slået det fra :)

Det er bedst at bruge is_file()
file_exists() "godkender" også et dir

<?php
$fildir = "/sti/til/dir/";  // Her ligger de filer som kan includes

if(isset($_GET["inc"])) {
    if(is_file($fildir . $_GET["inc"] .".php")) {
    include($fildir . $_GET["inc"] .".php");
    }else{
    include($fildir . "start.php");
    }
}else{
include($fildir . "start.php");
}

?>

sukos, hvis man nu også vil bruges dirs såå :D

donslund lukketid ?

dirs?
Vil du include et dir?

mente som i index.php?page=galleriet/side1.php :)

Jeps. Hyperpreprocessor, smid et svar.

Så laver jeg selv et svar. Så kan jeg da selv tage pointene hvis der ikke er nogen der vil have dem ;-)