Conversation
|
|
||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
전혀 중요하진 않지만, 여기 간격이 두 칸 이상 띄워져 있어서? 라인 일관성? 자체가 조금 다른 것 같아요!
|
|
||
| @Serial | ||
| private static final long serialVersionUID = 1L; | ||
| private transient final UUID id; |
There was a problem hiding this comment.
여기에서 transient 는 빼셔야 해요. 직렬화 할 떄 빠지거든요?
나중에 파일에서 다시 읽을떄(역 직렬화) 그 필드는 타입의 기본값으로 채워지게 되서, uuid 는 null 이 됩니다!
There was a problem hiding this comment.
Message, User 도 마찬가지에요~!
| import java.util.List; | ||
| import java.util.UUID; | ||
|
|
||
| public interface UserRepository { |
There was a problem hiding this comment.
인터페이스 잘 선언하셧어요. 이제 이거를 구현하는 구현 객체가 있어야겠죠!? 그대로 구현해보세요! 그러면 자동으로 구현해야할 메서드가 뜰거에요! 컴파일 에러로 잡힐겁니다.
| import java.util.List; | ||
| import java.util.UUID; | ||
|
|
||
| public interface ChannelRepository { |
| import java.util.List; | ||
| import java.util.UUID; | ||
|
|
||
| public interface MessageRepository { |
| import java.util.List; | ||
| import java.util.UUID; | ||
|
|
||
| public class FileUserService implements UserService { |
There was a problem hiding this comment.
이거는 서비스인데 제일 위의 패키지가 repository 이네요.
그런데 FileChannelService.java, FileMessageService.java -> 이거는 service/file/에 동일본 있어서
repository/file/의 서비스 3개 삭제가 필요합니다~!
| import com.sprint.mission.discodeit.entity.User; | ||
| import com.sprint.mission.discodeit.repository.UserRepository; | ||
| import com.sprint.mission.discodeit.service.UserService; | ||
| import java.io.FileReader; |
There was a problem hiding this comment.
사용하지 않는 import 는 지워주세요~!
There was a problem hiding this comment.
intelliJ 에서 특정 클래스에서 사용하지 않는 import 정리하는 단축키가 있습니다~!
|
현석님, sprint2-2 뼈대 잘 잡으셨어요! 서비스 인터페이스 나누고, Repository 인터페이스 선언하고, 서비스가 Repository를 생성자로 1단계 — id의 transient 먼저 제거 -> 리뷰 남겻습니다! 일단 요렇게만 먼저 해보세요! 사실 나머지는 파일 정리 (service 에 정의되어 잇는데 rpeository 에 또 선언햇다던지) 이런 것들이라 수고하셨습니다 현석님!! |
2 participants
요구사항
기본
심화
주요 변경사항
스크린샷
멘토에게