N Tech Service 인턴십 1주차 회고

N Tech Service(이하 NTS) SW개발분야 인턴십을 1주일간 진행하며 느낀점과 배운점.

인턴십 진행방식

NTS SW개발 분야 인턴십은 부스트코스 강의를 수강하고, 미션을 제출하면 멘토들이 리뷰해주는 방식으로 진행된다. 대부분 인턴 4명당 멘토 2명이 배정되었고, 우리조는 인턴2명에 멘토 2명이다.

부스트코스

부스트코스는 자바기반 풀스택 과정이다. 온라인 강의는 윤지수님과 강경미님께서 해주신다. 나는 Node.Js 중심으로 학습 해왔기에 새로 배울 내용이 많지만, 코스 진행방식이 내가 학습했던 코드스쿼드와 비슷하기 때문에 어렵지 않게 따라갈 수 있을것 같다.(윤지수님은 코드스쿼드에서 웹프론트엔드 마스터로 강의하고 계신다.)

아쉬웠던 점

코드의 의미파악 X

가장 아쉬운 부분이다. 정확한 내용은 모르고 돌아가는 코드를 만들었다. IDE가 자동 생성해준 코드를 그대로 사용했고, 정확한 동작방식을 모르는채 코드를 작성했다. Java 학습이 부족한상태에서 미션 제출에 급급해서 하면 안되는 실수를했다. 큰 문제되는 코드는 없었지만, 미션 제출보다는 학습에 중점을 두자.

내 코가 석자

인턴을 하면서 같은 인턴들과 모르는 내용을 공유하고 내가 잘 아는 부분을 나누고 싶었다. 하지만 내 코가 석자다. 수많은 강의를 듣고 미션을 진행할 시간도 부족하다. 강의는 주말이나 회사 오며가며 수강하고, 회사에서는 미션과 커뮤니케이션에 집중하도록하자.

같은 실수 반복

이전 리뷰에서 받은내용을 똑같이 실수했다. a태그의 외부링크에는 target:_blank 속성을 주라는 내용이였는데 footer부분만 수정하고 body에 들어가는 내용을 수정하지 않은채 PR을 작성했다. 급하게 하지말고 차근차근 꼼꼼히 읽어보고 제출하자.

알고 있는내용 실수

매직 스트링을 사용했다. 코드 품질을 너무 신경쓰지 않은것 같다. 정신차리자.

배운 점

Git branch 전략

Git branch 전략 중 하나인 git flow에 대해서 배웠다. 팀프로젝트를 진행하면서 비슷한 전략을 사용하고 있긴 했다. issue 별로 브랜치를 생성해서 해당 기능을 구현 완료하면 dev 브랜치에 merge 하는 방식으로. 여기에 추가로 네이밍 방식을 배웠다. 기능을 구현하는 브랜치라면 feature/do_something 같은식으로 브랜치를 생성하면 가독성도 좋고 관리하기도 좋다. 이 방식으로 브랜치를 생성하면 feat브랜치는 feature 폴더 내부에 생성이된다.(.git 폴더에서 확인 가능하다.)

Script 태그 위치 및 외부파일 참조

HTML 에서 Script 태그 위치는 body 태그가 끝나기 전에 위치 하는게 좋다. HTML이 전부 생성 된 뒤에 script 파일을 다운로드 하고 동작하는게 유저 입장에서 로딩속도를 빠르게 할 수 있다.

외부파일 참조는 지양하도록 하자. 외부링크가 손상되어도 내 서비스는 제대로 동작하게 하기 위해서 내부파일을 참조하는게 좋다.

HTML의 Data attribute

id 나 class 를 사용하면 마크업부서와 협업시 문제가 발생할 수 있다. 따라서 스크립트에서만 사용하기 위한 selector를 별도로 두는것이 좋다.

코드리뷰 내용

Java

- 메소드 선언 순서, 호출 순서와 매칭

- servlet의 printwriter는 close 할 필요 없다. servlet이 종료 되면서 writer 를 종료 하기 때문에 개발자가 해줄 필요 없다. <https://stackoverflow.com/questions/1159168/should-one-call-close-on-httpservletresponse-getoutputstream-getwriter> 참고

- 클래스와 메소드 주석 잊지 말자.

- 매직 스트링, 넘버 사용하지 말자. 상수로 등록하자.

HTML/CSS

- div 등 태그 를 이용해서 요소 그룹화

- header, footer 중복요소 제거

- a태그를 사용할만한 상황이라면 button 대신 a태그 사용

- CSS파일을 페이지별로 분할 할 필요는 없음

- 많아야 두개(common, project_name)

- float 해제방식 : 해제하는 class하나 지정

- 외부 링크는 새창에서 열리도록 target 속성 부여-

- 외부 참조파일 대신 내부 파일 사용하도록

Git

- git-flow 도입