[refactor] 약속 추가 플로우 데이터 전달 간소화

[JinUng41]

🔗 연결된 이슈

• Connected: [refactor] 약속 추가 플로우 데이터 전달 방식 간결화 #399

📄 작업 내용

• 연속된 화면 간 데이터 전달을 간소화 하고자 빌더 패턴을 차용하여 약속을 만들기 위해 입력받고 전달되는 내용을 수정하였습니다.

💻 주요 코드 설명

AddPromiseRequestModel 내용에 따른 빌더 객체 구현과 사용

• 여러 화면에서 입력받아야 하는 정보를 보다 간단하게 관리하기 위해서 빌더 패턴을 도입하였습니다.
• 각 단계에서 필요한 내용을 빌더 객체를 통해 구성하고 뷰모델은 빌더 객체를 이용하여 네트워크 통신 시, 필요한 구성물을 만들어 낼 수 있습니다.
• 각 뷰모델의 생성자가 매우 간단하게 바뀌었습니다.

AddPromiseRequestModelBuilder.swift

extension AddPromiseRequestModel {
    final class Builder {
        private(set) var id = 0
        private var name = ""
        private var placeName = ""
        private var address = ""
        private var roadAddress = ""
        private var time = ""
        private var dressUpLevel = ""
        private var penalty = ""
        private var x = 0.0
        private var y = 0.0
        private var participants = [Int]()
        
        @discardableResult
        func setName(_ name: String) -> Self {
            self.name = name
            return self
        }
        
        @discardableResult
        func setPlaceName(_ placeName: String) -> Self {
            self.placeName = placeName
            return self
        }
        
        @discardableResult
        func setAddress(_ address: String) -> Self {
            self.address = address
            return self
        }
        
        @discardableResult
        func setRoadAddress(_ roadAddress: String) -> Self {
            self.roadAddress = roadAddress
            return self
        }
        
        @discardableResult
        func setTime(_ time: String) -> Self {
            self.time = time
            return self
        }
        
        @discardableResult
        func setDressUpLevel(_ dressUpLevel: String) -> Self {
            self.dressUpLevel = dressUpLevel
            return self
        }
        
        @discardableResult
        func setPenalty(_ penalty: String) -> Self {
            self.penalty = penalty
            return self
        }
        
        @discardableResult
        func setCoordinates(x: Double, y: Double) -> Self {
            self.x = x
            self.y = y
            return self
        }
        
        @discardableResult
        func setId(_ id: Int) -> Self {
            self.id = id
            return self
        }
        
        @discardableResult
        func setParticipants(_ participants: [Int]) -> Self {
            self.participants = participants
            return self
        }
        
        func build() -> AddPromiseRequestModel {
            return AddPromiseRequestModel(
                name: name,
                placeName: placeName,
                address: address,
                roadAddress: roadAddress,
                time: time,
                dressUpLevel: dressUpLevel,
                penalty: penalty,
                x: x,
                y: y,
                id: id,
                participants: participants
            )
        }
    }
}

[youz2me]

빌더 패턴은 처음 알았는데 객체 생성 시에 간편하게 활용해볼 수 있겠네요! 저도 약속 수정이나 약속 상세 뷰모델 생성자로 많은 객체를 넘기는 게 힘들었는데 빌더 패턴 도입해보면 좋을 것 같습니다.
고생하셨습니다 🔥

[youz2me]

@discardableResult는 왜 사용하신건지 궁금해요!

[mmaybei]

방금 막 아요 코드리뷰하면서 @discardableResult 보고 똑같이 리뷰 남겼는데 여기 또 있어서 소름

[JinUng41]

함수의 리턴값에 대해서 사용하지 않아도 경고문을 표시하지 않는 @discardableResult 어노테이션을 사용한 이유는 아래의 코드처럼 메서드 체이닝으로 보다 간단하게 사용하면서도 경고문이 표시되지 않게 하기 위함이었습니다.

메서드 체이닝을 하기 위해서는 인스턴스를 메서드의 리턴값으로 내보내야 하는데요.
만약 메서드 체이닝이 아닌 형태로 구현된다면 빌더의 구현이 아래와 같아질 것 입니다.

final class Builder {
    // 생략..

    // 리턴형이 없는 메서드
    func setName(_ name: String) {
       self.name = name
    }
    
    func setPlaceName(_ placeName: String) {
        self.placeName = placeName
    }
}

// 실제 사용 시

let builder = Builder()
builder.setName(name)
builder.setPlaceName(place.location)

return builder

[youz2me]

오 ... 메서드 체이닝을 하려면 무조건 리턴값이 존재해야 하는군요! 좋은 정보 감사합니다 👍

[youz2me]

👍

[youz2me]

너무 간편해졌네요!

[hooni0918]

LGTM

[mmaybei]

빌더 패턴.. 오렵당 수고하셨어유~~

[mmaybei]

방금 막 아요 코드리뷰하면서 @discardableResult 보고 똑같이 리뷰 남겼는데 여기 또 있어서 소름