CLIEN

본문 바로가기 메뉴 바로가기 보기설정 테마설정
톺아보기 공감글
커뮤니티 커뮤니티전체 C 모두의광장 F 모두의공원 I 사진게시판 Q 아무거나질문 D 정보와자료 N 새로운소식 T 유용한사이트 P 자료실 E 강좌/사용기 L 팁과강좌 U 사용기 · 체험단사용기 W 사고팔고 J 알뜰구매 S 회원중고장터 B 직접홍보 · 보험상담실 H 클리앙홈
소모임 소모임전체 ·굴러간당 ·주식한당 ·아이포니앙 ·MaClien ·방탄소년당 ·일본산당 ·자전거당 ·개발한당 ·이륜차당 ·소시당 ·패스오브엑자일당 ·AI당 ·안드로메당 ·나스당 ·걸그룹당 ·바다건너당 ·사과시계당 ·클다방 ·물고기당 ·노키앙 ·전기자전거당 ·노젓는당 ·축구당 ·윈폰이당 ·IoT당 ·창업한당 ·가상화폐당 ·여행을떠난당 ·곰돌이당 ·키보드당 ·리눅서당 ·골프당 ·콘솔한당 ·3D메이킹 ·X세대당 ·ADHD당 ·AI그림당 ·날아간당 ·육아당 ·배드민턴당 ·야구당 ·농구당 ·블랙베리당 ·비어있당 ·FM당구당 ·블록체인당 ·보드게임당 ·활자중독당 ·볼링친당 ·캠핑간당 ·냐옹이당 ·문명하셨당 ·클래시앙 ·요리한당 ·쿠키런당 ·대구당 ·DANGER당 ·뚝딱뚝당 ·디아블로당 ·개판이당 ·동숲한당 ·날아올랑 ·e북본당 ·갖고다닌당 ·이브한당 ·패셔니앙 ·도시어부당 ·FM한당 ·맛있겠당 ·포뮬러당 ·젬워한당 ·안경쓴당 ·차턴당 ·총쏜당 ·땀흘린당 ·하스스톤한당 ·히어로즈한당 ·인스타한당 ·KARA당 ·꼬들한당 ·덕질한당 ·어학당 ·가죽당 ·레고당 ·LOLien ·Mabinogien ·임시소모임 ·미드당 ·밀리터리당 ·땅판당 ·헌팅한당 ·오른당 ·영화본당 ·MTG한당 ·소리당 ·적는당 ·방송한당 ·PC튜닝한당 ·찰칵찍당 ·그림그린당 ·소풍간당 ·심는당 ·라즈베리파이당 ·품앱이당 ·리듬탄당 ·달린당 ·Sea마당 ·SimSim하당 ·심야식당 ·윈태블릿당 ·미끄러진당 ·나혼자산당 ·스타한당 ·스팀한당 ·파도탄당 ·퐁당퐁당 ·테니스친당 ·테스트당 ·빨콩이당 ·공대시계당 ·터치패드당 ·트윗당 ·VR당 ·시계찬당 ·WebOs당 ·소셜게임한당 ·위스키당 ·와인마신당 ·WOW당
임시소모임
고객지원
  • 게시물 삭제 요청
  • 불법촬영물등 신고
  • 쪽지 신고
  • 닉네임 신고
  • 제보 및 기타 제안
© CLIEN.NET
공지[점검] 잠시후 서비스 점검을 위해 약 30분간 접속이 차단됩니다. (금일 18:15 ~ 18:45)

모두의공원

(저도 코딩중..) 전임자가 코드로 지뢰를 만들어놨네요... 23

1
2021-05-06 23:36:01 223.♡.162.43
TongMax


와 이거 회사 코드라 올릴수도 없고...

이전에 몇명의 전임자들을 거쳐간 코드라고는 하지만

깃 어노테이션으로 히스토리로 보면 한 사람의 소행입니다..


나이는 저보다 어리고(29살정도..) 이 코드를 받은 순간부터 1년동안 10만줄의 코드를 양성해 냈는데


그 코드를 만든 패턴이나 이유를 전혀 모르겠습니다.. 


같은 로직인데 순서가 틀립니다, 예를 들자면 

유저 정보를 저장한다 > 유저 롤을 저장한다 > 유저 작품을 저장한다 순서인데


다른api에서는 유저 롤 > 작품저장 > 유저정보..


그리고 여러곳에서 쓰는 코드를 공통화를 해볼라고 노력은 한거 같은데


그 내부 코드가 정말 엉망 진창입니다..

(Collections.singletonList 메소드를 쓴 코드는 처음봤습니다..)


다 뜯어고쳐야지! 마음먹었는데

마음이랑 다르게 제가 뭘 고치려고 했었는지조차 헷갈리게 만드는 악마의 코드를 보고 있자니..

하소연을 하게 되네요... 모두 즐코딩 하십시오..ㅠㅠ


TongMax 님의 게시글 댓글
SIGNATURE
개인 IT 블로그 : https://blog.naver.com/gingsero
Github : https://github.com/gingsero
개발자디버깅카톡방 : https://open.kakao.com/o/gR6Fa7F
서명 더 보기 서명 가리기
  • 주소복사
  • Facebook
  • X(Twitter)
댓글 • [23]
카슈밀
IP 125.♡.36.114
05-06 2021-05-06 23:38:52
·
제일 빠른 방법은 새로 만들거나 아니면 최대한 쪼개면서 기는구현이죠
TongMax
IP 223.♡.202.150
05-06 2021-05-06 23:43:11
·
@카슈밀님 다시 집중해서 쪼개보겠습니다🤧
gumdung
IP 218.♡.31.18
05-06 2021-05-06 23:38:55
·
저장 순서가 다르면 데드락 걸릴 확률이 높씹니더.

A-B-C의 순서로 저장되는 루틴이 실행중인데 A-C-B도 같이 실행중이믄 백프로 데드락입니더.
isaiah1
IP 175.♡.69.176
05-06 2021-05-06 23:39:35
·
정교한 능력이 없는데 "몇일까지는 안 될까요? 우리 스캐줄이,,,,," 를 몇번 당하면...
니파
IP 218.♡.220.80
05-06 2021-05-06 23:40:02
·
의외로 그 코드 자체도 리팩토링의 결과일수도 있고..

당시에는 오히려 그 방법 밖에는 없었을수도 있고... 그렇습니다.

같은 로직인데 순서를 다르게 짤 수 밖에 없는 환경을 경험한적이 있어서..
TongMax
IP 223.♡.202.150
05-06 2021-05-06 23:44:07
·
@니파님 이게 리팩토링이라면... 경찰서에 신고해야되나 싶을 정도의...
ㅠㅠ
mairoo
IP 171.♡.224.139
05-06 2021-05-06 23:42:39
·
1년 동안 10만 줄이요??? 설마 혼자서라면... 엄청난 거 아닌가요. ㄷㄷㄷ
TongMax
IP 223.♡.202.150
05-06 2021-05-06 23:45:33
·
@mairoo님 코드로 공부한거 같은데 음... 개발못하는 사람이 만든 코드가 서비스에 나가있다고 생각하시면 됩니다...ㅠㅠ
그 코드를 수정하는건 제 실력 향상에도 도움이 되지만, 스트레스가😫
삭제 되었습니다.
TongMax
IP 223.♡.202.150
05-06 2021-05-06 23:48:00
·
@별미에요님 음.. 이렇게 글을 쓸걸 그랬습니다..
복붙한겁니다 소스를...
자바8기반에 어떤 소스에는 stream을 떡칠을 해놓거나 한줄이면 될걸 굳이 메소드를 빼놨거나(제가 합치고..)
라인하르트뮤젤
IP 211.♡.21.53
05-06 2021-05-06 23:44:46
·
시간의 압박... 일단 돌아가게는 해야한다는 조급함...
Eastfield
IP 67.♡.121.26
05-06 2021-05-06 23:57:32 / 수정일: 2021-05-06 23:58:22
·
1년에 10만줄이요???? 제가 초대형 프로젝트 처음부터 재 개발하는 걸 했는데 (11개월 째) 그래도 다 합치고 합쳐서 2만 6천줄정도인데... ㄷㄷ
들판에서
IP 14.♡.84.142
05-06 2021-05-06 23:58:28
·
이게 아마 사연이 있었을 겁니다. 그거 모르고 만지다가 판도라 여는 수가 있어요.
갑자기 뜬금없는 부서에서 뭐가 안되요.. 뭐 그런거요. 인수인계에도 없는 부서인데 나타날 수 있습니다.
물론, 깔끔한 소스는 그런 사연조차도 깔끔하게 보이죠.
NULLNONE
IP 121.♡.240.130
05-07 2021-05-07 00:43:46
·
@들판에서님 +1 그것은 그럴만한 이유가 있다 라고 저도생각합니다. ㅎ
TongMax
IP 223.♡.202.133
05-07 2021-05-07 07:11:48
·
@들판에서님 제가 서비스 맡은지 6개월 정도 됐구, 6개월 내내한건 아니구요
한두달 리펙토링하고 기능 수정해주고 끄적끄적 했는데 3만줄 수정했습니다;;
아마 사연있는거라면 그분이 spring, jsp를 처음 만져봤다일거 같습니다..
방금 또 코드보다가 느꼈는데
메소드 명, api명이 검색을 못하게 만들어놨습니다..
나름대로 정리를 한다고는 했는데
메소드명은 insert() 로 박아놓거나
api는 타입스크립트 객체 타입으로 만들어왔는데.. 기가 막히는 ㅠ...
들판에서
IP 210.♡.105.1
05-07 2021-05-07 09:26:56
·
@TongMax님
3만줄 수정이면 원하시는대로 하셔도 되겠네요.
방향을 잡으셨으면 밀고 가시는데 윗 사람한테 생색을 엄청내세요. 약간의 뒷담화는 양념으로 곁들이면서..
뭐 재미있게 코딩하시기 바랍니다.
판디
IP 24.♡.126.243
05-06 2021-05-06 23:58:29
·
기능에 문제 없으면 그냥 두는게 나은 방법인거 같은데요..
삭제 되었습니다.
TongMax
IP 223.♡.202.133
05-07 2021-05-07 07:07:16
·
@Oh호라님 대단한건 아니고 어드민 서비스 입니다, 작품 제안하고 승인하고 그런..
근데 저도 이 서비스 맡은지 6개월 됐는데 그닥 열심히 고치지는 않고 한두달 정도 리팩토링 해볼라고 끄적끄적하니 3만줄 정도 나오더라구요..
안쓰는 라이브러리 정리, 잘못뺀 매소드 정리, 이상한 쿼리문 수정..
화면단 jsp는 더 기가 막힙니다 ㅠㅠ
삭제 되었습니다.
라면N계란
IP 211.♡.28.198
05-07 2021-05-07 06:38:34
·
반영 후...

???: 어? 이게 아닌데요?

이제와서 그러면 어째요?

???: 고쳐주던가 이전 상태로 돌려요!

18 @@@&&&

이렇게 수정해서 나온 코드는 고칠래야 고칠수가 없...
musicsh
IP 211.♡.70.116
05-07 2021-05-07 07:06:49
·
유닛테스트 부터 작성해보세요.
삭제 되었습니다.
TongMax
IP 223.♡.202.219
05-07 2021-05-07 17:06:18
·
@팜의추억님 깃헙 인사이트 기능으로 보면 +라인수 -라인수가 있습니다^^;;
라이브러리 안에 소스는 깃헙 인사이트에 포함되지 않는 것으로 알고 있습니다!
소스보면 진짜 10만 줄 맞습니다..ㅠㅠ
위에 분들 참조해서 수정 먼저 안하고 유닉테스트 만들고 있습니다...
삭제 되었습니다.
TongMax
IP 211.♡.69.183
05-08 2021-05-08 23:09:16 / 수정일: 2021-05-08 23:09:48
·
@팜의추억님 일단 제가 급하게 확인하느라 제가 수정한 코드수를 3만줄이라 적었는데 제가 아니고 그 이전 작업자의 코드가 3만이고 저는 4천라인정도 되네요
못믿는다는 걸 증명시켜드립니다, 코드 라인수로 회사가 잘되고 그랬으면 SI 업계가 제일 잘나가야하는거 아닐까요..?
이렇게 전임자 코드 라인수까지 찍어드렸습니다. 코드도 직접보여드리고 싶은데 그럴 수 없는거 아시죠..?

저 더 비싼회사 갈.. 수 있긴합니다만, 그만큼 무게를 견디기 싫고, 지금 회사도 나쁘지는 않습니다, 나쁜 코드를 만들어내는 사람들이 있을뿐이죠..
힘순찐 = 신계 라는 말도 언짢고.. 제가 신계라고 비꼬시는거 같은데, 기분이 언짢네요ㅎ..
창업하고 좋은 서비스면 1만라인으로도 성공해야죠.. 그럴 서비스 아닌데 라인수로 신계네 힘순찐이네, 하신게 기분이 나쁜데 사과부탁드립니다~
TongMax
IP 211.♡.69.183
05-08 2021-05-08 23:11:23
·
@팜의추억님 어,, 저 그리고 3만라인 코치는걸 손으로 직접 수정하신다고 생각하시는걸까요..?
IDE로 리플레이스 기능 쓰면 꽤 많은 라인을 1분만에 수정가능합니다만..
삭제 되었습니다.
TongMax
IP 59.♡.197.67
05-09 2021-05-09 12:57:50
·
@팜의추억님 음,, 일단 글 감사드립니다. 제가 글을 잘못쓴건가.. 믿지못하게 쓴건가.. 아니면 제가 코딩을 했다는 내용이 어려운건가... 여러가지로 생각을 해봤습니다.
만, 지금 이 댓글을 읽고 제가 또 다는 이유는 아래와 같은 생각 때문입니다.

1. 무엇을 보고 신계라고 질문 하신건지,, 개발언어 스펙정의서를 보면 알 수 있는 내용들입니다..제가 어딜봐서 신계인지..
2. 카운트 기능을 파일로 돌려서 돌리는 방법도 있습니다만 깃헙에서 제공해주고 있습니다
3. 10만 라인이든 100만 라인이든 라인수로 회사가 잘되고 못되고를 판단하시는건 어떤 경험으로 얻으신건지..
4. 힘순찐, 신계라는 단어로 비싼자리를 갈 수 있다 떠나라 라는 판단을 어떻게 하신걸까요? 주변에 잘하는 사람들이 돈을 다 많이 받는다고 단언할 수 있으신가요..? 회사에서 잘하는 직원들 보면 저랑 비슷하게 받거나 못받는 직원들도 보입니다 이건 실력의 문제가 아니라 사회 시스템의 문제인거 같습니다만..
4. 못 믿는다시길래 저는 증거를 드렸습니다. 10만라인이요, 이럴필요까진 없지만 증거가 있어야 주장을 해야하는 클리앙이니..
5. 증거를 드렸는데 "본인의 의견을 받아들이고 못받아들이고"라는 대답을 하셔서 이게 무슨 말씀이신가 생각해봐도 잘 모르겠습니다
6. 저,, 분노를 긍정적인 결과를 얻는데 이용하는 방법은 제가 주로 사용하는 방법입니다;;
7. 저도 나이가 좀 있어서 어린 친구들이나 젊은 친구들과 코드리뷰 하는데 많은 도움을 받습니다. 그 간에 저 스스로 잘못만든 코드를 보고 지적을 받고 고치기로 했구요
8. 기분의 요인이라고 말씀하신게 돈인데.. 저는 이걸 찬성하지 않습니다;; 저는 연봉에 딱히 불만이 없을정도로 받고 있다고 생각하는데 안좋은 코드를 보면 화가 납니다. 그리고 전 직장에서 지금의 반도 안되는 연봉 받으면서 밤새면서 코딩했을때는 아침에 팀장님 말 한마디에 웃으면서 퇴근했구요.. 돈의 문제도 있겠지만 돈'만의' 문제로만 보시면 생각이 전혀 다르시구나 정도로 이해가 될거 같습니다..

이 중에 제가 잘못 생각한 것도 있을거고 잘못 말한 것도 있을거 같은데, 그런 부분들이 있으면 말해주시면 고마울거 같습니다
저는 이상 8가지로 님의 댓글에 잘못이 있다고 말씀드렸고 사과를 요청했습니다. 사과를 해주시면 더 고마울거 같습니다.

그리고 현명한 판단을 바란다는 말씀이 저랑 연배차이가 얼마 나지 않는 분에게 듣는 조언으로는 들리지 않습니다..
더 현명한 판단이라면 시대가 달라졌구나, 내 경험을 더 조리있게 말하고 쓸 수 있었으면 좋았을텐데,
상대의 말을 믿지 못하겠다라고 상대의 글을 격하시키는 말보다 더 좋은 단어를 썼으면 좋았을텐데 라는 말을 남기셨으면 더 좋았을거 같습니다..
새로운 댓글이 없습니다.
이미지 최대 업로드 용량 15 MB / 업로드 가능 확장자 jpg,gif,png,jpeg,webp
지나치게 큰 이미지의 크기는 조정될 수 있습니다.
목록으로
글쓰기
글쓰기
목록으로 댓글보기 이전글 다음글
아이디  ·  비밀번호 찾기 회원가입
이용규칙 운영알림판 운영소통 재검토요청 도움말 버그신고
개인정보처리방침 이용약관 책임의 한계와 법적고지 청소년 보호정책
©   •  CLIEN.NET
보안 강화를 위한 이메일 인증
안전한 서비스 이용을 위해 이메일 인증을 완료해 주세요. 현재 회원님은 이메일 인증이 완료되지 않은 상태입니다.
최근 급증하는 해킹 및 도용 시도로부터 계정을 보호하기 위해 인증 절차가 강화되었습니다.

  • 이메일 미인증 시 글쓰기, 댓글 작성 등 게시판 활동이 제한됩니다.
  • 이후 새로운 기기에서 로그인할 때마다 반드시 이메일 인증을 거쳐야 합니다.
  • 2단계 인증 사용 회원도 최초 1회는 반드시 인증하여야 합니다.
  • 개인정보에서도 이메일 인증을 할 수 있습니다.
지금 이메일 인증하기
등록된 이메일 주소를 확인하고 인증번호를 입력하여
인증을 완료해 주세요.