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

console.log -> Logger Util 이용하기 #7

Open
kgwangrae opened this issue Jan 23, 2017 · 1 comment
Open

console.log -> Logger Util 이용하기 #7

kgwangrae opened this issue Jan 23, 2017 · 1 comment
Assignees

Comments

@kgwangrae
Copy link
Contributor

Debug 목적으로 console.log를 일부러 남겨둔 경우 현재는 eslint-disable no-console을 이용하여 lint warning을 제거하고 있습니다.

다만 로그를 남길 때 마다 eslint-disable no-console를 추가해야하는 문제가 있어 코드가 깔끔하지 못해지는 문제가 있어 아래와 같은 해결 방안을 검토할 필요가 있습니다.

  • src/common/_Util.es6의 static method를 추가하는 방법 (333d5d2)
    • _Util을 통해 log method에 접근하는 것이 어색한 느낌을 줄 수 있습니다. (또한 이를 위해 매번 _Util을 import해야 합니다.)
    • _Util에 method를 추가하다보니 log는 적절하지만 error, warn의 명칭이 적절하지 않게 됩니다. 하지만 logError 등과 같이 이름을 변경하면 log method를 이용하기 번거로워집니다.
  • src/common/_Util.es6의 하단에 window.log = { log: console.log, warn: console.warn, error: console.error }를 추가하는 방법
    • global namespace에 logger object를 추가한다는 단점이 있습니다.
@namenu
Copy link
Contributor

namenu commented Jan 23, 2017

첫 번째 방법의 보완책으로, Logger 클래스를 추가하는것은 어떨까요?
_Util.es6 파일 안에 포함한다면 아래와 같은 식으로 사용할 수 있을거 같아요.

import { Logger } from '../_Util';

Logger.warn(...);

@freekering freekering assigned freekering and kgwangrae and unassigned freekering Jan 25, 2017
@DavinAhn DavinAhn self-assigned this Nov 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants