Skip to content

고희준 [sprint3] #78

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 15 commits into from
May 10, 2025

Conversation

barrackkk
Copy link
Collaborator

추가 기능 요구사항

시간 타입 변경하기

  • 시간을 다루는 필드의 타입은 Instant로 통일합니다.
  • 기존에 사용하던 Long보다 가독성이 뛰어나며, 시간대(Time Zone) 변환과 정밀한 시간 연산이 가능해 확장성이 높습니다.

새로운 도메인 추가하기

image

  • 공통: 앞서 정의한 도메인 모델과 동일하게 공통 필드(id, createdAt, updatedAt)를 포함합니다.

  • ReadStatus
    사용자가 채널 별 마지막으로 메시지를 읽은 시간을 표현하는 도메인 모델입니다. 사용자별 각 채널에 읽지 않은 메시지를 확인하기 위해 활용합니다.

  • UserStatus
    사용자 별 마지막으로 확인된 접속 시간을 표현하는 도메인 모델입니다. 사용자의 온라인 상태를 확인하기 위해 활용합니다.

  • 마지막 접속 시간을 기준으로 현재 로그인한 유저로 판단할 수 있는 메소드를 정의하세요.
    마지막 접속 시간이 현재 시간으로부터 5분 이내이면 현재 접속 중인 유저로 간주합니다.

  • BinaryContent
    이미지, 파일 등 바이너리 데이터를 표현하는 도메인 모델입니다. 사용자의 프로필 이미지, 메시지에 첨부된 파일을 저장하기 위해 활용합니다.

  • 수정 불가능한 도메인 모델로 간주합니다. 따라서 updatedAt 필드는 정의하지 않습니다.

  • User, Message 도메인 모델과의 의존 관계 방향성을 잘 고려하여 id 참조 필드를 추가하세요.

  • 각 도메인 모델 별 레포지토리 인터페이스를 선언하세요.
    레포지토리 구현체(File, JCF)는 아직 구현하지 마세요. 이어지는 서비스 고도화 요구사항에 따라 레포지토리 인터페이스에 메소드가 추가될 수 있어요.


DTO 활용하기

  • DTO란?

UserService 고도화

create

  • 선택적으로 프로필 이미지를 같이 등록할 수 있습니다.
  • DTO를 활용해 파라미터를 그룹화합니다.
    유저를 등록하기 위해 필요한 파라미터, 프로필 이미지를 등록하기 위해 필요한 파라미터 등
  • username과 email은 다른 유저와 같으면 안됩니다.
  • UserStatus를 같이 생성합니다.

find, findAll

  • DTO를 활용하여: 사용자의 온라인 상태 정보를 같이 포함하세요.
  • 패스워드 정보는 제외하세요.

update

  • 선택적으로 프로필 이미지를 대체할 수 있습니다.
  • DTO를 활용해 파라미터를 그룹화합니다.
    수정 대상 객체의 id 파라미터, 수정할 값 파라미터

delete

  • 관련된 도메인도 같이 삭제합니다.
    BinaryContent(프로필), UserStatus

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

AuthService 구현

login

  • username, password과 일치하는 유저가 있는지 확인합니다.
  • 일치하는 유저가 있는 경우: 유저 정보 반환
  • 일치하는 유저가 없는 경우: 예외 발생
  • DTO를 활용해 파라미터를 그룹화합니다.

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

ChannelService 고도화

create

  • PRIVATE 채널과 PUBLIC 채널을 생성하는 메소드를 분리합니다.
  • 분리된 각각의 메소드를 DTO를 활용해 파라미터를 그룹화합니다.

PRIVATE 채널을 생성할 때:

  • 채널에 참여하는 User의 정보를 받아 User 별 ReadStatus 정보를 생성합니다.
  • name과 description 속성은 생략합니다.

PUBLIC 채널을 생성할 때에는 기존 로직을 유지합니다.

find

  • DTO를 활용하여: 해당 채널의 가장 최근 메시지의 시간 정보를 포함합니다.
  • PRIVATE 채널인 경우 참여한 User의 id 정보를 포함합니다.

findAll

  • DTO를 활용하여: 해당 채널의 가장 최근 메시지의 시간 정보를 포함합니다.
  • PRIVATE 채널인 경우 참여한 User의 id 정보를 포함합니다.
  • 특정 User가 볼 수 있는 Channel 목록을 조회하도록 조회 조건을 추가하고, 메소드 명을 변경합니다. findAllByUserId
  • PUBLIC 채널 목록은 전체 조회합니다.
  • PRIVATE 채널은 조회한 User가 참여한 채널만 조회합니다.

update

  • DTO를 활용해 파라미터를 그룹화합니다.
    수정 대상 객체의 id 파라미터, 수정할 값 파라미터
  • PRIVATE 채널은 수정할 수 없습니다.

delete

  • 관련된 도메인도 같이 삭제합니다.
    Message, ReadStatus

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

MessageService 고도화

create

  • 선택적으로 여러 개의 첨부파일을 같이 등록할 수 있습니다.
  • DTO를 활용해 파라미터를 그룹화합니다.

findAll

  • 특정 Channel의 Message 목록을 조회하도록 조회 조건을 추가하고, 메소드 명을 변경합니다. findAllByChannelId

update

  • DTO를 활용해 파라미터를 그룹화합니다.
    수정 대상 객체의 id 파라미터, 수정할 값 파라미터

delete

  • 관련된 도메인도 같이 삭제합니다.
    첨부파일(BinaryContent)

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

ReadStatusService 구현

create

  • DTO를 활용해 파라미터를 그룹화합니다.
  • 관련된 Channel이나 User가 존재하지 않으면 예외를 발생시킵니다.
  • 같은 Channel과 User와 관련된 객체가 이미 존재하면 예외를 발생시킵니다.

find

  • id로 조회합니다.

findAllByUserId

  • userId를 조건으로 조회합니다.

update

  • DTO를 활용해 파라미터를 그룹화합니다.
    수정 대상 객체의 id 파라미터, 수정할 값 파라미터

delete

  • id로 삭제합니다.

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

UserStatusService 고도화

create

  • DTO를 활용해 파라미터를 그룹화합니다.
  • 관련된 User가 존재하지 않으면 예외를 발생시킵니다.
  • 같은 User와 관련된 객체가 이미 존재하면 예외를 발생시킵니다.

find

  • id로 조회합니다.

findAll

  • 모든 객체를 조회합니다.

update

  • DTO를 활용해 파라미터를 그룹화합니다.
    수정 대상 객체의 id 파라미터, 수정할 값 파라미터

updateByUserId

  • userId 로 특정 User의 객체를 업데이트합니다.

delete

  • id로 삭제합니다.

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

BinaryContentService 구현

create

  • DTO를 활용해 파라미터를 그룹화합니다.

find

  • id로 조회합니다.

findAllByIdIn

  • id 목록으로 조회합니다.

delete

  • id로 삭제합니다.

의존성

  • 같은 레이어 간 의존성 주입은 순환 참조 방지를 위해 지양합니다. 다른 Service 대신 필요한 Repository 의존성을 주입해보세요.
image

새로운 도메인 Repository 구현체 구현

  • 지금까지 인터페이스로 설계한 각각의 Repository를 JCF, File로 각각 구현하세요.
image

심화 요구사항

Bean 다루기

  • Repository 구현체 중에 어떤 구현체를 Bean으로 등록할지 Java 코드의 변경 없이 application.yaml 설정 값을 통해 제어해보세요.
```yaml
# application.yaml
discodeit:
    repository: 
        type: jcf   # jcf | file
  • discodeit.repository.type 설정값에 따라 Repository 구현체가 정해집니다.

  • 값이 jcf 이거나 없으면 JCF*Repository 구현체가 Bean으로 등록되어야 합니다.

  • 값이 file 이면 File*Repository 구현체가 Bean으로 등록되어야 합니다.

  • File*Repository 구현체의 파일을 저장할 디렉토리 경로를 application.yaml 설정 값을 통해 제어해보세요.

# application.yaml
discodeit:
    repository: 
        type: jcf   # jcf | file
        file-directory: .discodeit

멘토에게

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

@barrackkk barrackkk requested a review from ssjf409 May 4, 2025 12:59
@barrackkk barrackkk added the 매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다. label May 4, 2025

if (request.content()) { //주강사님이 동적정적 바인딩? 불러오는거 안배웠다고 null로 표기하라고 하셨습니다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

넵 ㅎㅎ

@ssjf409
Copy link
Collaborator

ssjf409 commented May 6, 2025

안녕하세요. 희준님. 희준님이 이렇게 코드를 잘 작성하시는 줄 몰랐네요.
전체적으로 보기 깔끔하고 거의 완벽하다고 느꼈습니다.
다만, 일부 로직은 실제 운영에서 문제가 될만한 로직들이 있었어요. 모든 데이터를 조회해와서 반복문으로 살펴보는 로직들이 그런것들이었는데요.
제가 코멘트 달지 않은것들 중에도 그런게 남아있는지 모르겠네요. 보이는대로 코멘트 달긴했는데.. 한번 확인부탁드려요

고생하셨고 희준님이 잘해지시고 있는것 같아 저도 뿌듯하네요 ㅎㅎ

@barrackkk
Copy link
Collaborator Author

덕분에 실력이 느네요 ㅎ.ㅎ 칭찬 감사합니다 !!
제가 코드를 작성할 때 그냥 편의상 모든 데이터를 조회하게 했는데.. 이런 습관은 고쳐야겠네요 !!!
저도 다시한번 검토해보겠습니다 !!

@ssjf409
Copy link
Collaborator

ssjf409 commented May 10, 2025

고생하셨습니다. 희준님 딱히 더 고칠건 없을 것 같아 머지할게요!

@ssjf409 ssjf409 merged commit 4a766f0 into codeit-bootcamp-spring:고희준 May 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
매운맛🔥 뒤는 없습니다. 그냥 필터 없이 말해주세요. 책임은 제가 집니다.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants