rename_recursive

Kleine toevoeging, als je toch minimaal PHP 5.3 moet hebben kun je beter __DIR__ gebruiken in plaats van dirname( __FILE__ ). :-)

Een aantal dingen zijn mijn niet duidelijk of kunnen beter als je het mij vraagt.

- echo in een functie is not done! Misschien een reference meegeven voor eventueel loggen van de output
- je gebruikt een RecursiveDirectoryIterator maar waarom dan geen RecursiveIteratorIterator?
- is_dir( $sFileInfo ) is onnodig omdat FileInfo een isDir functie heeft.
- Waarom zou je hier php5.3 voor nodig hebben? 5.2 voor de iterators misschien, maar zie verder niks spannends.
- false === is_dir( $psFolder ) is dat echt nodig? controleerd de itterator dat niet?
- je gebruikt is_array() voor een paar parameters. je kan dit afdwingen met typehinting function a(array $option){

@Richard
Inderdaad, zal het veranderen :)

@Lode
1. Die echo is ook alleen om te debuggen, heb hem hier in een comment gezet. Als je de echo gebruikt i.p.v. de rename(), kun je eerst kijken wat er gaat veranderen, voordat het ook daadwerkelijk veranderd ;) En om hier nu echt een logger te gaan voor maken, neu, dat was te veel van het goede voor mijn situatie (be my guest als je dit graag nog wilt toevoegen!)

2. Wat is het verschil? De enige reden dat ik dit ben gaan gebruiken is omdat ik glob niet fatsoenlijk werkend kreeg! Ben geen held in de standaard php classes :D

3. Wat is het verschil dan tussen de normale is_dir functie, en de functie die de Iterator biedt?

4. Voor de functie strstr() heb ik gebruik gemaakt van de derde parameter, welke alleen beschikbaar is vanaf 5.3

5. Leek me handig om eerst alles te controleren, voordat je verder gaat met een functie. En daarnaast: beter te veel controleren dat te weinig, toch?

6. Typehinting kan ik inderdaad gebruiken, niet aan gedacht, dus wordt er bij gezet.

Thanks voor de opmerkingen :)

Even snel ook een versie gemaakt.
Draai geen 5.3 op m'n werk pc dus kan jou versie niet testen en weet niet helemaal wat ie moet doen.
Ook een beetje overbodig misschien om voor die ene functie 5.3 af te dwingen en m'dan jier te posten.

Als er verschillen in zitten mag het geen probleem zijn die aan te passen denk ik.

Is zo toch een stukje netter en simpeler als je het mij vraagt?

<?php
ini_set('display_errors', 1);
ini_set('error_reporting', E_ALL);

function renamer($directory, $length, array $exclude = array(), array $extensions = array(), $recursive = true){
    static $characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789';
    
    if(!ctype_digit((string) $length) && ($length < 5 ^ $length > 128)){
        trigger_error('length must be a number', E_USER_ERROR);
    }
    $directory = realpath($directory).DIRECTORY_SEPARATOR;
    try{
        $iterator = ($recursive) ?
            new RecursiveIteratorIterator(new RecursiveDirectoryIterator($directory)):
            new DirectoryIterator($directory);
    }
    catch(Exception $e){
        trigger_error($e, E_USER_ERROR);
    }
    foreach($iterator as $file){
        $extension     = strtolower(pathinfo($file, PATHINFO_EXTENSION));
        if(    $file->isDir() ||     //skip directories
            (!!$extensions && in_array($extension, $extensions)) || //skip extension
            (!!$exclude && in_array($file->getFilename(), $exclude)) //skip excluded filesname
        ){
            continue;
        }
        while(true){
            $destination = $file->getPath().DIRECTORY_SEPARATOR.substr(str_shuffle($characters), 0, $length).'.'.$extension;
            if(!file_exists($destination)){
                if(false === rename($file, $destination)){
                    trigger_error(sprintf('failed renaming file "%s"', $file), E_USER_ERROR);
                }
                break;
            }
        }
    }
}
renamer(dirname(__FILE__).'/files', 10);
?>

@ Lode

Persoonlijk vind ik mijn script dan behoorlijk duidelijker :)
Als die try/catch/if/else zo wild door elkaar

1. Waarom gebruik jij niet range() om de karakters te "selecteren"?

2. Waarom dubbele uitroeptekens gebruiken? Als $bool true is, lever !$bool false op, maar !!$bool toch weer true? Dan kun je toch net zo goed meteen $bool neerzetten?

3. Kun je dat iterator-gedoe eens verder toelichten? Ik heb het bij mij werkend gekregen, maar snap vanuit de handleiding eigenlijk 0,0 waarvoor het bruikbaar is =/

@Sjaak,

Die try catch is om errors af te vangen die je bij een niet bestaand pad bijvoorbeeld zou krijgen vanuit de iterator zonder daar zelf nog een keer op te gaan controleren. Wiel opnieuw uitvinden komt ter gedachte. Gewoon gebruiken wat er al is dus.

Range kan ook wellicht, maar str_shuffle wil een string.
Scheelt misschien weer een overbodige functie aanroep.

!! gebruik je om iets positief te casten naar boolean.
Scheelt een is_bool aanroep en is daardoor weer losser zodat je ook 0 of 1 zou kunnen gebruiken ook.

Een RecursiveIterator kan je als een gewone iterator benaderen door hem in een RecursiveIteratorIterator te stoppen. Hier wel makkelijk omdat we switchen tussen een DirectoryIterator en een RecursiveDirectoryIterator. Anders zou je voor beide een aparte procedure moeten schrijven om ze te doorlopen.
Zijn inderdaad niet altijd even goed gedocumenteerd.
Ik weet door er heel veel mee te werken inmiddels aardig hoe ze werken en waarvoor je ze kan gebruiken. Wat wil je precies weten? Misschien een keer een wiki artikel schrijven anders?

Is het niet makkelijker om gewoon een volgorde na te lopen? Wanneer de naamlengte niet erg ruim gekozen wordt, kan er op het einde een hoop iteraties uitgespaard worden op deze manier.

Misschien is het ook een idee om een check toe te voegen om te kijken of de naamlengte wel groot genoeg is voor het aantal bestanden.

Daarnaast misschien een optie voor een pre- en suffix?