[Fix] #289 - 2차 스프린트 기능 QA 반영

[EunsuSeo01]

🔥 Pull requests

👷 작업한 내용

• 미완성이었던 메뉴 추가 기능을 완성했습니다.
• 1차 및 2차 기능 QA를 반영했습니다.
• ModifyMenuTextField 컴포넌트 로직 개선을 진행했습니다.

🖥️ 주요 코드 설명

ModifyMenuTextField 이벤트에 따른 스타일 활성/비활성 로직 개선

• 메뉴 수정 텍스트필드가 가질 수 있는 스타일은 총 4개로, enum을 통해 구분했습니다.

ModifyMenuTextFieldType.swift

enum ModifyMenuTextFieldType {
    case focused
    case unfocused
    case focusedWithError
    case unfocusedWithError
}

• 각 type에 따라 함수를 다르게 호출함으로써 다른 스타일을 적용하고자 했습니다.

ModifyMenuTextField.swift

func updateStyle(type: ModifyMenuTextFieldType) {
    switch type {
    case .focused:
        updateStyleBy(isFocus: true)
        toggleAccessoryViewVisibility(isDeleteHidden: true)
    case .unfocused:
        updateStyleBy(isFocus: false)
    case .focusedWithError:
        updateStyleBy(isFocus: true)
        updateErrorStyleBy(isFocus: true)
        toggleAccessoryViewVisibility(isDeleteHidden: false)
    case .unfocusedWithError:
        updateStyleBy(isFocus: false)
        updateErrorStyleBy(isFocus: false)
    }
}

메뉴 추가/수정/삭제 완료 후, 식당 상세 페이지의 메뉴 목록 or 메뉴 편집의 메뉴 목록 내용 업데이트

• 메뉴 추가/수정/삭제 완료 화면에서 버튼 클릭 시 postNotification 함수를 실행하여 식당 상세 화면 또는 메뉴 편집 화면의 메뉴 목록을 업데이트 하도록 구현했습니다.

MenuCompleteView.swift

private extension MenuCompleteView {
    
    func postNotification() {
        NotificationCenter.default.post(Notification(name: NSNotification.Name(StringLiterals.NotificationName.reloadHankkiDetail)))
    }
    
    @objc func doThisAgainButtonDidTap() {
        postNotification()
        doThisAgainButtonAction?()
    }
}

메뉴 모델 변환 로직 작성

• 메뉴 관련 모델들이 형태가 조금씩 다릅니다. 예를 들어 요청으로 보낼 때는 id 값이 없어야 하지만, 응답에서는 id 값이 있다든가.
• 그래서 데이터 간의 형태 변환이 필요했고, 이를 MenuData의 extension에 함수로 작성했습니다.

GetHankkiDetailResponseDTO.swift

struct MenuData: Codable {
    var id: Int = 0
    var name: String = ""
    var price: Int = 0
}

extension MenuData {
    func toMenuRequestDTO() -> MenuRequestDTO {
        return MenuRequestDTO(
            name: self.name,
            price: self.price
        )
    }
}

✅ Check List

• Merge 대상 브랜치가 올바른가?
• 최종 코드가 에러 없이 잘 동작하는가?
• 전체 변경사항이 500줄을 넘지 않는가?

📟 관련 이슈

• Resolved: [Fix] 2차 스프린트 기능 QA 반영 #289

[shimseohyun]

많은 변경사항에 군말 없이 묵묵하게 해내어주는 모습에 오늘도 감동...
어서 나도 1인분을 하는 날이 오길 바라며...
approve를 드립니다

[shimseohyun]

멋진 식이에요

[shimseohyun]

이것또한 깔끔하고 좋네요