240618 : TIL

과제에 대한 피드백

 

- UserContollerTest는 모두 실패입니다. 세팅이 정상적으로 안 된 것으로 보이네요. 이부분은 MockMvc 적용하는 부분을 다시 한번 찾아보시기 바랍니다.

- 테스트 클래스에 public은 필요 없습니다. 인텔리J에 Sonarlint로 알려주는 내용은 도움이 되는 내용이라 노란 밑줄이 그어진 부분은 한번씩 확인해보는 습관을 들이시면 좋습니다.

- 변수 선언 후에 바로 리턴하는 부분이 있다면 이건 변수 선언하지 않고 바로 리턴하는게 좋습니다. AOP 구현 내부를 보면 아래와 같은 부분이 있습니다.

- String url 할 필요 없이 바로 return 하시면 됩니다.

String url = Stream.of( GetMapping.class, PutMapping.class, PostMapping.class, PatchMapping.class, DeleteMapping.class, RequestMapping.class) .filter(mappingClass -> method.isAnnotationPresent(mappingClass)) .map(mappingClass -> getUrl(method, mappingClass, baseUrl)) .findFirst().orElse(null); return url;