Ht4v2 #37
Ht4v2 #37
Conversation
|
|
||
| switch (type) { | ||
| case SET_ACTIVE_RESTAURANT: | ||
| return { ...restaurants, activeRestaurantId: payload.id }; |
There was a problem hiding this comment.
теперь знание о какой ресторан активный храниться в двух местах, в этом редьюсере и в компоненте Tabs, я расскажу на занятие какие могут возникнуть с этим проблемы
| } | ||
| const activeRestaurantId = payload.activeRestaurantId; | ||
| const restaurant = restaurants[activeRestaurantId]; | ||
| restaurant.reviews.push(payload.id); |
There was a problem hiding this comment.
мутации в редьюсере недопустимы, на любом уровне вложености
| } | ||
| return { | ||
| ...reviews, | ||
| error: false, |
There was a problem hiding this comment.
лучше для этого сделать на одну вложенность больше, чтобы на одном уровне не было разных типов данных (и ревью по id и дополнительные данные про ошибки), у нас это как раз будет сегодня на занятии
| if (action.type === 'POST_REVIEW') { | ||
| const newReviewId = uuidv4(); | ||
| const newUserId = uuidv4(); | ||
| action.payload.id = newReviewId; |
There was a problem hiding this comment.
не стоит мутировать action, лучше создать новый
|
|
||
| const ReviewForm = ({ onSubmit }) => { | ||
| const { values, handlers, reset } = useForm(INITIAL_VALUES); | ||
|
|
||
| const activeRestaurantId = useSelector( | ||
| (state) => state.restaurants.activeRestaurantId |
There was a problem hiding this comment.
это нужно выносить в селекторы
| const { name, menu, reviews } = restaurant; | ||
| const allReviews = useSelector((state) => state.reviews); |
There was a problem hiding this comment.
все это нужно выносить в селекторы, чтобы компонент занимался только рендером
В целом таск выполнен, возникли небольшие затруднения с reselect - в некоторых местах пришлось обойтись без нее, но там где нет селекторов я меморизировал через useMemo.