inflearn logo
강의

講義

知識共有

キム・ヨンハンの実戦Java - 高級編1、マルチスレッドと並行性

出金 例 - 開始

중복 코드와 유지보수성 사이에서 어떤 기준으로 리팩토링해야 할지 궁금합니다

3

hsg9911199884

投稿した質問数 5

0

안녕하세요.
백엔드/서버 개발자로 이직을 준비하고 있는 1년차 SI 개발자입니다.

강의를 들으면서 늘 느끼는 점이 있습니다. 강의에서 보여주시는 코드는 정말 간결하고 효율적이고, 특히 “한 사람이 조금 더 고민해서 코드를 작성하면 다음 사람이 유지보수하기 편하다”는 말씀에 많이 공감하고 있습니다.

저도 실무에서 신규 기능을 추가하거나 기존 기능을 수정할 때 다른 사람이 작성한 코드를 보게 되는데, 그때마다 유지보수하기 쉬운 코드의 중요성을 많이 느끼고 있습니다.

그런데 최근 작업을 하면서 한 가지 의문이 생겼습니다.

어떤 코드에서 분기 조건만 다르고, 마지막에 호출하는 메서드 형식은 완전히 같은 상황이 있었습니다. 예를 들면 if문 안에서 조건에 따라 넘겨주는 값만 다르고, 최종적으로 호출하는 메서드는 동일한 구조였습니다.

처음에는 각 분기 안에서 바로 메서드를 호출하면 코드 흐름이 단순해 보였습니다.

if (조건1)
{
    Method(a, b, c);
}
else if (조건2)
{
    Method(d, e, f);
}

하지만 이렇게 작성하면 호출 형식이 계속 반복되고, 나중에 파라미터가 추가되거나 호출 방식이 바뀌면 여러 군데를 수정해야 해서 유지보수성이 떨어질 것 같았습니다.

그래서 조건에 따라 필요한 값만 먼저 변수에 세팅하고, 마지막에 메서드는 한 번만 호출하는 방식이 더 낫다고 생각했습니다.

if (조건1)
{
    value1 = a;
    value2 = b;
    value3 = c;
}
else if (조건2)
{
    value1 = d;
    value2 = e;
    value3 = f;
}

Method(value1, value2, value3);

이 방식은 호출부 중복은 줄어들지만, 반대로 코드 길이는 늘어나고 중간 변수가 많아집니다.

또 단순한 if문 하나로 처리할 수 있는 로직을 굳이 변수 세팅 방식으로 바꾸는 게 항상 좋은 선택인지 고민이 됩니다.

결국 궁금한 점은 이것입니다.

분기마다 값만 다르고 호출 형식이 같은 경우, 실무에서는 어떤 기준으로 코드를 작성하는 것이 더 좋은가요?

단순히 코드 줄 수가 짧은 것이 좋은 코드인지, 아니면 조금 길어지더라도 중복 호출을 줄이고 변경 지점을 한 곳으로 모으는 것이 좋은지 아직 실무 경험이 부족해서 판단이 어렵습니다.

이런 상황에서 실무적으로 어떤 기준으로 리팩토링 여부를 판단하면 좋을지 궁금합니다.


java 객체지향 동시성 multithread thread

回答 1

0

communityai8509

안녕하세요, 인프런 AI 인턴입니다. 탕면이님.

코드 리팩토링 결정에는 여러 측면을 고려해야 하며, 특히 유지보수성과 가독성이 중요한 요소가 됩니다. 아래는 이런 부분에 대한 좋은 기준을 제공하는 참고 게시물입니다.

  1. 코드의 중복 줄이기와 유지보수성에 대한 일반적인 가이드라인은 다음 링크에서 확인하실 수 있습니다: 동일한 코드의 반복을 줄이는 방법에 대한 논의. 이 질문에서는 중복되는 코드의 문제점과 그에 대한 해결책에 대한 경험담과 조언이 잘 설명되어 있습니다.

  2. 함수의 호출방식 변화가 유지보수성에 미치는 영향에서 주의할 점은, 변경의 라이프사이클에 맞춰 코드를 리팩토링하는 것입니다. 예를 들어 이 게시물에서는 리팩토링 시 성능과 유지보수성 간의 트레이드오프를 어떻게 고려해야 하는지를 다루고 있습니다.

실제로 메서드를 여러 곳에서 반복하여 호출하는 경우, 추후 변경이나 파라미터 추가시에 모든 호출부를 수정해야 하는 추가 작업을 막기 위해 중복 호출을 피하는 것이 유리할 수 있습니다. 중복을 줄이면, 한 군데에서만 수정해도 되기 때문에 유지보수 하기도 더 편리해집니다.

한편, 코드가 명료성을 잃지 않도록 관리하는 것도 중요합니다. 코드가 길어진다고 해서 무조건 나쁜 것은 아니며, 가독성을 해치지 않는 범위에서 유지보수성을 높이는 방향으로 리팩토링하는 것이 좋습니다.

최종적으로는, 작은 코드 변경이 다른 부분에 미치는 영향을 최소화하는 방향으로 설계하는 것이 바람직합니다. 이 점을 고려하여 불가피한 중복은 줄이고, 명확한 변수명 사용과 주석으로 코드의 이해도를 높이는 것도 좋은 방법입니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

모니터와 synchronized, ReentrantLock, 원자적연산 CAS관련해서 추가적으로 더 깊게 공부했는데 제가 이해한 것이 맞나요??

0

122

1

Future cancel기능을 사용했지만 interrupt가 발생하지 않을 때 어떻게 처리해야하나요?

0

85

2

2.스레드 생성과 실행 - 17P 그림 문의

0

93

2

자바 동시성 현업에서 사용예시가 궁금합니다

0

165

1

ReentrantLock 동작 원리에 대한 질문입니다.

0

99

1

synchronized 대신 join 을 사용하여 순서를 보장할 경우 해법이 아닌가요?

0

113

2

메모리 가시성 문제 질문

0

86

1

Future4-이유 질문

0

75

1

오타 제보

0

83

2

runnable 질문

0

107

2

자바가 아닌 다른 언어에서도

0

129

1

메모리 가시성 문제가 발생하는 이유

0

144

1

데몬스레드가 어떨 때 사용이 되는지 궁금합니다.

0

91

1

Future.cancel() 취소를 사용하는 예가 궁금합니다.

0

100

1

synchronized 와 인스턴스의 lock

0

108

2

synchronized 키워드만으로 메모리 가시성이 해결되는 이유가 궁급합니다.

0

78

1

[스레드 풀과 Executor 프레임워크1][문제와 풀이] `log("일부 작업이 실패했습니다.");` 실행되는 경우가 있을까요?

0

71

1

원자적 연산이 멀티스레드 상황에서 안전한 이유를 제대로 이해했는지 확인 부탁드립니다

0

82

2

자바 고급 1,2,3편 같이 진행해도 문제없나요

0

98

2

CPU 캐시 때문에도 잔액이 200원이 될 수 있나요?

0

119

2

synchronized 메서드와 join 관련 질문이 있습니다.

0

78

2

섹션 14. 115번 강의 내용 질문

0

90

2

교안에 있는 7페이지 데이터 영역

0

58

1

Executor 스레드 풀 관리 질문

0

119

2