Как правильно записать в переменную первый элемент массива?

Question

[kachurinets]

Часто возникают ситуации, когда возвращается массив, который имеет всего лишь один элемент.

К примеру, такой случай

this.selectedRegion = this.regions.filter(region => this.area.region_id === region.region_id);
this.selectedRegion = this.selectedRegion[0];

Метод filter возвратит массив с одним элементом (в данном случае это объект). Далее мне нужно записать этот объект в переменную и приходится обращатся к нему по индексу this.selectedRegion[0];
Считается ли это нормальной практикой? Есть ли лучший способ переделать этот код? На сколько, я знаю, то использывание так званных "магических чисел" в программировании считается плохим тоном, считается ли плохой практикой в данном случае?

Comments

[Сергей Соколов]

.unshift(), он же .pop() для массива с единственным эл.

Answer 1

[Arman]

Если ждете один элемент, то зачем фильтр? Чем find не нравится, тем более он не будет обходить все данные, а остановится на найденом, что хорошо будет на производительности

Comments

[Никита Егоров]

find здесь не поможет. Он будет перебирать элементы массива regions, а нужно в каждом из этих элементов находить region_id и уже его сравнивать с area.region_id

[kachurinets]

Никита Егоров, а если использовать как-то foreach чтобы сразу возвращать нужный обьект? такой вариант возможен?

[Антон Спирин]

Никита Егоров, еще как поможет.

[Никита Егоров]

Антон Спирин, о, не знал что туда можно callback функцию передать.

[abberati]

kachurinets, forEach ничего не возвращает, не стоит даже пытаться.
Никита Егоров, использовать циклы для обхода массива - такое себе решение

kachurinets многие утилитарные библиотеки предоставляют функцию head(array), которая возвращает первый элемент массива. также в es6 есть деструктуризация, которая позволяет взять из массива нужные элементы вот так:

const [ selectedRegion ] = this.regions.filter(region => this.area.region_id === region.region_id);

[Никита Егоров]

abberati, как бы уже разобрались, есть find

[abberati]

Никита Егоров, как бы для общего развития было бы неплохо знать разные подходы

[Никита Егоров]

abberati,

использовать циклы для обхода массива - такое себе решение

я это имею ввиду

Answer 2

[abberati]

this.selectedRegion = this.regions.find(({ region_id }) => this.area.region_id === region_id);