Dennis Wronka
Soulcollector
Ich wuerde die beiden IFs zusammenpacken, so sparst Du Dir dann auch das continue.
Ausserdem wird bei Dir folders() immer aufgerufen, auch wenn es sich beim gefundenen Eintrag um eine Datei handelt, was ja nicht gerade viel Sinn macht, oder?
Anhand Deines Codes nehme ich mal an, dass nur Verzeichnisse ausgegeben werden sollen und keine Dateien. Dementsprechend wuerde ich den Code so gestalten:
Ausserdem wird bei Dir folders() immer aufgerufen, auch wenn es sich beim gefundenen Eintrag um eine Datei handelt, was ja nicht gerade viel Sinn macht, oder?
Anhand Deines Codes nehme ich mal an, dass nur Verzeichnisse ausgegeben werden sollen und keine Dateien. Dementsprechend wuerde ich den Code so gestalten:
PHP:
function folders($directory)
{
$opendir=opendir($directory);
while($dir=readdir($opendir))
{
if (($dir!='.') && ($dir!='..') && (is_dir($directory.$dir)))
{
echo '<option value="'.$directory.$dir.'">'.$dir.'</option>';
folders($directory.$dir.'/');
}
}
closedir($opendir);
}