레거시 코드 리팩터링하다가 멘탈 나감

변수명 a, b, aa, aaa, aaaa... 주석은 딱 하나 있음. '// 여기 수정하지 마세요'

차라리 새로 짜는 게 빠를 것 같은데 팀장님은 건드리지만 말래ㅋㅋ 하...

작성자 무한도전러890

9 개의 답변

ㅋㅋㅋㅋㅋㅋ aaaa 보고 빵터졌네

작성자 코딩하는곰492 · ▲0

아 그 '여기 수정하지 마세요' 주석이 제일 무섭지. 십중팔구 그거 건드리면 프로덕션 터짐

작성자 디지털노마드142 · ▲0

팀장님 말이 틀린 건 아닌 게, 레거시 리팩터링은 테스트 커버리지 없으면 그냥 도박임. 새로 짜는 게 빠르다고 느껴지는 순간이 제일 위험한 순간이야. 최소한 그 aaaa가 하는 일을 문서로 뽑아내는 것부터 시작해봐.

작성자 주말개발자624 · ▲0

출처 있음?

작성자 밤샘코더800 · ▲0

글쎄 그건 좀. 건드리지 마라는 게 리팩터링 하지 말라는 뜻은 아닐 텐데 그냥 안정권 유지하자는 소리 아님? 팀장 말도 좀 들어보고 판단해라.

작성자 밤샘코더840 · ▲0

저도 그래요... 제 코드엔 변수명이 data, data2, data_final, data_final_real 이렇게 있습니다. 웃프다

작성자 월급루팡139 · ▲0

동의 안 됨. 새로 짜는 게 빠르다는 건 착각인 경우가 태반임. 요구사항 다 파악했다고 생각하는 그 시점에 이미 절반은 놓치고 있는 거거든. 리팩터링이 지루해서 새로 짜고 싶은 거면 그건 그냥 도피임.

작성자 밤샘코더373 · ▲0

그 주석 쓴 사람 지금 회사에 있음? 있으면 밥 사주면서 왜 건드리면 안 되는지 물어봐라. 그게 제일 빠름 ㅇㅈ

작성자 데이터덕후672 · ▲0

실용적인 팁 하나: a, b, aa를 한 번에 다 바꾸려 하지 말고 IDE 리네임으로 한 변수씩, 커밋 하나씩 쪼개서 올려. 리뷰어도 안 무서워하고 팀장님도 '건드리지 마' 소리 덜 함. 주석은 지우지 말고 '// 왜: ' 형태로 이유라도 남겨둬.

작성자 클라우드러버352 · ▲0