🛠️ refactor: (BE) - create room 메서드에서 중복 아이디 체크

Author: anzmin

backend/socket/src/room/room.gateway.ts

@@ -42,8 +42,8 @@ export class RoomGateway implements OnGatewayConnection, OnGatewayDisconnect {
 	// connection
 
 	@SubscribeMessage(EVENT.CREATE_ROOM)
-	handleCreateRoom() {
-		return this.connectionService.createRoom();
+	handleCreateRoom(@ConnectedSocket() client: Socket) {
+		return this.connectionService.createRoom(client);
 	}
 
 	@SubscribeMessage(EVENT.ENTER_ROOM)

backend/socket/src/room/service/connection/connection.service.ts

@@ -26,7 +26,12 @@ export class ConnectionService {
 	 * uuid를 기반으로 방을 생성하고 저장하는 메서드입니다.
 	 * @returns uuid - 방의 uuid
 	 */
-	async createRoom() {
+	async createRoom(client: Socket) {
+		const exception = await this.isEnterableClient(client.data.authId);
+		if (exception) {
+			return exception;
+		}
+
 		const defaultRoom = this.createDefaultRoom();
 		const room = await this.roomRepository.createRoom({
 			roomUUID: defaultRoom.roomUUID,
@@ -45,7 +50,7 @@ export class ConnectionService {
 	async enterRoom({ client, roomUUID }: { client: Socket; roomUUID: string }) {
 		const room = await this.roomRepository.getRoom(roomUUID);
 
-		const exception = await this.isEnterable({ client, room });
+		const exception = await this.isEnterable({ authId: client.data.userId, room });
 		if (exception) return exception;
 
 		const user = await this.createDefaultUser({ client, roomUUID });
@@ -66,8 +71,8 @@ export class ConnectionService {
 	 * client가 들어갈 수 있는지 & room이 현재 들어갈 수 있는 상황인지 체크하는 메서드입니다.
 	 * @returns
 	 */
-	async isEnterable({ client, room }: { client: Socket; room: Room }) {
-		return (await this.isEnterableClient(client)) ?? (await this.isEnterableRoom(room)) ?? null;
+	async isEnterable({ authId, room }: { authId: string; room: Room }) {
+		return (await this.isEnterableClient(authId)) ?? (await this.isEnterableRoom(room)) ?? null;
 	}
 
 	/**
@@ -94,12 +99,12 @@ export class ConnectionService {
 	}
 
 	/**
-	 * 해당 socket client가 interview에 참석할 수 있는지 체크하는 메서드입니다.
-	 * @param client
+	 * 회원이 interview에 참석할 수 있는지 체크하는 메서드입니다.
+	 * @param authId
 	 * @returns
 	 */
-	async isEnterableClient(client: Socket) {
-		const prevUser = await this.roomRepository.getUserIdByAuthId(client.data.authId);
+	async isEnterableClient(authId: string) {
+		const prevUser = await this.roomRepository.getUserIdByAuthId(authId);
 		if (prevUser) {
 			return { success: false, message: SOCKET_MESSAGE.EXIST_SAME_AUTH_ID };
 		}