Java Assignment3 upload by HoonSubKim#12
Java Assignment3 upload by HoonSubKim#12khsrla9806 wants to merge 9 commits intoFastCampusKDTBackend:mainfrom
Conversation
Jungdahee
left a comment
Jungdahee
left a comment
There was a problem hiding this comment.
안녕하세요 훈섭님~
먼저 과제 수행하신다고 고생 많으셨어요!
이번에도 역시나 빠르게 제출하는 성실함!ㅎㅎ아주 칭찬해 드리고 싶습니다.
일단 코드에서 고민한 흔적이 많이 보여서 좋았는데요, equals()나 hashCode()를 완벽하게 구현하지 못한다면 차라리 IDE의 자동 완성 기능을 활용하는 것도 좋을 것 같아요!
하나 하나 다 비교를 하게 되면 분명 놓치는 부분이 생길 수도 있으니, 이 부분은 한번 고민해 보시길 바라요!
그리고 commit 단위가 너무 커보여요. 지금은 과제 제출이니 세세하게 나누지 않아도 크게 문제되진 않지만, 현업에 가게 되면 commit message로 어떤 작업이 수행되었는지 쉽게 알 수 있도록 한답니다.
다음 과제 제출 시에는 이 부분 한번 고민해서 반영해 보시죠!
feat, fix 등과 같이 작업 내용을 보다 한 눈에 파악하기 쉽게 한번 표현해 봅시다:)
수고 많으셨어요~
| import java.util.Objects; | ||
|
|
||
| public class Electronic { | ||
| private static int autoIncrementNumber = 0; |
There was a problem hiding this comment.
static 변수를 가장 상위에 위치하도록 해주셨네요. 아주 굿!
| private String modelName; | ||
| private Company companyName; | ||
| private String dateOfDate; | ||
| private AuthMethod[] authMethod; |
There was a problem hiding this comment.
사소한 부분이지만, 배열의 형태이니 복수형을 의미하는 변수로 변경해 보는 건 어떨까요?
There was a problem hiding this comment.
변수 이름도 신경쓰면서 작업해봐야겠네요. 감사합니다!
| private String modelName; | ||
| private Company companyName; | ||
| private String dateOfDate; | ||
| private AuthMethod[] authMethod; |
There was a problem hiding this comment.
| private AuthMethod[] authMethod; | |
| private AuthMethod[] authMethods; |
| Company companyName, | ||
| String dateOfDate | ||
| ) { | ||
| this.productNo = this.generateProductNo(); |
| } | ||
|
|
||
| private String generateProductNo() { | ||
| String id = String.format("%04d", ++autoIncrementNumber); |
There was a problem hiding this comment.
정규식 사용으로 코드가 한결 간단해 졌어요 아주 잘하셨습니다! 👍
| Users users = (Users) obj; | ||
|
|
||
| if (userList.length != users.userList.length) return false; | ||
| for (int i = 0; i < users.userList.length; i++) { |
|
|
||
| @Override | ||
| public int hashCode() { | ||
| return Arrays.hashCode(userList); |
| public class Electronics { | ||
| private static Electronics instance; | ||
| private Electronic[] electronicList; | ||
| private static final int DEFAULT_SIZE = 50; |
| return electronic; | ||
| } | ||
| } | ||
| throw new IllegalArgumentException("찾는 " + productNo + "에 해당하는 제품은 존재하지 않습니다."); |
| public Electronic[] groupByAuthMethod(AuthMethod authMethod) { | ||
| List<Electronic> electronics = new ArrayList<>(); | ||
| for (Electronic electronic : electronicList) { | ||
| for (AuthMethod method : electronic.getAuthMethod()) { |
There was a problem hiding this comment.
놓치기 쉬운 부분인데 잘 캐치하셨어요 잘하셨습니다:) 이 부분도 역시나 stream으로 변경해 볼 수 있겠죠?
2 participants
코드 리뷰 빡세게 부탁드립니다 ! 🔥🔥🔥