[IT-정보] Go 변수 안티패턴과 리팩터링 전략 – 유지보수하기 쉬운 깨끗한 코드를 만드는 실전 가이드

작은 변수가 부르는 거대한 재앙, 왜 지금 변수와 상수를 점검해야 할까요

협업 중인 프로젝트의 코드를 열었는데, 도대체 이 숫자가 무엇을 의미하는지 몰라 한참을 헤맨 적이 있나요? 혹은 전역 변수 하나를 수정했더니 전혀 상관없어 보이던 다른 모듈에서 갑자기 에러가 터져 밤을 지새운 경험이 있으신가요? 처음에는 아주 작은 실수처럼 보이지만, 이러한 Go 변수 안티패턴은 시간이 흐를수록 눈덩이처럼 불어나 프로젝트 전체의 유지보수 비용을 폭등시켜요.

Go 언어는 단순함과 명확성을 지향하는 언어예요. 하지만 입문 단계에서는 변수의 범위를 잘못 설정하거나, 상수로 관리해야 할 값을 변수로 방치하는 실수를 자주 저지르곤 해요. 이런 사소한 습관들이 모여 코드의 예측 불가능성을 높이고, 결국에는 기술 부채로 돌아오게 돼요. 지금 당장은 코드가 돌아가니까 괜찮다고 생각할 수도 있지만, 서비스의 규모가 커지는 순간 이 문제는 반드시 발목을 잡게 돼요.

이 글에서는 단순히 문법적인 정의를 넘어, 실제 프로덕션 환경에서 개발자를 괴롭히는 나쁜 습관들을 하나씩 파헤쳐 보려고 해요. 코드를 더 안전하고 읽기 쉽게 만드는 리팩터링 전략을 통해, 여러분의 코드를 한 단계 더 높은 수준으로 끌어올려 드릴게요.

이 글을 끝까지 읽고 나면 다음과 같은 능력을 갖추게 돼요.

• 코드 곳곳에 숨겨진 위험한 변수 사용 패턴을 식별할 수 있어요.
• 상수와 변수를 구분하는 명확한 기준을 세울 수 있어요.
• 복잡한 로직을 깔끔한 코드로 바꾸는 리팩터링 기술을 익혀요.
• 동시성 프로그래밍 환경에서 안전한 변수 관리 방법을 이해해요.

리팩터링 전 필수 체크리스트: 변수와 상수의 올바른 기준 세우기

무작정 코드를 고치기 시작하면 오히려 더 큰 버그를 만들어낼 수 있어요. 리팩터링의 핵심은 기존의 동작을 유지하면서 내부 구조만 개선하는 것이기 때문이에요. 본격적인 작업에 들어가기 전에, 우리가 다루는 데이터가 어떤 성격을 가졌는지 명확히 정의하는 과정이 반드시 필요해요.

먼저, 우리가 다루는 값이 변경 가능성을 가지고 있는지 확인해야 해요. 프로그램이 실행되는 동안 값이 변해야 한다면 변수를 사용하고, 한 번 정해지면 절대 변하지 않는 값이라면 상수를 사용해야 해요. 만약 상수로 선언해야 할 값을 변수로 두면, 실수로 값이 변경되었을 때 시스템 전체에 예측할 수 없는 부작용을 일으킬 수 있어요.

다음은 어떤 상황에서 무엇을 선택해야 하는지 판단을 돕기 위한 기준표예요. 이 표를 머릿속에 넣어두면 코드를 작성할 때 훨씬 빠르게 결정할 수 있어요.

이 기준을 바탕으로 현재 작성된 코드를 훑어보세요. 만약 상수로 충분한 값이 변수로 선언되어 있다면, 그것이 바로 여러분이 가장 먼저 해결해야 할 리팩터링 대상이에요. 또한, 변수의 범위(Scope)가 너무 넓지는 않은지도 함께 체크해야 해요. 함수 내부에서만 쓰일 변수가 패키지 수준의 전역 변수로 선언되어 있다면, 그것 역시 잠재적인 폭탄이 될 수 있어요.

실전 리팩터링: 안티패턴을 제거하고 클린 코드로 나아가는 5단계

이제 본격적으로 실전 예제를 통해 나쁜 패턴을 어떻게 찾아내고, 어떻게 우아하게 고칠 수 있는지 알아볼게요. 각 단계는 실제 개발 현장에서 가장 자주 발생하는 문제들을 중심으로 구성했어요.

STEP 1. 매직 넘버(Magic Number)를 상수로 전환하기

코드 중간에 갑자기 나타나는 의미를 알 수 없는 숫자, 이를 매직 넘버라고 불러요. 예를 들어, 결제 로직 중에 if amount > 10000이라는 코드가 있다면, 이 ‘10000’이 무엇을 의미하는지 코드를 읽는 사람은 알 길이 없어요. 무료 배송 기준인지, VIP 등급 기준인지 말이에요.

이런 코드는 가독성을 떨어뜨릴 뿐만 아니라, 나중에 기준이 15,000원으로 바뀌었을 때 코드 전체를 뒤져서 숫자를 찾아 바꿔야 하는 위험을 초래해요. 이를 해결하기 위해서는 의미 있는 이름을 가진 상수를 선언해야 해요.

[나쁜 예시]
func applyDiscount(price int) int {
if price > 50000 {
return price - 5000
}
return price
}

[개선된 예시]
const (
VIPDiscountThreshold = 50000
VIPDiscountAmount = 5000
)

func applyDiscount(price int) int {
if price > VIPDiscountThreshold {
return price - VIPDiscountAmount
}
return price
}

이렇게 바꾸면 코드를 읽자마자 ‘VIP 할인 임계값’을 기준으로 할인이 적용된다는 것을 누구나 알 수 있어요. 나중에 기준이 바뀌어도 상수의 값만 수정하면 되니 매우 안전하죠.

STEP 2. 전역 변수의 오남용을 캡슐화로 해결하기

패키지 수준에서 선언된 전역 변수는 가장 위험한 안티패턴 중 하나예요. 전역 변수는 프로그램 어디에서든 접근할 수 있기 때문에, 어떤 함수가 이 값을 언제 바꿨는지 추적하기가 매우 어렵거든요. 특히 Go는 고루틴(Goroutine)을 이용한 동시성 프로그래밍이 기본인데, 여러 고루틴이 동시에 전역 변수에 접근하면 데이터 경합(Race Condition)이 발생하여 프로그램이 죽거나 엉뚱한 결과를 내놓을 수 있어요.

전역 변수를 사용해야만 한다면, 직접적인 접근을 막고 함수나 구조체를 통해서만 접근할 수 있도록 캡슐화를 적용해야 해요. 변수를 소문자로 시작하여 패키지 외부에서는 보이지 않게 만들고, 필요한 기능만 메서드로 노출하는 방식이죠.

[나쁜 예시]
var UserCount int // 어디서든 수정 가능

func Increment() {
UserCount++
}

[개선된 예시]
type UserManager struct {
mu sync.Mutex
userCount int // 패키지 내부에서만 접근 가능
}