Skip to content

Homework 4 radio songs app#5

Open
cucumberian wants to merge 7 commits into
masterfrom
homework-4-radio-songs-app
Open

Homework 4 radio songs app#5
cucumberian wants to merge 7 commits into
masterfrom
homework-4-radio-songs-app

Conversation

@cucumberian

Copy link
Copy Markdown
Owner

Вопросы.

  1. Почему вы на занятии конвертировали id методом toString() в строку, чтобы его использовать как key? Почему нельзя просто оставить случайное число? (Вопрос от Анны)
  2. В компоненте SongListSmart я получаю сверху список песен songListи фильтрую его чтобы отобразить.
    Если я использую в состоянии этого компонента [filteredList, setFilteredList] = useState(songList), то при добавлении песни в songList выше уровнем он изменяется и в этом компоненте, но фильтрации еще раз не происходит. И песни в filteredList не обновляются. Я сделал через состояние этого элемента, которое хранит значение фильтра. Тогда при изменении songList выше, происходит перерасчет фильтра. Хочется проясниить почему в одном случае проихсодит перерасчет, а в другом нет.
  // const [filteredList, setFilteredList] = useState(songList);
  // Так не работает, при измененении songList сверху не пересчитывается filterdList
  
  const [genreFilterName, setGenreFilterName] = useState("");
  
  const filteredList = songList.filter((song) =>
    genreFilterName === "" ? true : song.genre === genreFilterName
  );

image
image

@cucumberian cucumberian requested a review from takhanov August 6, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant