zpět k lekci

Špatný styl

důležitost: 4

Co je špatného na stylu níže uvedeného kódu?

function mocnina(x,n)
{
  let výsledek=1;
  for(let i=0;i<n;i++) {výsledek*=x;}
  return výsledek;
}

let x=prompt("x?",''), n=prompt("n?",'')
if (n<=0)
{
  alert(`${n}-tá mocnina není podporována, zadejte prosím celé číslo větší než nula`);
}
else
{
  alert(mocnina(x,n))
}

Opravte ho.

Měli byste si všimnout následujícího:

function mocnina(x,n)  // <- chybí mezera mezi argumenty
{  // <- levá složená závorka na zvláštním řádku
  let výsledek=1;   // <- chybějí mezery před a za =
  for(let i=0;i<n;i++) {výsledek*=x;}   // <- chybějí mezery
  // obsah { ... } by měl být na novém řádku
  return výsledek;
}

let x=prompt("x?",''), n=prompt("n?",'') // <-- technicky je to možné,
// ale lepší je to rozdělit na 2 řádky, navíc tam chybějí mezery a ;
if (n<=0)  // <- chybějí mezery uvnitř (n <= 0) a nad ním by měl být prázdný řádek
{   // <- levá složená závorka na zvláštním řádku
  // níže - dlouhé řádky by měly být rozděleny na více řádků pro lepší čitelnost
  alert(`${n}-tá mocnina není podporována, zadejte prosím celé číslo větší než nula`);
}
else // <- toto může být na jediném řádku: "} else {"
{
  alert(mocnina(x,n))  // chybějí mezery a ;
}

Opravená varianta:

function mocnina(x, n) {
  let výsledek = 1;

  for (let i = 0; i < n; i++) {
    výsledek *= x;
  }

  return výsledek;
}

let x = prompt("x?", "");
let n = prompt("n?", "");

if (n <= 0) {
  alert(`${n}-tá mocnina není podporována,
    zadejte prosím celé číslo větší než nula`);
} else {
  alert( mocnina(x, n) );
}