我应该在哪里更新过滤后的数据，例如，重新选择或归约？

Question

我目前正在使用reselect，redux-sagas和减速器。我有一个状态为filterParameter的状态，其中包含filteredParameter值filterReducer.js：

initialState = {
  filteredParameters: { ... } // null by default
}

过滤后的数据用于我的组件filters.js：

// On change function
  const statusChange = event => {
    event.preventDefault();
    const { name, value } = event.target;

    setFilterParameters({ ...filterParameters, [name]: parseInt(value) });
  };

// Render
 {statuses && (
        <select name="statusId" onChange={statusChange}>
          <option>All</option>
          {statuses.map(status => (
            <option
              selected={
                filteredParameters && filteredParameters.statusId === status.id
              }
              key={status.id}
              name={status.name}
              value={status.id}
            >
              {status.icon} {status.name}
            </option>
          ))}
        </select>
      )}

// MapDispatchToProps
const mapDispatchToProps = dispatch => ({
  filterIdeas: parameters => dispatch(filterIdeas(parameters)),
  fetchCategories: () => dispatch(fetchCategoriesStart())
});

然后我有一个saga设置可以在filter.sagas.js中捕获此操作：

function* filterCurrentIdeas({ payload: filteredParameters }) {
  try {
    yield put(setFilterParameters(filteredParameters));
    const ideas = yield select(selectBoardIdeasItems);

    // Custom function that does the filtering handling, this is shown below 

    const filterResults = yield call(
      filterIdeasByStatus,
      ideas,
      filteredParameters
    ); 
    yield put(filterIdeasSuccess(filterResults)); // This then gets fed into the reducer
  } catch (err) {
    console.log(err);
    return;
  }
}

这是我的减速器filter.reducer.js

    case BoardActionTypes.FILTER_IDEAS_SUCCESS:
      return {
        ...state,
        filteredItems: action.payload
      };

这里是用于过滤filter.utils.js的自定义实用程序帮助程序：

export const filterIdeasByStatus = (ideas, filter) => {
  const { statusId, categoryId, orderBy, query } = filter;
  const orderedIdeas = orderBy ? _.orderBy(ideas, ["voteCount"]) : ideas;
  const regex = new RegExp(`${query}`, "gmius");

  const filtered = orderedIdeas.filter(
    idea =>
      (!statusId || (statusId && idea.IdeaStatus.id === statusId)) &&
      (!categoryId ||
        (categoryId && _.includes(idea.categoryIds, categoryId))) &&
      (!query || (query && idea.title.match(regex)))
  );

  return convertIdeasToObject(filtered);
};

目前，这一切正常，但是我做错了吗？或者有没有更好的方法或指南可以为我提供更好的例子？

提前感谢！

Answer 1

助手功能filterIdeasByStatus是使代码保持模块化和组织化的好方法。

但是，给定setFilterParameters是一个动作：

setFilterParameters({ ...filterParameters, [name]: parseInt(value) });

你为什么叫两次？

1. 在filter.sagas.js传奇中的filterCurrentIdeas处>
2. 在filters.js组件处

通常，您将希望在statusChange上发送一个动作，并让传奇故事过滤这些项目（如您所做的那样）。但是两次调用同一动作可能会导致副作用。

细心提示：请记住，React组件名称空间通常是大写的，即filter.js应该是Filters.js。

您也可以在Stack Exchange Code Review中寻求最佳实践和/或寻求反馈

希望您获得了一些见解：）