image weergeven

Door kai stevens op 09-04-2007 15:31

445 views

Ik ben bezig met het maken van een website voor een battlefield 2 clan. Het is mogenlijk om in het spel screenshots te maken die dan meteen geupload worden naar de server en via een simpel php script laat ik ze zien op de website.

Het overzicht van screenshots heb ik werkend met de volgende code:

<?

$handle=opendir($_SERVER['DOCUMENT_ROOT'] .'/images/screenshots/zatezeeman/');

while (false!==($file = readdir($handle))) {
if ($file != "." && $file != ".." ) {
echo "<a href='screen2.php?screen=". $file."'><img src=\"/images/screenshots/zatezeeman/". $file ."\" alt=\"AFS_Action\" id=". $file." width='280px' height='210px' border='0'/></a>\n";
}
}

closedir($handle); ?>

Als je in deze pagina op een thumbnail klikt dan krijg je de afbeelding op ware grootte te zien. Dit krijg ik niet aan het werk:

<?

echo "<img src=\"/images/screenshots/zatezeeman/". $_GET['screen']"\" alt=\"foto\"/>"

?>

ik krijg nu gewoon een witte pagina te zien en in de broncode is te zien dat de php niet verwerkt is omdat deze blanco is.

Kan iemand mij misschien vertellen wat ik verkeerd doe?

alvast bedankt!

Gr. Kszitterd

Frank -

09-04-2007 15:34

De PHP-code wordt dus niet geparsed? Heb je eigenlijk wel php op de server? Welke bestanden worden door de server als php geparsed, .php? Zo ja, heeft het bestand dan wel de extensie .php ?

Begin eens simpel te debuggen:
<?php // geen shorttags!
echo 'Hello world';
?>

kai stevens

09-04-2007 15:42

Ja Ik heb de code uitgeprobeerd met een statisch pad naar de afbeelding en dan geeft hij hem wel weer.

Frank schreef op 09.04.2007 15:34
De PHP-code wordt dus niet geparsed? Heb je eigenlijk wel php op de server? Welke bestanden worden door de server als php geparsed, .php? Zo ja, heeft het bestand dan wel de extensie .php ?

Begin eens simpel te debuggen:
<?php // geen shorttags!
echo 'Hello world';
?>

Frank -

09-04-2007 16:16

Waar moet $_GET['screen'] vandaan komen? De aanname dat deze altijd bestaat en een geldige waarde heeft, is een hele foute aanname. Deze garantie heb je namelijk niet, sterker nog, je hebt de garantie dat dit vreselijk fout gaat.

Oplossing: controleer of $_GET['screen'] bestaat (met isset) en een geldige waarde heeft.

Een GET ga je nooit echoen, dat is vragen om problemen.

Reageren

Inloggen om te reageren