[PHP] Probleem met functie eregi

[Verwijderd]

Hallo beste gotters.

Ik heb een klein probleempje met de functie eregi in php.

Ik ben een functie aan het maken om een alle files in een directory in een array te proppen. Nu wil ik filteren op files met een bepaalde extensie. Hiervoor dacht ik aan de functie eregi te gebruiken. Hieronder het betreffende stukje.

code:

eregi('\.jpg$',$file)

Dit werkt verder allemaal perfect alle files met de extensie jpg worden in de array gepropt.

Nu wilde ik de functie een beetje uitbreiden zodat ik ook op andere extensies kan zoeken. Daarom dacht ik eraan om een variable op te nemen in de regexp. Helaas krijg ik dat niet voor elkaar.

code:

$extension = "jpg";
eregi('\.$extension$',$file)

Ik heb al geprobeert een \ voor de variable te zetten maar dan nog werkt het niet. Heeft iemand hier een oplossing voor?

[ Voor 3% gewijzigd door Verwijderd op 23-10-2004 21:54 ]

[ludo]

• Code tussen enkele quotes wordt niet geparsed, gebruik dubbele quotes.
• Kijk ook even naar de PCRE functies, deze zijn (meestal) een stuk sneller.
• Het dollarteken in je expressie zal denk ik ook voor problemen zorgen.

[ Voor 6% gewijzigd door ludo op 23-10-2004 21:28 ]

[Verwijderd]

Verwijderd schreef op 23 oktober 2004 @ 20:57:
Hallo beste gotters.

Ik heb een klein probleempje met de functie eregi in php.

Ik ben een functie aan het maken om een alle files in een directory in een array te proppen. Nu wil ik filteren op files met een bepaalde extensie. Hiervoor dacht ik aan de functie eregi te gebruiken. Hieronder het betreffende stukje.

code:

eregi('\.jpg$',$file)

Dit werkt verder allemaal perfect alle files met de extensie jpg worden in de array gepropt.

Nu wilde ik de functie een beetje uitbreiden zodat ik ook op andere extensies kan zoeken. Daarom dacht ik eraan om een variable op te nemen in de regexp. Helaas krijg ik dat niet voor elkaar.
[code]

hier heb je een M

$extemsion = "jpg";

en hier een N

eregi('\.$extension$',$file)

[/code]
Ik heb al geprobeert een \ voor de variable te zetten maar dan nog werkt het niet. Heeft iemand hier een oplossing voor?

[Verwijderd]

Ja sorry dat was een overtik foutje. Heb 't even verbeterd in de start post.

[kvdveer]

Tussen enkele quotes worden variabelen niet vervangen door hun waarde... Dat gaat er dus mis.

PHP:

// zo werkt het wel:
preg_match('/\.' . $extensie . '$/i',$file);
// of:
preg_match("/\.$extensie\$/i",$file);
// of:
eregi("\.$extensie\$",$file);

[ Voor 11% gewijzigd door kvdveer op 23-10-2004 22:07 ]

[PrisonerOfPain]

Voor dat soort dingen gebruik ik meestal zoiets:

PHP:

$Ext = '.jpg';
$File = 'afbeelding1.jpg';
if (substr ($File, -strlen($Ext)) == $Ext)
{
//Doe
}

Met eregi zal het zoiets worden

PHP:

$File = 'afbeelding.jpg';
$Ext = 'jpg';
if (eregi ("\.{$Ext}$", $File))
{
//Doe
}

[Billie]

Wat is er mis met in_array?

[Verwijderd]

Met eregi zal het zoiets worden

PHP:

$File = 'afbeelding.jpg';
$Ext = 'jpg';
if (eregi ("\.{$Ext}$", $File))
{
//Doe
}

Hey ja dat werkt perfect. Mag ik je hartelijk bedanken voor je hulp. En de rest natuurlijk ook.

[chem]

Billie schreef op 23 oktober 2004 @ 22:11:
Wat is er mis met in_array?

idd, een in_array en een substr() zijn vele malen sneller dan een regexp er op loslaten.

Enige randvoorwaarde is dat de extentie altijd 3 karakters is; iets wat wel weer te overkomen is door naar de laatste . te zoeken in de bestandsnaam.

Vergeet trouwens niet dat er besturingssystemen bestaan die geen extentie gebruiken; mocht je dus uploads willen "syncen" dan is een extentie check niet afdoende.

[PrisonerOfPain]

chem schreef op 23 oktober 2004 @ 22:31:
Enige randvoorwaarde is dat de extentie altijd 3 karakters is; iets wat wel weer te overkomen is door naar de laatste . te zoeken in de bestandsnaam.

Waarom?

PHP:

substr ($File, -strlen($Ext))

Pakt de laatste 'x' characters naargelange $Ext lang is

Mijn aanpak is meestal

PHP:

function hasValidExt ($File)
{
       $AllowedExts = array ('.jpg','.gif','.jpeg');
       foreach ($AllowedExts as $Ext)
       {
              if (substr ($File, -strlen($Ext)) == $Ext)
              {
                     return true;
              }
       }
       return false;
}

[ Voor 33% gewijzigd door PrisonerOfPain op 23-10-2004 23:13 ]

[Janoz]

Verwijderd schreef op 23 oktober 2004 @ 22:30:
[...]


Hey ja dat werkt perfect. Mag ik je hartelijk bedanken voor je hulp. En de rest natuurlijk ook.

Nee, dat doet ie niet. Binnen " worden de \ ook verwerkt. De regexp daar zal ook "file.nojpg" pikken. Waneer je een \ in die string wilt zetten zul je de \ ook weer moeten escapen Daarnaast geldt dit eigenlijk ook voor de $. Hij zal dus uiteindelijk "\\.$extension\$" worden.