Как перебрать элементы массива?

Question

[Никита //]

Есть массив с объектами типа

[
{
site_id: "qwe",
url: [
"eee.eee.ru",
"www.www.ru",
],
phone_num: "123456789",
source: "qwe",
ops: "velc"
},
{
site_id: "qwe_pc",
url: [
"sss.sss.ru",
"aaa.aaa.ru"
],
phone_num: "1029384756",
source: "qwe",
ops: "mt"
}
]

и есть переменная типа let userUrl = 'sss.sss.ru';

Необходимо:
1) Перебрать циклом этот массив и сравнить свойство url каждого объекта в массиве с переменной userUrl
2) Если свойство url из объекта массива совпадает с переменной userUrl и свойство ops = 'velc' , то записать свойство phone_num этого объекта в одну переменную, а если ops = 'mt', то в другую переменную

Помогите кто чем сможет :)

Comments

[Никита //]

MaxKorz: Ну если для Вас это не составляет особой сложности, то может поможете? У меня с этим возникли проблемы, просто так я бы не стал писать

Answer 1

[Сергей Сосновский]

Первый undefined, второй 1029384756
Если я конечно правильно понял суть задания!
Так же можно немного сократить код в циклах(как по мне), ну это уже на любителя

let userUrl = 'sss.sss.ru', phone_num_1, phone_num_2;
	let arr = [{
		site_id: "qwe",
		url: [ "eee.eee.ru", "www.www.ru" ],
		phone_num: "123456789",
		source: "qwe",
		ops: "velc"
	}, {
		site_id: "qwe_pc",
		url: [ "sss.sss.ru", "aaa.aaa.ru" ],
		phone_num: "1029384756",
		source: "qwe",
		ops: "mt"
	}];

	arr.forEach(function(item) {
		item.url.indexOf(userUrl) > -1 && item.ops == "velc" && (phone_num = item.phone_num);
		item.url.indexOf(userUrl) > -1 && item.ops == "mt"   && (phone_num_2 = item.phone_num);
	});

	console.log(phone_num_1);
	console.log(phone_num_2);

Comments

[Илья Мясин]

Вместо вложенного цикла достаточно использовать проверку indexOf, будет как минимум аккуратнее выглядеть ;)

[Сергей Сосновский]

Илья Мясин: Полностью с вами согласен

arr.forEach(function(item) {
		item.url.indexOf(userUrl) > -1 && item.ops == "velc" && (phone_num = item.phone_num);
		item.url.indexOf(userUrl) > -1 && item.ops == "mt"   && (phone_num_2 = item.phone_num);
	});

[Илья Мясин]

Ну и вместо двух одинаковых проверок я бы сделал return если не сработало. Не готов писать код с мобильного, но вы меня поняли )))

[Илья Мясин]

Либо если уж писать в одну строку - можно выбирать имя переменной тернарным оператором )

window[ cond ? 'v1' : 'v2'] = val

Но зачем?)))

[Сергей Сосновский]

Илья Мясин: )) но в таком случае автор ни@рена не понял бы

item.url.indexOf(userUrl) > -1 ? item.ops == "velc" ? phone_num = item.phone_num : item.ops == "mt" ? phone_num_2 = item.phone_num : null : null;

[Илья Мясин]

desiigner: ага, поэтому из соображений читаемости я бы не побрезговал сделать if else вместо присвоения в конце цепочки && - будет смотреться слегка по лоховски и совсем не похоже на perl, зато понятно чо делает ))

[Сергей Сосновский]

малясь переборщил с ?:
вот немного проще!

item.url.indexOf(userUrl) > -1 && item.ops == "velc" ? phone_num = item.phone_num : item.ops == "mt" ? phone_num_2 = item.phone_num : null;

Пусть автор учится!