Привет.
После нажатия на определённую кнопку открывается блок div с уникальным id. Код js получается большой, потому что повторяется функция. Можно как то уменьшить код?
document.querySelector('button.1').onclick = function() {
var ob = document.getElementById('div1');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
document.querySelector('button.2').onclick = function() {
var ob = document.getElementById('div2');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
document.querySelector('button.3').onclick = function() {
var ob = document.getElementById('div3');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
document.querySelector('#type').onclick = function() {
var ob = document.getElementById('hid1');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
document.querySelector('#type2').onclick = function() {
var ob = document.getElementById('hid2');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
Пробовал вот так:
document.querySelector('button.1','button.2','button.3','#type','#type2').onclick = function() {
var ob = document.getElementById('div1','div2','div3','hid1','hid2');
if( ob.style.display=='block') { ob.style.display='none'; }
else { ob.style.display='block'; }
}
но почему-то работает только первый в списке селектор?
HTML
<button class="1"></button>
<div id="div1">блаблабла</div>
...
<span id="type"></span>
<div id="hid1">блаблабла</div>
...
CSS
стилей для элементов нет, скрипт js скрывает div1,div2,div3,hid1,hid2
onclick
вешается не пробовали отдельно вынести? Передаете параметр (div_id
) по которому изменяете стили. Уже меньше было бы кода. – BOPOH 19 апр '15 в 12:04onclick
через цикл по массиву селекторов (если грубо, то как-то так[1,2,3].forEach(function(x) { bind() })
). Можно и не выносить отдельно, а сразу вforEach
вставлять, но мало ли где исходная функция может еще понадобиться – BOPOH 19 апр '15 в 12:21var array = [ div1,div2,div3,hid1,hid2]
но функцияonclick
не работает. О forEach ничего не знаю. – Viher 19 апр '15 в 12:36