리팩토링이란?
- 새로운 소스코드를 만들어 내는 것이 아니다.
- 외부 동작은 변경하지 않고, 내부 구조를 변경하는 작업이다.
- 코드의 가독성을 좋게 한다.
- 개발자들마다 결과가 다를 수 있다.
리팩토링 주의사항
- 기능 추가 구현과 동시에 리팩토링을 하는것을 권장하지 않는다.
- 기능을 추가하기 전, 리팩토링을 먼저 하는 것을 추천
리팩토링 팁
아래 코드를 리팩토링하면서 리팩토링 팁을 알아보겠습니다.
public class Main {
public static int splitAndSum(String text) {
int result = 0;
if (text == null || text.isEmpty()) {
result = 0;
}
else {
String[] values = text.split("-");
for (String value : values) {
result += Integer.parseInt(value);
}
}
return result;
}
public static void main(String[] args) {
int ret = splitAndSum("11-22-33");
System.out.println(ret);
}
}1. 한 단계의 들여쓰기를 한다.
- 한 단계 더 들여쓰기를 할때 함수로 빼서 들여쓰기를 없앤다.
...
else {
String[] values = text.split("-");
for (String value : values) {
result += Integer.parseInt(value);
}
}
......
else {
String[] values = text.split("-");
result += getSum(values);
}
...
public static int getSum(String[] values) {
int sum = 0;
for (String value : values) {
sum += Integer.parseInt(value);
}
return sum;
}2. else를 없앤다.
- return 을 사용하여, 필요없는 else를 지운다.
...
if (text == null || text.isEmpty()) {
result = 0;
}
else {
String[] values = text.split("-");
result += getSum(values);
}
......
if (text == null || text.isEmpty()) {
return 0;
}
String[] values = text.split("-");
result += getSum(values);
...3. 하나의 역할을 하는 메소드로 만든다.
...
public static int splitAndSum(String text) {
int result = 0;
if (text == null || text.isEmpty()) {
return 0;
}
else {
String[] values = text.split("-");
result += getSum(values);
}
return result;
}
...
// 현재 getSum 메소드는 String 타입을 int로 바꿔주는 역할과 덧셈까지 수행하고 있다.
public static int getSum(String[] values) {
int sum = 0;
for (String value : values) {
sum += Integer.parseInt(value);
}
return sum;
}...
public static int splitAndSum(String text) {
int result = 0;
if (text == null || text.isEmpty()) {
return 0;
}
String[] values = text.split("-");
int[] numbers = toInts(values);
result += getSum(numbers);
return result;
}
...
// String 타입을 int로 바꿔주는 메소드
public static int[] toInts(String[] values) {
int[] numbers = new int[values.length];
for (int i = 0; i < values.length; i++) {
numbers[i] = Integer.parseInt(values[i]);
}
return numbers;
}
// int로 바꾼 숫자를 더하는 메소드
public static int getSum(int[] numbers) {
int sum = 0;
for (int number : numbers) {
sum += number;
}
return sum;
}4. 임시변수를 제거한다.
- 의미 파악에 도움이 되지 않는 변수를 제거한다.
...
int result = 0;
if (text == null || text.isEmpty()) {
return 0;
}
String[] values = text.split("-");
int[] numbers = toInts(values);
result += getSum(numbers);
return result;
......
if (text == null || text.isEmpty()) {
return 0;
}
return getSum(toInts(text.split("-")));
...5. 추상화 Level을 맞춘다.
...
public static int splitAndSum(String text) {
if (text == null || text.isEmpty()) {
return 0;
}
return getSum(toInts(text.split("-")));
}
......
public static int splitAndSum(String text) {
if (isEmpty(text)) return 0;
return getSum(toInts(text.split("-")));
}
...
public static boolean isEmpty(String text) {
if (text == null) return true;
return text.isEmpty();
}6. 최종 코드
public class Main {
public static int splitAndSum(String text) {
if (isEmpty(text)) return 0;
return getSum(toInts(text.split("-")));
}
public static int[] toInts(String[] values) {
int[] numbers = new int[values.length];
for (int i = 0; i < values.length; i++) {
numbers[i] = Integer.parseInt(values[i]);
}
return numbers;
}
public static int getSum(int[] numbers) {
int sum = 0;
for (int number : numbers) {
sum += number;
}
return sum;
}
public static boolean isEmpty(String text) {
if (text == null) return true;
return text.isEmpty();
}
public static void main(String[] args) {
int ret = splitAndSum("11-22-33");
System.out.println(ret);
}
}'Spring' 카테고리의 다른 글
| [Spring] @AuthenticationPrincipal null (0) | 2023.01.12 |
|---|---|
| 동시성 이슈 해결방법 (Synchronized, DB Lock, Redis Lock) (0) | 2022.12.20 |
| [Spring] 스프링 빈 설정 (XML, JAVA, Component Scan) (0) | 2022.11.08 |
| [Spring] Spring Framework, Spring Web MVC (0) | 2022.11.08 |
| [Spring] SecurityConfig, OAuth2UserDetailsService 순환 참조 오류 (0) | 2022.11.06 |
