我如何重构我的 JS 函数以确保这个 console.log 结构有效？

Question

我正在使用这种方法来解决这个问题，但我的 console.log 仍然没有返回预期的结果，我应该改变什么？

const executeCalculator = ({ x, y, operation }) => {
  let calculator = {
    x: this.x,
    y: this.y,
    operation: {
      "sum": (x, y) => x + y,
      "subtract": (x, y) => x - y,
      "multiply": (x, y) => x * y,
      "divide": (x, y) => x / y,
    },
  }
  if (operation !== 'sum' || 'multiply' || 'subtract' || 'divide') {
    console.error('undefined operation');
  } else {
    return;
  };
};

console.log(executeCalculator({
  operation: 'sum',
  x: 1,
  y: 1
}));

Answer 1

• 首先，您的operation 对象格式错误，有一个多余的逗号
• 这不是必需的：

x: this.x,
y: this.y,

...因为您的函数参数中已经有了参数。

• "sum": (x, y)，x、y 参数不是必需的。
• 使用includes()检查是否有有效操作，条件如下：

!['sum', 'multiply', 'subtract', 'divide'].includes(operation)

• executeCalculator() 函数没有返回任何内容，您需要返回操作，这就是您得到 undefined 结果的原因。

const executeCalculator = ({ x, y, operation }) => {

  let calculator = {
    operation: {
      "sum": () => x + y,
      "subtract": () => x - y,
      "multiply": () => x * y,
      "divide": () => x / y
    }
  }
  if (!['sum', 'multiply', 'subtract', 'divide'].includes(operation)) {
    console.error('undefined operation');
  } else {
    return calculator.operation[operation]();
  };
};

console.log(executeCalculator({
  operation: 'sum',
  x: 1,
  y: 1
}));