Problemer med inkludering...

[AnaXyd]

Hei!

 

Jeg har akkurat begynt med .php, og har nå fått tak i ett inkluderingsscript. (Fant det på forumet her)

 

<?php
if(isset($_GET['id']))
{
	$id = $_GET['id'];
	if (file_exists('artikler/'.$id.'.php') && preg_match("/^[a-z]*$/i", $id) == 1)
			include_once 'artikler/'.$id.'.php';
	else
			echo 'Siden finnes ikke, prøv igjen';
}
else
	include_once 'artikler/index.php';
?>

 

Jeg prøver å linke til dette, men det fungerer ikke.

Jeg linker med ?id=artikkel1, og det ligger en fil ved navn artikkel1.php i en mappe ved navn artikler.

 

Hva er galt?

[Mats Danielsen]

Fordi regulær utrykket ditt ikke tillater tall.

[AnaXyd]

Så, det fungerer om .php filene ikke har tall i seg? Hvordan kan jeg få dette til å fungere med tall?

[Mats Danielsen]

<?php
if(isset($_GET['id']))
{
	$id = $_GET['id'];
	if (file_exists('artikler/'.$id.'.php') && preg_match("/^[a-z+1-9]*$/i", $id) == 1)
			include_once 'artikler/'.$id.'.php';
	else
			echo 'Siden finnes ikke, prøv igjen';
}
else
	include_once 'artikler/index.php';
?>

[AnaXyd]

Ah! Takk! Du bare la på +1-9 i preg_match? Da skjønte jeg det også..

[gxi]

Fjern plussen, den har ikke noe der å gjøre med mindre det også skal være lovlig med plusstegn i navnet. Legg også tallrekken fra 0 dersom det skal være mulig å ha 0 i navnet også. Regexpen tillater også at filnavnet kan være tomt (altså i et forsøk på å inkludere filen ".php"). *-tegnet byttes ut til +-tegn for å unngå dette.

 

Altså "/^[a-z0-9]+$/i"