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

@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...) - удалите их, чего код захламлять ;]

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 11.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@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