Skip to content

Hotfix/loseminho : terminated 상태 조회 제거 #315

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.

Sign up for GitHub

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 1 commit into from
Oct 16, 2025
Merged

Hotfix/loseminho : terminated 상태 조회 제거 #315

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
61 changes: 40 additions & 21 deletions src/main/java/com/back/domain/studyroom/repository/RoomRepositoryImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@ public class RoomRepositoryImpl implements RoomRepositoryCustom {
private final QUser user = QUser.user;

/**
* 공개 방 중 입장 가능한 방들 조회 (페이징)
* 공개 방 중 입장 가능한 방들 조회 (페이징, TERMINATED 제외)
* 조회 조건:
* - 비공개가 아닌 방 (isPrivate = false)
* - 활성화된 방 (isActive = true)
* - 입장 가능한 상태 (WAITING 또는 ACTIVE)
*
*
* 참고: 정원 체크는 Redis 기반으로 프론트엔드/서비스에서 수행
* @param pageable 페이징 정보
* @return 페이징된 방 목록
Expand All @@ -52,7 +52,8 @@ public Page<Room> findJoinablePublicRooms(Pageable pageable) {
.where(
room.isPrivate.eq(false),
room.isActive.eq(true),
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE)
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE), // WAITING, ACTIVE만 (TERMINATED, PAUSED 제외)
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 명시적 제외
)
.orderBy(room.createdAt.desc())
.offset(pageable.getOffset())
Expand All @@ -66,15 +67,16 @@ public Page<Room> findJoinablePublicRooms(Pageable pageable) {
.where(
room.isPrivate.eq(false),
room.isActive.eq(true),
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE)
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE),
room.status.ne(RoomStatus.TERMINATED)
)
.fetchOne();

return new PageImpl<>(rooms, pageable, totalCount != null ? totalCount : 0);
}

/**
* 사용자가 참여 중인 방 조회 (MEMBER 이상만)
* 사용자가 참여 중인 방 조회 (MEMBER 이상만, TERMINATED 제외)
* 조회 조건:
* - 특정 사용자가 MEMBER 이상으로 등록된 방 (VISITOR 제외)
* - DB에 저장된 멤버십만 조회
Expand All @@ -89,7 +91,8 @@ public List<Room> findRoomsByUserId(Long userId) {
.join(room.roomMembers, roomMember) // 멤버 조인
.where(
roomMember.user.id.eq(userId),
roomMember.role.ne(com.back.domain.studyroom.entity.RoomRole.VISITOR) // VISITOR 제외
roomMember.role.ne(com.back.domain.studyroom.entity.RoomRole.VISITOR), // VISITOR 제외
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.fetch();
}
Expand Down Expand Up @@ -160,7 +163,7 @@ public Page<Room> findRoomsWithFilters(String title, RoomStatus status, Boolean
}

/**
* 인기 방 조회 (참가자 수 기준) - 공개+비공개 포함
* 인기 방 조회 (참가자 수 기준, TERMINATED 제외)
*
* 참고: 참가자 수는 Redis에서 조회하므로 DB에서는 정렬 불가
* 서비스 레이어에서 Redis 데이터로 정렬 필요
Expand All @@ -177,7 +180,8 @@ public Page<Room> findPopularRooms(Pageable pageable) {
.selectFrom(room)
.leftJoin(room.createdBy, user).fetchJoin() // N+1 방지
.where(
room.isActive.eq(true)
room.isActive.eq(true),
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.orderBy(room.createdAt.desc()) // 최신순 (서비스에서 Redis 기반으로 재정렬)
.offset(pageable.getOffset())
Expand All @@ -189,7 +193,8 @@ public Page<Room> findPopularRooms(Pageable pageable) {
.select(room.count())
.from(room)
.where(
room.isActive.eq(true)
room.isActive.eq(true),
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.fetchOne();

Expand Down Expand Up @@ -272,23 +277,26 @@ public Optional<Room> findByIdWithLock(Long roomId) {
}

/**
* 모든 방 조회 (공개 + 비공개 전체)
* 모든 방 조회 (공개 + 비공개 전체, TERMINATED 제외)
* 조회 조건:
* - 모든 방 (공개 + 비공개)
* 정렬:
* 1. 열린 방(WAITING, ACTIVE) 우선
* 2. 닫힌 방(PAUSED, TERMINATED) 뒤로
* 3. 최신 생성순
*
*
* 비공개 방은 컨트롤러/서비스 레이어에서 정보 마스킹 합니당
*/
@Override
public Page<Room> findAllRooms(Pageable pageable) {
List<Room> rooms = queryFactory
.selectFrom(room)
.leftJoin(room.createdBy, user).fetchJoin()
.where(
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.orderBy(
// 열린 방 우선 (0), 닫힌 방 뒤로 (1)
// 열린 방 우선 (0), 일시정지 방 뒤로 (1)
room.status.when(RoomStatus.WAITING).then(0)
.when(RoomStatus.ACTIVE).then(0)
.otherwise(1).asc(),
Expand All @@ -302,23 +310,27 @@ public Page<Room> findAllRooms(Pageable pageable) {
Long totalCount = queryFactory
.select(room.count())
.from(room)
.where(
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.fetchOne();

return new PageImpl<>(rooms, pageable, totalCount != null ? totalCount : 0);
}

/**
* 공개 방 전체 조회
* 공개 방 전체 조회 (TERMINATED 제외)
* 조회 조건:
* - isPrivate = false
* - includeInactive에 따라 닫힌 방 포함 여부 결정
* 정렬: 열린 방 우선 → 최신순
*/
@Override
public Page<Room> findPublicRoomsWithStatus(boolean includeInactive, Pageable pageable) {
BooleanExpression whereClause = room.isPrivate.eq(false);
BooleanExpression whereClause = room.isPrivate.eq(false)
.and(room.status.ne(RoomStatus.TERMINATED)); // TERMINATED 제외

// 닫힌 방 제외 옵션
// 일시정지 방 제외 옵션
if (!includeInactive) {
whereClause = whereClause.and(
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE)
Expand Down Expand Up @@ -349,7 +361,7 @@ public Page<Room> findPublicRoomsWithStatus(boolean includeInactive, Pageable pa
}

/**
* 내가 멤버인 비공개 방 조회
* 내가 멤버인 비공개 방 조회 (TERMINATED 제외)
* 조회 조건:
* - isPrivate = true
* - 내가 멤버로 등록된 방
Expand All @@ -359,9 +371,10 @@ public Page<Room> findPublicRoomsWithStatus(boolean includeInactive, Pageable pa
@Override
public Page<Room> findMyPrivateRooms(Long userId, boolean includeInactive, Pageable pageable) {
BooleanExpression whereClause = room.isPrivate.eq(true)
.and(roomMember.user.id.eq(userId));
.and(roomMember.user.id.eq(userId))
.and(room.status.ne(RoomStatus.TERMINATED)); // TERMINATED 제외

// 닫힌 방 제외 옵션
// 일시정지 방 제외 옵션
if (!includeInactive) {
whereClause = whereClause.and(
room.status.in(RoomStatus.WAITING, RoomStatus.ACTIVE)
Expand Down Expand Up @@ -394,7 +407,7 @@ public Page<Room> findMyPrivateRooms(Long userId, boolean includeInactive, Pagea
}

/**
* 내가 호스트(방장)인 방 조회
* 내가 호스트(방장)인 방 조회 (TERMINATED 제외)
* 조회 조건:
* - room.createdBy.id = userId
* 정렬: 열린 방 우선 → 최신순
Expand All @@ -404,7 +417,10 @@ public Page<Room> findRoomsByHostId(Long userId, Pageable pageable) {
List<Room> rooms = queryFactory
.selectFrom(room)
.leftJoin(room.createdBy, user).fetchJoin()
.where(room.createdBy.id.eq(userId))
.where(
room.createdBy.id.eq(userId),
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.orderBy(
room.status.when(RoomStatus.WAITING).then(0)
.when(RoomStatus.ACTIVE).then(0)
Expand All @@ -418,7 +434,10 @@ public Page<Room> findRoomsByHostId(Long userId, Pageable pageable) {
Long totalCount = queryFactory
.select(room.count())
.from(room)
.where(room.createdBy.id.eq(userId))
.where(
room.createdBy.id.eq(userId),
room.status.ne(RoomStatus.TERMINATED) // TERMINATED 제외
)
.fetchOne();

return new PageImpl<>(rooms, pageable, totalCount != null ? totalCount : 0);
Expand Down