使用 JavaScript 中的函数进行代码重构

Question

我正在尝试将 JS 代码重构为函数。我正在比较两个数组和 return 一个新数组，其中包含仅在两个给定数组之一中找到的任何项目，而不是两个。

下面的代码有效，但显然它不是 DRY 主体。

function diffArray(arr1, arr2) {
  let newArr = []
  for(let el2 of arr2) {
    if(!arr1.includes(el2)) {
      newArr.push(el2)
    }
  }
  for(let el1 of arr1) {
    if(!arr2.includes(el1)) {
      newArr.push(el1)
    }
  }
  return newArr;
}

我想将循环块变成一个函数，下面是我想出的。但是，它 return 是一个空数组。这里有什么问题？

let newArr = []

function singleEl(arrA, arrB) {
  for(let el of arrA) {
    if(!arrB.includes(el)) {
      newArr.push(el)
    }
  return newArr
}}

function diffArray(arr1, arr2) {
  singleEl(arr2, arr1)
  singleEl(arr1, arr2)
  return newArr;
}

  diffArray([1, "calf", 3, "piglet"], [1, "calf", 3, 4])

Answer 1

你的问题是return newArr在循环中。

function singleEl(arrA, arrB) {
  let newArr = [];
  for (let el of arrA) {
    if (!arrB.includes(el))
      newArr.push(el);
  }
  return newArr;
}

function diffArray(arr1, arr2) {
  let result = [];
  result = [...singleEl(arr1, arr2), ...singleEl(arr2, arr1)];

  return result;
}

console.log(diffArray([1, 2], [1, 5, 6]));
console.log(diffArray([1, "calf", 3, "piglet"], [1, "calf", 3, 4]));

使用 JavaScript 中的函数进行代码重构

Code refactor with functions in JavaScript

javascript

refactoring

function