펩시마이쪙
2020-01-02 22:40:58
19
3766

새로들어온 신입이 ajax 떡칠을 해놧습니다... ㅋㅋ


회사 웹 사이트가 탭 전환을 통한 페이지 전환등에 특화되어 있는 사이트입니다.

(이걸 UI 적인 용어로 뭐라고 부르는데 잘 생각이 안나네요...)


여러 기능들을 저희가 만들어둔 탭UI에 얹은다음 iframe을 쌓아올려 페이지를 로드하는 방식입니다.


그냥 브라우저 탭이랑 같아요. 네이버 하다가 웹툰 탭으로 넘어가서 웹툰좀 보다가 다시 네이버검색좀하고


그래서 이곳저곳 iframe 의존도가 좀 높은 편입니다. ajax를 사용하긴 합니다만,


로드되야할 리소스가 많아지면 크롬의 경우엔 괜찮지만 익스의 경우엔 끔찍할 정도로 느려지는 경우가 있기 때문에 왠만해선 사용을 안합니다.


사이트에 사용되는 스크립트 코딩들도 전부 iframe에 맞춰져 있기도하고요.


근데 이걸 사이트소스 공부도 하고, 겸사겸사 개선사항도 처리할겸 쉬운거 몇개 신입한테 던져줫더니


페이지에 있는 iframe을 몽땅 걷어버리고 블로그에서 가져온듯한 따끈따끈한 ajax 코드로 떡칠해놧습니다 ㅋㅋ


당연히 UI 다 깨지고, 되던 기능들 안되니 맨붕해서 공통 CSS, Script에도 손을 대고, 능력도 좋게 단 10시간 만에 커밋을 50개나 했더라고요 ㄷㄷ...


이 정도면 한 6개월 지나면 굉장한 복붙 실력을 갖춘 훌륭한 프로그래머로 성장하겠구나 라는 생각이 들었습니다.


궁금한거 있으면 언제든지 물어보라 해놓고, 바빠서 하루 손을 놨더니 완전 혼돈 그 자체를 만들어 놨더라고요 ㅋㅋ


자바 스크립트에  코드로 debugger; console.log 는 왜 그렇게 많이 찍어서 커밋해놨는지


아 이건 다른거 다 떠나서 그냥 전체 revert가 빠르겠구나 싶어 일한거 아깝지만 전부 되돌린 후, 왜 이렇게 하면 안되는지 1시간동안 설명해줬습니다.


저 처음 들어왔을때 똥싸던거 생각나서 재밌었던 하루였습니다.

0
  • 댓글 19

  • 로봇이 아니라예
    81
    2020-01-02 22:45:53

    저도 신입인데 새끼 손가락 아프네요

  • 시인들
    1k
    2020-01-02 22:51:31

    이럴 수도 있고 저럴 수도 있고 신입이 그러면 다행이죠...

  • Anonymousss
    193
    2020-01-02 23:01:29

    남 얘기 같지 않네요..... ㅎㅎ

  • pooq
    5k
    2020-01-03 01:19:18

    신입한테 마스터 브랜치를 그냥 맡기나요? ㅎㄷㄷ

    왠만하면 브랜치 하나 만들어서 그거가지고 씹고 뜯고 맛보고 즐기라고 하세요.

  • Frudy
    5k
    2020-01-03 05:33:37

    페이지로드 쪽으로는 제가 지식이 전무해서

    글 내용이 하나도 이해되지않네요 ㅜㅜ


    와드남기고 추후에 다시볼게요.

    잘못된 사례 올려주셔서 감사합니다.

  • Chaed
    2k
    2020-01-03 08:53:26

    리뷰시스템이 없나보네요 ㄷㄷ

  • 믹스베리
    162
    2020-01-03 09:07:24

    pooq 님 말씀처럼 브랜치 하나 따서 거기서 작업한 후 ,

    시간 날때 코드리뷰 한번 하는게 젤 베스트일거 같네요 하하

  • 퓨리오사
    2k
    2020-01-03 09:30:56 작성 2020-01-03 10:02:12 수정됨

    탭이 굉장히 많은가보네요.

    iframe 에서 ajax로 전환해서 깨진건지

    iframe 에서 ajax로 전환을 했는데 동적으로 그릴 화면을 잘못 그려서 깨진건지는 궁금하긴하네요.

    아니면 ajax 로 요청해서 오는 응답을 처리 못할정도로 요청/응답이 많아 iframe을 쓰는걸까요?


  • 코드리뷰
    104
    2020-01-03 09:38:55

    저는 신입이 들어올수도 있는 이 사이트에, 그리고 다른 사람들에게 잘못된 사실을 알릴 정도로 민망을 줄 정도(신입이 이 글을 본다면)로 신입이 잘못했단 생각이 들지 않습니다. 

    우선적으로 위에서 다른분이 언급했던것처럼 신입에게 마스터 브랜치를 맡긴것도 이해가 가지 않고, 사이트에 대한 자세한 설명, 예를들어 개발중이신 사이트가 ajax같은 게 아닌 iframe으로 개발해야 한다고 개발전에 먼저 일러주셨다는 내용도 없고 다짜고짜 맡겼더니 ajax가 떡칠되었네요, 하면서 오직 신입만 잘못했다는 식으로 얘기하시고 ㅋㅋ,ㄷㄷ 이런식으로 표현하시는건 조롱글마냥 좀 아니지 않나 생각이 드네요.

    후임도 후임 나름대로 열심히 했던것 같은데, 이런식으로 조롱글로 만드는 사수도 문제가 있는것 같습니다.

  • 콘푸로스트
    1k
    2020-01-03 09:59:14

    신입이 훌륭하네요.

    iframe을 뜯어내고 자기 주도적으로 작업하네요.

  • 가을
    1k
    2020-01-03 10:16:38 작성 2020-01-03 10:17:30 수정됨

    iframe을 사용하면 안된다는걸 알고있는 고수신입의 의도된 신의한수이지도요...

  • M마이클
    24
    2020-01-03 10:28:44

    ajax가 나쁜건 아닐텐데....

    그나저나 iframe 오랜만에 들어보는군요.

    안쓴지 6~7년 된것 같다는..

  • 찐한농도
    988
    2020-01-03 11:03:50

    근데 .., 코드짜고 에러 없으면 푸시하지 않나요..? ㅋㅋ 

  • rezigrene
    1k
    2020-01-03 11:32:36

    방향은 나쁘지 않아보이는데 분석이 덜된상태에서 너무많이 바꾼모양이네요..


    //퓨리오사

    아마 css규칙을 공유하면서 생길문제일 가능성이 높겠죠?

  • onimusha
    8k
    2020-01-03 12:39:45

    하나부터 열까지 알아서 척척척 해내는 슈퍼 루키 신입이면 좋겠지만,

    적어도 시스템의 코딩 표준은 아무리 날고 기는 신입이 들어왔어도 스스로 알아차리길 기대하기보다는 반드시 "떠먹여" 줘야 할 부분이라고 생각합니다.

  • TotalBox
    138
    2020-01-03 13:04:04

    신입 조롱글 같네요... 좋은 사수 였다면 업무 중에도 관심을 주었어야줘...

    하물며 잘되가? 뭐 안되? 이렇게요... 

    과연 글쓰신 분의 글 목적 무엇인지 궁금하네요...

  • 자라선
    1k
    2020-01-03 13:12:07 작성 2020-01-03 13:14:27 수정됨

    일반 개발자가 마스터 브랜치의 권한을 갖나요?

    저도 신입이지만 새로운 방향으로 개발하는 의도자체는 좋지만

    기존의 베이스를 싸그리 무시하고 다른 개발자의 의사를 묻지않은체 진행한건

    잘못된것같네요.

    기존에 iframe으로 되어있으면 왜? 굳이 ajax를 사용하지 않고 iframe으로 구성한거지? 에 대한 의문도 갖지 않은게 좀 그렇네요.

    오히려 사수님이 성격이 좋으신것 같습니다.

  • staticVar
    513
    2020-01-03 13:58:08

    신입이 자기나름 실력있다고 생각하고 자신의 실력을 과시하려고? iframe은 무조건 나쁘다고만 배워서? 

    적어도 본인이 작업할 수정사항에 대해 선임에게 보고는 해야죠.

    크리티컬한 문제인지 간단한 문제인지도 수정사항에대한 side effect도 파악못하고 10시간동안 멋대로 수정해놓고 개판5분전으로 만들어논 것이면 개념이 없는거죠.

    보고도 하지않고 공통과 css 마음대로 막 손대다가 장애 발생해서 되지게 욕얻어 먹고 시말서 써야 정신차릴지도 모르겠네여


  • mirheeoj
    11k
    2020-01-03 14:49:51

    이건 개발자를 탓하기보단 아키텍트 공유, 코드리뷰, 스테이징 등의 시스템을 도입하는게 맞는 방향으로 보입니다. 지적 내용은 모두 타당합니다만, 미리 걸러질 수 있는 기회를 확보해야 된다는 것. (제가 적은 것은 여러 방법론중 하나일 뿐이고, 회사에 맞는 모델이 분명 따로 존재할 거예요.) 그래야 다음에 새로운 인력이 등장했을 경우 동일한 일이 발생하지 않겠지요. 그 하룻동안에 회사에 큰 문제나 손해가 발생했을 수도 있지 않겠습니까? 지금의 기회를 잘 활용해 미래의 문제를 미리 막을 수 있다면 회사에도 큰 공헌을 하시는 거지요. 

    물론 이미 공유된 아키텍트를 무시하고, 코딩 스탠다드를 무시하고, 코드 리뷰에서 나온 지적도 무시하고 커밋을 한다면 그것은 확실히 개인의 문제가 되겠지요. 

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