E-ROOM

Refactoring with SonarLint - Repository

SonarLint라는 툴이 권장하는 방향으로 코드 리팩토링을 하면서 리팩토링 규칙들을 기록하는 글입니다! 중복되는 규칙들은 한 번만 작성하였습니다. 💡 BuildingCustomRepositoryImpl 반복되는 문자열 리터럴을 상수로 정의하라 private static final 키워드로 상수를 정의하여 코드에 반영했습니다. final vs static final final만 사용하면 해당 필드는 변경이 불가능하지만 클래스의 각 인스턴스마다 그 필드의 복사본이 생성됩니다. 이는 필요 이상으로 메모리를 사용하게 되므로, 특히 그 필드의 값이 모든 인스턴스에서 동일할 경우, 비효율적일 수 있습니다. static final을 사용하면 해당 필드는 클래스 레벨에서 한번만 초기화되며 모든 인스턴스에서 공유하게 ..

Refactoring with SonarLint - Service

SonarLint라는 툴이 권장하는 방향으로 코드 리팩토링을 하면서 리팩토링 규칙들을 기록하는 글입니다! 중복되는 규칙들은 한 번만 작성하였습니다. 💡 BuildingServiceImpl 사용하지 않는 메소드 파라미터는 삭제하라 해당 메소드 자체를 사용하지 않아서 삭제했습니다. // Noncompliant Code public Building updateBuilding(BuildingOptionalDto buildingOptionalDto) { return null; } 💡 FileProcessServiceImpl replaceAll 함수를 호출하는 대신 replace 함수를 호출하라 replace 함수로 대체했습니다. 설명 String::replaceAll 메소드는 호출될 때마다 java.util.re..

Refactoring with SonarLint - Controller

SonarLint라는 툴이 권장하는 방향으로 코드 리팩토링을 하면서 리팩토링 규칙들을 기록하는 글입니다! 중복되는 규칙들은 한 번만 작성하였습니다. 💡 BuildingRestController 하나의 파라미터만 가지는 람다이면 파라미터의 괄호를 없애라 // Noncompliant Code (building) -> BuildingSerializer.toBuildingListResponse(building) // Compliant Solution building -> BuildingSerializer.toBuildingListResponse(building) 람다를 메소드 참조로 바꿔라 // Noncompliant Code building -> BuildingSerializer.toBuildingListRe..