[javascript] 프라 미스 체인에서 여러 캐치 처리

이 동작은 동기 발생과 똑같습니다.

try{
    throw new Error();
} catch(e){
    // handle
}
// this code will run, since you recovered from the error!

그것은 .catch오류로부터 복구 할 수 있다는 점의 절반입니다 . 상태가 여전히 오류임을 알리기 위해 다시 던지는 것이 바람직 할 수 있습니다.

try{
    throw new Error();
} catch(e){
    // handle
    throw e; // or a wrapper over e so we know it wasn't handled
}
// this code will not run

그러나 나중에 처리기가 오류를 포착하기 때문에 이것만으로는 귀하의 경우에 작동하지 않습니다. 여기서 진짜 문제는 일반화 된 “아무것도 처리”오류 처리기는 일반적으로 나쁜 관행이며 다른 프로그래밍 언어와 생태계에서 극도로 눈살을 찌푸린다는 것입니다. 이러한 이유로 Bluebird는 형식화 및 술어 catch를 제공합니다.

추가 된 이점은 비즈니스 로직이 요청 / 응답주기를 전혀 인식하지 않아도된다는 것입니다. 클라이언트에 어떤 HTTP 상태와 오류가 발생하는지 결정하는 것은 쿼리의 책임이 아니며 나중에 앱이 커짐에 따라 클라이언트에 보내는 내용과 비즈니스 논리 (DB 쿼리 방법 및 데이터 처리 방법)를 분리 할 수 ​​있습니다. (어떤 http 상태 코드, 텍스트 및 응답).

다음은 코드를 작성하는 방법입니다.

먼저, 나는 Bluebird가 이미 제공하는 하위 클래스 .Query를 던질 NoSuchAccountError것 Promise.OperationalError입니다. 오류를 하위 분류하는 방법을 잘 모르겠 으면 알려주세요.

추가로 하위 클래스를 AuthenticationError만들고 다음과 같은 작업을 수행합니다.

function changePassword(queryDataEtc){
    return repository.Query(getAccountByIdQuery)
                     .then(convertDocumentToModel)
                     .then(verifyOldPassword)
                     .then(changePassword);
}

보시다시피-매우 깨끗하고 프로세스에서 일어나는 일에 대한 지침 매뉴얼처럼 텍스트를 읽을 수 있습니다. 요청 / 응답과도 분리됩니다.

이제 경로 처리기에서 다음과 같이 호출합니다.

 changePassword(params)
 .catch(NoSuchAccountError, function(e){
     res.status(404).send({ error: "No account found with this Id" });
 }).catch(AuthenticationError, function(e){
     res.status(406).send({ OldPassword: error });
 }).error(function(e){ // catches any remaining operational errors
     res.status(500).send({ error: "Unable to change password" });
 }).catch(function(e){
     res.status(500).send({ error: "Unknown internal server error" });
 });

이런 식으로 논리가 모두 한곳에 있고 클라이언트에 대한 오류를 처리하는 방법에 대한 결정이 모두 한곳에 있으며 서로 어수선하지 않습니다.

답변

repository.Query(getAccountByIdQuery)
        .then(convertDocumentToModel)
        .then(verifyOldPassword)
        .then(changePassword)
        .then(function(){
            res.status(200).send();
        })
        .catch(function(error) {
            if (/*see if error is not found error*/) {
                res.status(404).send({ error: "No account found with this Id" });
            } else if (/*see if error is verification error*/) {
                res.status(406).send({ OldPassword: error });
            } else {
                console.log(error);
                res.status(500).send({ error: "Unable to change password" });
            }
        });

repository.Query(getAccountByIdQuery)
.then(function(account) {
    return convertDocumentToModel(account)
    .then(verifyOldPassword)
    .then(function(verification) {
        return changePassword(verification)
        .then(function() {
            res.status(200).send();
        })
    }, function(verificationError) {
        res.status(406).send({ OldPassword: error });
    })
}, function(accountError){
    res.status(404).send({ error: "No account found with this Id" });
})
.catch(function(error){
    console.log(error);
    res.status(500).send({ error: "Unable to change password" });
});

    repository.Query(getAccountByIdQuery)
    .then((resultOfQuery) => convertDocumentToModel(resultOfQuery)) //inside convertDocumentToModel() you check for empty and then throw new Error('no_account')
    .then((model) => verifyOldPassword(model)) //inside convertDocumentToModel() you check for empty and then throw new Error('no_account')        
    .then(changePassword)
    .then(function(){
        res.status(200).send();
    })
    .catch((error) => {
    if (error.name === 'no_account'){
        res.status(404).send({ error: "No account found with this Id" });

    } else  if (error.name === 'wrong_old_password'){
        res.status(406).send({ OldPassword: error });

    } else {
         res.status(500).send({ error: "Unable to change password" });

    }
});

function convertDocumentToModel(resultOfQuery) {
    if (!resultOfQuery){
        throw new Error('no_account');
    } else {
    return new Promise(function(resolve) {
        //do stuff then resolve
        resolve(model);
    }
}

repository.Query(getAccountByIdQuery)
    .then(
        convertDocumentToModel,
        () => {
            res.status(404).send({ error: "No account found with this Id" });
            return Promise.reject(null)
        }
    )
    .then(
        verifyOldPassword,
        () => Promise.reject(null)
    )
    .then(
        changePassword,
        (error) => {
            if (error != null) {
                res.status(406).send({ OldPassword: error });
            }
            return Promise.Promise.reject(null);
        }
    )
    .then(
        _ => res.status(200).send(),
        error => {
            if (error != null) {
                console.error(error);
                res.status(500).send({ error: "Unable to change password" });
            }
        }
    );

let errorEncountered = false;
someCall({
  /* do stuff */
})
.catch({
  /* handle error from someCall*/
  errorEncountered = true;
  return Promise.reject();
})
.then({
  /* do other stuff */
  /* this is skipped if the preceding catch was triggered, due to Promise.reject */
})
.catch({
  if (errorEncountered) {
    return;
  }
  /* handle error from preceding then, if it was executed */
  /* if the preceding catch was executed, this is skipped due to the errorEncountered flag */
});