var filterBy = window.location.search.substring(10);
if(filterBy !==""){
if(filterBy ==='all'){
$('#filterItems').find('[data-type]').show();
}else{
$('#filterBy li').removeClass('active');
$('#filterBy').find('li[data-filter= "'+ filterBy + '"]').addClass('active');
$('#filterItems').find('[data-type]').hide().end().find('[data-type*="' + filterBy + '"]').show();
}
}
我想让它优化。 filterBy ===""和filterBy ===全部。我觉得它检查了两次状况。我想只有条件检查
答案 0 :(得分:4)
如果filterBy
等于"all"
,则不为空。所以你也不需要检查它。
if (filterBy === 'all') {
// your code here
} else if (filterBy !== '') {
// your code here
}
答案 1 :(得分:0)
你可以这样做。
if (filterBy === "" || filterBy === 'all'){
// if condition is met
} else {
// if condition is not met
}