Currently I am creating a filemanager.

What I want is to check if the user has selected a video file. The file can be mov, f4v, flv, mp4 and swf.

I want to check if my var ext is one of these.

What I have is:

if(ext == ('mov' || 'f4v' || 'flv' || 'mp4' || 'swf'))
{
    //Do something
}

Does anyone know how I can get this working. I don't want to use a switch because I will get a lot of cases.

Currently I am creating a filemanager.

What I want is to check if the user has selected a video file. The file can be mov, f4v, flv, mp4 and swf.

I want to check if my var ext is one of these.

What I have is:

if(ext == ('mov' || 'f4v' || 'flv' || 'mp4' || 'swf'))
{
    //Do something
}

Does anyone know how I can get this working. I don't want to use a switch because I will get a lot of cases.

• javascript
• nested-statement

Share Improve this question Follow asked Oct 5, 2012 at 9:20 Ron van der HeijdenRon van der Heijden 15.1k77 gold badges6262 silver badges8686 bronze badges

Add a ment  | 

3 Answers 3

Sorted by: Reset to default 9

You would need to explicitly pare the variable against each of those values.

if( ext === 'mov' || ext === 'f4v' || ... ) { 
}

..but, RegExp to the rescue, we can go like

if( /mov|f4v|flv|mp4|swf/.test( ext ) ) { 
}

you need to split them up, like so:

if(ext === "mov" || ext === "f4v" || ext === "flv" || ext === "mp4" || ext === "swf")
{
    // do stuff
}

you could also consider putting all the different extension in an array and checking whether ext exists in that array

Kinda nice way is this:

var exts = {
     "mov" : null,
     "f4v" : null,
     "flv" : null,
     "mp4" : null,
     "swf" : null,
}

if(ext in exts){
    // world peace
}