Browse door directory

Ik zou meer enkele quotes gebruiken. Voor de rest ziet het er goed uit. :-)

Ik ben persoonlijk niet zo'n voorstander van enkele quotes, omdat daarmee de variabelen niet omgezet worden.

Gaan we weer, hebben we nog niet genoeg van deze scripts?
Opmerkingen: Variabelen buiten quotes, gebruik emtpy(), er bestaan ook mappen met een . en bestanden zonder een . in de naam, gebruik is_dir(), waarom ereg() ipv het snellere strstr(), waarom sizeof() ipv count() en waarom () bij echo gebruiken.

Het gebruik van tabs is wel netjes. Goed gebruik van superglobals, variabelen vooraf gedefinieerd, prima indents, goede hoeveelheid commentaar.

if (!empty($_GET['mapnaam'])) zal trouwens een error opleveren. Gebruik isset() hiervoor. empty() moet op de plekken waar jij nu !="" doet.

@TS:
Nee klopt. Die hoor je, vind ik, buiten de quotes te houden. En anders doe dubbele quotes, en enkele quotes in de html. .-)

Jelmer, bedankt voor je uitgebreide commentaar, ik ga er mee aan de slag.

sizeof() is trouwens aan alias van count()

Voorbeeldje zal aardig zijn.
Veder denk ik één van de vele.
Misschien beter als functie

Webmakerij: Leuk idee die functie, heb ik toegevoegd (deel vanaf ---functie---)
Deze functie geeft een array terug met namen, gesplitst in mappen, plaatjes, en overig.

Zoals Jelmer al aangaf, een van de vele.
En maar 3 soorten bestanden herkenne (mappen, plaatjes, overig)?
Zou er toch wat meer van maken...

3 soorten lijkt me in dit geval meer dan genoeg, het gaat me namelijk om de handeling die er mee gedaan moet worden.
Mappen moeten als nieuwe pagina bereikbaar zijn, plaatjes direct getoond worden, en de rest als download link.

Nu is het uiteraard wel mogelijk om de rest te soorteren op extensie, en overal een titel bij te zetten, maar dat viel buiten mijn doel. Ik wou het script vooral kort houden.

Bedankt Edwin, als php-noob heb ik hier echt veel aan.
(Heb het al in gebruik op mn site, plesk list nm zelf geen mappen)
Is het veel werk om de bestanden op alfabet te laten sorteren?
Heb er zelf al naar gekeken maar ben te onervaren op php gebied.

Voor de tweede versie van het script (de functie) is het sorteren te doen met sort();
Ik weet het niet zeker, maar volgens mij moet het werken als je voor
return $namen;
De volgende regels zet:

Het sorteren van de namen binnen de lange string zou ik niet precies weten.

asort( $map_aantal ); reset( $map_aantal );
asort( $plaatje_aantal ); reset( $plaatje_aantal );
asort( $overig_aantal ); reset( $overig_aantal );

Probeer dit is :)

Ok, heb het op diverse plaatsen en op verschillende variabelen proberen toe te passen. Zowel de sort als de asort functie. Helaas niet gelukt, of ik heb een leeg scherm of de sortering veranderd niet.
Gewoon gebrek aan kennis en ervaring bij mij dus.
Wie weet is er nog iemand die zich hier op uitleeft....

Ik heb de sort() oplossing op de juiste plek gezet in de functie. Bij mij werkt het zo.
(in mijn vorige post stond ergens een ´ in plaats van ' )

typ foutje regel12: ipv