Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
에 맞춰 기능을 구현했습니다.
다만 발생한 오류가
인코딩이 제대로 되지 않아 출력되는 문자를 전부 영어로 바꿔줘야 했습니다. slack 에 올라온대로 해봤는데 잘 되지 않아서... 제출 후에도 더 시도해보겠습니다.
또한 Application 에서 System.exit()을 사용하면 안되서 return을 사용했는데 return이 회색으로 표시되며 제대로 작동하지 않았습니다. 이부분은 해결방법을 잘 모르겠습니다.
이 미니과제를 하며 느낀점은 코드를 단위별로, 함수별로 나눠서 구현한다는게 생각보다 까다롭다는걸 알게 되었고 코드의 전체적인 구조를 구상하는데 시간을 썼던것 같습니다. 또한 요구사항중 depth이 3 이상이 되지 않게 구현하는것도 까다로웠습니다. 제가 습관적으로 작성하는 코드는 3이 넘는 코드가 많은데 이는 가독성이 별로라는것을 알게 되었고 코드를 좀 더 깔끔하게 작성하는 법에 대해 배울 수 있었습니다. 마지막으로 테스트코드를 작성해보며 기존에 제 방식인 코드를 직접 실행해서 직접 입력하는 방식이 얼마나 비효율적인지 알게 되었고 기능 하나하나 테스트 할 수 있는 방법에 대해 알게 되었습니다.