중복된 아이템 생성 핸들링

[hyesungoh]

🤔 해결하려는 문제가 무엇인가요?

• 중복된 아이템 생성 방지

🎉 어떻게 해결했나요?

• useCardItemMutation에서 error를 던지도록 핸들링했어요.

  • onError 시 토스트 메세지가 재생되어 이렇게 했어요

  • 원래 내부에서 토스트 메세지를 재생하고, return Promise.resolve(undefined)로 함수를 종료하려 했는데, 바텀시트에서 onSuccess 시에 바텀시트가 닫치는데, 이를 따로 핸들링하는 건 DRY 하지 않다고 생각했어요

    근데 코드를 의도와 다르게 작성한 것 같아서 고민이긴해요 🤔

📚 Attachment (Option)

close #315

[github-actions[bot]]

Bundle Sizes

Compared against 795e319fd32ec5c76ae042e848ef44da3a3db695

Route: No significant changes found

Dynamic import: No significant changes found

[codecov-commenter]

Codecov Report

Patch and project coverage have no change.

Comparison is base (795e319) 95.95% compared to head (27b7f71) 95.95%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #316 +/- ## ======================================= Coverage 95.95% 95.95% ======================================= Files 24 24 Lines 247 247 Branches 48 48 ======================================= Hits 237 237 Misses 10 10 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=depromeet). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=depromeet)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

[eunddodi]

혹시 몸이 네 개신지? Dry하지 않다가 어떤 뜻인가요

[hyesungoh]

혹시 몸이 네 개신지? Dry하지 않다가 어떤 뜻인가요

Do not Repeat Yourself

소스 코드에서 동일한 코드가 반복이 된다는 것은 잠재적인 버그의 위협을 증가시킨다. 반복되는 코드 내용이 변경될 필요가 있는 경우, 반복되는 모든 코드에 찾아가서 수정을 해야 한다. 이 과정에서 실수가 발생한다면 버그가 발생하게 된다. 프로젝트의 규모가 커질수록 반복되는 코드로 인한 유지 보수 오버헤드가 커진다. 때문에 작은 프로젝트라도 코드를 반복해서 사용하지 않는 습관이 중요하다.

라고 검색한 결과 복붙드립니다

저는 같은 일을 여러번, 여러군데에서 한다는 뜻으로 이해하고 있어용

[kooku0]

https://blog.naver.com/complusblog/221163007357