chessta0126 / festivalBooking

0 stars 0 forks source link

기능 개선 , 유지 / 보수할 사항 #9

Open chessta0126 opened 1 year ago

chessta0126 commented 1 year ago

로그인

공연 등록 관련

공연 예매 관련

예매 확인 관련

Community 관련

관리자

추가 서비스

부실한 기능

chessta0126 commented 1 year ago

aws로 하면 공연 시간 설정 오류 있음 (시작 시간 < 마치는 시간 인데도 넘어간다고 나옴) -> 10시 이상으로 설정하면 그러는데, compareTo로 비교해서 앞자리가 1이라서 그런 것 같다.

chessta0126 commented 1 year ago

공연 시작 시간이 1자리 수일 때 2자리 수보다 크게 나오는 문제

ex ) 3시가 11시보다 크다고 나와서 에러 메시지가 이상하게 적용되었다.

앞에 있는 숫자부터 단순 비교하기 때문에 3>1로 계산된 것. -> 한 자리 수일 경우에 앞에 0을 추가하여 해결했다.

if(startTimeHour.length == 1){
    startTimeHour = "0" + startTimeHour;
}
chessta0126 commented 1 year ago

입력 가능한 공연 시간 통제

0시 ~ 23시까지, 0분~59분까지로 제한

if(startTimeHour < 0 || startTimeMinute < 0
        || startTimeHour > 23 || startTimeMinute > 59 ){
    alert("공연 시작 시간이 잘못 입력되었습니다.");
    return false;
}
chessta0126 commented 1 year ago

전화번호 형식 formatting

->비회원 예매 / 예매 확인 시, 아예 3가지 파트로 나누어 나중에 합치기로 했다.

chessta0126 commented 1 year ago

referrer가 로그인/회원가입 페이지일 경우 메인으로 이동

클릭한 페이지에 접근 권한이 없을 경우 로그인 화면으로 이동하게 해놓았는데 로그인 후 클릭했던 페이지로 이동하게 하기 위해 referrer를 사용했다. -> 그런데 회원가입 후 다시 회원가입 페이지가 나오면 이상하므로 예외적 통제

-> 그러나 전체 주소를 비교하기엔 포트 번호나 IP 주소가 변경될 수 있다.

-> /를 기준으로 split하여 user 관련된 페이지(로그인, 회원가입, 로그아웃)일 경우 메인으로 이동

ex) http://localhost:8080/user/sign_in_view

chessta0126 commented 1 year ago

페이징 다음 오류

<c:if test="${currentPage +1 < needPage}">
    <a href="/post/postList?postType=${postType}&page=${maxPage + 1}">>></a>
</c:if>

-> 이렇게 조건을 걸었더니, 3페이지씩 보인다고 가정했을 때 4페이지가 없는 경우 2페이지에서 다음(>>)을 누르면 4페이지로 이동, 없는 것이 사용자에게 보여져버린다.

<c:if test="${maxPage < needPage}">
    <a href="/post/postList?postType=${postType}&page=${maxPage + 1}">>></a>
</c:if>

-> 보여지고 있는 페이지의 최대페이지(maxPage)가 전체 필요한 페이지 수(needPage)보다 크거나 같으면 다음으로 넘어갈 필요가 없으므로, 조건을 변경한다.

chessta0126 commented 1 year ago

input 입력 비활성화

공연 날짜 입력 시 datepicker에서 지정한 형식으로 통일하기 위해, 입력의 자유도를 제한하고 input창을 비활성화 하기로 했다.

https://kkotkkio.tistory.com/22