[PHP] Functie geeft niet de juiste array terug.

[xehbit]

Hallo Tweakers,

Ik ben bezig met een project waarmee ik mappen kan zoeken op mijn webserver, daarvoor heb ik de volgende functie geschreven.

PHP:

    function SearchFolder($path = '.', $search){
        if($search == "" || !is_dir($path))         return false;
        if($dh = opendir($path)){
            while(false !== ($file = readdir($dh))){
                if($file != "." && $file != ".."){
                    if(is_dir($path.'/'.$file)){
                        $return[]=$path.'/'.$file.' [FOLDER]';
                        $tmp = SearchFolder($path.'/'.$file, $search);
                        echo print_r($tmp);
                        $return = array_merge($return, $tmp);
                    } else {
                        $return[]=$path.'/'.$file.' [FILE]';
                    }
                    
                }
            }
        }
        return $return;
    }

Het rare is, de $tmp word geen array, terwijl er op het laatst toch echt staat return $return; en e waarde $return word wel een array.

Ik zie hier eigenlijk geen fout in zitten, hopelijk andere wel

[MueR]

Ga eens debuggen dan? Ik je namelijk nergens return als array initialiseren. Leuk dat je in een loop besluit dat er een variabele $return is en dat je daar toevallig een array van maakt, maar als je een keer geen resultaten hebt wordt het geen array.

Overigens is het niet handig om een variabelen met een default in je functie call te zetten en daarna pas variabelen zonder, dan is het nut van de default weg.

[Koetjeboe]

Begin maar eens met je error reporting aan zetten, er zitten wel wat foutjes in, die je er zelf makkelijk kunt uithalen als je waarschuwingen van php krijgt.

code:

error_reporting(E_ALL);

Oa dus dat je $return nergens initialiseert.

[ Voor 10% gewijzigd door Koetjeboe op 18-06-2010 11:02 ]

[Matis]

@ TS, kijk eens hoe ze het oplossen in de RecursiveDirectoryIterator class van PHP

[HuHu]

Modbreak:

Wil je dat soort teksten voortaan achterwege laten?

Ik zou de code weggooien en de reactie van Matis goed lezen .

[ Voor 64% gewijzigd door MueR op 18-06-2010 12:02 ]

[xehbit]

Het is me toch gelukt om het werken te kunnen krijgen
er moest in de functie een $return = array(); komen. En nu werkt het
Bedankt voor e reacties

PHP:

<?php
    function SearchFolder($path = '.', $search){
        $return = array();
        if($search == "" || !is_dir($path))            return false;
        if($dh = opendir($path)){
            while(false !== ($file = readdir($dh))){
                if($file != "." && $file != ".."){
                    if(is_dir($path.'/'.$file)){
                        $return[]=$path.'/'.$file.' [FOLDER]';
                        $tmp = SearchFolder($path.'/'.$file, $search);
                        echo print_r($tmp);
                        $return = array_merge($return, $tmp);
                    } else {
                        $return[]=$path.'/'.$file.' [FILE]';
                    }
                    
                }
            }
        }
        return $return;
    }
?>

[HuHu]

Goed... ik zal mijn post dan nog maar even samenvatten, want ik ben van mening dat die toch best nuttig was:

• je gebruikt geen closedir() na de opendir()
• echo print_r is een hele vreemde constructie als je het tweede argument van print_r niet gebruikt
• recursie is nooit verstanding om zomaar aan te beginnen, zeker niet als beginnende programmeur
• vermijd dan ook de recursie en gebruik een SplQueue

[wackmaniac]

De fout zit oa hier:

PHP:

if($search == "" || !is_dir($path))            return false;

Maak daar maar eens van:

PHP:

if($search == "" || !is_dir($path))            return array();

Je moet even opletten dat je ALTIJD een array teruggeft als je die verwacht. En aangezien FALSE !== ARRAY ...

[afraca]

Het is gelukkig al eerder gezegd, waarom moeilijk doen als het makkelijk kan?

Matis schreef op vrijdag 18 juni 2010 @ 11:04:
@ TS, kijk eens hoe ze het oplossen in de RecursiveDirectoryIterator class van PHP

Zelf maak ik hiervan gebruik om grote bestanden recursief uit een directory te vissen (en filter filename is een zelf in elkaar geknutselde functie) :

PHP:

<?php

$fileSPLObjects = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($_POST['path']),RecursiveIteratorIterator::SELF_FIRST);
try {
    foreach( $fileSPLObjects as $fullFileName => $fileSPLObject ) {
        
        if(!$fileSPLObject->isDir() && $fileSPLObject->getSize() > 400000000){
            $list[] = ucwords(filter_filename($fileSPLObject->getFilename()));
        }  
    }
}
catch (UnexpectedValueException $e) {
        printf("Directory [%s] contained a directory we can not recurse into", $directory);
}

?>

Heb er overigens nog op GoT onlangs een topic voor moeten aanmaken, om dat die class nou niet héél logisch is, maar er is mee te werken....

edit: Jaja, $list word ooit daarvoor nog wel gedeclareerd.