Skip to content

DaeunLee#2

Open
daeun-ver wants to merge 1 commit intoLikeLion-Study:mainfrom
daeun-ver:mninm
Open

DaeunLee#2
daeun-ver wants to merge 1 commit intoLikeLion-Study:mainfrom
daeun-ver:mninm

Conversation

@daeun-ver
Copy link
Copy Markdown

예외발생은 아무리 해도 통과를 못하겠어서 그대로 제출합니다...ㅜ_ㅜ
예외발생부분은 어떻게 수정해야 통과될지 봐주셨으면 좋겠습니다.
제출과정에서 많이 헤매서 제대로 제출이 된건지 걱정이 됩니다..

Copy link
Copy Markdown
Member

@UiHyeon-Kim UiHyeon-Kim left a comment

Choose a reason for hiding this comment

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

1주차 고생하셨습니다!
금방 코틀린으로 넘어가겠지만, 자바 코딩 컨벤션을 한 번 찾아보시고 읽어보시면 비슷한 부분이 많아서 추후 코틀린으로 넘어가서도 도움이 될 거예요.
테스트 부분은 자기가 작성한 코드에 맞게 수정을 해야 해서 조금 어려울 수 있어요. 아직은 신경쓰지 않아도 되니 지금은 기본 코드 작성에 더 신경써 주시면 금새 잘 작성하실 수 있을 것 같아요

}
int result=0;
int num=0;
char b='+';
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

b하는 변수가 무슨 역할을 하는지 이름을 자세하게 지어주시면 좋을 것 같아요!

throw new IllegalArgumentException("0 입력 불가능");
}
result/=num; break;
}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

스위치를 활용해 깔끔하게 처리한 부분과 0으로 나눌 때 에러 처리한 부분이 좋네요.
자바 코딩 컨벤션이라고 구글에 검색해보시고, 컨벤션을 지켜 작성하신다면 더 보기 좋은 코드가 될 것 같아요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants