Skip to content

Commit 5f3271c

Browse files
Merge pull request #183 from alfill/code-style
1-js/03-code-quality/02-coding-style
2 parents 92c9a5b + 24c34c3 commit 5f3271c

5 files changed

Lines changed: 142 additions & 122 deletions

File tree

1-js/03-code-quality/02-coding-style/1-style-errors/solution.md

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,29 @@
11

2-
You could note the following:
2+
Вы могли заметить следующие недостатки, сверху вниз:
33

44
```js no-beautify
5-
function pow(x,n) // <- no space between arguments
6-
{ // <- figure bracket on a separate line
7-
let result=1; // <- no spaces before or after =
8-
for(let i=0;i<n;i++) {result*=x;} // <- no spaces
9-
// the contents of { ... } should be on a new line
5+
function pow(x,n) // <- отсутствует пробел между аргументами
6+
{ // <- фигурная скобка на отдельной строке
7+
let result=1; // <- нет пробелов вокруг знака =
8+
for(let i=0;i<n;i++) {result*=x;} // <- нет пробелов
9+
// содержимое скобок { ... } лучше вынести на отдельную строку
1010
return result;
1111
}
1212

13-
let x=prompt("x?",''), n=prompt("n?",'') // <-- technically possible,
14-
// but better make it 2 lines, also there's no spaces and missing ;
15-
if (n<0) // <- no spaces inside (n < 0), and should be extra line above it
16-
{ // <- figure bracket on a separate line
17-
// below - long lines can be split into multiple lines for improved readability
18-
alert(`Power ${n} is not supported, please enter an integer number greater than zero`);
13+
let x=prompt("x?",''), n=prompt("n?",'') // <-- технически допустимо,
14+
// но лучше написать в 2 строки, также нет пробелов и точки с запятой
15+
if (n<0) // <- нет пробелов, стоит добавить отступ в одну строку сверху
16+
{ // <- фигурная скобка на отдельной строке
17+
// ниже - слишком длинная строка, лучше разбить для улучшения читаемости
18+
alert(`Степень ${n} не поддерживается, введите целую степень, большую 0`);
1919
}
20-
else // <- could write it on a single line like "} else {"
20+
else // <- можно на одной строке, вместе: "} else {"
2121
{
22-
alert(pow(x,n)) // no spaces and missing ;
22+
alert(pow(x,n)) // вложенный вызов функции, нет пробелов и точки с запятой
2323
}
2424
```
2525

26-
The fixed variant:
26+
Исправленный вариант:
2727

2828
```js
2929
function pow(x, n) {
@@ -40,8 +40,8 @@ let x = prompt("x?", "");
4040
let n = prompt("n?", "");
4141

4242
if (n < 0) {
43-
alert(`Power ${n} is not supported,
44-
please enter an integer number greater than zero`);
43+
alert(`Степень ${n} не поддерживается,
44+
введите целую степень, большую 0`);
4545
} else {
4646
alert( pow(x, n) );
4747
}

1-js/03-code-quality/02-coding-style/1-style-errors/task.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ importance: 4
22

33
---
44

5-
# Bad style
5+
# Плохой стиль
66

7-
What's wrong with the code style below?
7+
Какие недостатки вы видите в стиле написания кода этого примера?
88

99
```js no-beautify
1010
function pow(x,n)
@@ -17,12 +17,10 @@ function pow(x,n)
1717
let x=prompt("x?",''), n=prompt("n?",'')
1818
if (n<=0)
1919
{
20-
alert(`Power ${n} is not supported, please enter an integer number greater than zero`);
20+
alert(`Степень ${n} не поддерживается, введите целую степень, большую 0`);
2121
}
2222
else
2323
{
2424
alert(pow(x,n))
2525
}
2626
```
27-
28-
Fix it.

0 commit comments

Comments
 (0)