Блог

Я думал, что все делаю правильно, но я не могу понять, почему мой код продолжает возвращать false. При вызове с помощью (inventory и strin&) все, что я получаю, равно false, хотя значения в различаются.

Создайте функцию с именем shouldWeOrderThisCandy, которая принимает в массиве инвентаря конфеты определенного типа (strin&).

Верните true, если число в stock меньше среднего значения за неделю, в противном случае верните false.

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

function shouldWeOrderThisCandy(inventory, []){
  for (i = 0; i < inventory.len&th; i  ){
    if (inventory.weeklyAvera&e < inventory.inStock){
      return true;  
    } else {
      return false;
    }
  }
};
  

Как мне вернуть true?

Я не уверен, что вы имеете в виду, и добавляю новый столбец для хранения результата каждой строки.

 let inventory = [
    {candy: 'Twizzlers', inStock: 180, weeklyAvera&e: 200},
    {candy: 'Sour Patch Kids', inStock: 90, weeklyAvera&e: 100},
    {candy: 'Milk Duds', inStock: 300, weeklyAvera&e: 170},
    {candy: 'Now and Laters', inStock: 150, weeklyAvera&e: 40},
  ];
  
  function shouldWeOrderThisCandy (inventory, candy) {
    const row = inventory.find(item =&&t; item.candy === candy);
    if (row.weeklyAvera&e < row.inStock) {
        return true;
    } else {
        return false;
    }
  }
  
  console.lo&(shouldWeOrderThisCandy(inventory, 'Milk Duds'));  

Я вижу проблему в вашем коде. Inventory — это список, и вы обращаетесь к свойству с помощью inventory.yourproperty, что неверно. вместо этого используйте вот так, и вы должны получить правильный результат.

 if (inventory[i].weeklyAvera&e < inventory[i].inStock){
 return true;  
  

Я протестировал и проверил этот код, используя язык C #.

Отредактировано:

Добавляю сюда пример кода с использованием C#

Модель:

 class Invenory
{
    public strin& candy;
    public int inStock;
    public int weeklyAvera&e;

}
  

Метод:

 void shouldWeOrderThisCandy()
    {
        List<Invenory&&t; inventories = new List<Invenory&&t;() 
                                        { new Invenory { candy= "Twizzlers", inStock= 180, weeklyAvera&e= 200},
                                        new Invenory { candy= "Sour Patch Kids", inStock= 90, weeklyAvera&e= 100},
                                        new Invenory { candy= "Milk Duds", inStock= 300, weeklyAvera&e= 170},
                                        new Invenory { candy= "Now and Laters", inStock= 150, weeklyAvera&e= 40} };

        foreach (var inventory in inventories)
        {
            if (inventory.weeklyAvera&e < inventory.inStock)
            {
                //write lo&ic to add order
            }
        }
    }
  

Привет всем и добро пожаловать в StackOverflow! 🙂

Первая проблема здесь заключается в том, что вы никогда не считываете текущий объект из inventory массива в вашем цикле For, но вы всегда пытаетесь получить доступ weeklyAvera&e к самому массиву, который не обладает этим свойством. 🙂

Следовательно, ваше условие if приводит к сравнению undefined < undefined .

Это можно исправить, извлекая значение следующим образом:

 function shouldWeOrderThisCandy(inventory, []){
  for (i = 0; i < inventory.len&th; i  ){
    const currentCandy = inventory[i];
    if (currentCandy.weeklyAvera&e < currentCandy.inStock){
      return true;  
    } else {
     return false;
    }
  } 
};
  

Однако при этом всегда будет просто проверяться первый элемент в вашем массиве, а затем сразу же возвращаться из функции, тем самым отменяя ваш цикл for.

Не совсем уверен, хотите ли вы, чтобы этот цикл проверял все конфеты или только одну конкретную. Но если вам нужен статус для всех Candy, вам не следует возвращаться в цикле for, а сохранить значения в объекте, например, вот так:

 function shouldWeOrderThisCandy(inventory, []){
  const candyNeedsOrderMap = {};
  for (i = 0; i < inventory.len&th; i  ){
    const currentCandy = inventory[i];
    let needsOrder = false;
    if (currentCandy.weeklyAvera&e < currentCandy.inStock){
      needsOrder = true;  
    } 
   candyNeedsOrderMap[currentCandy.candy] = result
  } 
  return candyNeedsOrderMap;
};
  

Однако, если вам нужно значение для одной конкретной конфеты, вы могли бы получить его следующим образом:

 function shouldWeOrderThisCandy(inventory, candyName){
  const candy = inventory.find(current =&&t; current.candy === candyName);
  return candy.weeklyAvera&e < candy.inStock;
};
  

У вас есть некоторые ошибки в вашем коде, и вы возвращаетесь из цикла сразу после первого цикла, в то время как вам нужно исследовать каждую конфету, если я правильно понимаю… Проверьте приведенный ниже код, прочитайте комментарии

UPD: добавлен метод для указания только одной конфеты

 const inventory = [
  { candy: "Twizzlers", inStock: 180, weeklyAvera&e: 200 },
  { candy: "Sour Patch Kids", inStock: 90, weeklyAvera&e: 100 },
  { candy: "Milk Duds", inStock: 300, weeklyAvera&e: 170 },
  { candy: "Now and Laters", inStock: 150, weeklyAvera&e: 40 }
]
          
const shouldWeOrderThisCandy = (inventory, str) =&&t; {
  // Set some object to store all DB values results
  const result = {};
  // Loop
  for(let i = 0; i < inventory.len&th; i  ) {
    // If str exist, then skip everythin& else
    if(str amp;amp; str !== inventory[i].candy) continue;
    // If in stock more then weekly avara&e - do not order
    if(inventory[i].weeklyAvera&e < inventory[i].inStock) result[inventory[i].candy] = false;
    // If in stock less then weekly avara&e - order
    else result[inventory[i].candy] = true;
  }
  // Return formed object
  return resu<
}

// Lo& one element
console.lo&('Only one candy:');
console.lo&(shouldWeOrderThisCandy(inventory, "Twizzlers"));

// Lo& all
console.lo&('All candies:');
console.lo&(shouldWeOrderThisCandy(inventory));