[TIL] 기술교육 - 코드 리뷰

작성일 | In |

코드 리뷰 : 정명주 책임님

  1. e.printStackTrace()
    • 처리를 해주지 않는다.
    • io가 동기로 처리. 너무 많이 뜬다. tps too high
    • log.info()를 쓴다.
  2. System.out, System.err
    • io를 동기로 처리.

로깅

  • logger를 이용
    • System.out.println(), e.printStackTrace() 금지
    • 적당한 로그 레벨
  • 좋은 로깅
1
2
3
4
5
try {

} catch (MailParseException e) {
    log.error("[ERROR] 메일 파싱에 오류가 발생하였습니다. {}", mail.getId(), e);
}
  1. 넓은 스코프
1
2
3
4
public class Coment {
    int id;
    int item;
}
  • 스코프는 최대한 좁게 -> private