我想我知道错误发生在哪里,但我能够为从fetch()返回的Promise制定正确的处理流程
我的消息减速器模块:-
import { fetchMessages } from '_helpers/api'
import { Map, fromJS } from 'immutable'
const FETCHING_MESSAGES = 'FETCHING_MESSAGES'
const FETCHING_MESSAGES_FAILURE = 'FETCHING_MESSAGES_FAILURE'
const FETCHING_MESSAGES_SUCCESS = 'FETCHING_MESSAGES_SUCCESS'
const ADD_MESSAGES = 'ADD_MESSAGES'
const ERROR_MESSAGE = 'There has been an error'
export const fetchAndHandleMessages = () => {
return (dispatch, getState) => {
dispatch(fetchingMessages())
fetchMessages()
.then((r) => {
if (!r.ok) {
dispatch(fetchingMessagesFailure(ERROR_MESSAGE))
}else{
return r.json()
}
})
.then((b) => {
dispatch(fetchingMessagesSuccess(b))
})
.catch(() => {
dispatch(fetchingMessagesFailure(ERROR_MESSAGE))
})
}
}
function fetchingMessagesSuccess(messages) {
return {
type: FETCHING_MESSAGES_SUCCESS,
messages,
lastUpdated: Date.now(),
}
}
function fetchingMessagesFailure(errMsg) {
return {
type: FETCHING_MESSAGES_FAILURE,
error: errMsg
}
}
const fetchingMessages = () => {
return {
type: FETCHING_MESSAGES,
}
}
const initialState = fromJS({
messages: [],
isFetching: true,
error: '',
})
export const messagesReducer = (state = initialState, action) => {
switch (action.type) {
case FETCHING_MESSAGES :
return state.merge({
isFetching: true,
})
case FETCHING_MESSAGES_SUCCESS :
return state.merge({
error: '',
isFetching: false,
messages: action.messages
})
case FETCHING_MESSAGES_FAILURE:
return state.merge({
error: action.error,
isFetching: false
})
default :
return state
}
}
export default messagesReducer
fetchMessages() 只返回一个承诺:-
export const fetchMessages = () => {
return fetch(baseUrl + 'messages')
}
我不会在这里发布组件代码,因为它与问题无关。
因此,如果我使用无效的 URL 调用 fetchMessages() 以返回 404,则 state.messages在我的组件中变得未定义。这似乎是由这部分功能引起的:-
if (!r.ok) {
dispatch(fetchingMessagesFailure(ERROR_MESSAGE))
}else{
return r.json()
}
我想我可能对如何正确检查和处理返回的 Promise 中的潜在错误感到困惑。根据 fetch() 的文档,404 不被视为错误,因为(与常规 AJAX 不同)只有网络问题被视为 catch() 类型的错误。
谁能帮我指出我的这部分代码有什么问题?我应该在dispatch(fetchingMessagesFailure(ERROR_MESSAGE))之后使用exit来停止以下.then()吗?此外,即使只有 404,.catch()块也正在运行。这似乎违背了文档的建议。
非常感谢任何帮助。谢谢。