레거시 코드 리팩터링하다가 멘탈 나감
변수명 a, b, aa, aaa, aaaa... 주석은 딱 하나 있음. '// 여기 수정하지 마세요'
차라리 새로 짜는 게 빠를 것 같은데 팀장님은 건드리지만 말래ㅋㅋ 하...
9 개의 답변
ㅋㅋㅋㅋㅋㅋ aaaa 보고 빵터졌네
아 그 '여기 수정하지 마세요' 주석이 제일 무섭지. 십중팔구 그거 건드리면 프로덕션 터짐
팀장님 말이 틀린 건 아닌 게, 레거시 리팩터링은 테스트 커버리지 없으면 그냥 도박임. 새로 짜는 게 빠르다고 느껴지는 순간이 제일 위험한 순간이야. 최소한 그 aaaa가 하는 일을 문서로 뽑아내는 것부터 시작해봐.
출처 있음?
글쎄 그건 좀. 건드리지 마라는 게 리팩터링 하지 말라는 뜻은 아닐 텐데 그냥 안정권 유지하자는 소리 아님? 팀장 말도 좀 들어보고 판단해라.
저도 그래요... 제 코드엔 변수명이 data, data2, data_final, data_final_real 이렇게 있습니다. 웃프다
동의 안 됨. 새로 짜는 게 빠르다는 건 착각인 경우가 태반임. 요구사항 다 파악했다고 생각하는 그 시점에 이미 절반은 놓치고 있는 거거든. 리팩터링이 지루해서 새로 짜고 싶은 거면 그건 그냥 도피임.
그 주석 쓴 사람 지금 회사에 있음? 있으면 밥 사주면서 왜 건드리면 안 되는지 물어봐라. 그게 제일 빠름 ㅇㅈ
실용적인 팁 하나: a, b, aa를 한 번에 다 바꾸려 하지 말고 IDE 리네임으로 한 변수씩, 커밋 하나씩 쪼개서 올려. 리뷰어도 안 무서워하고 팀장님도 '건드리지 마' 소리 덜 함. 주석은 지우지 말고 '// 왜: ' 형태로 이유라도 남겨둬.