Skip to content
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.

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

서버 에러 인스턴스 모듈화 #207

Open
skid901 opened this issue Nov 10, 2020 · 1 comment
Open

서버 에러 인스턴스 모듈화 #207

skid901 opened this issue Nov 10, 2020 · 1 comment

Comments

@skid901
Copy link
Collaborator

skid901 commented Nov 10, 2020

현재 코드에서는 에러를 발생시킬 때, 로직 내에서 new Error(/* error message */) 를 작성하는데,
이를 하단의 예시과 같이 별도의 모듈로 만들어 사용하면,
에러 객체의 단일 인스턴스(싱글톤)을 재사용할 수 있어 메모리 효율성을 높일 수 있습니다.
또한, 테스트 코드를 작성할 경우, 동일한 에러 인스턴스를 비교함으로써, 생산성을 높일 수 있습니다.

// errorHandler.js
const error = {
  NotFoundError: new Error('Not found'),
}

module.exports = {
  ...error,
  errorHandler,
};

// service.js
throw NotFoundError;
@skid901 skid901 changed the title 에러 인스턴스 모듈화 서버 에러 인스턴스 모듈화 Nov 10, 2020
@seokju2ng
Copy link
Collaborator

errorStatus도 이런식으로 바꾸어주면 동작하겠네요!

// errorHandler.js
const errorStatus = {
  [error.NotFoundError.message]: NOT_FOUND,
};

@seokju2ng seokju2ng added this to the week3-Web milestone Nov 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants