簡化JS，以便不需要在腳本中硬編碼散列

Question

我試圖找到一種方法來簡化此腳本，以便它不需要硬編碼的散列名稱。所以如果你添加一個帶有mytest的新鏈接和一個ID爲mytest的div，它就可以工作，而不必訪問腳本並將mytest添加到腳本中。希望我明確提出我的問題。

我非常友好的感謝任何幫助瞭解這一點。

  jQuery(function($) { 
       $('#menu-bottomnav li a').click(function(e) { 
       e.preventDefault(); 
       animateSlider(this.hash); 
       }); 

       function animateSlider(hash) { 
       if (!$('#popups div.open').length) { 
        if (hash == '#bio') { 
        openPopup(hash); 
        } 
        if (hash == '#resume') { 
        openPopup(hash); 
        } 
       } else { 
        if (hash == '#bio') { 
        openAndClose(hash) 
        } 
        if (hash == '#resume') { 
        openAndClose(hash) 
        } 
       } 
       } 


       function openPopup(hash) { 
       $(hash).slideToggle().addClass('open'); 
       } 


       function openAndClose(hash) { 
       if ($(hash).hasClass('open')) { 
        $($(hash)).slideToggle().removeClass('open'); 
       } else { 
        $('#popups div.open').slideToggle().removeClass('open'); 
        $(hash).slideToggle().addClass('open'); 
       } 
       } 
      }); 

已經在這裏工作的例子https://jsfiddle.net/a4y7z2fm/

Answer 1

怎麼是這樣的：

jQuery(function($) { 
    $('#menu-bottomnav li a').click(function(e) { 
    e.preventDefault(); 
    animateSlider(this.hash); 
    }); 

    function animateSlider(hash) { 
    $('.open').slideToggle().removeClass('open'); 
    $(hash).slideToggle().addClass('open'); 
    } 
}); 

Answer 2

也許我不明白的問題，但現在看來似乎會工作，如果只是罰款你只是通過了散列變量而沒有if語句：

jQuery(function($) { 
$('#menu-bottomnav li a').click(function(e) { 
    e.preventDefault(); 
    animateSlider(this.hash); 
}); 

function animateSlider(hash) { 
    if (!$('#popups div.open').length) { 
     openPopup(hash); 
    } else { 
     openAndClose(hash); 
    } 
} 


function openPopup(hash) { 
    $(hash).slideToggle().addClass('open'); 
} 


function openAndClose(hash) { 
    if ($(hash).hasClass('open')) { 
     $($(hash)).slideToggle().removeClass('open'); 
    } else { 
     $('#popups div.open').slideToggle().removeClass('open'); 
     $(hash).slideToggle().addClass('open'); 
    } 
    } 
});