Как в масиве найти значение и изменить его?

Question

[Данил]

Есть масив

[
	{
		"name": "крафт.gif",
		"offset": 964
	},
	{
		"name": "авария.gif",
		"offset": 681
	},
	{
		"name": "животные.gif",
		"offset": 777
	}
]

нужно изменить offset зная name в масиве
Как это можно реализовать?

Comments

[Israfil22]

Массив - ссылочный тип данных. Внутри у вас объекты, аналогично передающиеся по ссылке.
Это значит, что конструкция

data[index].offset

будет менять значение внутри массива.
В чем проблема заключается?

[WapSter]

Israfil22, не может пройтись по массиву видимо и найти нужный ему объект

Answer 1

[ivashjke]

const index = arr.findIndex((el) => el.name === name)
if (-1 !== index) {
  arr[index].offset = value
}

Comments

[WbICHA]

А что мешало сразу использовать find? Зачем тут findIndex?

[WapSter]

WbICHA, тут даже find не нужен. К тому же в условии вроде, изменить текущий массив, нафига вообще забивают память лишними телодвижениями

[Stalker_RED]

WapSter, forEach переберет все элементы, даже если искомый попадется в самом начале, а find прекратит перебор как только найдет первый подходящий.

[WbICHA]

WapSter, мне действительно нужно расписывать, зачем здесь файнд?

в условии вроде, изменить текущий массив

Куратор тега JavaScript

Куратор тега, не знающий основ. Куда катится тостер...

[ivashjke]

WbICHA, условие проверки один фиг будет, так ли важно ?

[WbICHA]

ivashjke, тебе нужен индекс? Нет. Так что ответ на твой вопрос очевиден.

[ivashjke]

WbICHA, не кипятись, не я же forEach'em прошелся)

Answer 2

[WapSter]

let arr = [
  {
    "name": "крафт.gif",
    "offset": 964
  },
  {
    "name": "авария.gif",
    "offset": 681
  },
  {
    "name": "животные.gif",
    "offset": 777
  }
];

arr.forEach(n => {
  if (n.name === 'животные.gif') {
    n.offset = 500
  }
})
console.log(arr)

Comments

[WbICHA]

Очень плохое решение. Давай теперь представим, что массив на 10000 элементов и искомый объект находится на индексе 2. Сколько лишних проверок будет?

[Данил]

WbICHA, может и решение так себе но оно в моем случае работает и масив никогда не привисит порог 20 элементов

[WbICHA]

Данил, это хорошо, это правильно. Ты, главное, дальше никак не развивайся, я не люблю большую конкуренцию.

[Григорий]

WbICHA, ваше решение в студию, или вы чисто так своим уровнем покрасоваться? ))

[WbICHA]

Григорий, зачем? Моё решение тоже самое, что и второй ответ, только с другим методом.
Хотя... Да, вы же обязательно что-то не так поймёте.

const element = arr.find((el) => el.name === name)
if (element) {
  element.offset = value
}

// or

arr.find((el) => el.name === name)?.offset = value;

[WapSter]

WbICHA, в условии сказано, что там будет хотя бы больше 100 элементов. Вроде нет. Решение достаточное для данной задачи.

[WbICHA]

WapSter, класс. Давай теперь писать говно, потому что условие позволяет.

[WapSter]

WbICHA, нет, я пишу исходя из условий всегда. А ты агрессируешь почему-то, хотя лично тебе ничего плохого не писал. Допусти хоть на секунду, что name может повториться)

[WbICHA]

WapSter, я? Агрессирую? Разве что совсем чуть-чуть.
Что же касается условия, то там чётко указано количество:

Как в масиве найти значениЕ и изменить его?

Так что нет, я не допускаю, что там могут быть повторения.

[WapSter]

WbICHA,

let arr = [
  {
    "name": "string",
    "offset": 964
  },
  {
    "name": "string",
    "offset": 681
  },
  {
    "name": "string"
    "offset": 777
  }
];

В массиве найти значение string. Я конечно не русский, но вроде к множественному или единственному числу элементов, это никак не относится. Понял бы если там был id, тогда по умолчанию это должен быть единственный элемент. Не лучше ли предложить несколько решений задачи. Нежели втюхивать одно единственно, верное по твоему мнению. Даже фашизмом попахивает.

[WbICHA]

Даже фашизмом попахивает.

WapSter, я Гитлер, не знал?

В массиве найти значение string. Я конечно не русский, но вроде к множественному или единственному числу элементов, это никак не относится.

Относится, причём напрямую: "значениЕ" ─ единственное число, "значениЯ" ─ множественное.

[WapSter]

WbICHA, как скажешь