Skip to content

Conversation

@BackJiwan
Copy link

4월 백엔드 블로그 포스팅 작성하여서 보냅니다.

@hoqn hoqn added the 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ label Apr 27, 2023
@hoqn hoqn requested review from iju1633 and seunghb320 April 27, 2023 11:39

## 마치며

일반적인 상황에서, 특히 Java 언어를 사용한 개발 환경에서라면 Java가 제공하는 Exception을 잘 활용하는 것은 에러를 핸들링하기에 적합한 방법으로 보입니다. 하지만 Exception이 항상 에러를 처리하는 정답은 아니며 상황과 환경에 따라서 적절한 에러처리 기술을 잘 활용할 수 있어야 할 것 입니다.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'것 입니다->것입니다'로 바꾸는데 어떨까 합니다!!

return "암호 길이 문제"
}
func RegisterAccount(n,p string) errer{
if ~~{
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Nit] if ~~{ 도 좋지만 if (조건문) { 이란 표현을 통해 예시를 더욱 가독성 있게 표현할 수 있을 것 같아요 !

Copy link

@iju1633 iju1633 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 별 생각없이 Exception을 주로 사용했던 것 같은데 좋은 글 정말 감사드립니다 !

@iju1633 iju1633 added ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 and removed 🙋‍♂️🙋‍♀️ 리뷰 요청 PR 리뷰를 요청합니다~ labels Apr 30, 2023
@BackJiwan
Copy link
Author

피드백 감사합니다~
반영하여서 push 하였습니다!

Copy link

@iju1633 iju1633 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

피드백 반영 감사합니다 ! LGTM 👍

@iju1633 iju1633 added 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 and removed 🔨 수정 요청 리뷰 과정 중 수정 사항이 생겼을 때 labels May 1, 2023
Copy link

@seunghb320 seunghb320 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

피드백 반영 및 좋은 글 감사합니다!!

@iju1633 iju1633 added 💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때 and removed 🛠 수정 완료 수정 요청 사항이 모두 해결됐을 때 labels May 24, 2023
@SSUHYUNKIM SSUHYUNKIM removed the ✅ 리뷰 완료 한 명 이상이 리뷰를 완료했을 때 label Jul 6, 2023
@hoqn hoqn merged commit f6b640e into GDSC-University-of-Seoul:master Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💯 최종 완료 모든 수정과 리뷰가 끝난 후 merge를 기다릴 때

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants