🔀 :: [#74] 참가자 관리 페이지 퍼블리싱 #75

Xixn2 · 2025-01-12T11:27:35Z

💡 개요

참가자 관리 페이지 퍼블리싱

📃 작업내용

참가자 관리 페이지 퍼블리싱 했습니다.
SceneApplicationInfoListDetail 컴포넌트를 제작 했습니다.
ExpoProgramDetail 컴포넌트를 제작 했습니다.
InfoListView와 SceneApplicationInfoListView를 재 사용 할 수 있도록 Func에서 Struct로 변경 했습니다.

🎸 기타

2025-01-12.8.24.53.mov

Summary by CodeRabbit

새로운 기능
- 사용자 관리 화면 추가
- 프로그램 상세 보기 기능 개선
- 참가자 정보 목록 레이아웃 업데이트
디자인 시스템
- 새로운 아이콘 및 버튼 이미지 추가
- 정보 목록 및 상세 보기 컴포넌트 개선
버그 수정
- 프로그램 상세 보기의 레이아웃 및 스크롤 기능 최적화

Management Top NavigationBar 추가

coderabbitai · 2025-01-12T11:27:41Z

Walkthrough

이 풀 리퀘스트는 앱의 디자인 시스템과 사용자 관리 기능에 대한 여러 변경 사항을 포함하고 있습니다. 새로운 이미지 에셋과 함께 InfoListView, SceneApplicationInfoListView, SceneApplicationInfoListDetail 등의 새로운 SwiftUI 뷰가 추가되었습니다. ExpoProgramDetail 뷰에는 퇴실 시간 속성이 추가되었으며, ProgramDetailView와 UserManagementView의 레이아웃이 업데이트되었습니다.

Changes

파일	변경 요약
`Projects/App/Resources/Assets.xcassets/blackBottomButton.imageset/Contents.json`	새 이미지 에셋 JSON 파일 추가
`Projects/App/Resources/Assets.xcassets/okIcon.imageset/Contents.json`	새 이미지 에셋 JSON 파일 추가
`Projects/App/Sources/DesignSystem/InfoList/InfoListView.swift`	새로운 정보 목록 뷰 생성
`Projects/App/Sources/DesignSystem/InfoList/SceneApplicationInfoListView.swift`	새로운 현장 신청 정보 목록 뷰 생성
`Projects/App/Sources/DesignSystem/Program/ExpoProgramDetail.swift`	퇴실 시간 속성 추가 및 레이아웃 수정
`Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift`	새로운 현장 신청 정보 상세 뷰 생성
`Projects/App/Sources/Feature/ProgramDetailFeature/Sources/ProgramDetailView.swift`	레이아웃 재구성 및 스크롤 뷰 추가
`Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift`	새로운 사용자 관리 뷰 생성

Poem

🐰 코드의 토끼, 새 기능을 뛰어다녀
에셋과 뷰를 깔끔히 정리해
퇴실 시간도 추가하고
스크롤은 부드럽게 흘러가네
개발의 즐거움, 토끼가 노래해! 🚀

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

coderabbitai

Actionable comments posted: 8

🧹 Nitpick comments (11)

Projects/App/Sources/DesignSystem/InfoList/SceneApplicationInfoListView.swift (2)
13-43: 접근성 및 유지보수성 개선이 필요합니다

다음 사항들을 개선하면 좋을 것 같습니다:

고정된 너비값과 간격을 사용하면 디바이스 크기에 따라 레이아웃이 깨질 수 있습니다. 상대적인 크기나 GeometryReader 사용을 고려해보세요.

하드코딩된 문자열은 지역화를 위해 별도 상수로 분리하는 것이 좋습니다.

반복되는 Text 스타일링을 위한 modifier 함수를 만들면 코드 중복을 줄일 수 있습니다.

예시 개선안:
// 문자열 상수
private enum Strings {
    static let name = "성명"
    static let organization = "소속"
    static let position = "직급"
    static let contact = "안내문자 발송용 연락처"
    static let consent = "개인정보동의제공 동의"
}

// 공통 Text 스타일 modifier
extension View {
    func columnHeaderStyle(width: CGFloat) -> some View {
        self
            .frame(width: width, alignment: .leading)
            .expoFont(.caption1B)
            .expoColor(ExpoColor.gray600)
    }
}
11-45: 재사용성 향상을 위한 제안

현재 뷰가 고정된 형태로 구현되어 있어 다른 상황에서 재사용하기 어려울 수 있습니다. 데이터 모델과 설정을 주입받는 형태로 변경하면 좋을 것 같습니다.

예시:
struct ColumnConfig {
    let title: String
    let width: CGFloat
}

struct SceneApplicationInfoListView: View {
    let columns: [ColumnConfig]
    
    var body: some View {
        VStack(spacing: 16) {
            // ... 기존 코드 ...
            HStack(spacing: 45) {
                ForEach(columns, id: \.title) { column in
                    Text(column.title)
                        .columnHeaderStyle(width: column.width)
                }
                Spacer()
            }
            // ... 기존 코드 ...
        }
    }
}
Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift (2)
11-20: 속성들의 접근 제어자를 명시적으로 선언해주세요.

현재 모든 속성들이 암시적 internal 접근 수준을 가지고 있습니다. 외부에서 접근할 필요가 없는 속성들은 private으로 선언하는 것이 좋습니다.
public struct SceneApplicationInfoListDetail: View {
-    var programNum: Int
-    var userName: String
-    var affiliation: String
-    var position: String
-    var phoneNumber: String
-    var consentStatus: Bool
-    var action: () -> Void
+    private let programNum: Int
+    private let userName: String
+    private let affiliation: String
+    private let position: String
+    private let phoneNumber: String
+    private let consentStatus: Bool
+    private let action: () -> Void
80-82: 프리뷰를 현재 뷰에 맞게 수정해주세요.

UserManagementView 대신 SceneApplicationInfoListDetail의 프리뷰를 보여주는 것이 더 적절합니다.
 #Preview {
-    UserManagementView()
+    SceneApplicationInfoListDetail(
+        programNum: 1,
+        userName: "홍길동",
+        affiliation: "회사",
+        position: "개발자",
+        phoneNumber: "01012345678",
+        consentStatus: true
+    )
 }
Projects/App/Sources/Feature/ProgramDetailFeature/Sources/ProgramDetailView.swift (1)

79-89: 하드코딩된 데이터 대신에 모델 사용 권장

ForEach 내부에서 하드코딩된 데이터를 사용하고 있습니다. 실제 데이터 모델을 활용하여 뷰에 데이터를 바인딩하면 유지보수성과 확장성이 향상됩니다.

Projects/App/Sources/DesignSystem/InfoList/InfoListView.swift (1)

18-39: 하드코딩된 프레임 크기 대신 유연한 레이아웃 사용 권장

Text 뷰에 고정된 width와 padding 값을 사용하면 다양한 기기 화면 크기에서 레이아웃 문제가 발생할 수 있습니다. Spacer()나 GeometryReader를 활용하여 동적인 레이아웃을 구성하는 것이 좋습니다.

Projects/App/Sources/DesignSystem/Program/ExpoProgramDetail.swift (1)

51-74: 하드코딩된 프레임 크기 대신 반응형 레이아웃 사용 권장

Text 뷰에 고정된 width를 설정하면 화면 크기에 따라 레이아웃이 깨질 수 있습니다. HStack 내에서 Spacer()를 사용하거나 LazyVGrid 등을 활용하여 다양한 화면 크기에 대응하는 반응형 레이아웃을 구현하는 것을 권장합니다.
Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift (4)
1-10: 문서화 개선 제안

뷰의 목적과 주요 기능을 설명하는 문서 주석을 추가하면 좋겠습니다.

다음과 같이 문서 주석을 추가해보세요:
 //
 //  UserManagementView.swift
 //  Expo-iOS
 //
 //  Created by 서지완 on 1/5/25.
 //  Copyright © 2025 SchoolofCompany. All rights reserved.
 //

+/// `UserManagementView`는 이벤트 참가자 관리를 위한 뷰입니다.
+/// - 사전 행사 참가자, 현장 신청 참가자, 사전 교원 원수 참가자를 구분하여 관리할 수 있습니다.
+/// - 참가자 목록을 확인하고 상세 정보를 볼 수 있습니다.
+
 import SwiftUI
11-15: 상태 변수 이름 개선 및 문서화 제안

상태 변수의 이름을 더 명확하게 하고, 각 변수의 용도를 설명하는 문서 주석을 추가하면 좋겠습니다.

다음과 같이 변경해보세요:
 struct UserManagementView: View {
     @Environment(\.dismiss) var dismiss
-    @State private var userInfoListName: String = "사전 행사"
-    @State private var userInfoListButtonAction: Bool = false
+    /// 선택된 참가자 목록 유형 (사전 행사, 현장 신청, 사전 교원 원수)
+    @State private var selectedParticipantType: String = "사전 행사"
+    /// 참가자 유형 선택 시트 표시 여부
+    @State private var isParticipantTypeSheetPresented: Bool = false
16-32: 네비게이션 구현 개선 제안

현재 ZStack을 사용한 네비게이션 구현보다 SwiftUI의 네비게이션 컴포넌트를 활용하면 더 좋을 것 같습니다.

다음과 같이 NavigationStack을 사용하여 구현해보세요:
-    var body: some View {
-        ZStack {
-            HStack(spacing: 0) {
-                Button {
-                    dismiss()
-                } label: {
-                    ExpoIOSAsset.Assets.leftBackButton.swiftUIImage
-                        .padding(.leading, 4)
-                }
-
-                Spacer()
-            }
-
-            Text("참가자 관리")
-                .expoFont(.body1R)
-        }
+    var body: some View {
+        NavigationStack {
+            VStack {
+                // 기존 컨텐츠
+            }
+            .navigationTitle("참가자 관리")
+            .navigationBarTitleDisplayMode(.center)
+            .navigationBarBackButtonHidden(true)
+            .toolbar {
+                ToolbarItem(placement: .navigationBarLeading) {
+                    Button {
+                        dismiss()
+                    } label: {
+                        ExpoIOSAsset.Assets.leftBackButton.swiftUIImage
+                    }
+                }
+            }
+        }
159-161: 프리뷰 구현 개선 제안

프리뷰에 샘플 데이터와 다양한 상태를 추가하면 개발 시 더 유용할 것 같습니다.

다음과 같이 프리뷰를 개선해보세요:
#Preview {
    Group {
        UserManagementView(viewModel: ParticipantViewModel(participants: PreviewData.sampleParticipants))
            .previewDisplayName("기본 상태")
        
        UserManagementView(viewModel: ParticipantViewModel(participants: []))
            .previewDisplayName("빈 상태")
    }
}

enum PreviewData {
    static let sampleParticipants: [Participant] = [
        // 샘플 데이터 추가
    ]
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 889055f and f9c685f.

⛔ Files ignored due to path filters (2)

Projects/App/Resources/Assets.xcassets/blackBottomButton.imageset/blackBottomButton.svg is excluded by !**/*.svg
Projects/App/Resources/Assets.xcassets/okIcon.imageset/okIcon.svg is excluded by !**/*.svg

📒 Files selected for processing (9)

Projects/App/Resources/Assets.xcassets/blackBottomButton.imageset/Contents.json (1 hunks)
Projects/App/Resources/Assets.xcassets/okIcon.imageset/Contents.json (1 hunks)
Projects/App/Sources/DesignSystem/InfoList/InfoListView.swift (1 hunks)
Projects/App/Sources/DesignSystem/InfoList/SceneApplicationInfoListView.swift (1 hunks)
Projects/App/Sources/DesignSystem/Program/ExpoProgramDetail.swift (5 hunks)
Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift (1 hunks)
Projects/App/Sources/Feature/ProgramDetailFeature/Sources/ProgramDetailView.swift (1 hunks)
Projects/App/Sources/Feature/ProgramFeature/Sources/ProgramView.swift (0 hunks)
Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift (1 hunks)

💤 Files with no reviewable changes (1)

Projects/App/Sources/Feature/ProgramFeature/Sources/ProgramView.swift

✅ Files skipped from review due to trivial changes (2)

Projects/App/Resources/Assets.xcassets/okIcon.imageset/Contents.json
Projects/App/Resources/Assets.xcassets/blackBottomButton.imageset/Contents.json

🔇 Additional comments (3)

Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift (2)

1-10: 파일 헤더와 import 문이 적절합니다.

기본적인 파일 구조가 잘 갖춰져 있습니다.

22-38: 초기화 구문이 잘 구현되어 있습니다.

매개변수와 기본값이 적절하게 설정되어 있습니다.

Projects/App/Sources/DesignSystem/Program/ExpoProgramDetail.swift (1)

92-92: Preview 변경 사항 확인 필요

#Preview에서 ProgramDetailView() 대신 UserManagementView()를 사용하고 있습니다. 의도된 변경 사항인지 확인해주세요.

coderabbitai · 2025-01-12T11:30:12Z

Projects/App/Sources/DesignSystem/InfoList/SceneApplicationInfoListView.swift

+#Preview {
+    UserManagementView()
+}


🛠️ Refactor suggestion

프리뷰 수정이 필요합니다

현재 프리뷰가 UserManagementView()를 표시하고 있는데, 이는 SceneApplicationInfoListView의 동작을 확인하기에 적절하지 않습니다. 컴포넌트를 직접 프리뷰하도록 수정해주세요.

#Preview { - UserManagementView() + SceneApplicationInfoListView() }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

#Preview {

UserManagementView()

}

#Preview {

SceneApplicationInfoListView()

}

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift

+    var userName: String
+    var affiliation: String
+    var position: String
+    var phoneNumber: String


🛠️ Refactor suggestion

전화번호 형식 검증이 필요합니다.

phoneNumber 속성에 대한 형식 검증이 없습니다. 올바른 전화번호 형식인지 확인하는 로직을 추가하는 것이 좋습니다.

private func isValidPhoneNumber(_ number: String) -> Bool { let pattern = "^01[0-1, 7][0-9]{7,8}$" let predicate = NSPredicate(format: "SELF MATCHES %@", pattern) return predicate.evaluate(with: number) }

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift

+            HStack(spacing: 0) {
+                Text("\(programNum)")
+                    .expoFont(.caption1B)
+                    .padding(.leading, 8)
+                    .frame(width: 50, alignment: .leading)
+
+                Text(userName)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(affiliation)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(position)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(phoneNumber)
+                    .frame(width: 170, alignment: .leading)
+
+                Image(consentStatus ? "okIcon" : "ButtonX")
+                    .frame(width: 130, alignment: .center)
+            }
+            .padding(.bottom, 36)
+            .expoFont(.caption2R)


🛠️ Refactor suggestion

레이아웃 값을 상수로 분리하고 접근성을 개선해주세요.

하드코딩된 프레임 너비값들을 상수로 분리하면 유지보수가 용이해집니다.

VoiceOver 지원을 위한 접근성 레이블이 누락되어 있습니다.

+ private enum Layout { + static let programNumWidth: CGFloat = 50 + static let nameWidth: CGFloat = 70 + static let affiliationWidth: CGFloat = 70 + static let positionWidth: CGFloat = 70 + static let phoneWidth: CGFloat = 170 + static let statusWidth: CGFloat = 130 + } HStack(spacing: 0) { Text("\(programNum)") .expoFont(.caption1B) .padding(.leading, 8) - .frame(width: 50, alignment: .leading) + .frame(width: Layout.programNumWidth, alignment: .leading) + .accessibilityLabel("프로그램 번호 \(programNum)")

Committable suggestion skipped: line range outside the PR's diff.

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/DesignSystem/Program/SceneApplicationInfoListDetail.swift

+        Button(action: {
+            self.action()
+        }) {
+            HStack(spacing: 0) {
+                Text("\(programNum)")
+                    .expoFont(.caption1B)
+                    .padding(.leading, 8)
+                    .frame(width: 50, alignment: .leading)
+
+                Text(userName)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(affiliation)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(position)
+                    .frame(width: 70, alignment: .leading)
+
+                Text(phoneNumber)
+                    .frame(width: 170, alignment: .leading)
+
+                Image(consentStatus ? "okIcon" : "ButtonX")
+                    .frame(width: 130, alignment: .center)
+            }
+            .padding(.bottom, 36)
+            .expoFont(.caption2R)
+        }
+        .buttonStyle(PlainButtonStyle())
+        .gesture(
+            DragGesture(minimumDistance: 0)
+                .onChanged { _ in self.isPressed = true }
+                .onEnded { _ in
+                    self.isPressed = false
+                    self.action()
+                }


⚠️ Potential issue

중복 액션 호출을 제거해주세요.

Button의 action과 gesture의 onEnded에서 모두 action()을 호출하고 있어 중복 실행될 수 있습니다.

- Button(action: { - self.action() - }) { + Button(action: {}) {

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

Button(action: {

self.action()

}) {

HStack(spacing: 0) {

Text("\(programNum)")

.expoFont(.caption1B)

.padding(.leading, 8)

.frame(width: 50, alignment: .leading)

Text(userName)

.frame(width: 70, alignment: .leading)

Text(affiliation)

.frame(width: 70, alignment: .leading)

Text(position)

.frame(width: 70, alignment: .leading)

Text(phoneNumber)

.frame(width: 170, alignment: .leading)

Image(consentStatus ? "okIcon" : "ButtonX")

.frame(width: 130, alignment: .center)

}

.padding(.bottom, 36)

.expoFont(.caption2R)

}

.buttonStyle(PlainButtonStyle())

.gesture(

DragGesture(minimumDistance: 0)

.onChanged { _ in self.isPressed = true }

.onEnded { _ in

self.isPressed = false

self.action()

}

Button(action: {}) {

HStack(spacing: 0) {

Text("\(programNum)")

.expoFont(.caption1B)

.padding(.leading, 8)

.frame(width: 50, alignment: .leading)

Text(userName)

.frame(width: 70, alignment: .leading)

Text(affiliation)

.frame(width: 70, alignment: .leading)

Text(position)

.frame(width: 70, alignment: .leading)

Text(phoneNumber)

.frame(width: 170, alignment: .leading)

Image(consentStatus ? "okIcon" : "ButtonX")

.frame(width: 130, alignment: .center)

}

.padding(.bottom, 36)

.expoFont(.caption2R)

}

.buttonStyle(PlainButtonStyle())

.gesture(

DragGesture(minimumDistance: 0)

.onChanged { _ in self.isPressed = true }

.onEnded { _ in

self.isPressed = false

self.action()

}

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/Feature/ProgramDetailFeature/Sources/ProgramDetailView.swift

+            ScrollView(.horizontal, showsIndicators: false) {
+                InfoListView()
+                    .padding(.top, 12)
+
+                ScrollView(showsIndicators: false) {
+                    ForEach(1..<10, id: \.self) { index in
+                        ExpoProgramDetail(
+                            programNum: index,
+                            userName: "서지완",
+                            affiliation: "고등학교",
+                            position: "연수자",
+                            attendanceState: true,
+                            inTime: "10:30",
+                            outTime: "11:30"
+                        )
+                    }
+                    .frame(maxWidth: .infinity, alignment: .leading)
+                }


🛠️ Refactor suggestion

중첩된 ScrollView로 인한 사용자 경험 및 성능 문제 가능성 확인 필요

ScrollView(.horizontal) 내에 ScrollView를 중첩하여 사용하면 스크롤 이벤트가 충돌하여 사용자 경험에 부정적인 영향을 줄 수 있습니다. 특히 스크롤 방향이 다른 ScrollView를 중첩할 경우 예상치 못한 동작이 발생할 수 있습니다. 레이아웃을 재구성하여 중첩된 ScrollView를 피하는 것이 좋습니다.

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift

+                Text("100명")
+                    .expoColor(ExpoColor.main)
+                    .expoFont(.caption2R)
+                    .padding(.trailing, 16)
+            }


⚠️ Potential issue

참가자 수 하드코딩 개선 필요

참가자 수가 하드코딩되어 있습니다. 실제 데이터를 반영하도록 수정이 필요합니다.

참가자 수를 동적으로 계산하여 표시하도록 수정해주세요. 또한 데이터 모델을 추가하여 상태 관리를 구현하는 것이 좋겠습니다.

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift

+            ScrollView(.horizontal, showsIndicators: false) {
+                if userInfoListName == "현장 신청" {
+                    SceneApplicationInfoListView()
+                        .padding(.top, 12)
+                } else {
+                    InfoListView()
+                        .padding(.top, 12)
+                }
+
+                ScrollView(showsIndicators: false) {
+                    ForEach(1..<10, id: \.self) { index in
+                        if userInfoListName == "현장 신청" {
+                            SceneApplicationInfoListDetail(
+                                programNum: index,
+                                userName: "서지완",
+                                affiliation: "고등학교",
+                                position: "연수자",
+                                phoneNumber: "01062860063",
+                                consentStatus: true
+                            )
+                        } else {
+                            ExpoProgramDetail(
+                                programNum: index,
+                                userName: "서지완",
+                                affiliation: "고등학교",
+                                position: "연수자",
+                                attendanceState: true,
+                                inTime: "10:30",
+                                outTime: "11:30"
+                            )
+                        }
+                    }
+                    .frame(maxWidth: .infinity, alignment: .leading)
+                }
+                .padding(.top, 15)
+            }
+


🛠️ Refactor suggestion

스크롤 뷰 구현 및 데이터 처리 개선 필요

중첩된 ScrollView는 성능 문제를 일으킬 수 있습니다.

테스트 데이터가 하드코딩되어 있습니다.

다음과 같은 개선을 제안드립니다:

중첩된 ScrollView 대신 LazyVStack과 단일 ScrollView를 사용하세요.

실제 데이터 모델을 생성하고 ObservableObject를 활용한 데이터 바인딩을 구현하세요.

예시 코드:

class ParticipantViewModel: ObservableObject { @Published var participants: [Participant] = [] func fetchParticipants(type: String) { // API 호출 및 데이터 로딩 구현 } } struct Participant: Identifiable { let id: UUID let programNum: Int let userName: String let affiliation: String // ... 기타 필요한 속성들 }

coderabbitai · 2025-01-12T11:30:13Z

Projects/App/Sources/Feature/UserManagementFeature/Sources/UserManagementView.swift

+        .sheet(isPresented: $userInfoListButtonAction) {
+            VStack(spacing: 0){
+                Button {
+                    userInfoListName = "사전 행사"
+                    userInfoListButtonAction.toggle()
+                } label: {
+                    Text("사전 행사 참가자")
+                        .frame(maxWidth: .infinity, minHeight: 58)
+                        .foregroundStyle(.black)
+                }
+                .padding(.top, 15)
+
+                Button {
+                    userInfoListName = "현장 신청"
+                    userInfoListButtonAction.toggle()
+                } label: {
+                    Text("현장 신청 참가자")
+                        .frame(maxWidth: .infinity, minHeight: 58)
+                        .foregroundStyle(.black)
+                }
+
+                Button {
+                    userInfoListName = "사전 교원 원수"
+                    userInfoListButtonAction.toggle()
+                } label: {
+                    Text("사전 교원 원수 참가자")
+                        .frame(maxWidth: .infinity, minHeight: 58)
+                        .foregroundStyle(.black)
+                }
+
+                Button {
+                    userInfoListButtonAction.toggle()
+                } label: {
+                    Text("취소")
+                        .frame(maxWidth: .infinity, minHeight: 58)
+                        .expoColor(ExpoColor.error)
+                }
+
+                Spacer()
+            }
+            .expoFont(.body2R)
+            .presentationDetents([.height(230)])
+        }


🛠️ Refactor suggestion

시트 구현 개선 제안

버튼 구현이 반복되고 있으며, 참가자 유형이 하드코딩되어 있습니다.

다음과 같은 개선을 제안드립니다:

참가자 유형을 열거형으로 정의

공통 버튼 스타일을 추출

동적 시트 높이 계산

enum ParticipantType: String, CaseIterable { case preEvent = "사전 행사" case onSite = "현장 신청" case preTeacher = "사전 교원 원수" var description: String { return "\(rawValue) 참가자" } } struct ParticipantTypeButton: View { let type: ParticipantType let action: () -> Void var body: some View { Button(action: action) { Text(type.description) .frame(maxWidth: .infinity, minHeight: 58) .foregroundStyle(.black) } } }

Xixn2 added 6 commits January 5, 2025 00:14

💄 :: [#74] User

0e5be46

Management Top NavigationBar 추가

💄 :: [#74] BlackBottomButton Asset 추가

6bd041f

💄 :: [#74] UserInfoList State 변경 버튼 추가

6e398ec

💄 :: [#74] InfoList Func -> Struct로 변경

8922d0e

💄 :: [#74] UserManagementView Publishing

885d3fd

💄 :: [#74] SceneApplicationInfoListDetail Component 추가

f9c685f

Xixn2 added 2️⃣ Priority: Medium 우선순위 - 중 ✨ Feature 신규 기능 labels Jan 12, 2025

Xixn2 requested a review from uuuunseo January 12, 2025 11:27

Xixn2 self-assigned this Jan 12, 2025

Xixn2 linked an issue Jan 12, 2025 that may be closed by this pull request

참가자 관리 페이지 퍼블리싱 #74

Closed

coderabbitai bot reviewed Jan 12, 2025

View reviewed changes

happytaeyoon approved these changes Jan 12, 2025

View reviewed changes

Xixn2 merged commit a2ba17d into develop Jan 12, 2025
3 checks passed

Xixn2 deleted the 74-UserManagement-Publishing branch January 12, 2025 13:14

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔀 :: [#74] 참가자 관리 페이지 퍼블리싱 #75

🔀 :: [#74] 참가자 관리 페이지 퍼블리싱 #75

Xixn2 commented Jan 12, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 12, 2025 •

edited

Loading

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

coderabbitai bot left a comment

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

coderabbitai bot Jan 12, 2025

🔀 :: [#74] 참가자 관리 페이지 퍼블리싱 #75

🔀 :: [#74] 참가자 관리 페이지 퍼블리싱 #75

Conversation

Xixn2 commented Jan 12, 2025 • edited by coderabbitai bot Loading

💡 개요

참가자 관리 페이지 퍼블리싱

📃 작업내용

🎸 기타

Summary by CodeRabbit

coderabbitai bot commented Jan 12, 2025 • edited Loading

Walkthrough

Changes

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

coderabbitai bot Jan 12, 2025

Choose a reason for hiding this comment

Xixn2 commented Jan 12, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Jan 12, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)