Sous-répertoire non reconnu comme étant des répertoires Résolu

Question

Bonjour,



J'ulise la fonction suivante :



function recup_repertoires(&$liste_repertoires, $dir= './', $recursif=true, $inc=0)

{    

        $rep = opendir($dir);        

        while ($file = readdir($rep))

        {

            if($file != '..' && $file !='.' && $file !='')

            { 

                 if (is_dir($file)) 

                {    

                    $liste_inc  = '';

                    

           
        for($i=0; $i

malalam · Accepted Answer

Hello,

<?php
function recup_repertoires(&$liste_repertoires, $dir='.', $recursif=true)
{
        $rep = opendir($dir);        while (false !($file readdir($rep)))
        {
            if(!in_array ($file, array ('.', '..')))
            {
                if (is_dir($dir.'/'.$file))
                {
                    array_push($liste_repertoires,$dir.'/'.$file);


                    if ($recursif === true)
                        recup_repertoires($liste_repertoires, $dir.'/'.$file, true);
                }
            }
        }


        closedir($rep);
        clearstatcache();
 }


$liste = array(); //Initialisation
recup_repertoires($liste);
print_r ($liste);
?>

WhiteDwarf · Answer

Premièrement je nepense pas que l'on puisse directement dans l'intitulé de la fontcion enter une variable et directement donner sa valeur, ca ne marche pas comme ca... tu devrai plutôt l'écrire comme ca :

---------------------- La lumière étant plus rapide que le son, un homme peu paraître brillant avant qu'il se mette a parler -----------------

cs_darkduck · Answer

Premi&#232;rement je nepense pas que l'on puisse directement dans l'intitul&#233;
de la fontcion enter une variable et directement donner sa valeur, ca
ne marche pas comme ca...



Si &#231;a marche comme &#231;a, qd on met une
valeur dans l'intitul&#233; de la fonction, &#231;a permet de donner une valeur
par d&#233;faut &#224; la variable pass&#233;e en paramettre ce qui fait que si on ne
lui donne pas de valeur dans l'appel, elle prend la valeur "par d&#233;faut"



Ex : 

Si j'appelle recup_repertoires(&$liste_repertoires)

$dir sera &#233;gal &#224; './'

$r&#233;cursif sera &#233;gal &#224; true

$inc sera &#233;gal &#224; 0



Alors que si j'appelle recup_repertoires(&$liste_repertoires, 'mon_repertoire/', false)

$dir sera &#233;gal &#224; 'mon_repertoire/'


$r&#233;cursif sera &#233;gal &#224; false


$inc sera &#233;gal &#224; 0



Esp&#233;rant en avoir aid&#233; plus d'un.....



...je n'ai cependant toujours pas la r&#233;ponse &#224; ma question (pourquoi is_dir($file) retourne false quand $file est un sous-repertoire alors que is_dir($file) retourne true, quand $file est un r&#233;pertoire "simple" ?)

cs_darkduck · Answer

Cool, merci &#231;a marche tr&#232;s bien !!!! Chapeau !



J'ai juste optimiser un peu la fonction :



function recup_repertoires(&$liste_repertoires, $dir= '.', $recursif=true)

    {

        $rep = opendir($dir);

         while ($file = readdir($rep)) 

        {

            if(!in_array ($file, array ('.', '..')))

            {

                if (is_dir($dir.'/'.$file))

                {

                   
array_push($liste_repertoires,$dir.'/'.$file);



                    if ($recursif)

                       
recup_repertoires($liste_repertoires, $dir.'/'.$file, true);

                }

            }

        }



        closedir($rep);

        clearstatcache();

     }



$liste  = array(); //Initialisation

recup_repertoires($liste);

print_r ($liste);





J'ai remplac&#233; :

    while (false !($file readdir($rep)))

Par :

    while ($file = readdir($rep))



Et : 

     if ($recursif === true)

Par :

    if ($recursif)



Raison : Etant donn&#233; que les tests sont fait sur des variables
(ou m&#233;thodes) qui sont (ou retournent) des boul&#233;ens, ce n'est pas la
peine de v&#233;rifier $true (ou $false) et != $true (ou != $false)



Merci encore !

malalam · Answer

C'est de l'optimisation, &#231;a ? 
Lol, en l'occurence, ma version est plus rapide.
Et j'explique :

"J'ai remplac&#233; :    while (false !($file readdir($rep)))
Par :
    while ($file = readdir($rep))
"
> tu as grand tort ;-) Partce que si tu as un r&#233;pertoire ou un fichier s'appellant '0', ta fonction, avec ta version, va s'aarr&#234;ter...'0' = false en php. Donc la condition de ton while ne sera plus remplie.
Mais, en php, '0' !== false (on teste sur le type).

Idem pour la 2&#232;me r&#233;flexion : 
"Et : 
    if ($recursif === true)
Par :
    if ($recursif)""Raison : Etant donn&#233; que les tests sont fait sur des variables (ou m&#233;thodes) qui sont (ou retournent) des boul&#233;ens, ce n'est pas la peine de v&#233;rifier $true (ou $false) et != $true (ou != $false)"

Certes, sauf que lorsqu'on code, on essaye d'&#234;tre strict : tu attends un booleen, true, ou false. Donc, tu testes que c'en est bien un. Et encore une fois, avec ton code, je peux passer 'bla' au lieu de true, ton code prendra &#231;a comme un true. Ce qui est incorrect.
D'ailleurs, si tu avais test&#233; le type correctement, tu te serais rendu compte, dans ta 1&#232;re version, que ta fonction ne rentrait pas dans l'appel rt&#233;cursif : tu avais invers&#233;, dans l'appel r&#233;cursif, le true et le chemin du sous-r&#233;pertoire. Or, ta fonction a pris ce chemin comme &#233;quivalent &#224; true puisque tu ne pr&#233;cisais pas ce que tu attendais. 
Or, si $recursif = 'dir/sous-dir', if ($recursif) renvoie true.

cs_darkduck · Answer

Ok.

Je n'aime pas trop le ton que tu prends mais merci pour les explications.



Je ne connaissais pas !et je croyais que tu avais saisi trop vite.

WhiteDwarf · Answer

Waaa, on apprends des trucs tous les jours en tout cas tr&#232;s instructif, bon code bravo et merci, je vais faire quelques modifs sur mon "Web FTP" ... :p

@++


---------------------- 
La lumi&#232;re &#233;tant plus rapide que le son, un homme peu para&#238;tre brillant avant qu'il se mette a parler 
-----------------

malalam · Answer

D&#233;sol&#233; pour le ton : le but n'&#233;tait pas d'&#234;tre blessant, du tout. Mais si tu l'as ressenti ainsi, je suppose que je l'ai &#233;t&#233;. 
D&#233;sol&#233;, donc.

cs_darkduck · Answer

Excuses accept&#233;es...



...sans rancune 



@+ et encore merci pour ton aide

Sous-répertoire non reconnu comme étant des répertoires

9 réponses

Votre réponse

Discussions similaires