빈지노
17
2017-12-02 00:08:07.0
27
3079

회사의 코드품질이 너무 떨어집니다


모 솔루션 회사에 다니고있는데 관리하는 소스들이 너무 속된말로 개판입니다

단일책임 원칙까지는 바라지도 않습니다만 이건 뭐 분리라는 개념이 아예없는건지 

하나의 큰기능은 메소드 하나에 다 때려박고 그 이외에는 유틸성의 메소드만 존재하네여..

때문에 몇백줄 천단위까지 넘어가는 메소드는 흔하구요 

진심 극악인건 plsql로 비지니스로직 주구절절이 구현해놓은 욕밖에 안나오는 쿼리들인데 

아예 쳐다보기도 싫습니다.....

테스트코드는 본적도 없고 단위테스트가 가능한 코드도 없습니다. 테스트란 그저 프로젝트 run시키고 마우스로 기능테스트 하는것이겠죠..

여기 개발자들은 도메인 모델이라는 개념자체가 없고 모든건 데이터의 흐름인지 의식의 흐름인지 모를것이 기준인거 같네요

아예 근본적인 아키텍쳐라는게 존재도 안하다보니 이건 뭐 리팩토링은 엄두도 안나지만

그래도 메소드 추출 분리, 언어의 컨벤션은 개나준 빌어먹을 암호같은 네이밍들 사람이 읽을 수 있을 정도로 정리 수준으로 꾸준히 리팩토링하고는 있습니다.

반전은 이게 우리나라에서 꽤나 잘나가는 솔루션이라는 건데 참 그 나름대로 놀라운 일입니다..

사실상 첫 회사인데 우리나라 it의 현실인건지 

제가 속한 이곳의 문제인건지 고민되네여...

그럼 굿밤되세요...


3
0
  • 댓글 27

  • kin5299
    163
    2017-12-02 00:40:53.0

    품질을 올려보세요.

    하지만 회사의 지원은 바라지마세요.

    회사는 이율만 생각 회사이니깐요.

    님만 그렇게 생각한다고 하지마세요.

    님 전 사람들도 똑같은 생각, 불만을 했을테니깐

    회사에 애정이 있다면, 회사내 사람들과 소통 후 솔루션 품질을 올려보세요.

    하지만 애정은 없겠지요.

    그러면, 불만 갖지마세요. 그냥 돈받고 다니세요.

    어짜피 다른 둥지를 찾으면 되니깐요.

    이게 현실이니깐요.

    2
  • 돌팅이
    79
    2017-12-02 01:44:24.0

    자 아무것도 없으니 작성자가 이렇게 해야한다고 그 회사를 리딩하면 됩니다.

    욕하는건 쉬워요. 


    2
  • 에르딘트
    1k
    2017-12-02 03:11:50.0

    욕할꺼 있나요~

    선택하시면 됩니다


    1. 내가 다 고쳐주마!

    2. 도망가자!!~

    1
  • 구텐모겐
    200
    2017-12-02 04:56:51.0 작성 2017-12-02 04:58:25.0 수정됨

    어딜가나 그런 코드가 있고 품질에 관심없는 개발자가 있습니다.

    그런다고 본인이 나서서 리팩토링 하다가 버그 생기면 욕먹는게 현실이고 리팩토링 잘 했다고 알아주거나 칭찬 받지도 읺습니다. 오히려 니 코드 바꿔서 못 알아 먹겠다는 불평 듣거나 필요한 일 안하고 허튼일 했다는 소리 듣습니다 

    크게 리팩토링 하진 말고 천천히 조금씩 바꿔서

    버그안나게 리팩토링 하세요. 리팩토링전에 다량의 유닛테스트 만들어 두고 리팩토링 후 유닛테스트가 문제 없이 잘 도는지 확인하면 리팩토링으로 인한 버그 걱정은 줄 수 있습니다

    0
  • 스피드스타
    185
    2017-12-02 09:11:48.0 작성 2017-12-02 09:13:16.0 수정됨

    윗댓글님들이 적절하게 답변해주셨네요.

    정리하자면

    1. 욕하는건 쉽다.

    2. 리팩토링 하다가 버그 생기면 욕먹고, 실력도 없는게 리팩토링 한다고 입으로만 설친다고 볼지도.

        소스는 원래대로 롤백이고, 다시는 리팩토링 얘기도 못꺼내게 됩니다.

        물론 앞으로도 좋은 평가는 기대할순없죠.. 자신을 돋보일수있는 뭔가가 있지 않는한..

    3. 리팩토링을 잘한다고쳐도, 그걸 성과로 인정해줄까요? 그건.. 글쎼요입니다..


    왜 이렇게 코드들이 계속 유지됐는지, 보자면 저런 이유들이 큽니다.

    코드가 개판이던뭐던, 돈을 잘 벌고있기떄문에 윗선에선 문제없다고 볼겁니다.

    그리고 전임자들도 당연히 싫어했을겁니다.

    그러나 왜 코드를 건들지 않았을까요?

    나름대로 다 이유는 있습니다..


    이제 그걸 글쓴님이 리팩토링 하는건 글쓴님의 선택입니다.

    다만 리팩토링 한 코드가, 글쓴님만 보기 편한 스타일로 리팩토링 된건지는

    그 후임자가 평가할것입니다.


    결론.

    실력에 자신있고, 리팩토링을 통해서 더 실력을 끌어올리는 목적이라면 리팩토링 하세요.

    암튼 전체를 리팩토링한다면 덤으로 그 솔루션 자체를 만들 실력이 되시겠네요~

    0
  • LichKing
    6k
    2017-12-02 09:31:41.0

    코드 품질이 훌륭한 솔루션 몇 없을겁니다.

    하시는것처럼 지속적으로 개선해나가고 사내스터디 등을 통해 팀원들에게 그런 마인드를 심어주는게 중요할것같네요.

    0
  • lllllllllllllll
    3k
    2017-12-02 09:58:50.0

    빌어먹을 암호같은 네이밍들 ㅋㅋㅋㅋㅋ

    0
  • Soos00
    254
    2017-12-02 14:16:10.0

    좋으신분이네요.ㅎㅎ
    나중에 글쓴이같은 사수님 만나기를 바래야겠습니다!

    화이팅이에요!

    0
  • 무지렁이
    226
    2017-12-02 17:06:25.0

    와 어떻게 저랑 똑같은 고민을 가지고계시네요..

    저도 고민입니다...

    0
  • 빈지노
    17
    2017-12-02 18:14:46.0
    답변들 감사합니다.
    금요일에 회의 후 너무 자괴감도 들고 화도 나고 해서 배설하듯 글을 썻는데 
    혹여나 불쾌하셨던 분들에게는 죄송합니다.
    어차피 들어온 회사, 혼자만의 원칙이 아닌 보편적인 원칙을 기준으로 개선해 나가겠습니다 
    1
  • 잘부셔지는쿠쿠다스
    162
    2017-12-02 18:15:46.0

    저는 어느 기관이 모 회사에 외주를 줘서 만들어진 솔루션 코드를 봤는데요. 한 메소드 안에 엄청 큰 다다다다다다다다다다중 이프문이 있네요. 무려 그 이프문의 길이는 1500줄이더라고요.


    제가 신입으로 입사했는데 원래 보통 규모가 있는 프로그램에서 저러는지 궁금합니다. 

    0
  • coding8282
    1k
    2017-12-03 02:08:27.0
    어지간한 중소기업은 다 그럴 겁니다.. 저는 리팩토링 하지 말라고 권해드리고 싶네요. 1500라인 메소드 complexity가 무지 높게 나올 것 같은데요, 단위테스트 불가/이해 불가능입니다... 그리고 이런 환경에서는 개발자가 할 수 있는 게 별로 없어요. 아무리 날아다녀도 1000라인짜리 메소드로 범벅되어 있는 데서는 안 될 거에요.. 그저 만능 if문으로 대응할 수밖에 없는 게 현실이구요... 중소기업 중에 또 인력이 자주 교체되는 경우라면 상황은 더 악화되는데요, 보통 소스를 한두명이 만지는 게 아니라 후임자가 와서 if문 추가하고, 또 다른 후임자가 와서 else문 추가하다가 보면 Big Ball Of Mud가 됩니다. 그리고 이런 환경에서는 객체지향 설계/아키텍처/신기술 등을 적용해볼 기회가 없게 되구요, 그저 if문 이리저리 추가해서 돌아가게 만드는 꼼수만 늘더라구요. 환경이 잘 갖추어진 회사에서 시작하는 개발자와 실력이 많이 벌어질 수밖에 없더군요... 환경 탓을 하면 안 되지만 저도 2년간 해보면서 느끼고 있어요.. 그래도 글쓴분은 자각을 하고 계셔서 괜찮으시네요.. 상당히 많은 사람들은 그런 생각도 안 해요.. 그저 if문 추가 해서 돌아가게 만드는 게 정상인 줄 알거든요..
    0
  • March
    1k
    2017-12-03 11:35:09.0

    그냥두세요

    이바닥의 80%는 거의 다 그렇고 또 고치려고 하는 개발자도 없습니다

    어쨋든 돌아가면 그만...이라는 오너가 상사의 생각인거죠

    0
  • 한수위
    628
    2017-12-04 00:29:16.0

    이 회사가 내가 평생 다닐 회사다: 천천히 고친다

    잠시 머물 회사다 : 그냥 둔다

    아마 전 개발자 분들은 잠시 머물다 가신거겠죠


    0
  • 나다
    2k
    2017-12-04 09:45:19.0
    아름다운 코드는 잘~ 돌아가는 코드이다......ㅋ
    0
  • Collin
    14
    2017-12-04 10:40:28.0

    본인이 작업한 코드는 상관없는데 다른 개발자가 진행중이거나 사용중인 건 상의 후 하세요.

    0
  • 7i
    796
    2017-12-04 14:04:37.0 작성 2017-12-04 14:10:57.0 수정됨
    리팩토링이 안되는 수준인데 어떻게 리팩토링을 합니까.....;;
    뭔가 잘못된 코드와 프로젝트들이 이런 저런 이유로 합리화되어서 어쩔수 없다는 이유로 유지되는게 안타깝네요

    누군가는 확 뒤집어서 개혁을 해야되죠...
    저 코드가 몇십줄에서...몇천줄이 될때까지 어쩔수 없는 핑계로 계속 관행되어 여기까지 왔다고 볼수 있죠
    자잘한 리팩토링은 해봤자 크게 소용이 없다는걸 잘알고들 계실겁니다.

    이런건 CTO나 개발 책임직책이 있는 윗분들이 총대를 메주셔야 사원들도 일할 맛이 나죠...
    그 스파게티코드를 매일 아침마다 볼 자신이 있으십니까...


    1
  • Dh. k
    1k
    2017-12-04 15:00:16.0

    ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ

    0
  • narise
    2k
    2017-12-04 21:39:55.0

    코드품질 이전에 설계서도 없을 겁니다.


    정확한 사양을 아무도 모른다는 겁니다.


    그냥 지금 돌아가는게 사양이다.


    이상황에서 어떻게 리팩토링합니까?

    0
  • Chaed
    786
    2017-12-04 22:47:09.0

    이런 지경에 이르면 리팩토링 안하는게 아니라 못하는거 아닙니까? 

    그냥 최소한 자기가 다루는 영역에서라도 하는게 최선이겠네요..

    0
  • imkh
    1k
    2017-12-05 08:53:13.0

    그냥 우리나라 현실중에 하나인거죠!! ㅜㅜ

    0
  • 퓨리오사
    1k
    2017-12-05 12:16:17.0

    오래다닐거면 리팩토링... 적당히 경력쌓을거면

    손대지마세요. 

    손대서 문제 생기면 다 글쓴이분 책임...

    잘 돌아가도 본전...

    왠만한 회사가 아닌 이상 리팩토링 중요하게

    생각안해요.

    0
  • shaffron
    2k
    2017-12-05 14:36:08.0

    리팩토링도 가능한 시점이 있는데 말씀 들어보면 이미 산송장이나 다름없는 코드인거 같네요.

    숨이 끊긴 코드에게 아무리 CPR을 해봤자 하는사람만 미친x 소리 들을 뿐입니다 ㅠ


    0
  • kkey21a
    2k
    2017-12-05 16:55:58.0

    하! 찔리네 ㅎㅎ

    0
  • 에그슈타인
    39
    2017-12-05 17:11:38.0

    이미 회생 불가 일것 같습니다. 그정도로 엉망이라면...이전 개발자들도 고쳐놓은것이 없다면...

    리팩토링이 귀찮아서가 아니라, 고칠수가 없었을 것 이라고 생각합니다. 

    처음 시스템 설계부터가 잘 못 되었을수 있겠네요. 테이블 구조라던지...

    그정도라면.. 아예 다시 설계 잡고 다시 개발을 해야하겠죠.

    몇몇 분들은 내가 계속 다닐거라면 천천히 고친다고 하셨는데. 그게 불가능한 상황도 있을것 같습니다.

    말단 개발자가 어찌 할 수 있는 상태가 아닌거죠. 그런 비정상적인 상황을 모두들 모른채 한다면 이직이 답입니다.

    '절'이 싫다면 중이 떠나야지요.. '절'을 바꾸는건 굉장히 어렵습니다. 그'절'이 그정도로 가치가 있을지도 모르겠네요.




    1
  • 소쿨렝보
    17
    2017-12-06 02:16:22.0

    새로 개발하지 않는이상 더욱 조악해져갈뿐...

    점점더 어려워지는 유지보수... 계속 바껴가는 사람들 ㅋ

    0
  • 냉이
    886
    2017-12-07 11:31:42.0

    조급해하지 말고 천천히 하세요~

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