[Feat/NST-13] 그룹상세화면 구현 #33

oyslucy · 2025-02-03T16:32:01Z

🔥 Issue

close #30

🔥 변경된 내용

Font 크기 변경 -> 했는데도 또 변경된 부분이 많은 것 같아서 갈아엎는게 좋을듯햅니다

🔥 PR Point

기존에 없었던 변수 추가
- Schedule은 주로 약속생성에, ExtendedSchedule은 주로 약속 확정에 쓰이는 구조체입니다

struct Schedule {
    let selectionDates: [Date]
    var selectionStartDate: Date? {
        return selectionDates.sorted().first
    }
    var selectionEndDate: Date? {
        return selectionDates.sorted().last
    }
    let selectionStartTime: Date
}

struct ExtendedSchedule {
    ///약속 날짜(1순위, 확정)
    let date: String
    ///약속 시작시각(1순위, 확정)
    let startTime: String
    ///약속 종료시각(1순위, 확정)
    let endTime: String
    ///그룹 총인원
    var groupMemberCount: Int
    ///가능한 인원
    var availableMemberCount: Int
}

GroupDetailViewController는 GroupDetailReactor, InProgressCVC는 InProgressReactor, ConfirmedCVC는 ConfirmedReactor에서 관리해줍니다. 따로 셀의 reactor를 만든 이유는 cell.reactor로 바로 바인딩하기 위해서입니당. GroupDetailReactor 로직 중심으로 확인해주시면 감사하겠습니당
Floating Btn과 Navigationbar, Tabbar는 추후 추가될 예정이예용

FpRaArNkK

캬 좋네요 야무지십니다 👍👍
멘트 몇 개 남겨놓았으니 확인해주시고, 트슈 기다릴게요~ ^^7

FpRaArNkK · 2025-02-03T17:38:53Z

Noostak_iOS/Noostak_iOS/Global/Extension/UIFont+.swift

이거 폰트 시스템은 언제 한번 날잡아서 기강 잡아야겠네요..

FpRaArNkK · 2025-02-03T17:40:38Z

Noostak_iOS/Noostak_iOS/Global/Resources/Assets.xcassets/ic_vector.imageset/Contents.json

vector는 너무 포괄적인 이름 같아요! ic_arrow_right 같은 네이밍도 괜찮지 않을까요?

FpRaArNkK · 2025-02-03T17:45:58Z

Noostak_iOS/Noostak_iOS/Global/Utils/NSTDateUtility.swift

+    ///타임테이블 뷰 : "요일 월/일"
    static func dateList(_ dateStrings: [String]) -> [String] {


NSTDateUtility는 모든 개발자들이 사용하는 유틸리티일거에요.
본 유틸리티에서 static func으로 추가하여 사용한다면, 불특정 다수의 개발자가 해당 함수를 사용할 수 있겠죠?

다른 영역에서도 넓게 쓰일 수 있도록 함수 이름과 사용 예시를 명확히 정해주세요
or

사용하는 뷰 내부에서 private하게 자체적으로 만들어 써도 될 것 같아요

둘 중 하나로 골라서 진행해보시면 좋을 것 같아요!

오우 이거 수정한다는걸 깜먹고 올렸네요 수정하고 올리겠슴당

FpRaArNkK · 2025-02-03T17:46:54Z

Noostak_iOS/Noostak_iOS/Global/Utils/NSTDateUtility.swift

+        // 날짜 포맷: "9월 7일 (일)"
+        dateFormatter.dateFormat = "M월 d일 (E)"
+
+        // 시간 포맷: "10:00"
+        let timeFormatter = DateFormatter()
+        timeFormatter.locale = Locale(identifier: "ko_KR")
+        timeFormatter.timeZone = TimeZone(identifier: "Asia/Seoul")
+        timeFormatter.dateFormat = "HH:mm"


해당 코드를 막기 위해 본 NSTDateUtility 코드가 존재한답니다!
해당 코드 문서와 PR 한번 읽고오시면 도움이 될거에요 :)

FpRaArNkK · 2025-02-03T17:51:09Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/Cell/ConfirmedCVC.swift

CollectionViewCell은 작성하실 때 prepareForReuse가 필요한 부분은 없는지 한 번씩 확인해보세요!
해당 코드에서 disposeBag은 사용되지 않아 지워도 되겠지만, 만약 실제 Rx subscribe나 bind가 되어 있는경우 PrepareForReuse에서 disposeBag을 초기화해주세요.

FpRaArNkK · 2025-02-03T17:56:25Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/Cell/ConfirmedCVC.swift

+    func bind(reactor: ConfirmedCellReactor) {
+        let schedule = reactor.currentState.schedule
+        chip.backgroundColor = schedule.schedule.category.displayColor
+        scheduleTitleLabel.text = schedule.schedule.name
+        timeLabel.text = NSTDateUtility.durationList(schedule.startTime, schedule.endTime)
+    }


rx 바인드하는 내용은 없으니 이름을 바꿔도 괜찮지 않을까요?!

뷰컨트롤러에서 self.reactor=reactor 로 할당하기 위해 bind 메소드를 썼습미다

FpRaArNkK · 2025-02-03T17:57:12Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/GroupDetailView.swift

+        let inProgressLayout = UICollectionViewFlowLayout()
+        let confirmedLayout = UICollectionViewFlowLayout()
+        self.inProgressCollectionView = UICollectionView(frame: .zero, collectionViewLayout: inProgressLayout)
+        self.confirmedCollectionView = UICollectionView(frame: .zero, collectionViewLayout: confirmedLayout)


트러블 슈팅 기다리고 있겠습니다 ^^7

2차스프린트 전에는..

FpRaArNkK · 2025-02-03T17:57:31Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/GroupDetailView.swift

+    // MARK: setUpUI
+    private func setUpUI() {
+        profileImageView.do {
+            $0.layer.cornerRadius = 7.14


이거 진짜에요?

FpRaArNkK · 2025-02-03T18:02:48Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailViewController.swift

+                if index == 0 {
+                    return Observable.concat([
+                        .just(.selectSegment(index)),
+                        .just(.loadInProgressData)
+                    ])
+                } else {
+                    return Observable.concat([
+                        .just(.selectSegment(index)),
+                        .just(.loadConfirmedData)
+                    ])
+                }


해당 부분은 삼항연산자로 묶어주면 3줄로 축약해서 가독성을 높일 수 있을 것 같네요!

FpRaArNkK · 2025-02-03T18:03:10Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailViewController.swift

+    // MARK: - Bind Reactor
+    func bind(reactor: GroupDetailReactor) {
+        rootView.segmentedControl.rx.selectedSegmentIndex
+            .distinctUntilChanged()


thingineeer · 2025-02-03T18:24:28Z

Noostak_iOS/Noostak_iOS/Global/Utils/NSTDateUtility.swift

+        // 날짜 포맷: "9월 7일 (일)"
+        dateFormatter.dateFormat = "M월 d일 (E)"


NSTDateUtility에 이 포맷 case로 추가 하면 돼요!

thingineeer · 2025-02-04T12:26:06Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/GroupDetailView.swift

+        let inProgressLayout = UICollectionViewFlowLayout()
+        let confirmedLayout = UICollectionViewFlowLayout()
+        self.inProgressCollectionView = UICollectionView(frame: .zero, collectionViewLayout: inProgressLayout)
+        self.confirmedCollectionView = UICollectionView(frame: .zero, collectionViewLayout: confirmedLayout)


이것도 함수로 묶어서 정리하면 좋을 거 같아요

thingineeer · 2025-02-04T12:27:32Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailReactor.swift

+import RxSwift
+import UIKit
+
+final class GroupDetailReactor: Reactor {


Reactor 는 Reactor 폴더로 따로 빼면 좋을 것 같습니다~

thingineeer · 2025-02-04T12:29:26Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailViewController.swift

+    }
+}
+
+extension GroupDetailViewController: UICollectionViewDelegateFlowLayout {


마크 주석을 잘 사용하시는거 같습니다. 여기도 부탁 드려요!

thingineeer · 2025-02-04T12:30:54Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailViewController.swift

+            .bind(to: rootView.inProgressCollectionView.rx.items(cellIdentifier: InProgressCVC.identifier, cellType: InProgressCVC.self)) { index, reactor, cell in
+                cell.reactor = reactor
+            }
+            .disposed(by: disposeBag)
+
+        // 확정된 약속 바인딩
+        reactor.state.map { $0.confirmedCellReactors }
+            .do(onNext: { _ in self.rootView.confirmedCollectionView.reloadData() })
+            .bind(to: rootView.confirmedCollectionView.rx.items(cellIdentifier: ConfirmedCVC.identifier, cellType: ConfirmedCVC.self)) { index, reactor, cell in


index 사용 안 한다면 와일드카드 패턴을 사용하면 좋을 것 같습니다.

thingineeer · 2025-02-04T12:31:21Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/ViewController/GroupDetailViewController.swift

+        DispatchQueue.main.async {
+            self.rootView.layoutIfNeeded()
+        }


여긴 꼭 필요한 Task 인가요? 꼭 필요하다면 왜 꼭 필요한가요??

thingineeer · 2025-02-04T12:35:34Z

Noostak_iOS/Noostak_iOS/Presentation/GroupDetail/View/Cell/InProgressCVC.swift

+import Then
+import ReactorKit
+
+final class InProgressCVC: UICollectionViewCell, View {


reactor를 매개변수로 받아서만 사용하는 것 같아요! View 프로토콜을 confirm안하면 bind 함수가 작동이 안되나요??

네 bind(reactor:)는 리액터 내장함수입니당

oyslucy added 2 commits February 4, 2025 01:22

[Feat/#30] 약속상세뷰.....

5466776

[Feat/#30] 원본

9e7e2f4

oyslucy requested review from thingineeer and FpRaArNkK February 3, 2025 16:32

oyslucy self-assigned this Feb 3, 2025

[Feat/#30] 디테일 수정 및 불필요한 코드 삭제

d6240f1

oyslucy added the Feat 새로운 기능 구현 label Feb 3, 2025

FpRaArNkK reviewed Feb 3, 2025

View reviewed changes

thingineeer approved these changes Feb 4, 2025

View reviewed changes

[Feat/#30] PR 반영

1fcba75

oyslucy merged commit 36ff4f5 into main Feb 8, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feat/NST-13] 그룹상세화면 구현 #33

[Feat/NST-13] 그룹상세화면 구현 #33

oyslucy commented Feb 3, 2025 •

edited

Loading

FpRaArNkK left a comment

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

oyslucy Feb 3, 2025

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

oyslucy Feb 8, 2025

FpRaArNkK Feb 3, 2025

oyslucy Feb 3, 2025

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

FpRaArNkK Feb 3, 2025

thingineeer Feb 3, 2025

thingineeer Feb 4, 2025

thingineeer Feb 4, 2025

thingineeer Feb 4, 2025

thingineeer Feb 4, 2025

thingineeer Feb 4, 2025

thingineeer Feb 4, 2025

oyslucy Feb 8, 2025

		///타임테이블 뷰 : "요일 월/일"
		static func dateList(_ dateStrings: [String]) -> [String] {

		// 날짜 포맷: "9월 7일 (일)"
		dateFormatter.dateFormat = "M월 d일 (E)"

[Feat/NST-13] 그룹상세화면 구현 #33

[Feat/NST-13] 그룹상세화면 구현 #33

Conversation

oyslucy commented Feb 3, 2025 • edited Loading

🔥 Issue

🔥 변경된 내용

🔥 PR Point

FpRaArNkK left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

oyslucy commented Feb 3, 2025 •

edited

Loading