Estou assumindo que é um exercício (ou um código para treinar), já que para calcular potências, é melhor usar Math.pow
, conforme sugerido na outra resposta (inclusive, ele trata outros casos que o seu código não, mas veremos isso em detalhes mais abaixo).
O problema é que sua variável resultado
começa com o valor zero. Então ao chegar nesse loop:
for (var i = 0; i < potencia; i++){
resultado *= numero;
}
Você estará multiplicando o número por zero várias vezes, e o resultado será zero (uma dica é fazer o teste de mesa, que erros como esse são descobertos rapidamente).
Para corrigir é simples, basta inicializar resultado
com 1 (que é o elemento neutro da multiplicação).
Outro detalhe é que na validação você estava fazendo:
if(numero !== undefined || potencia !== undefined){
throw new Error("Não aceitamos valores undefined...");
}
Ou seja, se numero
não for undefined
, ou se potencia
não for undefined
, dá erro. Mas o certo seria verificar se eles são undefined
, usando ===
em vez de !==
.
Mas na verdade você não precisa desse if
, pois anteriormente já foi usado parseInt
para transformar o texto digitado em número, então as variáveis não serão undefined
. Se a pessoa não digitar um número válido, parseInt
retorna NaN
, então basta verificar isso (aliás, é melhor verificar se é NaN
antes de ver se o valor é zero, pois não faz sentido testar se o valor é menor que zero se é NaN
).
Você também poderia evitar variáveis globais e fazer a função que lê os dados retornar os números lidos.
O código ficaria assim:
function lerDados(){
let numero = parseInt(prompt("Qual número será elevado?"));
let potencia = parseInt(prompt("Elevado a qual potência?"));
return validar(numero, potencia);
}
function validar(numero, potencia){
// verifica antes se é NaN (não faz sentido testar se é < 0 antes de saber se é NaN)
if (isNaN(numero) || isNaN(potencia)){
throw new Error("Os valores inseridos não são válidos...");
}
if (numero < 0 || potencia < 0){
throw new Error("Calcula somente números positivos...");
}
// retorna um array com os valores
return [numero, potencia];
}
function calcular(){
// obter os valores usando a sintaxe de destructuring assignment
// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/Destructuring_assignment
let [numero, potencia] = lerDados();
let resultado = 1;
for (let i = 0; i < potencia; i++){
resultado *= numero;
}
alert("O resultado é "+ resultado);
}
calcular();
Repare também que usei let
em vez de var
para declarar as variáveis. Para saber a diferença, veja esta pergunta.
Também coloquei nas verificações se a potência é negativa (já que você não está calculando isso). Mas se quiser expoentes negativos, basta mudar o cálculo para:
if (potencia < 0) {
potencia *= -1;
numero = 1 / numero;
}
for (let i = 0; i < potencia; i++){
resultado *= numero;
}
Por fim, não incluí uma verificação especial quando a base e o expoente são zero (já que zero elevado a zero é indefinido). Mas se quiser, pode incluir:
if (numero === 0 && potencia === 0) {
throw new Error("Zero elevado a zero é indefinido");
}
Sobre a conversão de string para número
A outra resposta dizia (antes de ser editada) para usar +
em vez de parseInt
para converter o texto digitado para número. Embora o resultado seja o mesmo para números inteiros, existem casos em que pode haver diferença).
Por exemplo, se a pessoa digitar 2.5
, a conversão com parseInt
resulta em 2
, enquanto com +
resulta em 2.5
:
let texto = "2.5";
console.log(parseInt(texto)); // 2
console.log(+texto); // 2.5
Para a base, não há problema ter um número com várias casas decimais, mas no seu caso, como você está fazendo um loop simples e multiplicando uma determinada quantidade de vezes, seu código não serve para expoentes com casas decimais.
Ou seja, seu código lidaria bem com 2.5
elevado a 3
(pois 2.5 seria multiplicado 3 vezes), mas não com 3
elevado a 2.5
, pois parseInt
vai arredondar a potência para 2
.
Mas o pior é se você converter com +
, pois nesse caso o for
iria multiplicar 3 vezes:
let numero = +"3";
let potencia = +"2.5";
let resultado = 1;
for (let i = 0; i < potencia; i++){
resultado *= numero;
}
console.log(resultado); // 27
Como o +
converte a string para 2.5, o for
itera 3 vezes (já que na terceira iteração, ele compara i < potencia
, e como i
vale 2
e potencia
vale 2.5
, ele faz uma iteração a mais do que deveria). Por isso o resultado é 27.
Sendo assim, para a base, não há problema em ter casas decimais, e você pode usar parseFloat
, que lê tanto números inteiros quanto com casas decimais. Já para o expoente, é melhor manter parseInt
mesmo, já que seu código só lida com expoentes inteiros:
let numero = parseFloat(prompt("Qual número será elevado?"));
let potencia = parseInt(prompt("Elevado a qual potência?"));
Claro que você pode usar +
em vez de parseFloat
para ler a base, mas eu ainda prefiro usar as funções, cujos nomes deixam bem claro o que está acontecendo. Um código mais claro e expressivo é melhor que um código mais curto (e nesse caso nem é tão mais curto assim).
Se quiser considerar expoentes como 2.5
, aí é melhor usar Math.pow
mesmo, que já sabe lidar com todos esses casos.
Tem ainda outro caso, que é quando o usuário só dá um ENTER no prompt
. Nesse caso, a string será vazia, e usar +
dá resultados diferentes de parseInt
e parseFloat
:
let texto = "";
console.log(parseInt(texto)); // NaN
console.log(parseFloat(texto)); // NaN
console.log(+texto); // 0
parseInt
e parseFloat
retornam NaN
, ou seja, se o usuário digitar uma string vazia, o código de validação dirá que as entradas são inválidas. Mas usar +
retorna zero, então no fim ele considera que uma string vazia é uma entrada válida.
O mesmo vale para quando a string só tem espaços (" "
). Então usar um ou outro faz diferença sim, principalmente nesses corner cases, que você tem que decidir se vai considerar válidos ou não. Se você pede que o usuário digite alguma coisa, o código tem que estar preparado para validar essa coisa, e só prosseguir se os dados forem válidos.
Enfim, não troque um método por outro só porque é "mais curto", ou porque parece "mais legal". Saiba as implicações de cada um e use o que fizer mais sentido para o que você pretende fazer.