[PHP]Extension naar lowercase

[antonboonstra]

Ik heb een functie gemaakt om de extensie van bestandsnamen naar kleine letters om te zetten. Ik vraag mij echter af of dit de makkelijkste/simpelste oplossing is. Of zie ik een standaard functie over het hoofd?

PHP:

function etl($file) //extension to lowercase
{
  $filename = strrev(strstr(strrev($file), '.')); // eerste gedeelte onveranderd
  $ext = substr(strtolower (strrchr ($file, '.')), 1); // extensiegedeelte naar kleine letters
  return $filename.$ext;
}

[CodeCaster]

Kijk eens naar pathinfo()

[Verwijderd]

Ik zou de filename + extentie dmv Regex eruit halen. Stukje duidelijker en wat veiliger

[NMe]

Je hebt duidelijke de file_convert_extension_to_lowercase() functie gemist...



Alle gekheid op een stokje: PHP heeft veel standaardfuncties maar voor iets dat zo specifiek is als dit zul je toch moeten combineren. In dit geval kun je pathinfo gebruiken om aan de extensie te komen en daar strtolower overheen gooien of je zou (beter dan wat je nu doet in elk geval) kunnen exploden/splitten op de . en daarna het laatste array-element dat je terug krijgt naar lowercase zetten. Je blijft verder wel aan het combineren tussen functies.

Verwijderd schreef op dinsdag 24 februari 2009 @ 12:16:
Ik zou de filename + extentie dmv Regex eruit halen. Stukje duidelijker en wat veiliger

Veiliger? Onzin, de code in de topicstart werkt ook prima voor het doel waar 'ie voor geschreven is en onveilig is het niet te noemen. Hooguit onhandig wanneer een file geen extensie heeft, maar daar moet je je regexp ook tegen wapenen. En als je dat doet is die ook verre van duidelijk.

[ Voor 29% gewijzigd door NMe op 24-02-2009 12:20 ]

[antonboonstra]

Ben (nog) niet zo thuis in reguliere expressies, maar pathinfo ziet er ook handig uit. Thanx!

Edit: Een explode waarbij je alleen de laatste element naar lowercase omzet, was iets handiger geweest dan mijn lastig leesbare oplossing.

[ Voor 43% gewijzigd door antonboonstra op 24-02-2009 12:22 ]

[Verwijderd]

NMe schreef op dinsdag 24 februari 2009 @ 12:18:
Je hebt duidelijke de file_convert_extension_to_lowercase() functie gemist...



Alle gekheid op een stokje: PHP heeft veel standaardfuncties maar voor iets dat zo specifiek is als dit zul je toch moeten combineren. In dit geval kun je pathinfo gebruiken om aan de extensie te komen en daar strtolower overheen gooien of je zou (beter dan wat je nu doet in elk geval) kunnen exploden/splitten op de . en daarna het laatste array-element dat je terug krijgt naar lowercase zetten. Je blijft verder wel aan het combineren tussen functies.

[...]

Veiliger? Onzin, de code in de topicstart werkt ook prima voor het doel waar 'ie voor geschreven is en onveilig is het niet te noemen. Hooguit onhandig wanneer een file geen extensie heeft, maar daar moet je je regexp ook tegen wapenen. En als je dat doet is die ook verre van duidelijk.

Nouja veiliger bedoel ik idd ook niet, maar meer failsave. Veiliger in die zin

[doeternietoe]

Verwijderd schreef op dinsdag 24 februari 2009 @ 12:24:
[...]


Nouja veiliger bedoel ik idd ook niet, maar meer failsave. Veiliger in die zin

Onzin, regexp geeft geen enkel voordeel bij zo´n duidelijke taak. Het verhoogt alleen de kans op fouten omdat regexp heel wat complexer zijn dan de andere functies.

Ik zou geen pathinfo gebruiken, strrchr om de positie van de laatste punt te bepalen en substr om deze uit te knippen, is weer een stukje sneller omdat je pathinfo niet allerlei onnodige informatie op hoeft te laten zoeken. Just my two cents... Het is opzich niet fout met explode of pathinfo.

[antonboonstra]

doeternietoe schreef op dinsdag 24 februari 2009 @ 13:01:
[...]

Onzin, regexp geeft geen enkel voordeel bij zo´n duidelijke taak. Het verhoogt alleen de kans op fouten omdat regexp heel wat complexer zijn dan de andere functies.

Ik zou geen pathinfo gebruiken, strrchr om de positie van de laatste punt te bepalen en substr om deze uit te knippen, is weer een stukje sneller omdat je pathinfo niet allerlei onnodige informatie op hoeft te laten zoeken. Just my two cents... Het is opzich niet fout met explode of pathinfo.

Ben het met je eens dat pathinfo misschien overbodige dingen doet. Mijn resulterende stukje code is:

PHP:

function etl($file) //extension to lowercase
{
  $arr = explode('.', $file);
  $arr[] = strtolower(array_pop($arr));
  return implode('.',$arr);
}

[NMe]

Pas wel op dat het voor zou kunnen komen dat een foldernaam een punt bevat terwijl de file geen extensie heeft. Dat zou je script ietwat in de war kunnen sturen.

[antonboonstra]

@NMe: Ik stuur alleen bestandsnamen zonder folder naar deze functie.

Ik heb (als echte tweaker natuurlijk ) echter een snelheidstest gedaan. Zie onderstaande code:

PHP:

<?php
$time_start = getmicrotime(); // start de teller voor de parsetime

if(isset($_GET['1']))
{
  for($i=0; $i<100000; $i++)
  {
    etl1('IMG_0001.JPG');
  }
}
elseif(isset($_GET['2']))
{
  for($i=0; $i<100000; $i++)
  {
    etl2('IMG_0001.JPG');
  }
}
elseif(isset($_GET['3']))
{
  for($i=0; $i<100000; $i++)
  {
    etl3('IMG_0001.JPG');
  }
}
else
{}

$time_end = getmicrotime(); // bereken de parsetime
$time = substr ( $time_end - $time_start, 0, 8 );
echo "Script is uitgevoerd in $time seconden";

function etl1($file) //extension to lowercase
{
  $filename = strrev(strstr(strrev($file), '.')); // eerste gedeelte onveranderd
  $ext = substr(strtolower (strrchr ($file, '.')), 1); // extensiegedeelte naar kleine letters
  return $filename.$ext;
}

function etl2($file) //extension to lowercase
{
  $file_parts = pathinfo($file);
  return $file_parts['filename'] . '.' . strtolower($file_parts['extension']);
}

function etl3($file) //extension to lowercase
{
  $arr = explode('.', $file);
  $arr[] = strtolower(array_pop($arr));
  return implode('.',$arr);
}

function getmicrotime() 
{
  list($usec, $sec) = explode(" ",microtime());
  return ((float)$usec + (float)$sec);
}
?>

Uitslagen op een P4 3.2GHz machine:
functie 1: 0,593 seconden
functie 2: 0,647 seconden
functie 3: 0,637 seconden

Uitslag verspringt elke keer een paar duizendsten, maar laat wel dit beeld zien. Mijn eerste oplossing, was dus eigenlijk wel de snelste

[Voutloos]

Micro-optimalisatie is wel de slechtste reden om tussen die functies te kiezen.

Hier heb je nog een optimalisatie: gewoon microtime(true) doen ipv suffe PHP 4 style getmicrotime() functies.

[NMe]

...en de minst leesbare.

Puur uit leesbaarheid zou ik hier toch voor pathinfo kiezen. Als je dit nou 100.000 keer op een pagina moet doen begrijp ik een eventuele wens voor optimalisatie wel, nu niet echt.

Overigens heb ik een beetje moeite met het accepteren van een tijd van ruim een halve seconde voor iets simpels als dit.

[CodeCaster]

NMe schreef op dinsdag 24 februari 2009 @ 14:28:
Overigens heb ik een beetje moeite met het accepteren van een tijd van ruim een halve seconde voor iets simpels als dit.

Hij doet 't wel 100.000 keer hè

[Patriot]

NMe schreef op dinsdag 24 februari 2009 @ 14:28:
Overigens heb ik een beetje moeite met het accepteren van een tijd van ruim een halve seconde voor iets simpels als dit.

Mis ik de grap of heb jij gemist dat hij het honderdduizend keer doet?

edit: spuit elf

[ Voor 4% gewijzigd door Patriot op 24-02-2009 14:33 ]

[MueR]

NMe is vandaag gewoon wat traagjes.

[NMe]

MueR schreef op dinsdag 24 februari 2009 @ 14:56:
NMe is vandaag gewoon wat traagjes.

Ben al een paar weken redelijk under the weather, dan krijg je dat.

[Verwijderd]

strtolower(end(explode($filename, ".")))

[Voutloos]

Daarmee krijg je enkel de extensie terug... Als je deze aanpak fixt, kom je op array_pop ed. uit, en die optie was reeds gegeven.

(en de argumenten van explode staan in de verkeerde volgorde )

[ Voor 26% gewijzigd door Voutloos op 25-02-2009 07:20 ]