Funkcja wyświetlania foto., Mógłby ktoś profesjonalnie na to spojrzeć. Opcje

[Ozarys]

Mam taki skrypt i działa ale ma dwa błędy nie wyświetla ostatniego zdjęcia tak jakby go nie było i nie chce pokazywać miniatur(czasami jakieś się pokazują albo są wszystkie takie same)nie wiem co dalej robić proszę o pomoc.
Tu jest link skryptu

[PHP] pobierz, plaintext 
<?
$dir = "fotos/ ";			   # katalog zdjec foto /
$ext = ".jpg|.gif|.png";		# rozszerzenia zdjęć/
$tytul = "Razem zdjęć:";		# Ilość zdjęć w katalogu $dir/
/*-----------------------------------------------------------------------*/
 
 
$puk=opendir($dir);
while ($fichero = readdir($puk)) {
if (is_file($dir.$fichero) && eregi($ext,$dir.$fichero)) {
	$fotos["nombre"][$imagen] = $fichero;
	$fotos["modi"][$imagen] = filemtime($dir.$fichero);
	$imagen++;
	}
}
closedir($puk);
/*-------------------------------------------------------------------------*/
 
function miniatury() {
global $dir,$tytul,$imagen,$bytes,$fotos;
$kb = number_format($bytes/1024);
echo "<hr color=#8BBCF7><br>$tytul$imagen<br><div id="nazwa"> </div>
";array_multisort($fotos["modi"],SORT_DESC,$fotos["nombre"]);
reset($fotos);
for ($i=0;$i<$imagen;$i++) {
echo "<a href="java script:click('".$fotos["nombre"][$i]."')">
<img height=50 src=".$dir.$fotos["nombre"][$i]." border=1></a>   ";
}
}
?>
<center><?
?>
<script>
function click(clicks) { document.images['zdjecie'].src="<?=$dir?>" + clicks; var nombre = clicks.substring(0,clicks.length-4); document.getElementById("nazwa").innerHTML = "Nazwa pliku:" + nombre.toUpperCase(); }
</script>
<tr>
<img  src=zdjecie.gif name="zdjecie" border=0>
<?miniatury()?></center>[code][/code][code]
[PHP] pobierz, plaintext 

Ten post edytował Ozarys 1.01.2007, 19:51:41

[ARJ]

[PHP] pobierz, plaintext 
<?
$dir = "images/";			   # katalog zdjec foto /
$ext = ".jpg|.gif|.png";		# rozszerzenia zdjęć/
$tytul = "Razem zdjęć:";		# Ilość zdjęć w katalogu $dir/
/*-----------------------------------------------------------------------*/
 
 
$puk=opendir($dir);
$imagen = 0;
while ($fichero = readdir($puk)) {
if (is_file($dir.$fichero) && eregi($ext,$dir.$fichero)) {
	$fotos["nombre"][$imagen] = $fichero;
	$fotos["modi"][$imagen] = filemtime($dir.$fichero);
	$imagen++;
	}
}
closedir($puk);
/*-------------------------------------------------------------------------*/
 
function miniatury() {
global $dir,$tytul,$imagen,$bytes,$fotos;
$kb = number_format($bytes/1024);
echo "<hr color=#8BBCF7><br>$tytul$imagen<br><div id="nazwa"> </div>";
array_multisort($fotos["modi"],SORT_DESC,$fotos["nombre"]);
reset($fotos);
for ($i=0;$i<$imagen;$i++) {
echo '<a href="java script:click(''.$fotos["nombre"][$i].'')"><img height="50" src="'.$dir.$fotos["nombre"][$i].'" border="1"></a>
';
}
}
?>
<center><?
?>
<script>
function click(clicks) { document.images['zdjecie'].src="<?=$dir?>" + clicks; var nombre = clicks.substring(0,clicks.length-4); document.getElementById("nazwa").innerHTML = "Nazwa pliku:" + nombre.toUpperCase(); }
</script>
<tr>
<img  src=zdjecie.gif name="zdjecie" border=0>
<?miniatury()?></center>
[PHP] pobierz, plaintext 

jeżeli chodzi o miniatury to poprawione jest echo wyświetlające link i zdjęcie. co do ostatniej miniaturki której nie było to byś do tego doszedł sprawdzając co zawiera tablica. był element bez indeksu. przy generacji tablic na początku skryptu dajesz $imagen++, ale gdzie jest zdefiniowana wartość tej zmiennej? wystarczyło wstawić przed pętlą wartość początkową: $imagen = 0; i wszystko gra.
w razie problemów pisz.