0

подскажите пожалуйста насколько правильна такая практика перебора в новый массив? Задача: Получить данные из базы данных , пройтись и вернуть новый массив из объектов с value и label ( Для рендеринга в React Select )

const categoriesPossible:any[] = []
  useEffect(() => {
    GetAllCategories.map((tower:any) => {
      const newObject:SelectInterface = {
        value: "",
        label: ""
      }
      newObject.value = tower.id
      newObject.label = tower.name
      categoriesPossible.push(newObject)
    })

  }, [GetAllCategories])

Можно ли данный код по другому более универсально написать?

1 ответ 1

1

Метод .map создает новый массив при переборе. Для подобных случаев использовать стоит или .map, или .forEach вместе с .push, как у вас. В общем, как-то так:

const categoriesPossible = GetAllCategories.map(({ id, name }) => ({
 value: id,
 label: name,
}))

Если не знаком такой способ обозначения свойств объекта - он называется деструктуризация.

Еще рассмотрите возможность записать результат перебора в state-хук. Это поможет избежать повторной операции при ререндере компонента.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.