Feature/crew

[bikooju]

📌 PR 요약 (하나 이상 선택해주세요)

• 기능 추가
• 기능 삭제
• 버그 수정
• 의존성, 환경 변수, 빌드 관련 코드 업데이트

---

💡 주요 변경사항

• ---

🔍 관련 이슈

---

🌿 브랜치 정보

• 병합 대상 브랜치:
• 현재 작업 브랜치:

---

🧪 테스트 방법

• ---

⚠️ 주의사항

[bikooju]

🤖 AI Code Review

코드 리뷰

✅ 잘된 점

• NotificationAsyncService 클래스를 분리하여 비동기 처리를 담당하도록 설계한 점
• @Async와 @Transactional(readOnly = true)를 사용하여 비동기 처리와 트랜잭션을 관리하는 점

⚠️ 개선이 필요한 부분

1. Null Pointer Exception 방지

문제점: notificationRepository.findUnread(userId)에서 반환되는 List<Notification>이 null일 경우 NullPointerException이 발생할 수 있음

개선 전:

List<Notification> unread = notificationRepository.findUnread(userId);
if (!unread.isEmpty()) {
    notificationSsePort.sendUnreadOnConnect(userId, emitter, unread);
}

개선 후:

List<Notification> unread = notificationRepository.findUnread(userId);
if (unread != null && !unread.isEmpty()) {
    notificationSsePort.sendUnreadOnConnect(userId, emitter, unread);
}

이유: notificationRepository.findUnread(userId)에서 반환되는 값이 null일 경우 isEmpty()를 호출하면 NullPointerException이 발생할 수 있으므로 null 체크를 추가하여 방지한다.

2. 트랜잭션 관리

문제점: NotificationService 클래스에서 @Transactional 어노테이션을 여러 번 사용하고 있음

개선 전:

@Override
@Transactional
public void createAndSend(CreateAndSendNotificationCommand command) {
    // ...
}

@Override
@Transactional
public void read(ReadNotificationCommand command) {
    // ...
}

개선 후:

@Service
@Transactional
public class NotificationService {
    // ...
}

이유: 클래스 전체에 @Transactional을 적용하여 트랜잭션 관리를 일원화하고 코드 중복을 줄일 수 있다.

3. 비동기 처리 예외 처리

문제점: NotificationAsyncService 클래스에서 비동기 처리 중 발생하는 예외를 처리하지 않고 있음

개선 전:

@Async
@Transactional(readOnly = true)
public void pushUnreadOnConnect(Long userId, SseEmitter emitter) {
    // ...
}

개선 후:

@Async
@Transactional(readOnly = true)
public void pushUnreadOnConnect(Long userId, SseEmitter emitter) {
    try {
        // ...
    } catch (Exception e) {
        // 예외 처리 로직
    }
}

이유: 비동기 처리 중 발생하는 예외를 처리하지 않으면 예외가 무시되고 로그에 기록되지 않으므로 예외 처리 로직을 추가하여 예외를 기록하고 처리할 수 있다.