eblo

Remove this "BigDecimal" constructor수정 전 public void serialize(Double value, JsonGenerator jgen, SerializerProvider provider) throws IOException { if (null == value) { jgen.writeNull(); } else { jgen.writeNumber(new BigDecimal(value).setScale(2, BigDecimal.ROUND_DOWN)); } }수정 후 public void serialize(Double value, JsonGenerator jgen, SerializerProvider provider) throws IOException { if (null == v..

Use "java.nio.Files#delete" here for better messages on error conditions.SonarQube를 이용하니 새로운 기술도 알려 주네요. ^^수정 전public static boolean deleteFile(String path, String name){ File file = new File(path + name);if(file.exists()){return file.delete();}return false;}수정 후 public static boolean deleteFile(String path, String name) { Path filePath = FileSystems.getDefault().getPath(path, name); try { Files..

"Add a private constructor to hide the implicit public one."유틸 클래스에서는 public 생성자가 없어야 합니다. 유틸 클래스 만들 때는 private 생성자를 추가해 주세요. public class ValidatorUtil { private ValidatorUtil() { throw new IllegalStateException("Utility class"); }...} Utility classes should not have public constructors (squid:S1118) Code smell MajorUtility classes, which are collections of static members, are not meant to be i..

Combine this catch with the one at line 82, which has the same body. 동일 처리를 하는 catch문 합치기 수정 전public boolean checkUser(UserParam userParam) {try {if(getUserInfo(userParam) != null) return true;}catch(DataNotFoundException e) {return false;}catch(UserNotFoundException e) {return false;}return false;}수정 후public boolean checkUser(UserParam userParam) {try {if(getUserInfo(userParam) != null) return ..

"Make this method "public" or remove the "@Transactional" annotation"@Transactional(propagation=Propagation.REQUIRED, readOnly=false, rollbackFor=Exception.class)

Iterate over the "entrySet" instead of the "keySet". Map에 있는 키 값을 꺼내서 루프를 돌리는데 만약 key값 이외 value도 필요하다며 keySet을 사용하지 말고 entrySet을 사용하라. "entrySet()" should be iterated when both the key and value are needed (squid:S2864) Code smell Major When only the keys from a map are needed in a loop, iterating the keySet makes sense. But when both the key and the value are needed, it's more efficient to iter..

Object를 일반적인 방법으로 복제를 하는 경우 문제가 발생할 수 있습니다. 아래 예제를 통해 확인해 보겠습니다. 테스트 코드 public class TestClone { public static void main(String[] args) { User user1 = new User("test1"); User cUser1 = (User)user1.clone(); cUser1.setName("test2"); System.out.println(user1.toString()); System.out.println(cUser1.toString()); } public static class User implements Cloneable, Serializable{ private String name; private..

검출 메시지 The type of the "map" object should be an interface such as "Map" rather than the implementation "HashMap".List, Set, Map 사용 시 자주 실수하는 내용 중 하나입니다. 심지어 List가 인터페이스인 줄도 모르는 개발자가 많습니다. 왜 인터페이스를 사용해야 하는지를 모르면서 습관처럼 만들고 사용합니다. 왜 그러해야하는가에 대한 것을 공부하고 좋은 습관을 만드는게 중요한거 같습니다. 잘못 사용 되는 예ArrayList list = new ArrayList();HashMap map = new HashMap();HashSet set = new HashSet(); public void test(HashMap..

리팩토링 검출 메시지 "Reorder the modifiers to comply with the Java Language Specification."Java class, method 선언 시 표현 순서 1. Annotations 2. 접근자 (public/protected/private) 3. abstract 4. static 5. final 6. transient 7. volatile 8. default 9. synchronized10. native11. strictfp 잘못된 예 : public synchronized static String replace(...} 변경 : public static synchronized String replace(...}참고 및 발췌8.1. Class Declara..

기존 코드 if(구글로그인){ // 로그인 처리}else if(인스타로그인){ // 로그인 처리}else if(페이스북로그인){ // 로그인 처리}else{ // 로그인 처리}문제점 새로운 추가가 발생할 때 마다 소스를 추가/변경해 주어야 한다.코드의 양이 커진다. 복잡도 증가한다. 가독성이 떨어진다. 부분 테스트 불가능하다. 리팩토링인터페이스를 생성하고 각각의 로그인 형태에 따라 클래스로 분리한다.flyweight pattern과 유사한 형태로 만든다. 각각의 클래스를 자료 구조에 담아 두고 꺼내 이용한다. 설계상에는 Factory를 별도로 만들지 않았습니다. Contorller123456789101112131415161718192021222324252627282930@RestController@Slf..