Нужен совет. Насколько мое решение лучше/хуже официального?

Question

[Nerevar96]

Решаю задачи из учебника.
Суть задачи: Напишите функцию pow(x,n), которая возвращает x в степени n. Иначе говоря, умножает x на себя n раз и возвращает результат.

Решение из учебника:

function pow(x, n) {
  let result = x;

  for (let i = 1; i < n; i++) {
    result *= x;
  }

  return result;
}

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

if (n < 1) {
  alert(`Степень ${n} не поддерживается, используйте натуральное число`);
} else {
  alert( pow(x, n) );
}

Вот мое решение:

let x = prompt('введи число', '')
let n = prompt('введи степень', '')

if (n < 0) {
    alert('степень должна быть больше нуля')
}

function pow(x, n) {
    return x ** n;
}

let result = x ** n
alert(result)
pow(x, n);

Мне мое решение кажется удачнее, элегантнее что ли... Я не пойму, зачем в этой задаче в принципе нужен цикл?
Можете пожалуйста написать комменты, что с моим решением не так? Или так? Что в нем плохого и хорошего?

Comments

[Lynn «Кофеман»]

Ну вы не написали функцию, а взяли готовую. К тому ещё и не вызвали её.

Как написано в ответе ниже, цель задания-то была научить вас чему-то. Само собой что в реальной программе вы не будете писать такую функцию.

[Сергей delphinpro]

В обучающих заданиях вы должны демонстрировать знания алгоритмов, а не использования готовых функций.

[Сергей delphinpro]

К примеру дадут вам задние вычислить 20 знаков числа пи. Написав в решении следующее, вы его провалите:

console.log(
  Math.PI.toFixed(20)
)

Answer 1

[Сергей Соколов]

ваше решение с комментариями:

// если это не сайт барбершопа, лучше обращаться к клиенту на «Вы»
let x = prompt('введи число', '')
let n = prompt('введи степень', '')

// не забывайте, что prompt() всегда возвращает строку, а не число

if (n < 0) {
    // сообщение вывели, но выполнять всё равно продолжили
    alert('степень должна быть больше нуля')
}

// переизобрели встроенную функцию Math.pow()
function pow(x, n) {
    return x ** n; // возвели в степень 1-й раз
}

let result = x ** n  // возвели в степень 2-й раз
alert(result)
// напоследок таки функцию вызывали, но результат — вникуда
pow(x, n);

Comments

[Nerevar96]

Переписал код с учетом комментариев. Да.. действительно мое предыдущее решение выглядит очень глупо( особенно возведение в степень 2 раза, и вызов функции который уже не имел смысла
Единственное, что я не знал на момент выполнения задания, что есть функция Math.pow, поэтому такой вариант я посчитал приемлемым.
Вот новое решение:

let x = +prompt('введите число', '')
let n = +prompt('введите степень', '')

let pow = (x, n) => x ** n;

if (n < 0) {
    alert(`Степень ${n} должна быть натуральной`)
} else {
    alert( pow(x, n) )
}

Уже гораздо лучше, осталось только написать свою функцию, точнее разобраться как это лучше сделать

[Сергей Соколов]

Nerevar96, попробуйте написать варианты:

1. с циклом, в лоб
   
2. с рекурсией (функция вызывает саму себя снова и снова)
   
3. с оптимизацией, чтобы минимум умножений, как описал в ответе rPman Возможно тут поможет разложить степень на биты.
   

Answer 2

[rPman]

использовать готовую функцию будет некрасиво, в правильной 'школе' за такой даже безупречный ответ должны ставить оценку 'хорошо'

для 'отлично' я предлагаю написать функцию, которая будет заметно эффективнее и вместо N умножений будет делать примерно log(N)
для этого попытаться представить возведение в степень не как N раз X*X*...*X а как (N/2 раз X*X*..*X)^2, а чтобы это делать в цикле, поступить так, если N четное, то делим его на два, вызываем рекурсивно свою же функцию и результат умнажаем на самого себя, иначе берем N-1 и делаем тоже самое но результат дополнительно умножаем на X

p.s. полгаю если сумеешь заменить рекурсию на цикл, будет однозначно плюсищем