Можно ли оптимизировать код

@Velidan · Регистрация: 14.10.2013

Студворк — интернет-сервис помощи студентам

Доброго вечера. Есть небольшой кусок кода, можно ли как-то его упростить? (потому что идет дубляж кода, а это не очень правильно, но в то же время, создавать массив и цикл для обхода всех элементов, чтобы добавить свойство - также не очень хорошо).
П.С. each jQuery использовать не хочу. Только javascript.

JavaScript
1
2
3
4
  document.getElementById('de').onclick = function() {
    document.querySelector('.slide1').style.height= "0px";
    document.querySelector('.slide2').style.height= "0px";
}

@arcmag · 08.02.2015, 21:59

Ну как вариант указать элементам которые у вас находятся внутри элемента с ID = de указать одинаковые классы .slide потому получить массив из этих классов getElementsByClassName и в цикле пробегать всю длинну массива присваивая каждому style.height= "0px";
Я конечно не спец по оптимизации но в принципе если вас беспокоит "дубляж" кода то мой вариант в принципе рабочий.

@Velidan · 08.02.2015, 22:43 **[ТС]**

Спасибо за ответ, только здесь незадача =) вместо 2 строчек кода придется писать цикл, обходить все элементы и т.д. Но мне главное скорость работы.
А какой вариант быстрее будет работать ?

@Padimanskas · 08.02.2015, 23:36

Можно как-то так:

JavaScript
1
2
3
4
5
6
7
var myUtils = {};
myUtils.setHeight = function(selector){
document.querySelector(selector).style.height= "0px";
return this;
}
 
myUtils.setHeight('.slide1').setHeight('.slide2');

Добавлено через 5 минут

Сообщение от Velidan

А какой вариант быстрее будет работать ?

Быстрее будет там где DOM элементы предварительно кешируются

for(var i = 0, dom = document.querySelector('.class'), domLength = dom.length; i<domLength ; i++){
// dom[i].style.height = '';
}

Новые блоги и статьи Все статьи Все блоги /
Перемещение выделенных строк ТЧ из одного документа в другой Maks 30.03.2026 Реализация из решения ниже выполнена на примере нетипового документа "ВыдачаОборудованияНаСпецтехнику" с единственной табличной частью "ОборудованиеИКомплектующие" разработанного в конфигурации КА2. . . .	Functional First Web Framework Suave DevAlt 30.03.2026 Sauve. IO Апнулись до NET10. Из зависимостей один пакет, работает одинаково хорошо как в режиме проекта так и в интерактивном режиме. из сложностей - чисто функциональный подход. Решил. . .	Автоматическое создание документа при проведении другого документа Maks 29.03.2026 Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .	Настройка движения справочника по регистру сведений Maks 29.03.2026 Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .

@arcmag 347 / 322 / 203 Регистрация: 27.06.2014 Сообщений: 762
	08.02.2015, 21:59
	Ну как вариант указать элементам которые у вас находятся внутри элемента с ID = de указать одинаковые классы .slide потому получить массив из этих классов getElementsByClassName и в цикле пробегать всю длинну массива присваивая каждому style.height= "0px"; Я конечно не спец по оптимизации но в принципе если вас беспокоит "дубляж" кода то мой вариант в принципе рабочий. 1

@Velidan 1 / 1 / 1 Регистрация: 14.10.2013 Сообщений: 106
	08.02.2015, 22:43 [ТС]
	Спасибо за ответ, только здесь незадача =) вместо 2 строчек кода придется писать цикл, обходить все элементы и т.д. Но мне главное скорость работы. А какой вариант быстрее будет работать ? 0