[SWEP-100] 메모 폴더명 공백 입력 시 에러 처리 (#210)

[asjasj3964]

Sweepic Server PR List

close #209

⚒️develop의 최신 커밋을 pull 받았나요?

• 최신 커밋 업데이트

🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현한건지, 이슈 대응이라면 어떤 이슈인지 PR이 열리게 된 계기와 목적을 Reviewer 들이 쉽게 이해할 수 있도록 적어 주세요
일감 백로그 링크나 다이어그램, 피그마를 첨부해도 좋아요

• 메모 폴더명 공백 입력 시 에러 처리
  • FolderValidationError 에러 핸들링을 만들어 입력한 폴더명이 공백으로 되어있을 경우 에러 처리
  • trim()을 이용해 앞뒤 공백을 제외하여 검색어의 길이가 0일 경우 검색 에러 응답을 반환한다.

✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요? (핵심 작업 내용)

문제를 해결하면서 주요하게 변경된 사항들을 적어 주세요

• 에러 핸들링

  // src/error.ts
  
  export class FolderValidationError extends BaseError {
    constructor(details: {folderName: string}) {
      super(400, 'FOL-400', '폴더명을 1자 이상 입력해야 합니다.', details);
    }
  }
  

• controller에서 메모 폴더 생성 또는 메모 폴더 이름 수정 시 프론트로부터 받아온 폴더명이 공백('' 또는 띄어쓰기)으로만 처리되어 있으면 에러 처리한다.

  // src/controllers/memo-folder.controller.ts & src/controllers/memo-image.controller.ts
  
  if (body.folderName === null || body.folderName.trim().length === 0) {
    throw new FolderValidationError({
      folderName: body.folderName,
    });
  }
  

• 유효하지 않은 폴더명 에러 예시 추가

  @Response<ITsoaErrorResponse>(
    StatusCodes.BAD_REQUEST,
    '유효하지 않은 데이터 에러',
    {
      resultType: 'FAIL',
      success: null,
      error: {
        errorCode: 'FOL-400',
        reason: '폴더명을 1자 이상 입력해야 합니다.',
        data: {folderName: ''},
      },
    },
  )
  

🤚 동작 확인

기능을 실행했을 때 정상 동작하는지 여부를 확인하고 스크린 샷을 올려주세요

• 폴더 생성 API & 폴더명 수정 API 에러 예시
  
• 폴더 생성 API
  
  
• 폴더명 수정 API
  
  

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

없으면 "없음" 이라고 기재해 주세요

• 없음

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

개발 과정에서 다른 분들의 의견은 어떠한지 궁금했거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요

🩺 이 PR에서 테스트 혹은 검증이 필요한 부분이 있을까요?

테스트가 필요한 항목이나 테스트 코드가 추가되었다면 함께 적어주세요

• 없음

📌 PR 진행 시 이러한 점들을 참고해 주세요

• Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
• Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
• Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 2일 이내에 진행해 주세요.
• Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
  • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
  • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
  • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

---

📝 Assignee를 위한 CheckList

• To-Do Item

[jjiinaaa]

폴더명을 입력하지 않고 0자일 때, 에러처리를 해야하나요??

[asjasj3964]

폴더명을 입력하지 않고 0자일 때, 에러처리를 해야하나요??

pm님이 폴더명을 공백(아무것도 안 썼거나 띄어쓰기만 있을 때)으로 지정하는 걸 허용하지 않는 걸로 하셨다고 해서 그렇게 했습니다

[jjiinaaa]

폴더명을 입력하지 않고 0자일 때, 에러처리를 해야하나요??

pm님이 폴더명을 공백(아무것도 안 썼거나 띄어쓰기만 있을 때)으로 지정하는 걸 허용하지 않는 걸로 하셨다고 해서 그렇게 했습니다

개인적인 생각으로 폴더명을 공백으로 지정하는 걸 허용하지 않는거는, 프론트에서 애초에 입력할 때 빈 칸이면 api를 불러오지 못하게 하는 게 더 나을 거 같아서요.

[asjasj3964]

폴더명을 입력하지 않고 0자일 때, 에러처리를 해야하나요??

pm님이 폴더명을 공백(아무것도 안 썼거나 띄어쓰기만 있을 때)으로 지정하는 걸 허용하지 않는 걸로 하셨다고 해서 그렇게 했습니다

개인적인 생각으로 폴더명을 공백으로 지정하는 걸 허용하지 않는거는, 프론트에서 애초에 입력할 때 빈 칸이면 api를 불러오지 못하게 하는 게 더 나을 거 같아서요.

그렇군요 그렇게 하는 것이 성능상에 더 좋으면 그 편이 좋을 것 같습니다. 해당 기능을 담당하시는 프론트 분께 말씀드리겠습니다