[#53] API 2차 구현 #56
[#53] API 2차 구현 #56
Conversation
- 피드 목록 조회 API 구현 - 히드 상세 조회 API 구현 및 UI 추가
- UseCase 파일 생성 - Repository 확장 - ViewModel 수정
- Comment 모델에 API 응답 매핑 메서드 추가
- CommentRepository에 대댓글 메서드 추가
- DefaultCommentDataSource 구현 (CodiveAuthMiddleware 포함)
- 댓글 조회/작성: GET, POST /comments
- 대댓글 조회/작성: GET, POST /comments/{commentId}/replies
- CommentRepositoryImpl에 대댓글 구현 추가
- FetchRepliesUseCase, PostReplyUseCase 생성 - CommentViewModel에 대댓글 관련 상태 추가 - replyingToCommentId, currentReplyText - reloadComments(), reloadReplies() 메서드로 실시간 업데이트 - setReplyingTo(), cancelReply(), postReply() 메서드 구현
- CommentRow에 viewModel 주입으로 대댓글 기능 활성화 - isMine일 때 '(작성자)' 표시 - 대댓글 입력 모드 시 댓글 미리보기 표시 - FeedDetailView에 CommentView 바텀시트 연결 - CommentDIContainer에서 DefaultCommentDataSource 사용 - FeedDIContainer에 CommentDIContainer 통합
- TextLiteral의 situationWork 문자열을 Constants와 일치시킴 ("출근복" -> "출근룩")
- SituationConstants에 getIds() 메서드 추가로 StyleConstants와 동일한 인터페이스 제공
- FeedView에서 상단 카테고리 선택 시 StyleConstants를 사용하여 스타일 ID로 변환
- FeedFilterBottomSheet의 필터 적용 시 StyleConstants/SituationConstants를 사용하여 ID 배열로 변환
- 선택된 필터가 없을 때 nil 처리로 전체 피드 조회 가능
- selectedCategory를 String에서 Set<String>으로 변경하여 여러 개 선택 지원 - FeedFilterBar에서 다중 선택 가능하도록 로직 수정 - 바텀시트에서 선택한 스타일이 상단 카테고리에도 자동으로 표시됨 - 필터 초기화 시 selectedCategory.removeAll() 사용
- FeedView와 FeedFilterBottomSheet의 스타일 순서 통일 - sortedCategories computed property 추가로 선택된 항목을 최우선으로 앞에 배치 - 비선택 항목들은 원래 순서대로 뒤에 배치 - FeedFilterBar에 정렬된 배열 전달
- Domain 계층 추가: * ProfileRepository protocol * FetchMyProfileUseCase / DefaultFetchMyProfileUseCase * FetchFollowsUseCase / DefaultFetchFollowsUseCase * ProfileEntity (도메인 모델) - Data 계층 확충: * ProfileDataSource - API 호출 추상화 * ProfileRepositoryImpl - Repository 구현 * 의존성 체인: Repository → DataSource → APIService - ProfileDIContainer 생성: * 모든 뷰 생성 메서드 제공 * 의존성 주입 관리 - ViewModel 개선: * ProfileViewModel: APIService → FetchMyProfileUseCase * FollowListViewModel: APIService → FetchFollowsUseCase * 모든 ViewModel이 DIContainer에서 주입됨 - View 구조 정리: * ProfileView, FollowListView, ProfileSettingView, OtherProfileView * 모두 DIContainer 주입 방식으로 변경 - 라우팅 일관성 확보: * MainTabView에서 profileDIContainer 사용 * FeedViewFactory에서 profileDIContainer.makeOtherProfileViewModel() 사용 * 모든 Profile 관련 라우팅이 DIContainer 경유
Summary of ChangesHello @Hrepay, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 Codive 앱의 핵심 백엔드 API 연동 2차 구현을 목표로 합니다. 주요 기능 영역인 피드, 댓글, 검색, 프로필, 옷장 등에 실제 API를 연결하고, 이에 맞춰 앱의 DI(Dependency Injection) 아키텍처를 개선하며 데이터 모델링을 확장합니다. 사용자 경험을 향상시키기 위한 UI 개선과 함께, 향후 기능 확장을 위한 기반을 마련하는 데 중점을 두었습니다. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. 코드의 강물 흐르네, API 다리 놓으니, 기능의 꽃 피어나라. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
이번 PR은 Mock 데이터를 실제 API로 교체하고 프로필 관련 기능을 모듈화하는 등 대규모 리팩터링과 기능 구현이 포함되어 있네요. 전반적으로 DI 컨테이너를 활용한 구조 개선과 API 연동이 깔끔하게 잘 이루어졌습니다. 특히 TaskGroup을 사용한 병렬 처리 등 성능을 고려한 점이 인상적입니다. 몇 가지 아쉬운 점(데이터소스의 반환 값, DateFormatter 성능, View의 결합도)에 대해 코멘트를 남겼으니 확인해보시면 좋겠습니다. 다음 3차 API 구현도 기대됩니다!
| // 새로운 댓글 객체 생성 (추가 정보는 필요하면 별도로 조회) | ||
| let currentUser = User(id: "", nickname: "현재 사용자", profileImageUrl: nil) | ||
| let newComment = Comment( | ||
| id: Int(commentId), | ||
| content: content, | ||
| author: currentUser, | ||
| isMine: true, | ||
| hasReplies: false, | ||
| replies: [] | ||
| ) | ||
|
|
||
| return newComment |
There was a problem hiding this comment.
postComment 메서드에서 댓글 생성 후 currentUser 정보를 하드코딩하여 Comment 객체를 반환하고 있습니다. ViewModel에서 목록을 새로고침하여 이 문제를 해결하고 있지만, 이는 비효율적이며 DataSource의 메서드가 일시적으로 부정확한 데이터를 반환하게 만듭니다. API가 생성된 댓글 객체 전체를 반환하거나, 이 메서드가 Void를 반환하여 호출 측에서 데이터 리프레시를 명확히 책임지도록 하는 것이 더 나은 설계입니다. postReply 메서드(170-181 라인)에도 동일한 문제가 있습니다.
| let dateFormatter = ISO8601DateFormatter() | ||
| dateFormatter.formatOptions = [.withFullDate, .withDashSeparatorInDate] |
There was a problem hiding this comment.
toDomain 메서드가 호출될 때마다 ISO8601DateFormatter 인스턴스가 새로 생성됩니다. DateFormatter 생성은 비용이 큰 작업이므로, extension HistoryDetailDTO 내부에 static let 프로퍼티로 만들어 재사용하는 것이 성능에 유리합니다.
private static let iso8601DateFormatter: ISO8601DateFormatter = {
let formatter = ISO8601DateFormatter()
formatter.formatOptions = [.withFullDate, .withDashSeparatorInDate]
return formatter
}()위와 같이 선언 후 Self.iso8601DateFormatter로 접근하여 사용하시는 것을 권장합니다.
| let comment: Comment | ||
| var isReply: Bool = false | ||
|
|
||
| @ObservedObject var viewModel: CommentViewModel |
There was a problem hiding this comment.
CommentRow 뷰가 CommentViewModel 전체에 의존하고 있습니다. 이는 CommentRow와 CommentViewModel 간의 강한 결합(tight coupling)을 만들어 재사용성을 낮추고, 독립적인 테스트 및 SwiftUI 프리뷰를 어렵게 만듭니다. 필요한 데이터와 액션 클로저(예: onReplyTap: (Int) -> Void, onFetchRepliesTap: (Int) -> Void)를 파라미터로 전달하는 방식이 더 유연한 구조입니다.
🔗 연결된 이슈
Resolved #53
✨ 주요 작업사항
DI/아키텍처 개선
ProfileDIContainer신규 추가: 프로필 기능 전용 DI 컨테이너로 API 서비스→레포→유스케이스→VM→View까지 구성해 모듈화/테스트 용이성 강화.AppDIContainer,FeedDIContainer정리:ProfileDIContainer생성/제공 연결, 네비게이션 라우터 주입 흐름을 factory/container 기준으로 일관되게 정돈.API 연동 & 데이터소스 리팩터링
FeedRepositoryImpl개선: DataSource 외에HistoryAPIService도 주입받도록 변경 → 피드 관련 데이터 관리/확장성 향상.기능 확장 (댓글/피드)
옷장(Closet) & 데이터 모델링
ProductItem개선:imageName뿐 아니라imageUrl도 지원하도록 확장, 초기화 유연화.DefaultClothDataSource리팩터: API에서 실제 의류 아이템을 조회하도록 변경(목업 제거), 필터링/매핑 지원.ClothDetailResult확장:seasons(계절) 데이터 포함하도록 API 매핑 강화 → 계절 정보 표시/처리 정확도 향상.ClothDetailViewModel: API 시즌 데이터가 있으면 우선 사용하도록 변경.UI/기타
CustomAIRecommendationView썸네일 영역: 가로 스크롤로 UX 개선.TextLiteral의 작업 상황 라벨 문구 오타/표현 수정.SplashViewModel의 자동 로그인 로직 임시 재활성화(개발 진행 목적).🔍 리뷰어에게 (선택)