곰이
10
2020-03-13 17:32:46
17
1265

코드리뷰가 없는 회사


어떻게 생각하세요 ? 




0
0
  • 댓글 17

  • devcrema
    962
    2020-03-13 17:36:48

    저는 리뷰가 없더라도 다른 문화가 좋다면 상관없을 것 같아요.

    다른 문화도 없고 리뷰도 안한다면 굳이 메리트가 없어보이네요.

    0
  • 빵상
    2k
    2020-03-13 17:37:16

    상황에따라 하는거죠..


    0
  • 7이닝
    255
    2020-03-13 17:43:01

    여유 있으면 하는거고 없으면 못하죠

    0
  • 이설란
    3k
    2020-03-13 17:43:08

    뭔 상관이죠?

    개발할 수 있고!! 정시에 퇴근할 수 있고!!! 월급만 잘 준다면

    있건없건 상관없어요.

    0
  • 곰이
    10
    2020-03-13 17:48:14 작성 2020-03-13 17:48:25 수정됨

    devcrema 개발자에 있어 다른 문화가 어떤 것인지는 잘모르겠지만, 흠 딱히 없는 것같네요.. 

    0
  • 곰이
    10
    2020-03-13 17:52:25 작성 2020-03-13 17:52:55 수정됨

    빵상 , 7이닝

    신입개발자가 와서 개발 코드에 commit 을 하고 리얼에 반영을 시도하다가 
    문제가 발생하는 사건이 종종 생기고 있습니다. 

    0
  • 7이닝
    255
    2020-03-13 17:56:22

    문제라면 커밋을 하다 svn 쫑나는거마냥 그런건가요

    혹은 남의 소스를 덮어버렸다던지...


    소스를 너무 괴상하게 짜서 서로 종속이 꼬인다던지

    아니면 잘돌아가는 공용함수에 이상한 기능을 심는다던지


    커밋 문제는 당연히 아닐거고, 계속 문제의 소스를 올린다면 리뷰 하는게 맞고

    우선 커밋전에 테스트를 빡시게 해보라고(기왕이면 기능 전수테스트) 해야할 거 같네요



    0
  • 이설란
    3k
    2020-03-13 17:56:56 작성 2020-03-13 17:58:28 수정됨

    "신입개발자가 와서 개발 코드에 commit 을 하고 리얼에 반영을 시도하다가 
    문제가 발생하는 사건이 종종 생기고 있습니다. "


    이업무는 PL이 해야할 일 아닌가요?(대부분 PL의 역할에 들어가있어서....-진짜 몰라서 물어봄 )


    수정으로 추가 - 아!! 운영이면...다를 수 있겠네요.



    0
  • 곰이
    10
    2020-03-13 18:01:54

    7이닝 

    우선 문제가 발생한 코드를 보면 로직상 예외처리가 가장 큰 문제인거 같습니다.

    이제 들어온 신입이기에 테스크코드를 작성해서 돌려라 라고 해도 뭐 어떻게 하는지 한계는 분명 보여서

    제 윗분들에게는 코드리뷰를 어느정도 하고 넘기자 라는 제안은 열심히 해봤습니다만, 

    터지면 수정하지 뭐 같은 마인드네요.. 

    0
  • 곰이
    10
    2020-03-13 18:03:32

    이설란

    음 현재 해당 프로젝트에는 PM 만 있고 PL 은 제 윗분인것 같습니다만,

    의욕이 없으신건지 이제까지 해온 언어가 달라서 인지.. 본인 코드가 아니면 관심이 없네요

    0
  • 닭강정
    244
    2020-03-13 18:11:45

    코드 리뷰... 저희 회사도 없어서... 그냥 제가 신입 분들 커밋 치면 한번씩 보고 수정할 거나 이상한점 말해주는 정도네요..


    회사에 규정이 있으면 좋을라만... 

    0
  • defult
    4k
    2020-03-13 19:39:47

    리뷰가 없는경우 사내 코딩규약이라도 확립되있어야하겠죠.

    코딩규약도 없는데 리뷰도 없다면 사후 관리개념이 없다에 가까우니 별로 기대할곳은 안될듯하고요.

    0
  • code-01
    1k
    2020-03-13 19:52:01 작성 2020-03-13 19:53:54 수정됨

    꼭 PR형태에 리뷰가 아니더라도

    신입사원이면 제대로 했는지정도는 봐줘야하는거 아닌가요?


    중요한 비지니스 로직, DB같은 민감한 부분을 신입이 사고치면

    회사가 어떻게 처리하려고 하는지.


    신입은 신입이고, 결국 큰 책임은 회사 선임들이 져야하는데 말이죠.

    생각해보니 신입한테 어떻게든 책임 몰아세우는 회사도 있겠네요

    0
  • Dive_Drink_Develope
    4k
    2020-03-13 19:55:25

    그 레벨의 신입한테 커밋까지 맡기는게 이상하네요

    신입이 케이스 테스트 하고 사수가 검수하고 커밋하는정도는 해야되는데

    리뷰가 아니라 아예 시스템이 없는거아닌가요.

    0
  • 원숭이부대
    800
    2020-03-14 04:58:57 작성 2020-03-14 05:56:19 수정됨

    리뷰가 없다는 건 컨벤션없이 자기 마음대로 되어있을 확률이 높고

    컨벤션이 있다고 해도 지켜지지 않을 가능성이 높고 그 컨벤션도 제대로 된게 아닐 수도 있습니다.

    테스트 코드도 없을 가능성이 높습니다.

    테스트는 당연히 디버깅 로그를 찍거나 직접 테스트 하는 방식으로 할겁니다.

    나중가면 개판돼서 관리 전혀 안되고 리팩토링도 불가능한 수준까지 갑니다.

    객체지향적인 코드도 아니라 알아보기도 힘들고

    일을 받을때마다 해당 부분을 소스코드를 계속해서 다시봐야 할겁니다.


    저희 회사가 그렇습니다.

    제일 편한건 이직이죠

    리팩토링같은건 직접 해도 인정을 못받아요

    티가 안나서..


    담당자가 이직을 결심하는 순간 대충 작업하고 집가서 쉬죠

    어차피 곧 안볼 소스니까


    반드시 개발자 탓만이라고 할 수는 없고

    그렇다고 반드시 회사 탓만이라고 할 수가 없죠

    다만 개인적으로는 회사쪽의 탓이 조금 더 크다고 보고

    팀장이나 파트장급의 인력관리능력이나 소스코드의 품질에 대한 태도에 따라 좌우된다고 생각합니다


    이런거 신경을 안쓰면.. 버그나 장애가 생겼을 시 개발자에게 앞뒤안따지고 책임을 물기도 하는데,

    이런 회사는 100% 개판입니다. 이러면 아무도 안고칩니다.

    내가 만든 소스도 아닌데, 굳이 내가 고쳤다가 인사평가 안좋게 받을 이유는 없죠..

    자기가 만든 소스면 100% 개발자 잘못인거구요

    0
  • 트렘플러
    14
    2020-03-14 05:30:02

    결국 조직장이 나서서 하지 않으며 안될일 같아요

    코드리뷰 - 리팩토링 이런게 없다면 쥬니어코드 시니어코드 뒤섞이게 되고

    이걸 못견뎌해서 이직하면 다른 사람이와서 또 똥치우게 되고..

    현명한 리더라면 당연히 해야 맞지 않을까 싶습니다


    0
  • 스텁
    1k
    2020-03-14 11:02:48

    코드 리뷰는 보통 애자일프로세스를 차용하거나 cicd가 도입되어 있다면 보통들 하는거라(필수는 아니지만) 그걸 안하는건 전체 프로세스에 문제있지 않을까 하는게 추측이 됩니다. 코드리뷰 자체가 문제는 아니지만 그리고 개발자들이 모두 일당백의 개발자라면 안해도 그만이지만...보통 안그렇기 때문에 시스템적으로 버그를 없애려고 쓰는 프로세스인데 그게 없다면...좀..

    유닛테스팅은 잘 되어 있나요? 왠지 이것도 잘 안되어 있을거 같은데 말이죠 

    0
  • 로그인을 하시면 댓글을 등록할 수 있습니다.