Проверка кода на корректность

@Aqua77 · Регистрация: 13.07.2015

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

Сделал выподающее меню. Всё работает, меня только один вопрос гложет: всё ли корректно, может код считаться хорошим?

HTML5
1
2
3
4
5
6
7
8
9
<ul class="q1">
 
<li class="q2">
<a href="#">123</a><ul class="q3"><li><a href="#">111</a></li><li><a href="#">222</a></li><li><a href="#">333</a></li></ul>
</li>
 
<li class="q2"><a href="#">123</a><ul class="q3"><li><a href="#">111</a></li><li><a href="#">222</a></li><li><a href="#">333</a></li></ul></li>
</ul>
 Выполни

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
* {
  box-sizing: border-box;
  margin:0;
  padding:0;  
}
.q1{
    margin-left:40px;
    margin-top:50px;
    border:1px black solid;
    text-align:center;
}
.q2{
    display:inline-block;
    position:relative;
    vertical-align:top;
    z-index:1000;
    overflow:hidden;
    height:42px;
    transition:all 1s;
    width:17%;
}
.q2 a{
    display:inline-block;
    text-decoration:none;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:black;
}
.q3{
    position:absolute;
    top:-90px;
    right:0;    
    width:100%;
    transition:all 1s;
    z-index:-1;
}
.q3 li{
    display:block;
}
.q3 a{
    display:inline-block;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:red;
    margin-bottom:3px;
}
.q2:hover .q3{
    top:45px;
}
.q2:hover{
    height:177px;
}
.q3 li:last-child a{
    margin-bottom:0;
}

@s_ · 18.11.2016, 10:43

Aqua77, Вы можете проверит свой код ... https://validator.w3.org/

@Fedor92 · 18.11.2016, 11:05

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

может код считаться хорошим?

Как минимум с точки зрения, читабельности - нет...

@Aqua77 · 18.11.2016, 21:53 **[ТС]**

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

с точки зрения, читабельности - нет

отел чтобы подменю плавно выползало и не влияло на нижестоящий контент, ещё и резиновое. В общем вот что я сделал https://jsfiddle.net/2ohktw18/2/

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
* {
  box-sizing: border-box;
  margin:0;
  padding:0;  
}
.q1{
    position:absolute;
    width:100%;
    top:0;left:0;
}
.q1>li{
    display:inline-block;
    position:relative;
    vertical-align:top;
    z-index:1000;
    overflow:hidden;
    height:42px;
    transition:all 1s;
    width:17%;
}
.q1>li>a{
    display:inline-block;
    text-decoration:none;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:black;
}
.q2{
    position:absolute;
    top:-90px;
    left:0; 
    width:100%;
    transition:all 1s;
    z-index:-1;
}
.q2>li{
    display:block;
}
.q2>li> a{
    display:inline-block;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:red;
    margin-bottom:3px;
}
.q1>li:hover .q2{
    top:45px;
}
.q1>li:hover{
    height:177px;
}
.q2>li:last-child a{
    margin-bottom:0;
}
nav{
    position:relative;
    height:48px;
    margin:50px 0 0 20px;
}

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 <nav>
 <ul class="q1">
 <li>
    <a href="#">123</a>
        <ul class="q2">
        <li><a href="#">111</a></li>
        <li><a href="#">222</a></li>
        <li><a href="#">333</a></li>
        </ul>
</li>
 <li>
    <a href="#">123</a>
        <ul class="q2">
        <li><a href="#">111</a></li>
        <li><a href="#">222</a></li>
        <li><a href="#">333</a></li>
        </ul>
</li>
</ul>
</nav>
<p>23212123</p>

Добавлено через 6 часов 53 минуты
Чтот не общительный чат

@mrtoxas · 18.11.2016, 22:15

Aqua77, что вы хотите услышать? Вам уже сказали про корректность в третьем сообщении и про валидность - во втором. Вы не понимаете что такое нечитабельный код? Попробуйте взглянуть на него как будто в первый раз. Вам там что-то понятно? Что это за классы q1,q2? Вот сколько времени постороннему человеку нужно, чтобы понять что ищет селектор .q1>li:hover .q2?
Итак, ответ на главный вопрос темы "может код считаться хорошим?" - нет, не может.
Дайте элементам "говорящие" наименования классов и возвращайтесь.

@Aqua77 · 19.11.2016, 00:21 **[ТС]**

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

нечитабельный код

Я имел в виду годноту кода в том смысле, что логично построены все значения в CSS для реализации резиново-выподающего меню, а давать логичные названия классам я принебрег. Впредь буду задумываться над этим больше. Вот окончательный вариант: https://jsfiddle.net/2ohktw18/4/ Я там ещё не успел задать имена классам

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
* {
  box-sizing: border-box;
  margin:0;
  padding:0;  
}
.q1{
    position:absolute;
    width:100%;
    top:0;left:0;
    overflow:hidden;
}
.q1>li{
    display:inline-block;
    position:relative;
    vertical-align:top;
    z-index:1000;
    height:42px;
    transition:all 1s;
    width:17%;
}
.q1>li>a{
    display:inline-block;
    text-decoration:none;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:#FF6633;
}
.q2{
    position:absolute;
    top:-90px;
    left:0; 
    width:100%;
    transition:all 1s;
    z-index:-1;
}
.q2>li{
    display:block;
}
.q2>li> a{
    display:inline-block;
    padding:12px 0px;
    width:100%;
    text-align:center;
    background:#336699;
    margin-bottom:3px;
}
.q1>li:hover .q2{
    top:45px;
}
.q1>li:hover{
    height:177px;
}
.q2>li:last-child a{
    margin-bottom:0;
}
nav{
    position:relative;
    height:48px;
    margin:50px 0 0 20px;
}
.q2>li{
    position:relative;
    width:200%;
    z-index:1000;
}
.q2>li>a{
    width:50%;
}
 
.q2>li>a.q4{
    position:absolute;
    top:0; left:0;
    background:blue;
    width:50%;
    transition:all 1s;
    z-index:-1;
}
.q2>li:hover a.q4{
    left:50%;
}

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 <nav>
 <ul class="q1">
 <li>
    <a href="#">123</a>
        <ul class="q2">
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        </ul>
</li>
 <li>
    <a href="#">123</a>
        <ul class="q2">
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        <li><a href="#">333</a><a href="#" class="q4">123</a></li>
        </ul>
</li>
</ul>
</nav>
<p>23212123</p>

Комментарий модератора

Код из песочницы дублируем на форуме!

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .	SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .

@s_ 2 / 2 / 4 Регистрация: 16.11.2016 Сообщений: 7
	18.11.2016, 10:43
	Aqua77, Вы можете проверит свой код ... https://validator.w3.org/ 0

@mrtoxas 3827 / 2677 / 1521 Регистрация: 12.07.2015 Сообщений: 6,672 Записей в блоге: 4
	18.11.2016, 22:15
	Aqua77, что вы хотите услышать? Вам уже сказали про корректность в третьем сообщении и про валидность - во втором. Вы не понимаете что такое нечитабельный код? Попробуйте взглянуть на него как будто в первый раз. Вам там что-то понятно? Что это за классы q1,q2? Вот сколько времени постороннему человеку нужно, чтобы понять что ищет селектор `.q1>li:hover .q2`? Итак, ответ на главный вопрос темы "может код считаться хорошим?" - нет, не может. Дайте элементам "говорящие" наименования классов и возвращайтесь. 1