在JavaScript中繁琐的问题

问题描述

     1. let inventory = [
  { candy: "Twizzlers",inStock: 180,weeklyAverage: 200 },{ candy: "Sour Patch Kids",inStock: 90,weeklyAverage: 100 },{ candy: "Milk Duds",inStock: 300,weeklyAverage: 170 },{ candy: "Now and Laters",inStock: 150,weeklyAverage: 40 }
];

// write the shouldWeOrderThisCandy function
function shouldWeOrderThisCandy(inventory){
  for (i = 0; i < inventory.length; i++)


if (inventory[i].inStock < inventory[i].weeklyAverage){ 
for (j = 0; j < inventory[i].weelyAverage.length; j++){
      return inventory[i].weeklyaverage[j] * 2;
   } else {
      return 0;
    }
  }
}

所以我想在JavaScript中做的是找出是否要订购更多糖果。到目前为止，我将“存货”与每周平均值进行比较，如果“存货”

解决方法

有两个问题立即引起我的注意。

第一个是您说要使用两个参数来调用该函数，但是编写函数的方式只接受一个。

我看到的第二个问题是在第二个for循环中。您拼写了“ weeklyAverage”。

我看到的第三个问题是，我什至看不到为什么您还有第二个for循环。看起来您正在尝试遍历weeklyAverage值的每个数字，然后将其乘以2，但是您可以简单地将weeklyAverage乘以2，而不是尝试逐个数字地进行操作。这也导致了第四个问题：

您将在第二个for循环内返回一个值。当代码到达该点时，它将立即返回那里的任何值，然后停止运行，因此，尽管您有一个希望运行多次的for循环，但它将只运行第一次迭代，然后返回。

因此，要使其正常工作，您需要做一些事情：

1. 更改函数声明，使其接受两个参数而不是一个。第一个参数可以是实际的库存对象（就像您现在设置它的方式），然后第二个参数可以是包含要查看的糖果名称的字符串。
2. 完成此操作后，您不再需要在函数中使用任何for循环，因为您将不再需要手动遍历库存对象中的每个糖果。相反，您需要找到一种可用于在库存对象内部搜索candy参数的方法。
3. 在库存对象中发现糖果后，只需比较inStock和weeklyAverage属性。如果inStock太低，则返回weeklyAverage乘以2，否则返回零。

,

1. 您需要接受商品名称作为第二个参数。
2. 您可以使用Array#find查找具有指定名称的库存项目。

let inventory = [
  { candy: "Twizzlers",inStock: 180,weeklyAverage: 200 },{ candy: "Sour Patch Kids",inStock: 90,weeklyAverage: 100 },{ candy: "Milk Duds",inStock: 300,weeklyAverage: 170 },{ candy: "Now and Laters",inStock: 150,weeklyAverage: 40 }
];
function shouldWeOrderThisCandy(inventory,candy){
  const obj = inventory.find(x=>x.candy===candy);
  if(obj?.inStock < obj?.weeklyAverage) return obj.weeklyAverage * 2;
  else return 0;
}
console.log(shouldWeOrderThisCandy(inventory,"Twizzlers"));