[김찬호] Sprint1 #2

cheis11 · 2025-06-02T01:59:58Z

요구사항

기본 요구사항

프로젝트 초기화

IntelliJ를 통해 다음의 조건으로 Java 프로젝트를 생성합니다.
- - IntelliJ에서 제공하는 프로젝트 템플릿 중 Java를 선택합니다.
- - 프로젝트의 경로는 스프린트 미션 리포지토리의 경로와 같게 설정합니다.
  - 예를 들어 스프린트 미션 리포지토리의 경로가 /some/path/1-sprint-mission 이라면:
    - Name은 1-sprint-mission
    - Location은 /some/path
      으로 설정합니다.
- - Create Git Repository 옵션은 체크하지 않습니다.
- - Build system은 Gradle을 사용합니다. Gradle DSL은 Groovy를 사용합니다.
- - JDK 17을 선택합니다.
- - GroupId는 com.sprint.mission로 설정합니다.
- - ArtifactId는 수정하지 않습니다.
- - .gitignore에 IntelliJ와 관련된 파일이 형상관리 되지 않도록 .idea디렉토리를 추가합니다.
    ... .idea ...

도메인 모델링

서비스 설계 및 구현

도메인 모델 별 CRUD(생성, 읽기, 모두 읽기, 수정, 삭제) 기능을 인터페이스로 선언하세요.
- - 인터페이스 패키지명: com.sprint.mission.discodeit.service
- - 인터페이스 네이밍 규칙: [도메인 모델 이름]Service
- 다음의 조건을 만족하는 서비스 인터페이스의 구현체를 작성하세요.
- - 클래스 패키지명: com.sprint.mission.discodeit.service.jcf
- - 클래스 네이밍 규칙: JCF[인터페이스 이름]
- - Java Collections Framework를 활용하여 데이터를 저장할 수 있는 필드(data)를 final로 선언하고 생성자에서 초기화하세요.
- - data 필드를 활용해 생성, 조회, 수정, 삭제하는 메소드를 구현하세요.

메인 클래스 구현

메인 메소드가 선언된 JavaApplication 클래스를 선언하고, 도메인 별 서비스 구현체를 테스트해보세요.
- - 등록
- - 조회(단건, 다건)
- - 수정
- - 수정된 데이터 조회
- - 삭제
- - 조회를 통해 삭제되었는지 확인

기본 요구사항 커밋 태그

여기까지 진행 후 반드시 커밋해주세요. 그리고 sprint1-basic 태그를 생성해주세요.

심화

서비스 간 의존성 주입

도메인 모델 간 관계를 고려해서 검증하는 로직을 추가하고, 테스트해보세요.
- 힌트: Message를 생성할 때 연관된 도메인 모델 데이터 확인하기

주요 변경사항

- 양방향 연결
- - User, Message, Channel끼리의 관계를 양방향으로 연결
- 클래스 상속
- - User, Message, Channel에서 중복된 멤버를 BaseEntity에 추가하여 각 클래스에 상속
- 클래스 상태 추가
- - User, Message, Channel 각각의 상태를 열거형으로 [도메인 모델 이름]State로 구현
  - - 삭제된 User, Message, Channel은 출력이나 수정이 불가능
- 팩토리 패턴 구현
- - 구현체를 메인에서 선언하는 것이 아닌 ServiceFactory라는 클래스를 만들어 팩토리 패턴을 구현

스크린샷

멘토에게

셀프 코드 리뷰를 통해 질문 이어가겠습니다.

spring-kang · 2025-06-10T16:32:49Z

some/path/1-sprint-mission/build.gradle

+
+test {
+    useJUnitPlatform()
+}


EOL 해결해주세요.
나중에 git에서 무의미한 diff 가 찍힐 수 있습니다.

spring-kang · 2025-06-10T16:33:04Z

some/path/1-sprint-mission/settings.gradle

@@ -0,0 +1 @@
+rootProject.name = '1-sprint-mission'


EOL 해결해주세요.

spring-kang · 2025-06-10T16:33:27Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/BaseEntity.java

+    protected final Long createdAt;
+    protected Long updatedAt;


멤버 변수는 private 으로 선언하시는게 좋습니다. 캡슐화를 위해서요.

spring-kang · 2025-06-10T16:33:56Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/Channel.java

+    private List<User> users;
+    private List<Message> messages;


리스트는 선언과 동시에 초기화 해주시는게 좋습니다.

spring-kang · 2025-06-10T16:34:39Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/Channel.java

+    public void addUser(User user) {
+        if(!users.contains(user)) {
+            users.add(user);
+            user.addChannel(this);
+        }
+    }
+    public void removeUser(User user) {
+        if(users.contains(user)) {
+            users.remove(user);
+            user.removeChannel(this);
+        }
+    }
+
+    public void addMessage(Message message) {
+        if(!messages.contains(message)) {
+            messages.add(message);
+            message.addChannel(this);
+        }
+    }
+    public void removeMessage(Message message) {
+        if(messages.contains(message)) {
+            messages.remove(message);
+            message.removeChannel(this);
+        }
+    }


if 문에서는 예외를 처리하고, 그 밖에서 비지니스 로직이 처리되도록 구성해보세요.

spring-kang · 2025-06-10T16:35:26Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/ChannelState.java

+    private ChannelState(String state) {
+        this.state = state;
+    }
+}


EOL 해결해주세요.

spring-kang · 2025-06-10T16:35:42Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/ChannelState.java

+    DEACTIVATED("비활성화"),
+    DELETED("삭제");
+
+    private String state;


description이라고 명명해도 괜찮을것 같습니다.

spring-kang · 2025-06-10T16:36:12Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/Message.java

+    private Channel channel;
+    private String contents;
+    private MessageState state;
+


이런 의미 없는 공백은 제거해주시는게 좋습니다.
spotless 같은 코드 포맷팅 툴 설치해서 돌려주시면 이런 문제는 해결할 수 있습니다.

spring-kang · 2025-06-10T16:36:33Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/Message.java

+        SimpleDateFormat sdf = new SimpleDateFormat("dd/MM/yyyy HH:mm:ss");
+        String created = sdf.format(new Date(createdAt));
+        String updated = sdf.format(new Date(updatedAt));


이런 코드가 중복되네요. 이런 경우는 Util 클래스로 빼는것도 괜찮습니다.

spring-kang · 2025-06-10T16:37:28Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/Message.java

+    public void addChannel(Channel channel) {
+        if(this.channel == null) {
+            this.channel = channel;
+            channel.addMessage(this);
+        }
+    }
+
+    public void removeChannel(Channel channel) {
+        if(this.channel != channel) {
+            this.channel = null;
+            channel.removeMessage(this);
+        }
+    }
+
+    public void addUser(User user) {
+        if(this.user == null) {
+            this.user = user;
+            user.addMessage(this);
+        }
+    }
+    public void removeUser(User user) {
+        if(this.user != user) {
+            this.user = null;
+            user.removeMessage(this);
+        }
+    }


if 안에서 조건만족할때만 비지니스로직이 돌면 나중에 실 운영에서 이슈가 있습니다.
if 안에서는 조건 만족하지 않을 경우에 대한 예외처리를 하고 바깥에서 비지니스 로직을 처리해보시기 바랍니다.

spring-kang · 2025-06-10T16:38:12Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/MessageState.java

+package com.sprint.mission.discodeit.entity;
+
+public enum MessageState {
+    VISIBLE(""),


"" 로 두지마시고 이것도 설명 달아주시면 좋을것 같습니다.

spring-kang · 2025-06-10T16:38:28Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/MessageState.java

+    INVISIBLE("숨김"),
+    DELETED("삭제");
+
+    private String state;


description 정도면 적당할듯합니다.

spring-kang · 2025-06-10T16:38:50Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/User.java

+    public void addChannel(Channel channel) {
+        if(!channels.contains(channel)) {
+            channels.add(channel);
+            channel.addUser(this);
+        }
+    }
+    public void removeChannel(Channel channel) {
+        if(channels.contains(channel)) {
+            channels.remove(channel);
+            channel.removeUser(this);
+        }
+    }
+
+    public void addMessage(Message message) {
+        if(!messages.contains(message)) {
+            messages.add(message);
+            message.addUser(this);
+        }
+    }
+    public void removeMessage(Message message) {
+        if(messages.contains(message)) {
+            messages.remove(message);
+            message.removeUser(this);
+        }
+    }


이것도 위의 코멘트 보시면서 수정해보세요!

spring-kang · 2025-06-10T16:39:02Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/entity/UserState.java

+    public String getState() {
+        return state;
+    }
+}


EOL 해결해주세요.

spring-kang · 2025-06-10T16:39:24Z

...path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/ChannelService.java

+    public void leaveUser(Channel channel, User user);//채널에서 유저 추방
+    public void deleteChannel(Channel channel);//채널 삭제
+
+}


eol 해결해주세요.

spring-kang · 2025-06-10T16:39:45Z

...path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/ChannelService.java

+
+public interface ChannelService {
+    public Channel createChannel(Channel channel);//채널 생성
+    public List<Channel> getChannels();//모든 채널 출력


Suggested change

public List<Channel> getChannels();//모든 채널 출력

public List<Channel> getAllChannels();//모든 채널 출력

spring-kang · 2025-06-10T16:40:00Z

...path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/MessageService.java

+
+public interface MessageService {
+    public Message addMessage(Message message);//메세지 입력
+    public List<Message> getMessages();//모든 메세지 출력


Suggested change

public List<Message> getMessages();//모든 메세지 출력

public List<Message> getAllMessages();//모든 메세지 출력

spring-kang · 2025-06-10T16:40:12Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/UserService.java

+
+public interface UserService {
+    public User addUser(User user);//유저 가입
+    public List<User> getUsers();//모든 유저 출력


Suggested change

public List<User> getUsers();//모든 유저 출력

public List<User> getAllUsers();//모든 유저 출력

spring-kang · 2025-06-10T16:40:21Z

some/path/1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/UserService.java

+    public void updateUser(UUID userId, String updatedText);//유저 이름 수정
+    public void deleteUser(User user);//유저 탈퇴
+    public List<Message> findMessagesByUser(User user);//유저가 쓴 메세지 출력
+}


EOL 해결해주세요.

spring-kang · 2025-06-10T16:41:10Z

...sprint-mission/src/main/java/com/sprint/mission/discodeit/service/jcf/JCFChannelService.java

+    public Optional<Channel> findChannelById(UUID channelId) {
+        return data.stream()
+                .filter(channel -> channel.getChannelId().equals(channelId))
+                .filter(channel -> channel.getState() != ChannelState.DELETED)
+                .findFirst();
+    }


만약 결과가 empty인 경우 예외처리를 하고 Channel을 리턴하도록 바꿔보세요.

spring-kang · 2025-06-10T16:41:32Z

.../1-sprint-mission/src/main/java/com/sprint/mission/discodeit/service/jcf/JCFUserService.java

+    }
+
+    @Override
+    public Optional<User> findUserById(UUID UserId) {


만약 결과가 empty인 경우 예외처리를 하고 User를 리턴하도록 바꿔보세요.

spring-kang · 2025-06-10T16:41:49Z

...sprint-mission/src/main/java/com/sprint/mission/discodeit/service/jcf/JCFMessageService.java

+    }
+
+    @Override
+    public Optional<Message> findMessageById(UUID messageId) {


만약 결과가 empty인 경우 예외처리를 하고 Message를 리턴하도록 바꿔보세요.

Sprint 2 과제 제출

cheis11 and others added 3 commits June 2, 2025 10:53

Sprint 1 과제 제출

6438967

Merge branch 'codeit-bootcamp-spring:main' into sprint1

1f2dad9

Sprint 1 과제 제출

b83bf31

cheis11 force-pushed the sprint1 branch from 7f3707c to b83bf31 Compare June 3, 2025 01:30

cheis11 added 3 commits June 4, 2025 09:35

Sprint 1 과제 제출

afe6b1d

Sprint 1 과제 제출

c92d04c

Sprint 1 과제 제출

4b51958

cheis11 changed the base branch from main to 김찬호 June 10, 2025 02:46

spring-kang reviewed Jun 10, 2025

View reviewed changes

some/path/1-sprint-mission/settings.gradle

@@ -0,0 +1 @@

rootProject.name = '1-sprint-mission'

Copy link

Collaborator

spring-kang Jun 10, 2025

Choose a reason for hiding this comment

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

EOL 해결해주세요.

spring-kang reviewed Jun 10, 2025

View reviewed changes

spring-kang approved these changes Jun 10, 2025

View reviewed changes

joongwonAn added a commit to joongwonAn/4-sprint-mission that referenced this pull request Jun 15, 2025

sprint mission codeit-bootcamp-spring#2

edff64c

jinho-yoo-jack pushed a commit that referenced this pull request Jun 16, 2025

Merge pull request #2 from Eunhye0k/sprint2

5ca045a

Sprint 2 과제 제출

	public List<Channel> getChannels();//모든 채널 출력
	public List<Channel> getAllChannels();//모든 채널 출력

	public List<Message> getMessages();//모든 메세지 출력
	public List<Message> getAllMessages();//모든 메세지 출력

	public List<User> getUsers();//모든 유저 출력
	public List<User> getAllUsers();//모든 유저 출력

[김찬호] Sprint1 #2

Are you sure you want to change the base?

[김찬호] Sprint1 #2

Uh oh!

Conversation

cheis11 commented Jun 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

요구사항

기본 요구사항

심화

주요 변경사항

스크린샷

멘토에게

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cheis11 commented Jun 2, 2025 •

edited

Loading