Java Assignment3 upload by Hyeonjinyun #18
Conversation
There was a problem hiding this comment.
안녕하세요 현진님~
과제 진행하신다고 너무 고생많으셨습니다~!
코드를 작성하실 때 코드 스타일에 대해서도 조금 신경 써주셨으면 좋겠습니다.
- 메서드, 변수들의 위치, 줄바꿈 등등
깃 커밋의 단위를 좀 더 잘개 쪼개면 좋을 것 같아요!
ex)
- feat: User 클래스 생성 및 필드 작성
- feat: 생성자 생성
- feat: 싱글톤 패턴 구현
- feat: 회원 조회 메서드 구현
- refactor: 회원 조회 메서드 로직 분리
요렇게요!!
커밋 내역만으로 어떤 작업을 했는지 알 수 있다면 추후 협업 과정에서 큰 도움을 받을 수 있을 거에요~!
다음 과제에서는 요구사항을 좀 더 꼼꼼히 읽어보셨으면 좋겠습니다!
그리고 pr을 올리실 때 본인만의 템플릿을 가지고 계시면 좋을 것 같습니다.
ex)
- 내가 중점적으로 생각한 부분
- 작업 내용
- 궁금한 점
- 함께 이야기하고 싶은 부분
| package Practice01; | ||
|
|
||
| public class AuthMethod { | ||
| public enum authMethod{Fingerprint, Pattern, Pin, Face}; |
| private String dateOfMade; | ||
|
|
||
| private Company companyName; | ||
| private AuthMethod[] arrAuth; |
There was a problem hiding this comment.
굳이 어떤 자료형인지를 변수명에 나타낼 필요는 없을 것 같아요~
| private AuthMethod[] arrAuth; | |
| private AuthMethod[] authMethods; |
| import java.util.Arrays; | ||
| import java.util.Objects; | ||
|
|
||
| public class Electronic { |
There was a problem hiding this comment.
요구사항을 꼼꼼히 읽어주시면 좋을 것 같아요!!
생성자가 빠져있네요 ㅠ
| public class Electronic { | ||
| private String productNo; | ||
| private String modelName; | ||
| private String dateOfMade; |
There was a problem hiding this comment.
LocalDateTime이나 LocalDate를 사용하는 게 어떨까요?
|
|
||
| //2-2 회원 아이디 userId를 통해 인자로 주어진 회원번호에 해당하는 회원을 반환하는 함수를 작성 | ||
| public User findByUserId(String userId){ | ||
| for(int i=0; i<userslist.length;i++){ |
| } | ||
|
|
||
|
|
||
| //2-3 인자로 주어진 회원 정보를 깊은 복사 (deepCopy - '실제 값'을 새로운 메모리 공간에 복사하는 것) 하는 함수를 작성 |
| // - 메소드 - 생성자, getter(), setter(), hashCode(), equals(), toString() | ||
| private static Electronic[] electroniclist; | ||
| private Electronics singleton; | ||
| private Electronics(){ |
There was a problem hiding this comment.
기본 생성자에서 electroniclist 을 초기화해야하지 않을까요?
| } | ||
|
|
||
| //3-3 전자제품들 중 인자로 주어진 제조 회사를 찾아서 하나의 배열에 반환하는 함수를 작성 | ||
| public static List<Electronic> groupByCompanyName(Company company){ |
|
|
||
| } | ||
|
|
||
| public Electronic[] getElectroniclist() { |
There was a problem hiding this comment.
여기서 문제!
이렇게 반환된 배열의 값을 바꾸면 원본 배열에도 영향이 있을까요 없을까요?
There was a problem hiding this comment.
안녕하세요 현진님~
과제 진행하신다고 너무 고생많으셨습니다~!
코드를 작성하실 때 코드 스타일에 대해서도 조금 신경 써주셨으면 좋겠습니다.
- 메서드, 변수들의 위치, 줄바꿈 등등
깃 커밋의 단위를 좀 더 잘개 쪼개면 좋을 것 같아요!
ex)
- feat: User 클래스 생성 및 필드 작성
- feat: 생성자 생성
- feat: 싱글톤 패턴 구현
- feat: 회원 조회 메서드 구현
- refactor: 회원 조회 메서드 로직 분리
요렇게요!!
커밋 내역만으로 어떤 작업을 했는지 알 수 있다면 추후 협업 과정에서 큰 도움을 받을 수 있을 거에요~!
다음 과제에서는 요구사항을 좀 더 꼼꼼히 읽어보셨으면 좋겠습니다!
그리고 pr을 올리실 때 본인만의 템플릿을 가지고 계시면 좋을 것 같습니다.
ex)
- 내가 중점적으로 생각한 부분
- 작업 내용
- 궁금한 점
- 함께 이야기하고 싶은 부분
|
|
||
| List<Electronic> groupByAuthMethodlist =new ArrayList<>(); | ||
| for(int i=0; i<electroniclist.length;i++){ | ||
| if(electroniclist[i].getArrAuth().equals(authMethod)){ |
| copy.setUserEmail(user.getUserEmail()); | ||
| copy.setUserBirthDate(user.getUserBirthDate()); | ||
| copy.setUserPhoneNumber(user.getUserPhoneNumber()); | ||
| copy.setElectronicDevices(user.getElectronicDevices()); |
제가 잘 이해해서 작성한건지 모르겠네요..틀린 부분이 있다면 지적해주세요 감사합니다!