초보개발자에게 하는 꼰대 개발자의 조언

gmltjd0911 2022-03-11 16:37:15 2 0
배포하기 전에 코드 리뷰나 QA 등 검증하는 단계가 없나요? 코드 수정한 사람을 탓할게 아니라 검증되지 않은 코드가 배포될 수 있는 환경을 탓해야하지 않을까요? 이렇게 뒷담까고 다니면 김대리는 무서워서 코드 수정하겠나요?
추천 답글 신고

추천완료
닭배달아저씨 2022-03-11 17:24:31 0 0
결국 형식적인 qa가 문제였던건 부인하지 못할 사실이긴 하죠. 그 문제의 김대리가 해당 프로그램의 1차qa담당이었고 검증을 위한 테스트에서 노말한 케이스만 테스트 했던 것이고 간혹 등장하는 특이한 케이스에 대한 테스트를 못했던 것이었습니다. 게다가 인사이동 된지 얼마 안된 상태라 해당 프로그램에 발생했던 이슈를 모두 인지하지 못한 상태에서 단지 소스가 너무 지저분 하다는 이유로 의욕이 앞서 수정을 했던 것이고 최종 qa는 1차qa의 결과만 보고 배포를 승인했던 것이었죠.
추천 신고

추천완료
닭배달아저씨 2022-03-11 17:27:03 0 0
익명이니 크게 뭐라한것 처럼 쓰긴했지만 사실 대리 본인이 가장 자책하고 오히려 제가 그럴수도 있다고 격려해주는 입장이었어요
추천 신고

추천완료
gmltjd0911 2022-03-11 22:26:42 2 0
팀에 온지 얼마 안된 사람이 기존 코드 뜯어고치는 일은 뭐... 사실 흔히 있는 일이죠... 거기다 대리급 정도 되면 얼마나 뜯어 고치고 싶었겠습니까.. 그런 사람이 한 작업일 수록 코드리뷰를 좀 해줬으면 어땠을까 하는 아쉬움이 남네요. 오히려 실제 배포 나가기 전에라도 빡세게 리뷰 해줬으면 더 나았을거라 생각듭니다. 김대리님은 코드짤때 한번이라도 더 생각해 볼 수 있는 기회가 될것이고, 기존 팀원들 입장에는 엄한 사람이 와서 히스토리가 있는 코드 맘대로 수정하지 못하게 막는 기회가 되겠고요.
추천 답글 신고

추천완료
원진아 2022-03-22 14:53:11 1 0
프로그램의 소스를 깔끔하게 수정하는건 잘아는사람의 몫인데, 수정안하니까 문제 ㅋㅋ
추천 답글 신고

추천완료
닭배달아저씨 2022-03-22 15:08:28 0 0
잘하는 사람은 그거 아니더라도 할일이 많은게 문제
추천 신고

추천완료
원진아 2022-03-22 15:09:37 0 0
그건 ㅇㅈ
추천 신고

추천완료
다쿠렉 2022-04-03 03:36:40 0 0
돌아가는 소스코드는 삼각형모양이더라도 냅두라는 격언이 있죠 ㅋㅋ
추천 답글 신고

추천완료

초보개발자에게 하는 꼰대 개발자의 조언 [8]