Плохой стиль
важность: 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) ); }