코드가 더럽고 문제가 많을 경우 적절한 대응 방식은 무엇일까요?

Question

https://okky.kr/questions/1542961?topic=questions&page=1일단 지난 글 과 이어서 질문합니다..오늘 애플리케이션 코드를 까보니 하드코딩 된 코드가 한 가득이더군요...분명 전임자가 저한테 던질때는 "작동은 한다" 라고 해서 던지긴 했는데. (실제로 원하는 로직으로 동작은 하는 걸로 보임 클라이언트는 만족중 이며 실사중)중첩 된 if elseif else 분기처리 에 컨트롤러에 난잡하게 작성되어 있는 트라이 캐치에..if(result.size == 144)(144 가 왜 정상로직 으로 처리되는건지...?) 와 같은 알수 없는 숫자들과.. 무분별하게 사용된 UNION 과 서브 쿼리 등등 후...말 그대로 동작은 합니다만... 간혹 알수 없는 오류로 특이하게 데이터가 나오거나 하는 오류등이 있습니다. 아직 까지는 크리티컬한 이슈는 없습니다.이러한 코드와 저는 못해도 반년은 함께 해야 합니다....제 생각엔 방법이 두가지 정도 생각납니다.어떻게든 땜질 처방으로 스파게티 코드를 만들던 말던 임시처리해 다음 전임자에게 던지기...본인이 욕먼던 가 해서 리팩토링 하기.ps : 이러한 코드에서 당연하지만 테스트 코드 따위는 없습니다.

캐티 · Answer

위에 거는 db 이번 거는 어플 나는 돈과 시간이 갈린다 하더라도 다 지우고 새로 해요.

정규분포 · Answer

저도 항상 겪던 문제였습니다.계속 담당하시게 되는 코드면 차근차근 보수 가능할 정도로 리펙토링을 시작하시는게 좋아요. 한번에 다 하는 것은 불가능하고, 그런 쓰레기 코드처럼 보이는 부분들, 이해 불가능한 부분을 하나하나 조사해가면서 제거하는 것이 좋음.절대로 사전조사, 이해 없이 코드 제거/리펙토링 하지 마세요. 제가 했었던 제일 큰 실수 중 하나였습니다.그리고 반드시 실수는 하게 될것입니다. 버그도 나올 것이고요. 그런 리스크 감수를 해서라도 리펙토링 해야 미래가 보이면 하시고, 아니면 하지 마세요. 판단은 결국 담당자가 하는 것이라.

코가더 · Answer

가능하다면 재개발

Kyeongrok Kim · Answer

이렇게 해놓고 던지고 자기는 다른일 하러 가고 정말 흔한 경우 같습니다. 인수인계 받는 사람만 죽어 납니다. 그래도 요즘은 클로드 코드로 side effect분석 해달라고 하면 많이 찾아주더라구요. 저는 이런식으로 비즈니스 로직 분석하고 코드의 특정 부분 unit 테스트 만들어 놓고 리팩토링 하면서 테스트 통과하는지 보면서 고치면서 개선 해나갑니다.하다가 안되면 퇴사를 고려합니다.

프레이야 · Answer

거진 파악 다 되신것 같으니결정이 남은듯 합니다사견으로는리팩토링이 레귤러한 방식이겠지만리팩토링 정말 조은 맘으로 하시던중 현업에서 쓸데없는 일로 에러만 만든다고 명예가 더러워지는 경우도 있을것 같은니6개월 잠시 거쳐가는곳이라면 땜질도 방법중 하나일수 있겠네여

코딩하는 두루미 · Answer

반년만 버티면 된다면..저라면 버티다 던지겠습니다..

킷캣맨 · Answer

반년이면 그냥 버티다 던져야죠

월급은나의빛 · Answer

리팩토링이라는 게 여러 층위가 있는데, 조금씩 고쳐나가는 게 좋다고 생각합니다. 우선은 커플링된 모듈들을 분리한 다음에, 중간에 멈추더라도 동작에는 아무런 영향이 없도록 하면서 하나씩 고쳐나가면 재미있게 작업할 수 있지 않을까요.

퓨리오사 · Answer

예전에 이런 케이스를 보면 화가 났는데 지금은 각자의 사정이 다 있었겠지 생각이 듭니다.촉박한 일정... 뭐 하여튼 일단 리팩토링에 대해서 언급을 해보세요.대부분의 경우 부정적일 수 있습니다.리팩토링에 대한 설득 성공하여 작업 허가가 나면 그때 계획을 세워서 스텝 밟아가며 해결해 나가는게 좋을 듯합니다. 물론 대부분 허가가 안 나는게 함정이죠.

집킨 · Answer

"그때는 사정이 다 있었겠지" 가스라이팅에 당하지 마세요통제 범위를 벗어나는 기술 부채는 1차적으로 무조건 개발자 잘못입니다"어떻게든 땜질 처방으로 스파게티 코드를 만들던 말던 임시처리해 다음 전임자에게 던지기..."를 시전하면 그 코드 또한 님 책임이구요당장 새로운 코드부터 잘 짜보세요그리고 눈에 보이는 간단한 코드들도 리팩토링 해보세요그렇게라도 해야합니다

Taeyeon · Answer

테스트코드안하고 운영에 배포가 가능한가요? 신기

nippori · Answer

'책임'이 생기기 때문에 솔직히 리펙토링 위험하다 생각합니다. 그것만 맡아서 안 바쁘시면 도전해볼하다 생각하고요, 다른일 까지 많아서 바쁘면 힘들죠.

dw kim · Answer

본인 사업 시스템인지?어떤 회사에서 어떻게 근무하시는지에 따라 다릅니다저희 시스템은 하드코딩도 문제인데온갖 워스트 프랙티스 떡칠 입니다보는 저도 역겹습니다더 큰 문제는 그런 코드보고 개선점 없이 태연하게비슷한 방식으로 일을 하십니다그런데도 조직이 멀쩡히 잘 굴러갑니다그런 사업이니까요그게 좋다는게 아니라님이 거기 1-2년 있는 동안은그런 코드가 얼마나 있던 님에게 큰 상관 없습니다그런 코드가 남발된데는 이유가 있습니다그것부터 찾아야 합니다개선을 하지 말라는게 아닙니다개선은 나중 일 입니다그러다 보면 개선을 못하게 되는거죠작은 개선도 구조, 컨벤션, 문화가 엉망인데모래성 위에 모래성 쌓는 격이죠그게 님 전임자들 상황입니다

코드가 더럽고 문제가 많을 경우 적절한 대응 방식은 무엇일까요?

13개의 답변

Q&A 목록