不好的风格
重要程度: 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(`Power ${n} is not supported, please enter an integer number greater than zero`);
}
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) // <- (n <= 0) 里面没有空格,并且应该在本行上面加一个空行
{ // <- 花括号独占了一行
// 下面的一行代码太长了,可以将其拆分成 2 行以提高可读性
alert(`Power ${n} is not supported, please enter an integer number greater than zero`);
}
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(`Power ${n} is not supported,
please enter an integer number greater than zero`);
} else {
alert( pow(x, n) );
}