Как правильно искать итемы через for с условием if?

Question

[Maxla93]

Реализовали поиск "нужных вещей" по инвентарю игрока.

• items[i].market_hash_name - вещи в инвентаре игрока
  
• sendItemName - вещи которые ищем в инвентаре
  

Если вещь найдена, тогда стоп цикла break;

Проблема заключается в следующем:

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

Например в инвентаре будет:
1. Name100
2. Name99
3. Name55

Искать будем:
1. Name99
2. Name99
3. Name99

Разумеется напишет, что всё найдены.
Но в инвентаре то есть только 1 вещь Name99, а считает что есть всё.

Как реализовать, чтобы получался такой результат:
1. Найдено
2. Не найдено
3. Не найдено

Сам код:

// Загрузка инвентаря с итемами
offers.loadMyInventory({appId: 730, contextId: 2}, function(err, items) {
var otvet = null;

// g_ItemName - массив с вещами которые будем искать в инвентаре
g_ItemName.forEach(function(sendItemName) {

if (typeof sendItemName != 'undefined') {
    for(var i = 0; i < items.length; i++){
     
if(sendItemName == items[i].market_hash_name){
    console.log('Вещь: ' + sendItemName + ' найдена в инвентаре');
    otvet = true;
    break;
}else{
    console.log('Вещь: ' + sendItemName + ' не найдена в инвентаре');
    otvet = false;
    }
    }
    }
});

Answer 1

[Алексей Ярков]

Сделайте вложенный цикл. А если строку ищете, то indexOf почему не юзать?
UPD: простите, невнимательно прочел вопрос.
Вот так ищите:
https://jsfiddle.net/3Ljv9d73/

Comments

[Maxla93]

можете показать как это выглядеть должно? ибо не совсем понимаю.

[Maxla93]

О цикле имеется ввиду.

[Maxla93]

prntscr.com/bnxg3p - так не получится.

[Алексей Ярков]

Maxla93: https://jsfiddle.net/3Ljv9d73/2/

[Maxla93]

prntscr.com/bnxr3r - также не работает

[Maxla93]

отблагодарю финансово, если найдете решение как сделать.

[Алексей Ярков]

Maxla93: Вы сравните свои 2 скриншота, найдите противоречие в ВАШИХ пометках как должно быть и переосмыслите задачу )))

[Maxla93]

так всё правильно написано) возможно вы не так понимаете задачу, со второго скриншота понятно что нужно?

[Алексей Ярков]

Maxla93: Ну мой первый вариант выводил повторения. Второй не выводит. Что не так?

[Maxla93]

А при чем здесь повторения? он должен всё g_ItemName искать в items

[Maxla93]

prntscr.com/bnxy6s - может так будет понятней.

[Алексей Ярков]

Maxla93: а сейчас как? Так и ищем, только убираем дубли. Если итем1 ищется больше 1 раза, то выводим результат прошлого поиска. Все верно. Мы друг друга не поняли. Распишите задачу по пунктам.

[Maxla93]

нам не нужно выводить результат прошлого поиска, каждый ищется отдельно.
prntscr.com/bnxy6s - посмотрите пожалуйста, уверен поймете.

[Алексей Ярков]

Maxla93: Вообще не понятно. Зачем вам каждый раз сравнивать с каждым элементом искомый элемент.
https://jsfiddle.net/3Ljv9d73/4/
Вот такая реализация мне кажется логичной. Или мы на разных волнах.

[Maxla93]

Да мы на разных волнах.
Я уже примерно понимаю, нужно вырезать с массива найденный item и при поиске уже второго итема в массиве 1 найденного не должно быть, можете такое реализовать?

[Алексей Ярков]

Maxla93: array.shift(), array.pop(). Но получится то же самое )))

[Алексей Ярков]

Maxla93: https://jsfiddle.net/3Ljv9d73/6/

[Maxla93]

https://jsfiddle.net/3Ljv9d73/6/ - тут не вырезается найденный массив, при поиске второго итема он такой же как и был.

[Maxla93]

var test = new Array(1,2,3,4,5);
test.splice(test.indexOf(3));
alert(test);

пример как вырезается значение с ключем.

[Алексей Ярков]

Maxla93: https://jsfiddle.net/3Ljv9d73/7/

[Maxla93]

Алексей Ярков: Теперь вы вообще уже не туда пошли, я не знаю, я ясно объяснил на скриншотах.. не знаю как вы не можете понять.

[Maxla93]

Есть скайп?

[Алексей Ярков]

Maxla93: Есть, но не могу щас говорить.
https://jsfiddle.net/3Ljv9d73/12/
Оно?

[Maxla93]

Алексей Ярков: да, сейчас протестирую, оставьте рублевой кошелек webmoney

[Алексей Ярков]

Maxla93: да боже упаси )) За 10 строк кода?

[Maxla93]

Алексей Ярков: ну мало ли, спасибо)

[Maxla93]

А не могли бы сделать по моей функции? (если есть время) с такой же структурой как в теме, а то что-то у меня на node.js не совсем корректно работает.

[Алексей Ярков]

Maxla93:

function searchInventory(name, items) {
	for(var i = 0; i < items.length; i++){
		if (name == items[i].market_hash_name) {
			items.splice(i, 1);
			return true;
		}
	}
	return false;
}

// Загрузка инвентаря с итемами
offers
	.loadMyInventory({
			appId: 730,
			contextId: 2
		}, function(err, items) {
				var otvet = null;
				var g_ItemName = ["Меч огня", "Меч огня", "Морковь снеговика", "Нож гопника", "Нож гопника", "Улыбка единорога"];

				for(var j = 0; j < g_ItemName.length; j++){
					var sendItemName = g_ItemName[j];
					if (searchInventory(sendItemName, items)) {
						console.log('Вещь "%s" найдена в инвентаре', sendItemName);
					}
					else{	
						console.log('Вещь "%s" не найдена в инвентаре', sendItemName);
					}
					//console.log(items.length);
				}
		});

[Maxla93]

Алексей Ярков: Ответь пожалуйста тут Почему в new_items2 нет всех вещей, которые были function(err, items)?, заранее спасибо)

Answer 2

[mazahistka16]

var count = 0;
for(i = 0 i < items.lengt; i++)
if(sendItemName == items[i].market_hash_name) {
   console.log('Вещь: ' + sendItemName + ' найдена в инвентаре');
   count++;
}

if (count == 0)
   console.log('Вещь: ' + sendItemName + ' не найдена в инвентаре');

Что то типа этого