fix(be): remove redundant problem controller handler (#1198)

* fix(be): remove redundant problem controller handler * fix(be): remove unused function * fix: include problem tags in problem list * docs: remove useless endpoint --------- Co-authored-by: Jaemin Choi <[email protected]>
skkuding · Jan 17, 2024 · d4854fc · d4854fc
1 parent d26897f
commit d4854fc
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 54 deletions.
diff --git a/backend/apps/client/src/problem/dto/problems.response.dto.ts b/backend/apps/client/src/problem/dto/problems.response.dto.ts
@@ -8,5 +8,5 @@ export class ProblemsResponseDto {
   @Expose() difficulty: Level
   @Expose() submissionCount: number
   @Expose() acceptedRate: number
-  @Expose() tags: Partial<Tag>
+  @Expose() tags: Partial<Tag>[]
 }
diff --git a/backend/apps/client/src/problem/problem.controller.ts b/backend/apps/client/src/problem/problem.controller.ts
@@ -22,15 +22,6 @@ export class ProblemController {
   private readonly logger = new Logger(ProblemController.name)
 
   constructor(private readonly problemService: ProblemService) {}
-  @Get()
-  async searchProblemTitle(@Query('search') search: string) {
-    try {
-      return await this.problemService.searchProblemTitle(search)
-    } catch (error) {
-      this.logger.error(error)
-      throw new InternalServerErrorException()
-    }
-  }
 
   @Get()
   async getProblems(

diff --git a/backend/apps/client/src/problem/problem.repository.ts b/backend/apps/client/src/problem/problem.repository.ts
@@ -91,6 +91,12 @@ export class ProblemRepository {
       where: {
         groupId,
         title: {
+          // TODO/FIXME: postgreSQL의 full text search를 사용하여 검색하려 했으나
+          // 그럴 경우 띄어쓰기를 기준으로 나눠진 단어 단위로만 검색이 가능하다
+          // ex) "hello world"를 검색하면 "hello"와 "world"로 검색이 된다.
+          // 글자 단위로 검색하기 위해서, 성능을 희생하더라도 contains를 사용하여 구현했다.
+          // 추후에 검색 성능을 개선할 수 있는 방법을 찾아보자
+          // 아니면 텍스트가 많은 field에서는 full-text search를 사용하고, 텍스트가 적은 field에서는 contains를 사용하는 방법도 고려해보자.
           contains: search
         }
       },
@@ -105,23 +111,6 @@ export class ProblemRepository {
     })
   }
 
-  // TODO/FIXME: postgreSQL의 full text search를 사용하여 검색하려 했으나
-  // 그럴 경우 띄어쓰기를 기준으로 나눠진 단어 단위로만 검색이 가능하다
-  // ex) "hello world"를 검색하면 "hello"와 "world"로 검색이 된다.
-  // 글자 단위로 검색하기 위해서, 성능을 희생하더라도 contains를 사용하여 구현했다.
-  // 추후에 검색 성능을 개선할 수 있는 방법을 찾아보자
-  // 아니면 텍스트가 많은 field에서는 full-text search를 사용하고, 텍스트가 적은 field에서는 contains를 사용하는 방법도 고려해보자.
-  async searchProblemTitle(search: string): Promise<Partial<Problem>[]> {
-    return await this.prisma.problem.findMany({
-      where: {
-        title: {
-          contains: search
-        }
-      },
-      select: this.problemsSelectOption
-    })
-  }
-
   async getProblemTags(problemId: number): Promise<Partial<Tag>[]> {
     return (
       await this.prisma.problemTag.findMany({

diff --git a/backend/apps/client/src/problem/problem.service.ts b/backend/apps/client/src/problem/problem.service.ts
@@ -53,10 +53,6 @@ export class ProblemService {
 
     return plainToInstance(ProblemsResponseDto, await Promise.all(problems))
   }
-  async searchProblemTitle(search: string) {
-    const data = await this.problemRepository.searchProblemTitle(search)
-    return plainToInstance(ProblemsResponseDto, data)
-  }
 
   async getProblem(problemId: number, groupId = OPEN_SPACE_ID) {
     const data = await this.problemRepository.getProblem(problemId, groupId)

diff --git a/collection/client/Problem/Search Problem Title/Succeed.bru b/collection/client/Problem/Search Problem Title/Succeed.bru