Skip to content
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

Open
wants to merge 16 commits into
base: sseung3424
Choose a base branch
from

Conversation

sseung3424
Copy link

RacingCar 클래스는 CarRace 클래스에서만 사용되도록 하고, 외부에서는 접근하지 못하게 하고 싶어 package-private으로 지정했습니다. 그런데, package-private은 현업에서 잘 사용되지 않는다는 글들을 보게 되었습니다.

RacingCar 클래스를 public으로 두는 것이 올바르지 않다고 생각되는데, 접근 권한을 어느 수준으로 제한하는 것이 가장 바람직할까요?

Copy link

@hong-sile hong-sile left a comment

Choose a reason for hiding this comment

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

승훈님 수고하셨습니다!!

몇몇부분에 대한 코멘트를 남겨봤어요.

실제 자동차 경주에 대한 테스트를 어떻게 할지 고민해보면 좋을 것 같습니다.
일요일에 뵐게요!!

README.md Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
src/main/java/RacingCar.java Outdated Show resolved Hide resolved
src/main/java/RacingCar.java Outdated Show resolved Hide resolved
src/test/java/CarRaceTest.java Outdated Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
src/main/java/CarRace.java Outdated Show resolved Hide resolved
@hong-sile
Copy link

RacingCar 클래스는 CarRace 클래스에서만 사용되도록 하고, 외부에서는 접근하지 못하게 하고 싶어 package-private으로 지정했습니다. 그런데, package-private은 현업에서 잘 사용되지 않는다는 글들을 보게 되었습니다.
RacingCar 클래스를 public으로 두는 것이 올바르지 않다고 생각되는데, 접근 권한을 어느 수준으로 제한하는 것이 가장 바람직할까요?

승훈님이 생각하기에 public으로 두면 안 좋을 것 같다는 이유는 어떤걸까요?
class의 접근제어자도 중요하지만, 필드의 접근제어자는 어떤 식으로 정하면 좋을까요?

@sseung3424 sseung3424 changed the title [자동차 경주] 최승훈 미션 제출합니다. [자동차 경주] 최승훈 4단계 미션 제출합니다. Oct 25, 2024
@sseung3424 sseung3424 changed the title [자동차 경주] 최승훈 4단계 미션 제출합니다. [자동차 경주] 최승훈 미션 제출합니다. Oct 25, 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