Плохой стиль
важность: 4
Какие недостатки вы видите в стиле написания кода этого примера?
function
pow
(
x,
n
)
{
let
result=
1
;
for
(
let
i=
0
;
i<
n;
i++
)
{
result*=
x;
}
return
result;
}
let
x=
prompt
(
"x?"
,
''
)
,
n=
prompt
(
"n?"
,
''
)
if
(
n<=
0
)
{
alert
(
`
Степень
${
n}
не поддерживается, введите целую степень, большую 0
`
)
;
}
else
{
alert
(
pow
(
x,
n)
)
}
Вы могли заметить следующие недостатки, сверху вниз:
function
pow
(
x,
n
)
// <- отсутствует пробел между аргументами
{
// <- фигурная скобка на отдельной строке
let
result=
1
;
// <- нет пробелов вокруг знака =
for
(
let
i=
0
;
i<
n;
i++
)
{
result*=
x;
}
// <- нет пробелов
// содержимое скобок { ... } лучше вынести на отдельную строку
return
result;
}
let
x=
prompt
(
"x?"
,
''
)
,
n=
prompt
(
"n?"
,
''
)
// <-- технически допустимо,
// но лучше написать в 2 строки, также нет пробелов и точки с запятой
if
(
n<=
0
)
// <- нет пробелов, стоит добавить отступ в одну строку сверху
{
// <- фигурная скобка на отдельной строке
// ниже - слишком длинная строка, лучше разбить для улучшения читаемости
alert
(
`
Степень
${
n}
не поддерживается, введите целую степень, большую 0
`
)
;
}
else
// <- можно на одной строке, вместе: "} else {"
{
alert
(
pow
(
x,
n)
)
// вложенный вызов функции, нет пробелов и точки с запятой
}
Исправленный вариант:
function
pow
(
x,
n
)
{
let
result =
1
;
for
(
let
i =
0
;
i <
n;
i++
)
{
result *=
x;
}
return
result;
}
let
x =
prompt
(
"x?"
,
""
)
;
let
n =
prompt
(
"n?"
,
""
)
;
if
(
n <=
0
)
{
alert
(
`
Степень
${
n}
не поддерживается,
введите целую степень, большую 0
`
)
;
}
else
{
alert
(
pow
(
x,
n)
)
;
}