Javascript: try-catch 중첩해서 사용하기

Question

안녕하세요. 아래의 쿼리 3개가 조건이어서 선행조건을 먼저 작성하고 조건문을 작성하려고 하는데 저 쿼리들 중에 하나라도 undefined이면 에러때문에 코드가 멈춰버리더라구요. const getEmptyCartQuery &#61; await shopping_cart.findOne({ (...) }); const needsUpdatedQuantityQuery &#61; await shopping_cart.findOne({ (...) }); const needsNewCartQuery &#61; await shopping_cart.findOne({ (...) }); 그래서 try-catch문으로 예외를 줘서 아래와 같이 코드를 짰는데요. const data &#61; await shopping_cart.findAll({ where: { cart_id } }); try { const getEmptyCart &#61; await shopping_cart.findOne({ (...) }); if (getEmptyCart) { await shopping_cart.update({ (...) }); } ctx.body &#61; data; } catch (e) { try { const needsUpdatedQuantity &#61; await shopping_cart.findOne({ (...) }); if (needsUpdatedQuantity) { await shopping_cart.update({ (...) }); } ctx.body &#61; data; } catch (e) { try { const needsNewCart &#61; await shopping_cart.findOne({ (...) }); if (needsNewCart) { await shopping_cart.create({ (...) }); } } catch (e) { ctx.status &#61; 400; ctx.body &#61; e.message; } } } 어째저째 돌아가긴 합니다만 저런 중첩 try-catch문을 사용하기는 하나요? 너무 보기 지저분한데 try-catch 말고 db조회중 에러나지 않고 코드가 흘러갈 수 있는 방법이 있을까요? 혹시 제가 올린 질문 중 미흡한 부분이 있어서 보충해야할 부분이 있다면 댓글 부탁드립니다. 감사합니다.

moonti · Answer

그냥 promise쓰시면 마지막에 catch 해주면 될껄요?? 질문을 잘못 이해했네요. 저기 catch로 빠지나요?? 그냥 update 치면 update 된 개수 넘어올텐데 0이면 아래꺼 타고 하는게 낫지 않을까요?

oneday · Answer

너무 이상해요. try-catch는 에러나 예외처리인데 마치 if-else처럼 로직을 처리하고 있으니...

Rawfish · Answer

&#64;moonti model.update()가 두개 있는건 둘다 액션이 달라서 두개 작성했습니다. promise쓰라고 말씀하신건 Promise.all() 안에 배열로 조건들을 묶어버리는걸 말씀하시는건가요? &#64;oneday 이상하죠.. 그래서 질문드렸답니다 ㅠㅠ if-else로 처음엔 접근했다가 db쿼리에서 에러가 나가서 말입니다.

개나소나고생 · Answer

try~catch 블록 1회만 감싸서 이용하면 되는거 아닌가요?왜 여러번 감싸고 있는건가요?

Rawfish · Answer

&#64;개나소나고생 update나 create하기 전에 find로 검증을 하는데 그게 가장 상단에 있는 코드입니다. 얘네들 중 하나라도 오류가 나면 거기서 코드가 멈춰버립니다. 개나소나고생님 말씀대로 try블록 안에 조건을 3개 다 넣으면 셋중 하나는 오류나는 코드라서 바로 catch블록으로 가버리겠죠. 그래서 중첩해서 사용했습니다.

Keloper245 · Answer

let errorHandler &#61; function(err) { throw new Error({ status : 400, body : err.message }) } try { const data &#61; await shopping_cart.findAll({ where: { cart_id } }); const getEmptyCart &#61; await shopping_cart.findOne() .catch(errorHandler(&#34;getEmptyCart is Empty&#34;)) .then(result &#61;> { return &#34;done&#34;; }) const needsUpdatedQuantity &#61; await shopping_cart.findOne() .catch(errorHandler(&#34;needsUpdatedQuantity is Empty&#34;)) .then(result &#61;> { return &#34;done&#34;; }) const needsNewCart &#61; await shopping_cart.findOne() .catch(errorHandler(&#34;needsNewCart is Empty&#34;)) .then(result &#61;> { return &#34;done&#34;; }) } catch (e) { ctx.status &#61; e.stauts; ctx.body &#61; e.message; } 요런느낌일까요. 테스트는 안해봣구 저라면 저런식으로 짤것같네요.

Rawfish · Answer

&#64;Keloper245 엄청 깔끔하네요. 도움주셔서 감사합니다.

Rawfish · Answer

댓글 달아주신분들께 감사드립니다. 최종적으로 아래와 같은 형태로 사용하려고 합니다. try { const [getEmptyCart, needsUpdatedQuantity, needsNewCart] &#61; await Promise.all([ shopping_cart.findOne({ (...) }), shopping_cart.findOne({ (...) }), shopping_cart.findOne({ (...) }) ]); // use the results } catch (e) { // catch the error }

Javascript: try-catch 중첩해서 사용하기

8개의 답변

#try-catch 목록