[곽민혁] Sprint 1 #8
Conversation
|
|
||
| //TIP To <b>Run</b> code, press <shortcut actionId="Run"/> or | ||
| // click the <icon src="AllIcons.Actions.Execute"/> icon in the gutter. | ||
| public class Main { |
There was a problem hiding this comment.
사용하지 않는 또는 불필요한 클래스 파일은 과감하게 삭제해주세요.
코드를 읽는 독자에게 잘못된 정보를 전달할 수 있습니다.
| import java.util.UUID; | ||
|
|
||
| public class BaseEntity { | ||
| private UUID id; |
There was a problem hiding this comment.
id 값은 절대 바뀌지 않죠? 불변성 설정을 위해서 final 키워드를 붙이는 것은 필수 입니다.
| return updatedAt; | ||
| } | ||
|
|
||
| public void touchUpdatedAt() { |
There was a problem hiding this comment.
touch라는 단어를 이용하여 적절하게 의미 전달이 잘되는 메서드명 좋았습니다.
|
|
||
| public class Channel extends BaseEntity { | ||
| private String channelName; | ||
| private List<User> users; |
There was a problem hiding this comment.
Collections 타입 변수에 저장되는 값은 변하지 않고 컬렉션 타입의 요소들만 변하기 때문에 final 키워드를 붙여주는 것이 올바른 방법입니다.
| } | ||
|
|
||
| public void addUser(User user) { | ||
| if (!users.contains(user)) { |
There was a problem hiding this comment.
users.contains(user) 문장이 자주 선언되는데, 이런 경우, 하나의 메서드로 빼서 구현하는 것이 코드의 가독성이나 유지보수성에 좋습니다.
| import java.util.List; | ||
|
|
||
| public class Message extends BaseEntity { | ||
| private User user; |
There was a problem hiding this comment.
Message 클래스에서 의존성 주입을 받는 객체들은 절대로 변할 일이 없죠?
Setter. 함수가 없으니깐요! 그렇다면 final로 불변성을 가지고 있다는 것을 표현해주는 것이 좋고
final 키워드를 통해서 변하지 않도록 설정을 해준다면 의도하지 오류를 만날 확률이 급격하게 떨어집니다..
|
|
||
| @Override | ||
| public User createUser(User user) { | ||
| data.add(user); |
There was a problem hiding this comment.
해당하는 User가 추가되어도 되는지 유효성 검사 로직이 누락되었습니다.
| @Override | ||
| public Optional<User> getUserByUserId(UUID userId) { | ||
| return data.stream() | ||
| .filter(u -> u.getStatus().equals(UserStatus.ACTIVE)) |
There was a problem hiding this comment.
equals로 문자열을 비교할 때, 비교문자열.equals(비교하고 싶은 변수 or 데이터)로 쓰는 것이 좀 더 안전합니다.
ex) UserStatus.ACTIVE.equals(u.getStatus()) 이런 식으로 말이죠.
이 이유는 한번 찾아서 공부해보시기 바랍니다.
| return data.stream() | ||
| .filter(ch -> ch.getId().equals(channelId)) | ||
| .findFirst() | ||
| .map(channel -> { |
There was a problem hiding this comment.
Stream API를 사용할 때, map 안에서 리턴문을 작성하는 것 보다는 생략하는 문법을 사용하는 것이 좋습니다.
아래 처럼 작성하면 return문을 생략해도 updateName에서 리턴하는 String 타입의 데이터를 반환할 수 있습니다.
.map(channel -> channel.updateName(newChannelName)) // updateName이 void면 X
요구사항
기본
프로젝트 초기화
IntelliJ를 통해 다음의 조건으로 Java 프로젝트를 생성합니다.
IntelliJ에서 제공하는 프로젝트 템플릿 중 Java를 선택합니다.
프로젝트의 경로는 스프린트 미션 리포지토리의 경로와 같게 설정합니다.
예를 들어 스프린트 미션 리포지토리의 경로가 /some/path/1-sprint-mission 이라면:
Create Git Repository 옵션은 체크하지 않습니다.
Build system은 Gradle을 사용합니다. Gradle DSL은 Groovy를 사용합니다.
JDK 17을 선택합니다.
GroupId는 com.sprint.mission로 설정합니다.
ArtifactId는 수정하지 않습니다.
.gitignore에 IntelliJ와 관련된 파일이 형상관리 되지 않도록 .idea디렉토리를 추가합니다.
도메인 모델링
생성자
메소드
서비스 설계 및 구현
메인 클래스 구현
심화 요구 사항
주요 변경 사항
멘토에게