Chore/#143 Design QA(ticket + Reservation)

[longlivedrgn]

작업한 내용

• 티켓 디테일 UI를 수정했어요.(Design QA에 나와있는 내용을 수정했어요!)
• 환불 하기 쪽 로직을 수정했어요.
  • 은행을 선택하고 돌아온 다음 다시 선택할 때 이전의 선택 사항을 유지할 수 있게 구현했어요.
  • 환불 계좌 유효성 검증 로직 변경 및 환불 계좌 확인 뷰의 UI를 수정했어요.

(중간에 Develop을 머지했는데, 변경된 파일이 현재 develop 이전 버전과 비교해서 체킹을 하는 거 같아요!.. 그래서 변경된 파일 보기 쫌 힘들 거 같은데.. 혹시 문제가 있으면 알려주시면 바로 달려가겠씁니다!!..)

스크린샷

TicketDetail	TicketReservation

관련 이슈

• Resolved: [CHORE] 디자인 QA 반영(TicketDetail-TicketList) #143

[wngus4296]

용재!! 생각보다 체크할 게 많아서 힘들었을텐데 수정하느라 고생많았어~~!!
코멘트 확인부탁해 🫠

[wngus4296]

요기는 이렇게 적어보는 건 어떨까?!

Suggested change

	let index = BankEntity.all.firstIndex { $0 == selectedBank }
	guard let index else { return }
	guard let index = BankEntity.all.firstIndex(where: { $0 == selectedBank }) else { return }

[longlivedrgn]

오오 원래 형태에서는 트레일링 클로져를 써서 바로 옵셔널 바인딩 안되더라구! 그냥 안쓰고 작성하면 되는구나! 고마오 주현🙇‍♂️

[wngus4296]

이거 todo인건가?!

[longlivedrgn]

움 뷰모델을 만들까 말까 고민할 때 써 놓은 건데! 내가 정리하려구 써놨어!~ push할 때는 제거할겡😆

[wngus4296]

현재 은행 선택 뷰에서 cell을 두 번 눌렀을 때 isSelected가 풀리는 방식이 아니기 때문에
여기 selectedItemIndex가 없는 경우 nil을 다시 보내주지 않아도 될 것 같아!

Suggested change

	if let index = owner.selectedItemIndex {
	owner.selectedItem?(BankEntity.all[index])
	} else {
	owner.selectedItem?(nil)
	}
	guard let index = owner.selectedItemIndex else { return }
	owner.selectedItem?(BankEntity.all[index])

[longlivedrgn]

웅응!! 생각해보니까 어차피 클로져에 input 값을 안 넣어주면 실행안되니까 굳이 Optional로 처리하지 않아도 될거같아서 코드 바꿨오!! 땡큐 주현😊

[wngus4296]

이거 취소/환불규정 - 환불 요청하기 사이 blank 때문에 넣었던 거 같은데 지금 환불 요청하기 버튼 아래에 들어가고 있더라구..!
214번 줄 <-> 215번 줄 이렇게 변경해도 되구

아니면 blankView를 삭제하고 아래 코드를 넣어줘도 될 거 같아!

self.contentStackView.setCustomSpacing(40, after: self.reversalPolicyView)

그리고 추가로 33번줄에 현재 이렇게 넣어져있는데

    private lazy var contentStackView: UIStackView = {
        let stackView = UIStackView()
        stackView.axis = .vertical
        stackView.distribution = .fill
        stackView.alignment = .center
        stackView.spacing = 12
        
        stackView.setCustomSpacing(0, after: self.concertInformationView) // 여기 제대로 작동 안됨!

        return stackView
    }()

아직 addArrangedSubViews가 되지 않은 상황이라 주석 달아놓은 부분이 작동이 안되는 오류가 있어서

1. lazy var -> let으로 변경해주고
2. addArrangedSubViews함수에 아래 코드 넣어주기
   로 수정하면 잘 작동하는 거 같아~!!

        self.contentStackView.setCustomSpacing(0, after: self.concertInformationView)
        self.contentStackView.setCustomSpacing(40, after: self.reversalPolicyView)

[longlivedrgn]

주현! setCumstomSpacing은 맨 마지막 view에대해서는 spacing을 못 넣어주는 거 같더라구?! 그래서 일부로 blankView를 구현했오! (공연 예매 마감 일에 따라서 button이 hidden처리가 될 때가 있어서)
혹시 더 좋은 방법이 있을까?!

[wngus4296]

아 버튼 hidden 처리 때문이었구나!!
configureRefundButton 함수 안에서 환불하기 버튼 숨김 처리에 따라 stackview에 inset을 주는 방법이 있을 것 같긴한데 ..
뭐가 더 좋은 방법일지는 고민이되네 ..!

용재가 보고 더 편한 쪽으로 가는 게 좋을 듯!!

private func configureRefundButton(with entity: TicketReservationDetailEntity) {
    switch entity.reservationStatus {
    case .reservationCompleted:
        if Date() <= entity.salesEndTime.formatToDate() {
            self.requestRefundButton.isHidden = false
        } else {
            self.requestRefundButton.isHidden = true
        }
    default:
        self.requestRefundButton.isHidden = true
    }
    
    // 여기!! addArrangedSubViews함수 안에서하면 hidden 값을 못가져오는 이슈가 있음 ..!!
    if self.requestRefundButton.isHidden {
        self.contentStackView.isLayoutMarginsRelativeArrangement = true
        self.contentStackView.directionalLayoutMargins = NSDirectionalEdgeInsets(top: 0, leading: 0, bottom: 40, trailing: 0)
    } else {
        self.contentStackView.setCustomSpacing(40, after: self.reversalPolicyView)
    }
}

아 그리구 concertInformationView 에서 spacing 뺀 높이를 height으로 지정해줬더라구!
그래서 아래 코드는 빼도 될 듯!!

self.contentStackView.setCustomSpacing(0, after: self.concertInformationView)