[PHP] if inkorten

[Verwijderd]

Ik vroeg me af of het volgende ook korter kan (niet specifiek voor extensies, maar zulke if's in het algemeen):

PHP:

if ($sExtension == "jpg" or $sExtension == "jpeg" or $sExtension == "gif" or $sExtension == "png") {

Ik had al wat gezocht, maar kreeg niks relevants.

[Megamind]

je kan or met || vervangen, alleen wordt het niet korter

[Brakkie]

De extensies in een array zetten en dan checken met in_array of de extensie die aangeboden wordt in je array zit.

[Joshua]

Misschien iets met regular expressions? (weet even niet zo snel uit me hoofd hoe je beste kan doen)

[amoen]

Brakkie schreef op donderdag 31 augustus 2006 @ 12:15:
De extensies in een array zetten en dan checken met in_array of de extensie die aangeboden wordt in je array zit.

zo dus:

PHP:

<?php
$ext = array("jpg", "jpeg", "gif", "png");
if (in_array($sExtension, $ext)) {
  // yup
} else {
  // helaas
}
?>

ik verveelde me

[Verwijderd]

Brakkie schreef op donderdag 31 augustus 2006 @ 12:15:
De extensies in een array zetten en dan checken met in_array of de extensie die aangeboden wordt in je array zit.

--knip--
amoen was me voor

[ Voor 16% gewijzigd door Verwijderd op 31-08-2006 12:21 ]

[Verwijderd]

PHP:

$ext = array("jpg","jpeg","gif","png");
if (in_array($sExtension, $ext)) {

Ietsje handiger misschien, maar vind het alleen niet echt korter.

Iets anders?

[Icelus]

Verwijderd schreef op donderdag 31 augustus 2006 @ 12:21:

PHP:

$ext = array("jpg","jpeg","gif","png");
if (in_array($sExtension, $ext)) {

Ietsje handiger misschien, maar vind het alleen niet echt korter.

Iets anders?

Is toch veel beter leesbaar en korter?

[ Voor 179% gewijzigd door Icelus op 31-08-2006 12:25 ]

[Janoz]

Hoe had je een kortere versie van

PHP:

if (in_array(array("jpeg","jpg","png","gif"))){
  //blaat
}

voorgesteld? Je zult toch alle extenties moeten benoemen. Tenzij je gebruik maakt van de Jomanda plugin waardoor de computer automagisch weet wat je bedoeld.

[Grijze Vos]

code:

<?php
switch($sExtension)
{
  case "jpg"  :
  case "jpeg" :
  case "gif"  :
  case "png"  : ifcode();break;
  default: elsecode()
}
?>

Niet echt korter, wel overzichtelijker.

[ Voor 22% gewijzigd door Grijze Vos op 31-08-2006 12:27 ]

[Verwijderd]

Icelus schreef op donderdag 31 augustus 2006 @ 12:22:
[...]
Is toch veel beter leesbaar en korter?

Dat zeker, maar ik dacht aan zoiets:

PHP:

<?php
if ($sExtension ("jpg" or "jpeg" or "gif" or "png")) {
?>

Er klopt niks van, maar om een idee te geven waar ik naar op zoek ben.

[Verwijderd]

Janoz schreef op donderdag 31 augustus 2006 @ 12:26:
Hoe had je een kortere versie van

PHP:

if (in_array(array("jpeg","jpg","png","gif"))){
  //blaat
}

voorgesteld? Je zult toch alle extenties moeten benoemen. Tenzij je gebruik maakt van de Jomanda plugin waardoor de computer automagisch weet wat je bedoeld.

Dat is wat ik zocht. Dank u vriendelijk!

[Janoz]

Tjongejonge, dat had je zelf niet kunnen extraheren uit de post van amoen? Het enige verschil is dat ik geen variabele heb geintroduceerd.

@hieronder, inderdaad, nu zie ik het ook . Krijg je als je snel ff uit het hoofd wat code tikt.

[ Voor 26% gewijzigd door Janoz op 31-08-2006 13:09 ]

[Icelus]

in_array verwacht toch altijd minimaal twee parameters?

[wackmaniac]

Icelus schreef op donderdag 31 augustus 2006 @ 12:43:
in_array verwacht toch altijd minimaal twee parameters?

Ja, de array en wat je zoekt in die array

[DemonTPx]

Spike1506 schreef op donderdag 31 augustus 2006 @ 12:16:
Misschien iets met regular expressions? (weet even niet zo snel uit me hoofd hoe je beste kan doen)

PHP:

if(preg_match("/^(jpe?g|png|gif)$/", $sExtension))

edit:
haakje en ^ en $ vergeten

[ Voor 7% gewijzigd door DemonTPx op 31-08-2006 14:37 ]

[.oisyn]

misschien moet je even ^ en $ toevoegen, anders matcht ie ook "ksejpngekjr"

[ReseTTim]

ik had laatst zelfs 'pjpeg' dus heb ik maar die ook maar toegevoegd.. je zult maar net zien dat die er ook weer tussen komt.. of het was iets raar / fouts..