Задать вопрос
Не нравятся результаты поиска? Попробуйте другой поиск!
DLE FAQ » Все вопросы » Ламерские вопросы » Как сократить или оптимизировать скрипт?

Как сократить или оптимизировать скрипт?

     14.09.2015    Все вопросы » Ламерские вопросы    2395

вопрос
Есть скрипт который скрывает или показывает блоки в зависимости от выбранного селекта. При большом количестве селектов в списке скрипт получается огромным, возможно ли его как то оптимизировать?
function Selected(a) {
        var label = a.value;
        if (label==1) {
            document.getElementById("xfield_holder_model").style.display='block';
            document.getElementById("xfield_holder_model1").style.display='none';             
        } else if (label==2) {
            document.getElementById("xfield_holder_model").style.display='none';
            document.getElementById("xfield_holder_model1").style.display='block';   
        } else if (label==3) {
            document.getElementById("xfield_holder_model").style.display='none';
            document.getElementById("xfield_holder_model1").style.display='none';
            document.getElementById("xfield_holder_model3").style.display='block';
        } else if (label==4) {
            document.getElementById("xfield_holder_model").style.display='none';
            document.getElementById("xfield_holder_model1").style.display='none';
            document.getElementById("xfield_holder_model3").style.display='none';
            document.getElementById("xfield_holder_model4").style.display='none';
            document.getElementById("xfield_holder_model5").style.display='block';
        } else {
            document.getElementById("xfield_holder_model").style.display='none';
            document.getElementById("xfield_holder_model1").style.display='none';
            document.getElementById("xfield_holder_model3").style.display='none';
            document.getElementById("xfield_holder_model4").style.display='none';
            document.getElementById("xfield_holder_model5").style.display='none';
        }   
         
}

Ответил: werty1001

function selected(v) {

    var list = document.getElementById('list').children,
        value = parseInt(v);

    for (var i=list.length; i--;) { list[i].style.display='none'; }

    if (value === 1) show(['xfield_holder_model']);
    else if (value === 2) show(['xfield_holder_model1']);
    else if (value === 3) show(['xfield_holder_model3']);
    else if (value === 4) show(['xfield_holder_model5']);

}

function show(el) {
    el.forEach(function(id) {
        document.getElementById(id).style.display='block';
    });
}
Живой пример тык

13 комментариев

werty1001
Юзер

werty1001 - 14 сентября 2015 21:56 -

Можете еще 
else if
заменить на 
if
и сократить идентификаторы, например просто 
model_1
и.т.д.
follor
Юзер

follor - 14 сентября 2015 22:34 -

werty1001, спасибо)
follor
Юзер

follor - 14 сентября 2015 23:51 -

Попытался его использовать, никак не хочет работать, в jsfiddle работает а на сайте не хочет, перепробовал всё а причины конфликта так и не нашёл(
werty1001
Юзер

werty1001 - 15 сентября 2015 15:19 -

А в консоли что-то есть? Возможно, у вас уже есть функция с названием "show", попробуйте переименовать ее как-нибудь по-другому.
follor
Юзер

follor - 15 сентября 2015 20:01 -

Смена имени так же не помогла, причину так и не нашел. Нашёл другой скрипт вроде как работающий, но так же требующий корректировок.
dj-avtosh
PHP-developer

dj-avtosh - 15 сентября 2015 07:54 -

Как вариант:



document.getElementById('selected').addEventListener('change', function() { selected(this.value); }, false);

function selected(v) {

    var list = document.getElementById('list').children, value = parseInt(v);
    
    for (var i=list.length; i--;) { list[i].style.display='none'; }

    obj = {1:'', 2:1, 3:3, 4:5}

    show(['xfield_holder_model' + obj[value]]);

}

function show(el) {
    el.forEach(function(id) {
        document.getElementById(id).style.display='block';
    });
}

По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
dj-avtosh
PHP-developer

dj-avtosh - 15 сентября 2015 08:09 -

Чуть короче
document.getElementById('selected').addEventListener('change', function() { selected(this.value); }, false);

function selected(v) {
    Array.prototype.forEach.call(document.getElementById('list').children, function(element){element.style.display='none';}); 
    
    obj = {1:'', 2:1, 3:3, 4:5}
    
    show(['xfield_holder_model' + obj[parseInt(v)]]);

}

function show(el) {
    el.forEach(function(id) {
        document.getElementById(id).style.display='block';
    });
}
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
follor
Юзер

follor - 15 сентября 2015 09:50 -

Опять же, в jsfiddle работает а на сайте не хочет, не пойму в чем проблема, буду искать конфликтующий сркипт, отключая их поочерёдно
dj-avtosh
PHP-developer

dj-avtosh - 15 сентября 2015 09:56 -

напиши в скайп, гляну.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
follor
Юзер

follor - 15 сентября 2015 19:53 -

Нашёл скрипт который вроде как работает на моем сайте, но не получается его оптимизировать под свои нужды
вот то что нашёл

А вот то что у меня
dj-avtosh
PHP-developer

dj-avtosh - 16 сентября 2015 02:51 -

Опишите логику, что хотите что бы делал скрипт с Вашими выпадающими списками.
По заказам пишем сюда: @Rud00y

ЯД: 41001679231462
Заказы в telegram (ремонт модулей, оптимизация нагрузок и т.п.):
В телегу писать сразу задачу и бюджет.
follor
Юзер

follor - 16 сентября 2015 10:02 -

По данным из этой ссылки, селект с именем Марка автомобиля - основной, при выборе в нём пункта 1 должен появиться второй селект который ниже модель, при выборе пункта 2 другой селект и т. д.
follor
Юзер

follor - 27 декабря 2015 21:38 -

Как к этому скрипту прикрутить второй набор селектов? С другим id
Чтобы комментировать - войдите или зарегистрируйтесь на сайте

Похожие вопросы

 01.03.2013 rahman  Ламерские вопросы
 07.05.2013 onneuha  Общие вопросы по PHP, Модули, Ламерские вопросы
 01.09.2013 stopani  Ламерские вопросы
 17.10.2013 voveolegovich  Общие вопросы, Ламерские вопросы
 15.12.2013 deffos  Общие вопросы, Шаблоны (TPL), Ламерские вопросы
 18.01.2014 stazilov  Ламерские вопросы
 05.07.2014 fh79655l123  Ламерские вопросы
 13.02.2015 arriah  Ламерские вопросы
 13.05.2015 veniamin  Общие вопросы по вёрстке, Ламерские вопросы
 13.07.2015 falcon5  Ламерские вопросы
  • TeraMoune | Как отключить миниатюры стандартной гале ...
    Фото TeraMoune
    TeraMoune
    Как отключить миниатюры стандартной галереи Fancybox?
    Сегодня, 06:00
    В файле engine/classes/fancybox/fancybox.js искать нужную инициализацию и добавлять соответствующие параметры, весь список параметров и их описание можно найти на этой странице https://fancyapps.com/fancybox/api/options/

    Например: параметр Thumbs: false отключает миниатюры.
    Fancybox.bind("[data-highslide]", {
	groupAttr: 'data-highslide',
	idle:2000,
	Thumbs: false,
	caption: (fancybox, slide) => {
		if(  slide.thumbEl && slide.thumbEl.alt ) {
			return slide.thumbEl.alt
		}
	}
});
  • derebas | Как реализовать шифрование трафика? 3
    Фото derebas
    derebas
    Как реализовать шифрование трафика?
    18-11-2024, 16:52
    Ну в теории, можно использовать src на site.com разместить прокси на site2.com а выдавать видео с site3.com, при этом прямой доступ на site3.com закрыть, отдача будет только если запрос с site2.com, таким образом палится только site2.com, НО источник потокового видео, все равно можно увидеть если захотеть, а ссылки, ну купить домен dsac-43efew-csdds.uz как пример
  • Gordi | Перенёс сайт - не работают кнопки, не от ... 8
    Фото Gordi
    Gordi
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:46
    я уже делал. Пользователь 1000 (это я так понимаю root) и chmod 777
    Так и было...
  • lutskboy | Перенёс сайт - не работают кнопки, не от ... 8
    Фото lutskboy
    lutskboy
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:39
    чтоб не удалять выстави права и пользователя какие там идут на файлы. .
  • Gordi | Перенёс сайт - не работают кнопки, не от ... 8
    Фото Gordi
    Gordi
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:37
    т.е. мне снести всё, что я уже загрузил как root, зайти под юзером и закачать снова?
    Видосы в директории upload можно хоть оставить? их там 80 Гб
  • lutskboy | Перенёс сайт - не работают кнопки, не от ... 8
    Фото lutskboy
    lutskboy
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:17
    фтп обычный юзер а не root
  • Gordi | Перенёс сайт - не работают кнопки, не от ... 8
    Фото Gordi
    Gordi
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:15
    дэк только так и загружаю
  • Gordi | Как реализовать шифрование трафика? 3
    Фото Gordi
    Gordi
    Как реализовать шифрование трафика?
    18-11-2024, 14:14
    я тоже не понимаю....товарищ один, представитель хостинга сказал, что даже есть модули такие специальные под популярные цмс
  • lutskboy | Перенёс сайт - не работают кнопки, не от ... 8
    Фото lutskboy
    lutskboy
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:13
    Gordi,
    через фтп загружай
  • Gordi | Перенёс сайт - не работают кнопки, не от ... 8
    Фото Gordi
    Gordi
    Перенёс сайт - не работают кнопки, не отображаются новости. Как вылечить?
    18-11-2024, 14:11
    Привет Николай!
    Всегда загружал от root... посмотришь может? твой плагин тоже не виден ((
    или направь на путь истинный, пжст
наверх