- 에러발생시 원인을 찾기 힘들어지며 롤백 또한 이루어지지 않는 경우가 생길 수 있음.
2.System.out.println 사용금지! 꼭 carryBean.info 를 사용 할 것!!
- 시스템 리소스를 차지하게 되며 로그제어에 문제가 됨
3. if 또는 for 문 사용 시 비교값은 하드코딩(예: "CMN") 으로 하지않고 Constant를 사용 할 것
- 하드코딩한 구문들은 코드값이나 비교값이 바뀌었을 시 "CMN"으로 코딩 된 부분을 모두 찾아 고쳐야되며 예외적인 부분도 있을 수 있기에 오류를 범할 수 있음.
Constant에 정의된 값만 변경하면 모든 값이 바뀔 수 있는 구조가 올바름
- 하드코딩한 구문들은 코드값이나 비교값이 바뀌었을 시 "CMN"으로 코딩 된 부분을 모두 찾아 고쳐야되며 예외적인 부분도 있을 수 있기에 오류를 범할 수 있음.
Constant에 정의된 값만 변경하면 모든 값이 바뀔 수 있는 구조가 올바름
4. equal 사용 시 앞에 비교되는 것은 상수로 사용 할 것 (예: ITSMConstants.BUSINESS.INCIDENT.equals(cmReviewDto.getBusiness()) )
- 앞에 비교되는 값이 null 이라면 null 포인트 Exception이 발생하지만 상수를 앞에쓰면 Exception을 방지 할 수 있다.
- 앞에 비교되는 값이 null 이라면 null 포인트 Exception이 발생하지만 상수를 앞에쓰면 Exception을 방지 할 수 있다.
5. for문에서 for(i=0; i < aaa.size(); i++) 일 경우 aaa.size() 는 Int형으로 선언하여 사용
- 기본적인 내용이지만 모르고 있는 사람들이 있기에... for(i=0; i < aaa.size(); i++) 이렇게 사용하는 것은 반복 될때 마다 aaa.size()를 읽어들여야 하지만
int size = aaa.size(); for(i=0; i < size; i++) 이와 같이 사용하게 되면 반복되는 작업을 줄 일 수 있다.
위의 내용들 중 1번은 크리티컬 하므로 즉시 수정 바라며 다른 내용들도 지켜주길 바람.
별거 아니라 생각되는 내용이 있을 수 있지만 잘못된 코딩 습관이기도 하며 외부프로젝트 나가서 개발 시
코드인스펙션(소스검증)에 의하여 지적 받게되어 어차피 수정하게 되어야 하는 부분이니 지금부터 습관을 들이면 좋을 것 같습니다.
- 기본적인 내용이지만 모르고 있는 사람들이 있기에... for(i=0; i < aaa.size(); i++) 이렇게 사용하는 것은 반복 될때 마다 aaa.size()를 읽어들여야 하지만
int size = aaa.size(); for(i=0; i < size; i++) 이와 같이 사용하게 되면 반복되는 작업을 줄 일 수 있다.
위의 내용들 중 1번은 크리티컬 하므로 즉시 수정 바라며 다른 내용들도 지켜주길 바람.
별거 아니라 생각되는 내용이 있을 수 있지만 잘못된 코딩 습관이기도 하며 외부프로젝트 나가서 개발 시
코드인스펙션(소스검증)에 의하여 지적 받게되어 어차피 수정하게 되어야 하는 부분이니 지금부터 습관을 들이면 좋을 것 같습니다.
'IT노트 > JAVA' 카테고리의 다른 글
equalsIn(String value, String... compares) (0) | 2015.03.04 |
---|---|
개발자 보수교육4일차 (0) | 2015.02.24 |
자바 공부를 위한 즐겨 찾기 (0) | 2015.02.19 |
개발자 보수교육1일차(5,6장) (0) | 2015.02.19 |
[POI] JAVA로 엑셀을 다뤄보자 : 네이버 카페 (0) | 2015.02.18 |