220819

🍮 오늘 한 일

파이널 프로젝트 개인 작업 :

• 내가 맡은 부분을 세부기능까지 모두 다 구현했다. 그리고 다른 파트의 진행중인 페이지 하나를 구현했다.
  어제오늘은 자잘한 UI 디벨롭, 세부기능 추가 등의 작업을 했다.
  객실 정보 탭에서는 정원 별 조회, 예약가능한 객실만 조회를 할 수 있게 바꾸었다.
  - 정렬을 정확하게 하자
  sql에서 order by를 쓸 때 컬럼이 unique가 아니라면 같은 값끼리는 랜덤하게 순서가 주어질 수 있다는 것을 잊지말자.
  페이징처리를 할 때 order by 객실정원을 했다가 원하는 대로 출력이 안되서 의아해했는데.. 객실정원이 2인 정보가 여러 개 있기 때문이었다. 이럴 때는 PK라던가 완전히 식별 가능한 컬럼을 추가로 정렬조건에 넣어주자.
  - 조인을 신중하게 하자
  다 완료하고 가뿐한 마음으로 머지까지 했다가, 2개 뿐인 리뷰정보가 400개 쯤 조회되는 걸 보고 식겁했다.
  리뷰 조회 쿼리를 restaurantNo를 넣으면 식당리뷰를, accoId를 넣으면 숙소리뷰를 조회할 수 있게끔 mybatis에서 동적sql문으로 짰는데, restaurant 테이블과 조인하는 내용을 if문에 넣지 않아서 그 테이블의 모든 값과 다른 행들이 전부 조인되어버린 것이었다.
  처음 구현할 때에는 식당리뷰 데이터가 하나도 없었기 때문에 문제를 알아차리지 못했다.
  작업 중 코딩 실수로 인해 발생한 결함 중 가장 큰 일로 느껴졌다.
  - 이름은 중요하다, 아니 애초에 중복을 줄여라
  모든 페이지에서 include해서 사용하는 네비바에도 검색창이 있고, 맛집 검색 페이지에도 내부 검색창이 하나 있다.
  내부 검색창에 구현된 최근검색어 기능을 위한 js코드를 네비바 검색창에도 적용시키기 위해
  공통적으로 include해서 쓰는 nav.jsp의 script태그에 생각없이 복사붙여넣기해서 쓰고 있었다.
  맛집페이지가 내 담당이 아니라 변수, 함수명이 같음으로써 생기는 문제를 뒤늦게야 깨달았다.
  사실 공통적으로 사용하는 js코드들도 별도 파일로 빼주어야 할 텐데... 급한대로 이름만 몽땅 수정했다. ㅜㅜ..
  (지금은 페이지마다 자바스크립트 코드에 중복이 꽤 많다)
  마감하고나서라도 정리해봐야겠다.
  

🥞 내일 할 일

• 잘못된 url입력에 대한 예외 처리
• 마이페이지 찜한숙소 조회 페이지 구현
• 적어도 일주일은 남겨두고 작업을 마친 뒤, 테스트나 더 추가적인 작업들을 해보자고 생각했는데 계획에 거의 근접해서 개인적으로는 만족스럽다. 하지만 담당 외에 현재 미구현 상태인 기능들이 좀 있어서 한편으로는 걱정스럽다. 그걸 이용해서 테스트 해봐야하는 부분도 있고..
  너무 조회 기능 위주로만 한 것 같아 채팅 api나 좀 색다른 걸 더 해보고 싶긴 한데.. 시간도 애매하고
  미구현 기능들 작업을 마저 돕고, 내가 추가적으로 하는 작업도 기본 기능을 보완하는 종류의 것으로 하기로 했다(통합검색 등)
• 여전히 재미있고 몰두하게 되는 반면, 더 깔끔하게 코딩하고 싶다는 생각이 강해진다. 내가 쓴 코드를 보며 아, 이런 게 냄새나는 코드인가.. 생각하게 된다 . . ^_^