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

부산대_ BE_ 이성훈 6주차 (2단계) #315

Open
wants to merge 37 commits into
base: nextrplue
Choose a base branch
from

Conversation

NextrPlue
Copy link

과제 제출을 위한 선 pr입니다.

NextrPlue and others added 30 commits July 31, 2024 14:52
Copy link

@cmst-steve cmst-steve left a comment

Choose a reason for hiding this comment

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

1단계에서 리뷰드린 내용 잘 적용해주시면 좋을 것 같습니다.
1단계와 리뷰드릴 내용이 같아 머지하겠습니다~

return name;
}

public void setName(String name) {

Choose a reason for hiding this comment

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

setter는 그냥 습관처럼 만들기보다는 필요한 경우에만 만들어주는게 좋습니다.

@cmst-steve
Copy link

@NextrPlue 컨플릭 해결후 디엠주세요~

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