Как поменять значение у определённых объектов в массиве?

Question

[babai19]

Есть 2 массива с объектами, у объектов есть значение id. Так вот: нужно пройтись по первому массиву и у объектов, которые так же имеются во втором массиве (можно проверять по id), изменить значение конкретного поля на true. Как такое реализовать?
p.s. можно использовать библиотеки
p.p.s. решение использовать только с оговорками относительно обхода массива

Answer 1

[Леонид Сысолетин]

var one = [{id: '1'}, {id: '2'}],
	two = [{id: '2'}, {id: '5'}],
	twoIds = [];

for ( i in two ) {
	if ( two.hasOwnProperty(i) ) {
		twoIds[two[i].id] = 1;
	}
}

for (i in one) {
	if ( one.hasOwnProperty(i) ) {
		if ( twoIds[one[i].id] !== undefined ) {
			one[i].field = true;
		}
	}
}

Comments

[Леонид Сысолетин]

M-ka: jsfiddle.net/waL3b5yk . С дополнением согласен, но только в этом частном случае.

[M-ka]

Пример не верный.
one и two, у вас обычные масивы, через in вы их не обойдете... только через обычные циклы. Необходимо предварительное конвертирование в ассоциативные масивы.
if ( twoIds[one[i].id] !== undefined ) -- можно заменить на if in

Дополнение: если нет функций внутри, а все однотипное, тогда нет нужны применять hasOwnProperty

[M-ka]

Леонид Сысолетин: вам придется согласиться полностью... если нужен кросбраузерный вариант, а не тот, который работает только у вас в вашей версии хрома...
Рекомендую все же повторно ознакомиться с применением for (variable in object)

[Леонид Сысолетин]

M-ka: С этим-то я согласен. Я не согласен с тем, что можно не проверять hasOwnProperty.

[M-ka]

Леонид Сысолетин: если данные в объекте вы формируете сами или получаете их с сервера в оговоренном формате, тогда не нужно... т.к. вы туда не додумаетесь запихнуть какую то функцию умышленно или случайно, хотя случайно, скорее через прототипы будет и только... потому, все зависит только от разработчика, а именно то, с какими данными в том или ином месте он собирается работать...

[Леонид Сысолетин]

M-ka: Вот примерно поэтому. Кто-нибудь прототип переопределит, какой-нибудь браузер что-нибудь новое придумает, или, в конце концов, со стороны сервера решат немного поменять формат данных =)

[M-ka]

Леонид Сысолетин: в браузере ничего нового, что бы поломало функционал текущей работы, не приедет... в отношении яваскрипта... Все новые версии поддерживают старые. А прототипы использовать или нет, это только решение разработчика да и в какой области видимости будут эти переменные, что бы на них влияли, это тоже нужно учитывать, потому hasOwnProperty это только лайт защита от дураков и не больше...

[M-ka]

Для copal:, скажу так, разработчики браузеров значит не знали в свое время, как работать с масивами через for in....

Answer 2

[M-ka]

Обход первого(или второго) масива, постройка объекта с ключами в виде ай ди.
При обходе другого масива проводить все действия сравнения и необходимые действия с новым объектом... Измененный объект запихнуть в масив с длинной в 1 или побить по ключах или как.... ну вообщем под нужную структуру...
Итого сделать функцию конвертации масива в ассоциативный, ну а второй производить сравнение объектов с возвратом общих ключей. Сравнение через поочередный обход обоих объектов и вынесением найденных ключей, которые находятся в другом объекте. Результат вернуть...
А вообще по хорошему следовало бы избавиться от масивов изначально, сделав ассоциативные масивы...