Как правильно написать (оптимизировать) код для меню?

@Astrox · Регистрация: 20.12.2008

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

Есть менюшка из заголовков, сам список выежает при нажатии на заголовок (метод .slideToggle ). Назначил каждому списку и заголовку свои идентификаторы и вот получается что я использую тот же код для каждого идентификатора, можно ли както оптимизировать? Я пока начал изучать скрипты, первое что приходит в голову это использовать циклы и\или регулярные выражения, но как не знаю.

Структура:

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 
<head>
    <meta http-equiv="content-type" content="text/html; charset=utf-8" />
    <link rel="stylesheet" type="text/css" href="style.css" />
    <script type="text/javascript" src="jquery.js"></script>
    <script type="text/javascript" src="scripts.js"></script>
    <title>МЕНЮ</title>
</head>
 
<body>
    <div id="menu">
        <div id="s1" class="sss">НОВОСТИ</div>
            <ul class="uli" id="uli1">
                <li><a href="#">НОВОСТИ1</a></li>
                <li><a href="#">НОВОСТИ2</a></li>
                <li><a href="#">НОВОСТИ3</a></li>
            </ul>
        <div id="s2" class="sss">НОВОСТИ</div>
            <ul class="uli" id="uli2">
                <li><a href="#">НОВОСТИ1</a></li>
                <li><a href="#">НОВОСТИ2</a></li>
                <li><a href="#">НОВОСТИ3</a></li>
            </ul>
    </div>
</body>
</html>

Таблица стилей:

CSS
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
#menu {
    background-color: gray;
    width: 200px;
    padding:10px;
}
.sss {
    margin: 10px;
    padding: 10px;
    border:  1px solid red;
}
ul {
    list-style-position: inside;
    margin: 0 0 0 10px;
    padding: 0;
}
li {
    margin: 0px;
    padding: 5px 10px;
    border-bottom:1px dotted #FFDD00;
}
.uli {
    display:none; width:180px;
}
.sss:hover {
    background-color: white;
}

Скрипт:

JavaScript
1
2
3
4
5
6
7
8
9
$(document).ready(function () {
    $('#s1').click(function () {
    $('ul#uli1').slideToggle('medium');
    });
    $('#s2').click(function () {
    $('ul#uli2').slideToggle('medium');
    });
    
});

Так вот в скрипте видно что код повторяется, меняются только идентификаторы, и это меня смущает

Есть ли какоето решение?

Файлы прикрепил.

@Astrox · 14.03.2012, 20:16 **[ТС]**

Что вообще все "ноль"?

@Egoist0 · 14.03.2012, 20:20

что-то я струдом понял что тебе надо, попробуй сделать через анимацию для высоты задай что в течении секунды будет высота 100% а при втором нажатии высота 0 или если допустим у блока есть бордер или у ли есть бордер то можно для блоков ставить мин высоту после чего просчитать сколько ли в блоке взять ихнию всю высоту в переменую и кинуть в анимацию после чего от заданых параметров уже регулировать как оно будет выходить заходить

@Astrox · 14.03.2012, 21:00 **[ТС]**

Мне нужно оптимизировать код, чтобы не повторялось вот так:

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
    $('#s2').click(function () {
    $('ul#uli2').slideToggle('medium');
    });
    $('#s3').click(function () {
    $('ul#uli3').slideToggle('medium');
    });
    $('#s4').click(function () {
    $('ul#uli4').slideToggle('medium');
    });
    $('#s5').click(function () {
    $('ul#uli5').slideToggle('medium');
    });
    $('#s6').click(function () {
    $('ul#uli6').slideToggle('medium');
    });
    $('#s7').click(function () {
    $('ul#uli7').slideToggle('medium');
    });

Видите для каждой выборки тот же код только идентификаторы меняются, нужно зделать покороче както...
Тоесть определить автоматически какой идентификатор был кликнут и выкатить соответствующий блок.
Нужно из 20 строк зделать пару-тройку))

@Daredevi1 · 14.03.2012, 21:38

JavaScript
1
2
3
$('.sss').click(function(){
  $(this).find('.uli').slideToggle('medium');
});

@Astrox · 14.03.2012, 21:43 **[ТС]**

Вопщем не дождался ни от кого помощи)
Целый день думал, пытался както выборки через родителя к следующему элементу зделать, но не удачно в итоге зделал так:

JavaScript
1
2
3
4
5
6
7
8
$(document).ready(function () {
 
    $('div.sss').click(function () {
        ch = $(this).attr('id');
        $('ul#uli' + ch.slice(1)).slideToggle('medium');
    })
 
});

Но если у когото есть идеи инной реализации, буду рад увидеть

Добавлено через 4 минуты
Daredevi1, Ваш код не работает

@Daredevi1 · 14.03.2012, 21:50

Извиняюсь, не увидел что ul вне div. Вот:

JavaScript
1
2
3
$('.sss').click(function(){
  $(this).next('.uli').slideToggle('medium'); 
});

@Weaver · 14.03.2012, 22:41

JavaScript
1
2
3
4
5
$(function() {
  $('#menu .sss').click(function() {
    $(this).next().slideToggle('medium');
  });
});

если вы не пользуетесь идентификаторами, которые определили в коде (#s1, #s2, #s3...) - удалите их, чего код захламлять ;]

Новые блоги и статьи Все статьи Все блоги /
Видеокарта простаивает ночами? Вот 4 проекта, которые загрузят её наукой Programma_Boinc 10.04.2026 Видеокарта простаивает ночами? Вот 4 проекта, которые загрузят её наукой Если на Windows стоит дискретная NVIDIA или AMD — можно отдать её вычислительную мощность реальным исследованиям. . . .	Очистка реквизитов документа при копировании Maks 09.04.2026 Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .	модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий anaschu 08.04.2026 https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI	Блокировка документа от изменений, если он открыт у другого пользователя Maks 08.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка. Hrethgir 08.04.2026 Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .	Модель ЗдрввоСохранения 7: больше работников, больше ресурсов. anaschu 08.04.2026 работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ	Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики. Hrethgir 07.04.2026 Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .	Горе от ума kumehtar 07.04.2026 Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .

@Astrox 13 / 13 / 2 Регистрация: 20.12.2008 Сообщений: 73
	14.03.2012, 20:16 [ТС]
	Что вообще все "ноль"? 0

@Egoist0 21 / 21 / 2 Регистрация: 17.02.2011 Сообщений: 303 Записей в блоге: 1
	14.03.2012, 20:20
	что-то я струдом понял что тебе надо, попробуй сделать через анимацию для высоты задай что в течении секунды будет высота 100% а при втором нажатии высота 0 или если допустим у блока есть бордер или у ли есть бордер то можно для блоков ставить мин высоту после чего просчитать сколько ли в блоке взять ихнию всю высоту в переменую и кинуть в анимацию после чего от заданых параметров уже регулировать как оно будет выходить заходить 0