请问这种写法 如何更加完美 更加精简

Question

let_suffix=res.fileExt||res.fileExt.substring(1);console.log(res.fileExt.substring(1))if(_suffix=='jpeg'||_suffix=='jpg'||_suffix=='png'||_suffix=='bmp'||_suffix=='gif'){alert(1);this.fileDetail.iFileType='img'}elseif(_suffix=='mov'||_suffix=='avi'||_suffix=='rm'||_suffix=='rmvb'||_suffix=='mpeg'||_suffix=='mpg'){this.fileDetail.iFileType='video'}else{this.fileDetail.iFileType='none'}

Answer 1

『Javascript策略模式』
const_suffix=res.fileExt||res.fileExt.substring(1)
constimg='img'
constvideo='video'
constmapping={
jpeg:img,
jpg:img,
png:img,
bmp:img,
gif:img,
mov:video,
avi:video,
rm:video,
rmvb:video,
mpeg:video,
mpg:video,
}
this.fileDetail.iFileType=mapping[_suffix]||'none'
                            

Answer 2

我抛个砖，应该可以竖过来写吧：
let_suffix=res.fileExt||res.fileExt.substring(1);
console.log(res.fileExt.substring(1))
switch(_suffix){
case'jpeg':
case'jpg':
case'png':
case'bmp':
case'gif':
alert(1);
this.fileDetail.iFileType='img'
break;
case'mov':
case'avi':
case'rm':
case'rmvb':
case'mpeg':
case'mpg':
this.fileDetail.iFileType='video'
break;
default:
this.fileDetail.iFileType='none'
}
不过仅仅是照顾下小屏码农，离“完美”还比较远……所以大家请轻拍~