Блог

Впервые публикую здесь. Я немного застрял. Назначение выглядит следующим образом:

Создайте функцию с именем priceLookup, чтобы найти цену одного товара. Функция должна принимать массив товаров и имя товара (строку) в качестве параметров и возвращать цену товара с этим именем. Если товар не может быть найден, верните строку «Товар с таким именем не найден»

** Предположим, что эти тестовые данные могут быть другими аргументами, введенными в функцию ** (я думаю, что я использовал правильную терминологию)

// Тестовые данные для проблем

 let items = [
  {
    itemName: "Effective Programming Habits",
    type: "book",
    price: 13.99
  },
  {
    itemName: "Creation 3005",
    type: "computer",
    price: 299.99
  },
  {
    itemName: "Finding Your Center",
    type: "book",
    price: 15.00
  }
]
  

Вот мой код и где я застрял:

 function priceLookup(items, name){
 let result = 0;
 for (i = 0; i < items.length; i  ){
   if (name == items[i].itemName){
   (result =  items[i].price)
   } else { result = "No item found with that name"
      }
  }
  return resu<
 }
  

Итак, я думаю, мой вопрос в том, почему мой оператор else отменяет if перед ним? Если я удалю это, я пройду тестирование при вызове имени, но потерплю неудачу, если имя не существует, и оно покажет «Товар не найден с этим именем»

 function priceLookup(items, name) {
  let result = 0;
  for (i = 0; i < items.length; i  ) {
    if (name == items[i].itemName) {
      result  =  items[i].price;
    }
  }
  if (result === 0) {
    return "No item found with that name";
  }
  return resu<
}
  

Когда вы найдете совпадение, немедленно верните его. Нет необходимости просматривать остальные товары.

Что касается «Товар не найден» … возвращайте его только после завершения цикла.

В настоящее время происходит то, что, как только найден элемент, который не соответствует, вы меняете результат с числа на строку сообщения, и цикл продолжает проходить через другие элементы в массиве

 function priceLookup(items, name) {

  for (let i = 0; i < items.length; i  ) {
    if (name === items[i].itemName) {
      // the return will break the loop and exit the function
      return items[i].price;
    }
  }
  // if loop completes no matches were found
  return "No item found with that name"
}
  

Другие примечания: не создавайте i глобальную переменную и не используйте === для проверки равенства

Массив.find — ваш друг. Обычно меньше ошибок и эффективнее использовать встроенный, а не использовать свой собственный.

Таким образом, ваша функция будет:

 function priceLookup(items, name) {
    let ret = items.find( x => x.itemName === name);
    return ret ? ret.price : "No such item";
}
  

 function priceLookup(items, obj) {
    for (let i = 0; i < items.length; i  ) {
        if (items[i].itemName == obj) {
            return items[i].price
        }
    }
    return undefined
}