Skip to content

Conversation

@growingJd
Copy link
Contributor

240516 김재동 0x02 풀이

@lee-JunR lee-JunR closed this May 16, 2024
@lee-JunR
Copy link
Owner

주석까지 남기면서 열심히 푸신게 보이네요! 저도 부족하지만 최선을 다해서 리뷰 남겨보겠습니다!

@lee-JunR lee-JunR reopened this May 16, 2024
Copy link
Owner

@lee-JunR lee-JunR left a comment

Choose a reason for hiding this comment

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

잘보고갑니당! 앞으로도 화이팅이에요!

Comment on lines +9 to +18
Copy link
Owner

Choose a reason for hiding this comment

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

문자열 입출력과 split 에 BufferedReader, InputStreamReader, StringTokenizer 를 사용하신게 인상적이네요!
저도 항상 Scanner를 사용해왔던 터라 이번에 익숙해지는데 시간이 많이 필요할 것 같아요..!

Comment on lines +9 to +11
Copy link
Owner

Choose a reason for hiding this comment

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

코드를 작성할때 주석을 남기는 습관은 정말정말 잘하고 계신 것 같아요!
하지만 주석을 남기실때 공백이 너무 긴 것 같아 보기 조금 힘든 감이 있습니다!
또한 주석을 남기실때 일관성에 대한 부분도 참고하시면 더 명확한 코드가 나올 것 같아요!

관련 아티클도 함께 첨부합니다!
좋은 주석읠 위한 9가지 규칙

Copy link
Owner

Choose a reason for hiding this comment

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

StringBuilder 객체를 만들어서 출력을 하신게 아주 인상깊네요!
저도 이 부분은 다음 주차부터는 참고해서 진행해야겠습니다!

Comment on lines +14 to +27
Copy link
Owner

Choose a reason for hiding this comment

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

해당 문제에서는 크게 상관이 없지만
배열 초기화 -> 해당 배열에 접근하면서 temp로 접근 후 최대값 추출
로 진행하셨는데 꼭 배열을 사용하지 않아도 index, temp 변수만으로도 최대값은 구할 수 있을 것 같아요!

또한 해당 코드에서 index 변수를 사용하지 않고 for문에서 생성된 i를 index로 사용하면 변수도 줄일 수 있겠어요~

@lee-JunR lee-JunR merged commit cb2fdcb into lee-JunR:main May 17, 2024
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