우리정보 공유해요~

[오류내용]

Define a constant instead of duplicating this literal "sndRqRspCd" 7 times.

String literals should not be duplicated.

[해결방안]

문자열이 중복되어 사용이 되는게 문제이며 해결방안으로는 멤버변수를 두고 재사용해야된다.

아래 샘플 소스코드는 해결방안으로만 참고하길 바란다.

package deploy;

public class Deploy3Controller {

    private static final String RETURN_ERR_CODE = "returnErrCd";    // 멤버변수
    private static final String RETURN_MESSAGE  = "returnMessage";  // 멤버변수
    
    @PostMapping ~~~~~~~~~
    public @ResponseBody Object sendMsg(@RequestBody Map<String, Object> paramMap) {
    
        Map<String, Object> retMap = new HashMap<>();
        
        if(StringUtil.isEmpty(paramMap.get("telNo"))) {
            retMap.put(RETURN_ERR_CODE, "-1");
            retMap.put(RETURN_MESSAGE, "전화번호가 없습니다.");	
        } else if(StringUtil.isEmpty(paramMap.get("addr"))) {
            retMap.put(RETURN_ERR_CODE, "-1");
            retMap.put(RETURN_MESSAGE, "주소가 없습니다.");
        } else {
            retMap.put(RETURN_ERR_CODE, "0000");
            retMap.put(RETURN_MESSAGE, "정상 처리되었습니다.");        
        }
        
        return retMap;
    }
    
}

소나큐브에서 개발 언어별 Rules 를 공유한게 있는거 같은데 아래 사이트를 참고하자.

https://rules.sonarsource.com/java/RSPEC-6437

[오류내용]

Refactor this method to reduce its Cognitive Complexity from 37 to the 15 allowed.

Cognitive Complexity of methods should not be too high.

[해결방안]

오류내용을 해석하면

너무 많은 if 문 사용으로 로직의 복잡성을 높이는데 한계와 유지의 어려움이 있어 수정을 하라는 말씀.

if 문의 사용빈도와 switch 문을 사용하여 로직을 변경해주면 되겠다.

[오류내용]

Why is this an issue? 클릭하면 해결 답안을 직접적으로 볼 수 있다. 아래 참고..

[수정전 코드]

@RequestMapping(path = "/greeting", method = RequestMethod.GET) // Noncompliant

[수정후 코드]

@GetMapping(path = "/greeting") // Compliant