Бросить разные ошибки в цепочку обещаний?

Question

У меня есть метод на сервере WebApp с помощью узла, который будет выполнять несколько асинхронных действий, например:

function saveTopValuePages(pageSize, clientId ) 
     setExistingCredentials(clientId) 
     .then(function() { 
      return getViewIdByClientId(clientId); 
     }, function(error) { 
      throw new customErrors.SetExistingCredentials('test123') 

     }) 
     .then(function(viewId) { 
      return fetch(viewId, options) 
     }, function(error) { 
      throw new customErrors.GetViewIdByClientId('abcasdasd') 
     }) 
     .then(function(response) { 
      return response; 
     }) 

После этого будет мой вызовом этого функция из моих маршрутов:

analytics.saveTopValuePages(pageSize, clientId) 
    .then(function(data) { 
     res.status(200)send({ message: 'success'}) 
     } 

    }).catch(customErrors.SetExistingCredentials, function(error) { 
     res.status(400).send({ error: error}) 
    }).catch(customErrors.GetViewIdByClientId, function(error) { 
     res.status(401).send({error: error}) 
    }).catch(function(e){ 
    res.status(500).send({ error: "Unknown internal server error" }); 
}); 

Как вы можете см. в моем примере, я пытаюсь выбросить ошибку в зависимости от того, какая часть процесса завершилась неудачно. Это будет работать, если все эти действия будут независимо друг от друга, но наоборот. Таким образом, если setExistingCredentials не работает, все следующие функции потерпят неудачу.

Это вызывает то, что если setExistingCredentials не удался, ошибка будет выбрана, а также ошибка GetViewIdByClientId. Наконец, ошибка, которую я поймаю, будет последней (GetViewIdByClientId в этом случае), а не правильной.

Итак, я не уверен, был ли это правильный подход, если есть какой-либо другой образец обещаний для достижения нужных мне результатов.

Заранее благодарен!

Answer 1

Не знаю, как «кошерного» это

function saveTopValuePages(pageSize, clientId ) { 
    setExistingCredentials(clientId) 
    .then(function() { 
     return getViewIdByClientId(clientId); 
    }) 
    .catch(function(error) { 
     throw { custom: customErrors, type: 'SetExistingCredentials', arg: 'test123' } 

    }) 
    .then(function(viewId) { 
     return fetch(viewId, options) 
    }) 
    .catch(function(error) { 
     if (error.custom == customErrors) { 
      throw new customErrors[error.type](error.arg); 
     } 
     throw new customErrors.GetViewIdByClientId('abcasdasd') 
    }); 
    // next 3 lines are actually redundant 
    //.then(function(response) { 
    // return response; 
    //}) 
} 

Если это не сработает, то вы не будете, как это - не нравится - но иногда это должно быть сделано - раскрой!

function saveTopValuePages(pageSize, clientId ) { 
    setExistingCredentials(clientId) 
    .then(function() { 
     return getViewIdByClientId(clientId).then(function(viewId) { 
      return fetch(viewId, options) 
     }, function(error) { 
      throw new customErrors.GetViewIdByClientId('abcasdasd') 
     }); 
    }, function(error) { 
     throw new customErrors.SetExistingCredentials('test123') 
    }) 
    .then(function(response) { 
     return response; 
    }) 
} 

Конечно, вы могли бы смягчить эту гадость, создав функцию

function getView(clientId) { 
    return getViewIdByClientId(clientId).then(function(viewId) { 
     return fetch(viewId, options) 
    }, function(error) { 
     throw new customErrors.GetViewIdByClientId('abcasdasd') 
    }); 
} 
function saveTopValuePages(pageSize, clientId ) { 
    setExistingCredentials(clientId) 
    .then(function() { 
     return getView(clientId); 
    }, function(error) { 
     throw new customErrors.SetExistingCredentials('test123') 
    }) 
    .then(function(response) { 
     return response; 
    }) 
}