StackOverflow에서 다시 게시를 요청했습니다.
개발 시간이 매우 짧은 소규모 개발 시간에 일하고 있습니다. 우리는 업무 결과에 중요하지만 매일 사용하지 않는 도구를 개발합니다. 나는 프로그래머로서의 배경을 가진 팀의 유일한 사람입니다.
내 문제는 1 년 이상 동안 트렁크로 병합하기 전에 코드 검토를 추진하고 있다는 것입니다. 모두가 이것에 동의했지만 여전히 검토 된 것은 내 코드 일뿐입니다. 긴 휴가에서 돌아온 후 "이것은 못생긴 솔루션입니다-가능한 빨리 제거하십시오"및 "빠른 수정"이라는 코드 주석이있는 트렁크로 돌아옵니다. 새로운 점은 도구에 대한 책임이 사람에게 지정되었다는 것입니다. (처음 나에게 제공된 역할이지만 업무와 관련이없는 이유로 거절했습니다.) 그리고 그는 이것이 일하기에 좋은 방법이라고 생각합니다.
저의 걱정은 다른 개발자가 못생긴 코드를 작성한다는 것입니다. 캡슐화를 깨고, 큰 클래스를 작성하고, 이상한 장소에 내부 클래스를 추가하거나, 단위 테스트가 거의 없거나 전혀없는 등. 결국 도구를 더 개발하는 것은 불가능합니다.
트렁크로 병합하기 전에 코드 검토를 수행해야한다고 주장해야합니까, 아니면 코드 품질에 문제가 있습니까?