函数被调用两次 - 一次具有正确的值，一次具有错误的值

Question

这是

第一篇文章，我也是编码新手，所以希望这个问题足够清楚。

我正在做一个基本的JavaScript练习。该程序在很大程度上按照我的需要工作，除了一个问题。

该程序会填充一系列警报。例如，如果您选择数学，然后在接下来的三个警报中输入：1、+、1，它将返回"您的答案是 2"(这是正确的(。然而，随后，它随后显示"呃哦！问题！！'，这是不对的。

我不知道为什么会这样？

我已经粘贴了下面的相关代码。此外，这里还有一个包含所有代码的 https://jsfiddle.net/poc4kzm9/。

//////////////// MATHS CALCULATOR //////////////////////////
// logic for maths calculation
function mathCalculate(operator, firstNumber, secondNumber) {
  console.log(operator);
  if (operator === '+') {
    add(firstNumber, secondNumber);
  } else if (operator === '-') {
    subtract(firstNumber, secondNumber);
  } else if (operator === '/') {
    divide(firstNumber, secondNumber);
  } else if (operator === '*') {
    multiply(firstNumber, secondNumber);
  } else if (operator === 'sq') {
    power(firstNumber, secondNumber);
  } else if (operator ==='sqrt') {
    squareRoot(firstNumber);
  } else {
    alert('Uh oh! Problem!!');
  }
}
// collect required values for math
function mathSetup() {
  // Set intial values for math calculator
  const firstNumber = prompt('What is your first number?');
  const operator = prompt('Which operator would you like to use? (+, -, /, *, sq, sqrt)');
  let secondNumber;
  // No need for a second number if sqrt-ing the first value
  if (operator !== 'sqrt') {
    secondNumber = prompt('What is your second number?');
  }
  mathCalculate(operator, firstNumber, secondNumber);
}
// The various functions by operator, one for each operator
function add(a,b) {
  alert('Answer is ' + (parseFloat(a) + parseFloat(b)));
}

任何帮助将不胜感激。

Answer 1

您调用mathCalculate方法两次。

第一次是在mathSetup，另一个是开关中的主回路。第二次在没有参数的情况下调用它。这就是为什么你会对问题保持警惕。

要解决此问题，您必须从switch中删除matchCalculate。另一种方法是从mathSetup返回数据并在mathCalculate中使用它

我还做了小提琴：https://jsfiddle.net/5bh9ndhc/

我从mathSetup返回了数据，并在mathCalculate中使用它：

const [operator, firstNumber, secondNumber] = mathSetup();
mathCalculate(operator, firstNumber, secondNumber);

请注意，我使用了解构赋值符。此语法是 ECMAScript 2015 的一部分，因此它只能在少数现代浏览器中使用。

UPD：我强烈建议在这种情况下使用debugger关键字。你的代码有很多行，我一生中第一次看到它，但由于debugger关键字和Chrome Devtools，我能够在30秒内找到错误。

Answer 2

你调用了两次计算函数

一个在设置功能内

function mathSetup() {
  // Set intial values for math calculator
  const firstNumber = prompt('What is your first number?');
  const operator = prompt('Which operator would you like to use? (+, -, /, *, sq, sqrt)');
  let secondNumber;
  // No need for a second number if sqrt-ing the first value
  if (operator !== 'sqrt') {
    secondNumber = prompt('What is your second number?');
  }
  mathCalculate(operator, firstNumber, secondNumber);
}

和设置后

  mathSetup();
  mathCalculate();

从开关大小写中删除计算函数 此时您没有有关计算的信息。小提琴链接 - https://jsfiddle.net/ezmhc0ff/