{{ message }}
Repository navigation
Expand file tree
/
Copy pathjavascript.html
More file actions
593 lines (446 loc) · 35.5 KB
/
Copy pathjavascript.html
File metadata and controls
593 lines (446 loc) · 35.5 KB
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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
<!DOCTYPE html>
<html lang="ru">
<head>
<meta charset="utf-8">
<title>Стиль кода Академии HTML</title>
<meta name="viewport" content="width=device-width, initial-scale=1">
<meta name="twitter:card" content="summary_large_image">
<meta name="twitter:site" content="@htmlacademy_ru">
<meta name="twitter:image" content="https://htmlacademy.github.io/codeguide/img/og.png">
<meta property="og:type" content="website">
<meta property="og:url" content="https://htmlacademy.github.io/codeguide/">
<meta property="og:title" itemprop="name" content="Стиль кода Академии HTML">
<meta property="og:image" itemprop="image" content="https://htmlacademy.github.io/codeguide/img/og.png">
<link rel="stylesheet" href="css/normalize.css">
<link rel="stylesheet" href="css/prism.css">
<link rel="stylesheet" href="css/style.css">
<script>(function(i,s,o,g,r,a,m){i['GoogleAnalyticsObject']=r;i[r]=i[r]||function(){(i[r].q=i[r].q||[]).push(arguments)},i[r].l=1*new Date();a=s.createElement(o),m=s.getElementsByTagName(o)[0];a.async=1;a.src=g;m.parentNode.insertBefore(a,m)})(window,document,'script','//www.google-analytics.com/analytics.js','ga');ga('create','UA-29640059-7','auto');ga('send','pageview');</script>
</head>
<body>
<header>
<a href="https://htmlacademy.ru" target="_blank">
<img class="logo" src="img/bane.svg" width="300" height="300" alt="HTML Academy" title="HTML Academy">
</a>
<h1>Стиль кода Академии HTML</h1>
<p><a href="https://github.com/htmlacademy/codeguide">github.com/htmlacademy/codeguide</a></p>
</header>
<main>
<!-- <section class="chapter">
<h2>Оглавление</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<h4>JS</h4>
<ul>
<li><a href="#html-syntax">Синтаксис</a></li>
</ul>
</div>
</article>
</section> -->
<section class="chapter">
<h2>Синтаксис</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Отступы, пробелы и переносы</h4>
<ul>
Для правильного форматирования используйте файл .editorconfig в вашем редакторе.
<li>Для отступов используются два пробела. Знак табуляции не используется. При переносах в продолжении строки используется четыре пробела.</li>
<li>Запрещено одновременное использование табов и пробелов для отступов.</li>
<li>Выражения должны заканчиваться точками с запятой.</li>
<li>Внутри строки не используется более одного пробела.</li>
<li>Открывающие скобки блоков кода находятся на одной строке с оператором, которых их использует
<pre class="language-js"><code>- if (condition)
- {
- // code
- }
+ if (condition) {
+ // code
+ }</code></pre>
</li>
<li>В однострочных блоках кода, код отделен от открывающей и закрывающей скобки пробелом</li>
<li>В однострочных массивах скобки не отделяются пробелами, первое значение идет непосредственно после закрывающей скобки, а закрывающая скобка идет непосредственно после последнего значения массива</li>
<li>После запятой всегда должен ставиться пробел, если запятая не в конце строки</li>
<li>Запрещено переносить запятую на новую строку при создании множественных переменных через один var, при объявлении массивов и объектов. Запрещено использовать запятую в конце списков, объектов или наоборов параметров функции.</li>
<li>Файл должен заканчиваться пустой строкой</li>
<li>Строка не должна заканчиваться пробелами</li>
<li>В однострочных объектах и при деструктуризации фигурные скобки отбиваются пробелами с двух сторон</li>
<li>В коде не используются совместно табы и пробелы</li>
<li>Блоки кода отделяются друг от друга не более чем двумя пустыми строками</li>
<li>В качестве символа строки используется стандартный символ, который используется в системе</li>
<li>При объявлении вычисляемых ключей в объектах с помощью синтаксиса ES2016 в квадратных скобках не используются пробелы</li>
<li>Оператор вызова функции () не отделяется круглыми скобками от названия функции</li>
<li>В объектах после объявления ключа не ставится пробел. После двоеточия, которое отделяет в объектах ключ от значения может стоять только один пробел, выравнивать несколько значений при объявлении многострочного объекта запрещено</li>
<li>Ключевые слова отделяются пробелами. Правило касается всех блоков, которые не оговорены отдельно в других правилах. В частности ключевые слова должны отделяться пробелами для условных операторов, циклов, операторов множественного выбора, блоков try..catch, объявлений классов и т. д.</li>
<li>В блоках кода первая и последняя строка не должны быть пустыми (код не отбивается от начала блока кода)</li>
<li>После двоеточий и точек с запятым ставятся пробелы. Перед ними — не ставятся.</li>
<li>Перед скобками начинающими новый блок кода должен ставиться пробел.</li>
<li>при объявлении анонимных функций, скобки с параметрами ставятся непосредственно после ключевого слова function. Если функция именована, перед названием ставится пробел, после — нет.</li>
<li>В комментариях текст отбивается пробелом от начала комментария.</li>
<li>При создании генераторов, звездочка идет сразу после ключевого слова function без пробелов</li>
<li>В spread-операторе точки не отделяются от названия коллекции</li>
<li>Звездочка после ключевого слова yield не отбивается пробелом. После звездочки проблел ставится всегда.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Строки</h4>
<ul>
<li>В строках, объявленных с помощью одинарных и двойных кавычек запрещено использовать переносы.</li>
<li>В строках запрещено указывать код спецсимволов в восьмеричной системе счисления <pre class="language-js"><code>- var foo = "Copyright \251";
+ var foo = "Copyright \u00A9";</code></pre></li>
<li>При создании объектов через литералы свойства должны объявляться в едином стиле: или без кавычек или с ними.</li>
<li>В строках используются одинарные кавычки. Разрешено использовать строковые шаблоны. Если позволяет поддержка, предпочтение отдается строковым шаблонам.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Числа</h4>
<ul>
<li>В числах запрещено опускать ноль в дробной и целой части.</li>
<li>Запрещено использовать ведущий ноль при создании чисел, поскольку это приводит к созданию числа в восьмеричной системе счисления.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Литералы</h4>
<ul>
<li>Массивы должны создаваться через литерал массива, а не через конструктор. Допустимое исключение — создание массива определенной длины.</li>
<li>Объекты должны создаваться через литерал объекта, а не через конструктор.</li>
<h5>Объекты</h5>
<li>При создании объектов запрещено дублирование ключей.</li>
<li>При описании функций запрещено дублировать названия параметров.</li>
<li>В операторе `switch` запрещено дублировать условия (case)</li>
<h5>Массивы</h5>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Операторы</h4>
<ul>
<li>В бинарных и тернарных операторах операнды и символы оператора отделяются пробелами.</li>
<li>Унарные операторы не отделяются от операнда пробелом. Исключения составляют операторы, состоящие из слов, а не символов, например оператор typeof.</li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>Именование</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<ul>
<li>Запрещено объявлять переменные, имена которых совпадают с именами используемых в коде лейблов</li>
<li>Все переменные должны быть названы в верблюжьем регистре. Исключения составляют константы которые должны именоваться прописными буквами в змеином регистре.</li>
<li>Запрещено использовать имена переменных, которые используются во внешних областях видимости.</li>
<li>Запрещено называть переменные и свойства ключевыми словами JS</li>
<li>Запрещено использовать переменные, не объявленные ранее. При использовании переменной, объявленной в другом модуле в глобальной области видимости нужно обращаться к ней как в свойству объекта `window/global`.</li>
<li>Запрещено объявлять переменные без значения.</li>
<li>Запрещено напрямую обращаться к значению `undefined`. Для проверки типа рекомендуется использовать `typeof`. Для прямого сравнения с `undefined` можно использовать конструкцию `void 0`</li>
<li>С заглавной буквы называются только функции-конструкторы.</li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>Предотвращение ошибок</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<ul>
<li>Все точки с запятой должны быть проставлены явно, не стоит рассчитывать на автоматическую расстановку точек с запятой ASI.</li>
<li>Не рекомендуется использовать указатель Unicode BOM в коде, потому что код должен быть сохранен не в кодировке UTF-16, а в кодировке UTF-8, в которой нет указателя порядка бит.</li>
<li>Обязательно используются блоки кода даже если в выражении содержится одна строчка</li>
<li>В коде нет пустых блоков кода</li>
<li>Код работает в строгом режиме: в начале всех файлов явно прописана директива `use strict`.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Условные операторы</h4>
<ul>
<li>Запрещена "проверка Йоды" — в условных операторах в блоке условия при сравнении переменной или свойства со значением сначала идет переменная или свойство объекта и только потом значения, а не наоброт. <pre class="language-js"><code>- if (1 === myValue) {}
+ if (myValue === 1) {}</code></pre></li>
<li>При сравнении двух значений используется оператор строгого сравнения вместо оператора нестрогого сравнения</li>
<li>В условных операторах не используется оператор присвоения</li>
<li>В условтях не используется небезопасное отрицание, например использование ! в in или instanceof без скобок.</li>
<li>Запрещено сравнение с NaN. Для проверки, является результат операции числовым, нужно использовать `isNaN`.</li>
<li>Запрещено использовать case без break или return в блоках switch.</li>
<li>Не используются вложенные тернарные операторы.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Функции</h4>
<ul>
<li>Если в функции используется ветвление, в котором есть возврат значения (`return`), `return` должен быть добавлен и в остальных ветках. Это правило предотвращает создание функций, при использовании которых непонятно, вернут ли они какое-то значение. <pre class="language-js"><code>- var do = function() {
- if (cond) {
- return 1;
- } else {
- // Проблема заключается в том, что
- // в первой ветке условия возвращается
- // единица, а в альтернативной нет никакого
- // возврата и при вызове функции неизвестно
- // будет ли из нее возвращего какое-то значение.
- doSomethingElse();
- }
- }
+
+ var do = function() {
+ if (cond) {
+ return 1;
+ } else {
+ return 2;
+ }
+ }</code></pre></li>
<li>В return не используется оператор присваивания.</li>
<li>В стрелочных функциях всегда используются скобки для параметров, даже если параметр один</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Операторы</h4>
<ul>
<li>В конструкторе классов-наследников обязательно вызывается super()</li>
<li>В конструкторе нет обращения к this до того, как будет вызван super()</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Объявление переменных и функций</h4>
<ul>
<li>При объявлении переменных предпочтение отдается ключевым словам let и const. Переменные объявляются через var только при наличии проблем с поддержкой кода.</li>
<li>Запрещено переопределять с помощью ключевого слова ранее созданные переменные</li>
<li>Запрещено использовать множественное объявление через одно ключевое слово. Для каждой переменной используется отдельный `var`, `let` или `const`. Это правило облегчает отладку, поскольку создание нескольких переменных через запятую трактуются движком как одна операция, что означает, что невозможно поставить отладчик на создание одной из переменных. <pre class="language-js"><code>- var a = 1, b = 2;
-
+ var a = 1;
+ var b = 2;
</code></pre></li>
</ul>
<h5 id="">Объявление функций</h5>
<ul>
'no-func-assign': 'error',
<li>Запрещено переопределение функций, созданных с помощью функционального объявления (function expression) <pre class="language-js"><code>- function myFunc() {};
- myFunc = 2;
</code></pre></li>
<li>Запрещено объявление значений внутри блоков</li>
<li>Новые функции не создаются с помощью конструктора `Function`</li>
<li>Вместо коллекции `arguments`, используется rest-оператор</li>
<li>Вместо вызова функции через `apply` используется spread-оператор</li>
</ul>
<h5 id="">Создание значений</h5>
<ul>
<li>Не вызываются служебные конструкторы, создающие объекты из глобальной области видимости `Math(), JSON(), Reflect()`.</li>
<li>При создании непустых массивов запрещено опускать пустые значения (ставить подряд несколько запятых)</li>
<li>При создании непустых объектов и массивов не используются «висячие» запятые</li>
<li>Не используются конструкторы примитивов, которые используются для автобоксинга `String`, `Number`, `Boolean`</li>
<li>Конструкторы вызываются со скобками, даже если у них нет параметров.</li>
<li>Символы (`Symbol`) создаются вызовом функции `Symbol` без ключевого слова `new`.</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Хорошие практики</h4>
<ul>
<li>Не используется конструкция `with`.</li>
<li>Не используются `alert`.</li>
<li>Не используется выполнение кода через `eval`.</li>
<li>Не используется неявный eval — в функциях, которые поддерживают передачу исполняемого кода как строку, например `setTimeout`.</li>
<li>Не используется протокол javascript.</li>
<li>Конструкторы не используются без ключевого слова `new`.</li>
</ul>
<h5>Манкипатчинг</h5>
<ul>
<li>Встроенные в язык объекты и прототипы не расширяются в рантайме</li>
<li>Не переопределяются глобальные значения (undefined, null, Object, window и прочие)</li>
</ul>
<h5>Контекст</h5>
<ul>
<li>bind не используется вне методов классов</li>
<li>в функциях, которые не являются методами никакого объекта или класса не используется `this`</li>
</ul>
<h5>Переменные</h5>
<ul>
<li>Переменные, объявленные через `var` не удаляются оператором `delete`</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Приведение типов</h4>
<ul>
<li>Не используется лишнее приведение к Boolean. Например, нет большого смысла переводить в boolean условия в конструкциях `if`, `while`, `for`, в первом операнде тернарного оператора</li>
<li>Оператор `typeof` используется корректно — используются только правильные значения, возвращаемые оператором, не производится сравнения со строковыми литералами, которые содержат некорректные значения `typeof`</li>
<li>В `parseInt` обязательно передается второй параметр — основание системы счисления, даже в случае с десятичной системой счисления</li>
</ul>
<h5>Константы в условиях</h5>
<ul>
<li>В блоки условия операторов if, while, for и тернарного оператора не передается константное значение, которое подразумевает, что условие выполняется (или не выполняется) всегда <pre class="language-js"><code>- if (true) {}
+ if (a > 1) {}</code></pre>
<pre class="language-js"><code>- var ternaryValue = true ? 'a' : 'b';
+ var ternaryValue = isA() ? 'a' : 'b';
</code></pre></li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>Чистый код</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Лишние символы (пробелы, точки с запятой и прочее)</h4>
<ul>
<li>Не используются лишние (множественные) точки с запятой</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Отладчик и консоль</h4>
<ul>
<li>В коде не используется оставленных выводов в консоль</li>
<li>В коде нет забытых инструкций debugger</li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Неиспользуемый код</h4>
<ul>
<li>
В проекте нет недоступного кода, который никогда не выполнится<pre class="language-js"><code>- if (false) {
- doSomething();
- }
+
+ if (needToDoSomething()) {
+ doSomething();
+ }</code></pre>
</li>
<li>В коде нет объявленных, но неиспользуемых переменных. Проверка не относится к параметрам функций.</li>
<li>
В коде нет выражений, значения которых не записываются в переменные, параметры функций, свойства объектов и не передаются как параметры функций <pre class="language-js"><code>- check === true && doSomething();
-
+ if (check) {
+ doSomething();
+ }</code></pre></li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Обработка ошибок и исключения</h4>
<ul>
<li>
Для выбрасывания исключения в оператор `throw` передаются только объекты Error. Передавать литералы запрещено.<pre class="language-js"><code>- throw 'Passed value is out of range';
+ throw new RangeError('Rassed value is out of range');</code></pre></li>
</ul>
<h5>try..catch с пустым try</h5>
<ul>
<li>В конструкции `try..catch` запрещен пустой блок `try`</li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>Документирование</h2>
<article class="chapter-part">
<div class="chapter-part-col">
При написании jsDoc используются следующие ограничения:
<ul>
<li>необязательно добавлять текстовое описание конструкциям @return и @param, достаточно просто указания типа и названия параметра для @param</li>
<li>описание возвращаемного типа @return требуется только в том случае, если функция возвращает какое-то значение. Правило не действует для конструкторов, для них указывать @return необязательно. Также необязательно указывать тег @return для функций, в которых конструкуия `return` используется без возвращаемого значения для выхода из функции</li>
<li>для описания возвращаемого значения из функции используется тег @return, а не @returns</li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>Прочее</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<ul>
<li>При итерировании по объектам через `for..in` при работе со свойствами используется конструкция `hasOwnProperty`</li>
<li>В функциях не используются обращения к `caller` и `callee`</li>
<li>В объектах напрямую не переопределяется свойство `__iterator__`</li>
<li>
В объектах напрямую не переопределяется свойство `__proto__`. Разрешено переопределять `__proto__` через `Object.create` или запись в прототип, объектов, созданных другими конструкторами, но напрямую редактировать `__proto__` нельзя<pre class="language-js"><code>- var obj = {
- __proto__: Parent
- };
+
+ var Obj = function() {};
+ Obj.prototype = new Parent();
+ var obj = new obj;
+
+ var obj = Object.create(Parent.prototype);</code></pre></li>
<li>В коде не используются лейблы. Лейблы используются с конструкциями `break` и `continue` для направленного выхода из цикла и могут привести к слишком сложному для понимания коду</li>
<li>Блоки, использующиеся в коде должны описывать тело функции, условия, оператора `switch` и прочие конструкции относящиеся к группировке множественных операций. Блоки не должны использваться сами по себе, любой блок должен быть частью другого выражения.</li>
<li>
В коде не используется оператор «запятая» для описания последовательностей действий. Для создания переменных используются отдельные ключевые слова `let`, `const`, `var`, в операторы группировки, условные операторы и прочие конструкции. Исключение составляет начальное условие оператора `for`<pre class="language-js"><code>- const result = (1, 2); // 2
+ const result = 2;
- switch (val = getVal(), val) {}
+ val = getVal();
+ switch(val) {}
+ for (var i = 0, l = 100; i < l; i++);</code></pre></li>
</ul>
</div>
</article>
<article class="chapter-part">
<div class="chapter-part-col">
<h4 id="">Регулярные выражения</h4>
<ul>
<li>В регулярных выражениях не используются «управляющие выражения»</li>
<li>В регулярных выражениях не используются пустые классы символов `[]` (блоки, ограниченные квадратными скобками)</li>
<li>В коде регулярные выражения не создаются через конструктор RegExp из строки, которая не может быть разобрана как правильное регулярное выражение</li>
<li>
В регулярных выражениях не используются нескольких последовательных пробелов. Вместо этого предлагается использовать модификаторы количества {3}
<pre class="language-js"><code>- / /.exec(myString);
+ /\s{3}/.exec(myString);</code></pre></li>
</ul>
</div>
</article>
</section>
<section class="chapter">
<h2>node.js</h2>
<article class="chapter-part">
<div class="chapter-part-col">
<ul>
<li>В node.js файлах не используется конструкция `process.exit`.</li>
</ul>
</div>
</article>
</section>
</main>
<footer>
<div class="contacts">
<a class="logo-full" href="https://htmlacademy.ru">
<img src="img/logo-full.svg" width="161" height="55" alt="HTML Academy" title="HTML Academy">
</a>
<div class="social-icons">
<a class="icon icon-vk" href="https://vk.com/htmlacademy" target="_blank">
Вконтакте
</a>
<a class="icon icon-fb" href="https://www.facebook.com/htmlacademy" target="_blank">
Facebook
</a>
<a class="icon icon-tw" href="https://twitter.com/htmlacademy_ru" target="_blank">
Twitter
</a>
<a class="icon icon-ig" href="https://instagram.com/htmlacademy" target="_blank">
Instagram
</a>
</div>
<div class="github">
<a class="github-link" href="https://github.com/htmlacademy" target="_blank">
GitHub
</a>
</div>
</div>
</footer>
<script src="js/prism.js"></script>
</body>
</html>
You can’t perform that action at this time.
