如何检查这个 function 是否为真，或者如果为假则返回一个空数组？

Question

我正在看这个作业，我的问题是关于最后加粗的部分：

 // Do not edit the code below. var myGroceryList = ['chips', 'pizza', 'hotpockets', 'MtnDew', 'corndogs']; // Do not edit the code above.

在这里，我们将编写一个 function 来模拟购物和检查我们的杂货清单上的东西并将新项目添加到我们的清单中。

编写一个名为 removeItem 的 function ，给定两个 arguments，第一个是myGroceryList ，第二个是要从myGroceryList中删除的项目。 如果第二个参数（或要添加或删除的项目）与myGroceryList中的项目匹配，则从您的购物清单中删除该项目并返回新的、更新的购物清单。

完成此操作后，编写另一个名为addItem的 function 并给出两个 arguments，第一个是myGroceryList ，第二个是要添加到您的购物清单的项目。 在addItem中，将您传入的项目添加到myGroceryList ，然后返回新的、更新的购物清单。

在 removeItem 和 addItem 中检查“myGroceryList”和“item”arguments 是否真实。 如果不是，则返回一个空数组。

以下是调用函数的一些示例以及应返回的内容：

 removeItem(myGroceryList, 'chips') --> ['pizza', 'hotpockets', 'MtnDew', 'corndogs']; addItem(myGroceryList, 'Jerky') --> ['pizza', 'hotpockets', 'MtnDew', 'corndogs', 'Jerky']; removeItem(myGroceryList) --> []; addItem() --> [];

这是我的代码：

removeItem=(myGroceryList,item)=>{
   return myGroceryList.filter((thing)=>{
      return thing != item
   })
}

addItem=(myGroceryList, item)=>{
   myGroceryList.push(item);
   return myGroceryList;
} 

我怎样才能让这最后一步起作用？

Answer 1

只需添加一个if即可验证 arguments 的真实性：

removeItem=(myGroceryList,item)=>{
   if (!myGroceryList || !item) return [];
   return myGroceryList.filter((thing)=>{
      return thing != item
   })
}

不改变论点

当您被要求返回新列表时，您很好地使用了filter并且没有改变原始列表。 但是您最好将相同的原则应用于addItem ，它也应该返回新列表，因此最好不要改变给出的列表：

addItem=(myGroceryList, item)=>{
   if (!myGroceryList || !item) return [];
   return myGroceryList.concat(item); // don't use push, but concat
} 

或者……改变论点？

然而，在分配结束时给出的示例代码，没有使用 function 的返回值（除了控制台报告它），从第二个调用中我们可以看到“芯片”实际上已经从第一次调用removeItem的列表。

所以这意味着你必须让你的函数有副作用。

在这种情况下，您不应该使用filter或concat ，而是将第一个 function 更改为使用splice ：

removeItem=(myGroceryList,item)=>{
   if (!myGroceryList || !item) return [];
   let i = myGroceryList.indexOf(item);
   if (i >= 0) myGroceryList.splice(i, 1); // mutate the list
   return myGroceryList;
}

...并且您可以将您的版本与push for addItem一起使用：

addItem=(myGroceryList, item)=>{
   if (!myGroceryList || !item) return [];
   myGroceryList.push(item); // mutate...
   return myGroceryList;
} 

Answer 2

您可以检查是否为假并立即返回一个空数组，而不是检查是否为真。

   addItem=(myGroceryList, item) => {
      if(!myGroceryList) { return [] }
      myGroceryList.push(item);
      return myGroceryList;
   }