Бесконечный l oop с использованием эффекта в React - PullRequest
Новая
       107

Бесконечный l oop с использованием эффекта в React

0 голосов
/ 11 апреля 2020

Это мой компонент: 

import React, { useState, useEffect } from "react";

export default function App() {
  const [countriesArray, setCountriesArray] = useState([]);

  useEffect(() => {
    getCountriesArray();
  }, []);

  const getCountriesArray = async () => {
    try {
      let response = await fetch(
        "https://coronavirus-19-api.herokuapp.com/countries"
      );
      if (response.status === 200) {
        const newCountriesArray = [...countriesArray];
        const data = await response.json();
        await data.forEach(item => newCountriesArray.push(item.country));
        setCountriesArray(newCountriesArray);
      } else {
        setErrorStatus(true);
        console.error("Error status");
      }
    } catch (err) {
      console.error(err);
    }
  };

  const optionItems = countriesArray.map((item) =>
        <option key={item}>{item}</option>
    )

  return (
    <div className="App">
      <select>{optionItems}</select>
    </div>
  );
}

При выборе я получаю названия стран при монтировании компонента, но в консоли у меня появляется сообщение об ошибке al oop: 

Warning: Encountered two children with the same key, `Total:`. Keys should be unique so that components maintain their identity across updates. Non-unique keys may cause children to be duplicated and/or omitted — the behavior is unsupported and could change in a future version.
    in select (at App.js:36)
    in div (at App.js:35)
    in App
    in StrictMode (at src/index.js:8)

Однако я использую пустой массив как второй параметр useEffect, чтобы выполнить его только при монтировании компонента

Ответы [ 2 ]

1 голос
/ 11 апреля 2020

Эта ошибка не имеет ничего общего с массивом зависимостей эффекта. Проблема в том, что item.country не уникален в этих данных. json включает 7 записей, где country === "Total:".

Пара возможностей:

1) Отфильтруйте дубликаты. Лучше всего, если вам не нужны эти записи «Всего:». 

if (response.status === 200) {
  const newCountriesArray = [...countriesArray];
  const data = await response.json();
  data.forEach(item => {
    if (item.country !== "Total:") {
      newCountriesArray.push(item.country)
    }
  });
  setCountriesArray(newCountriesArray);
}

2) Используйте другую клавишу. Поскольку эти данные не имеют уникального ключа, вам может понадобиться использовать индекс массива. Имейте в виду, что это не будет хорошим вариантом, если вы планируете сортировать этот список. 

const optionItems = countriesArray.map((item, index) =>
  <option key={index}>{item}</option>
)
1 голос
/ 11 апреля 2020

Вы можете использовать ключ, предоставленный самой картой, следующим образом: 

const optionItems = countriesArray.map((item, key) =>
    <option key={key}>{item}</option>
)

Это должно решить вашу проблему.

Кстати, это не бесконечный l oop проблема, это дублирующий ключ в вашей карте функции.

...