Feature/settings view

[jinwoong16]

📕 Issue Number

Close #49

📙 작업 내역

구현 내용 및 작업 했던 내역

• 메인을 TableView로 구성
• 텍스트만 표시해주는 필드 추가
• 클릭시, 화면이동이 가능한 필드 추가
• 토글 스위치를 가진 필드 추가

📘 작업 유형

• 신규 기능 추가
• 버그 수정
• 리펙토링
• 문서 업데이트

📋 체크리스트

• Merge 하는 브랜치가 올바른가?
• 코딩컨벤션을 준수하는가?
• PR과 관련없는 변경사항이 없는가?
• 내 코드에 대한 자기 검토가 되었는가?
• 변경사항이 효과적이거나 동작이 작동한다는 것을 보증하는 테스트를 추가하였는가?
• 새로운 테스트와 기존의 테스트가 변경사항에 대해 만족하는가?
  
  

📝 PR 특이 사항

PR을 볼 때 주의깊게 봐야하거나 말하고 싶은 점

• 현재는 ViewController에 있는 fields 프로퍼티를 통해서 델리게이트 메서드를 실행하는데, 나중에 fields부분을 따로 ViewModel 파트를 만들어서 처리해야할 것 같습니다. 로그인과 로그아웃은 상태에 따라 텍스트가 변경되어야 하고, 토글 역시 누를때마다 상태를 변경해야할테니까요.

final class SettingsViewController: UITableViewController {
    
    private let fields: [CommonField]  = [
        ToggleField(viewModel: .init(title: "둘러보기 허용", imageName: "person.crop.circle.badge.checkmark")),
        PlainField(viewModel: .init(title: "some", info: "some...?", imageName: "pencil")),
        NavigateField(viewModel: .init(title: "로그인", imageName: "person.circle.fill")),
        PlainField(viewModel: .init(title: "앱 버전", info: "0.0.1", imageName: "exclamationmark.transmission"))
    ]
    
    // MARK: - Life Cycle
    override func viewDidLoad() {
        super.viewDidLoad()
        tableView.separatorStyle = .singleLine
        
        register()
    }
    
    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return fields.count
    }
    
    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let field = fields[indexPath.row]
        return field.dequeue(for: tableView, at: indexPath)
    }
    
    private func register() {
        fields.forEach { field in
            field.register(for: self.tableView)
        }
    }
}

• 누락된 커밋은 StatusView의 PR이 제대로 안들어가서 추가했습니다.

[wickedRun]

오우 뚝딱뚝딱 야무지십니다. 고생하셨읍니다.

[wickedRun]

음 field가 동적으로 추가될 일이 있나요?

[jinwoong16]

viewdidload에서 fields에 초기값이 들어갑니다. 동적으로 추가될 일은 없지만, 프로피터 옵저버로 처리하면 view did load 메서드에서 따로 다른 메서드를 호출할 필요가 없어져서 저렇게 구현했어요 :)

[wickedRun]

음 저는 고정적인 값이 var로 선언되는 것이 좀 그렇기도 하고 제 생각으론 약간 쓸모없는 함수가 만들어졌다고도 생각을 해서 viewDidLoad에서 한번만 호출해주는게 좋을 것 같습니당

[jinwoong16]

피드백 반영했습니다! 이제 아래의 메서드를 호출해서 처리하는 코드로 바꿨습니다 :)

private func register() {
        fields.forEach { field in
            field.register(for: self.tableView)
        }
}

[Jeonhui]

고생 많으셨습니다! 혹시 Toggle같은 경우는 셀 내부에서 들고 있는데 Settings View Controller에서 어떤 식으로 처리하실 건지 여쭤봐도 될까요?

[jinwoong16]

고생 많으셨습니다! 혹시 Toggle같은 경우는 셀 내부에서 들고 있는데 Settings View Controller에서 어떤 식으로 처리하실 건지 여쭤봐도 될까요?

지금은 구조체로 되어있는 ToggleItemViewModel을 통해서 처리할 생각입니다. 그때가 되면 class로 바꿔서 처리해야할 것 같아요.
또한, 둘러보기 허용의 여부를 어디에 저장하고 불러올지를 결정을 못해서 지금의 상태가 되었습니다 :)

[Jeonhui]

ViewModel을 이용하면 조금더 데이터 관리하기가 편해지겠네요! 좋습니다!
저는 둘러보기 허용 / 거부도 사용자의 정보로 볼 수 있을 것 같아서 둘러보기 허용 여부도 UserInfo에 같이 저장을 하는 것이 좋을 것 같다고 생각을 합니다. 내일 다른 분들 의견도 들어보면 좋을 것 같아요

[sprituz]

벌써 설정뷰가 구성이 됐었네요 고생하셨습니다!!