-
Notifications
You must be signed in to change notification settings - Fork 49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[자동차 경주] 최승훈 미션 제출합니다. #60
base: sseung3424
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
승훈님 수고하셨습니다!!
몇몇부분에 대한 코멘트를 남겨봤어요.
실제 자동차 경주에 대한 테스트를 어떻게 할지 고민해보면 좋을 것 같습니다.
일요일에 뵐게요!!
승훈님이 생각하기에 public으로 두면 안 좋을 것 같다는 이유는 어떤걸까요? |
RacingCar 클래스는 CarRace 클래스에서만 사용되도록 하고, 외부에서는 접근하지 못하게 하고 싶어 package-private으로 지정했습니다. 그런데, package-private은 현업에서 잘 사용되지 않는다는 글들을 보게 되었습니다.
RacingCar 클래스를 public으로 두는 것이 올바르지 않다고 생각되는데, 접근 권한을 어느 수준으로 제한하는 것이 가장 바람직할까요?