[Feature] Implementation CoreData Behavior with View (User) Input Actions #39

keeplo · 2022-08-23T08:37:29Z

구현 배경

#33 RecordView 기능구현
close #34 ConversationUseCase 기능구현
close #38 NoteUseCase 기능 구현

기능 설명

공통 사항

Protocol 추상화로 인한 @published 변화
@published 로 데이터 최신화를 하는 UseCase의 dataSource를,
ViewModel에서 List 그려주는 @published 프로퍼티와 연결하는 작업

   protocol Maintainable {
      var dataSourcePublisher: Published<[Item]>.Publisher { get }
   }

   class UseCase: Maintainable {
        @Published private var dataSource: [Item] = []
        var dataSourcePublisher: Published<[Item]>.Publisher { $dataSource }
   }

   class ViewModel {
        @Published var list: [Conversation] = []

    init(dependency: Dependency) {
 	self.dependency = dependency

 	self.dependency.useCase.dataSourcePublisher
 		.assign(to: &self.$list)
      }
   }

Conversation

RecordView : Create 동작 구현
ConversationListView : Fetch List 연동 (@published) / Delete 동작 구현

Note

SelectionView : Create 동작 구현
NoteSet : Fetch List 연동 (@published) / Delete 동작 구현
NoteDetail : Update 동작 구현
NoteUseCase : isDone 연산프로퍼티로 입력여부 확인

UseCase 동작 이외에 수정

RecordView : Pin 찍기 기능 아이디어 적용
NoteDetailView : 취소 버튼 삭제 및 저장 버튼 disabled 기준 isEdited 구현
NoteSetViewRow : isDone 체크 -> TODO 리스트 형태 UI 체크
ConversationListRow : isChecked 체크 -> infomation 입력 여부 체크로 변경

Data Layer 수정사항

다른 역할의 두 객체가 Repository 이름으로 겹치는 현상
UseCase에서 다루는 Repository 용어의 객체 명칭 -> DataController 로 변경

Local Repository에서 데이터를 불러오기 및 변경요청을 하는 객체로
Domain Layer 에서 추상화된 DataController는 데이터 불러오기 및 변경요청을 담당

코멘트

추가적인 회의를 거치지 않았지만.. 임의의로 DataController 명칭으로 변경하였습니다.

- 각 상황에 맞는 기능 메서드 구현 (ViewModel) - pin 초기화 및 찍기, 제거 동작 구현 - CoreData에 저장 동작 (UseCase) 구현 - 테스트 용 코드 작성

- CoreData fetch 정렬 내림차순으로 변경 - UseCase DataSource View에 바로 연동 되도록 Publish 동작 구현 - onDelete 와 연동해서 제거 동작 구현

- 사용자 입력 조건 체크하는 LanguageCheckable 프로토콜 extension 기능 구현 - 사용자 입력에 따라 Note 데이터 저장 기능 구현 - 오타 수정 sentece -> sentence

- PresentationDIContatiner 테스트 추가 - RecordViewModel UnitTest 구성 - SelectionViewModel UnitTest 구현

- Conversation List Published 연동 - Note List Published 연동 - Debug Repository Published protocol 연동

- 제거 동작 ViewModel로 이동

- Local Repository를 다루는 Controller 명칭 변경 Repository -> DataController

- 사용자 입력 조건 체크하는 LanguageCheckable 프로토콜 extension 기능 구현 - 사용자 입력에 따라 Note 데이터 저장 기능 구현 - 오타 수정 sentece -> sentence

- Placeholder 수정 (기본 날짜, 안내문) - CoreData 와 연동해서 Update 기능 구현

- NoteDetailView 취소 버튼 제거 - NoteDetailView 완료 버튼 -> 저장 - 저장 버튼 사용성 체크 - NoteSetRow 컨텐츠 onTapGesture 동작 이슈 해결 - NoteUseCase edit(_:_:) 메서드 매개변수 수정

- 발음 관련 컨텐츠 삭제(MVP) - isDone 연산프로퍼티로 TODO 상태 처리

- Info 입력시 Row 우측에 체크해주는 UI 구현

dacodaco · 2022-08-27T07:31:02Z

CasualConversation/Targets/Common/Sources/Protocol/LanguageCheckable.swift

+	public func hasTwoMoreSpace(by text: String) -> Bool {
+		let trimmedText = text.trimmingCharacters(in: [" "])
+		return trimmedText.components(separatedBy:  " ").count > 2
+	}


모듈화를 할거면, 파라미터로 스페이스 개수를 지정해서 검사할 수 있게 하면 어떨까요?!
hasSpaces(lessThan: Int) -> Bool

dacodaco · 2022-08-27T07:33:25Z

CasualConversation/Targets/Domain/Sources/UseCase/ConversationUseCase.swift

@@ -17,44 +17,46 @@ public protocol ConversationRecodable {
 }

 public protocol ConversationMaintainable {
-	var list: [Conversation] { get }
+	var dataSourcePublisher: Published<[Conversation]>.Publisher { get }


@published를 프로토콜에서 사용하겠다는 의지 잘 봤습니다...

dacodaco · 2022-08-27T07:43:56Z

CasualConversation/Targets/Domain/Sources/UseCase/NoteUsecase.swift

-		} 
+			fetcedList = dependency.dataController.fetch(filter: item) ?? []
+		}
+		self.dataSource = fetcedList.filter({ !$0.isDone }) + fetcedList.filter({ $0.isDone })


CoreData에서 Fetch 해오는 시점에 바로 descending이 가능할 것 같은데,
성능적으로 무엇이 더 나은지까지는... 검사해봐야 알 것 같네요 ㅜㅜ
염두해봅시다.

dacodaco · 2022-08-27T07:53:10Z

...gets/Presentation/Sources/ConversationListView/ConversationListRow/ConversationListRow.swift

@@ -60,7 +60,7 @@ extension ConversationListRow {
 	@ViewBuilder
 	private func CheckMarkIndicator() -> some View {
 		if viewModel.isChecked {
-			Image(systemName: "checkmark.circle.fill")
+			Image(systemName: "info.circle.fill")


Presentation Layer 와 Data Layer 를 나눠서 Pull Request 보내는 습관을 들여봅시다!

dacodaco · 2022-08-27T07:54:31Z

...nversation/Targets/Presentation/Sources/ConversationListView/ConversationListViewModel.swift

@@ -19,21 +20,29 @@ final class ConversationListViewModel: Dependency, ObservableObject {

 	let dependency: Dependency

+	@Published var list: [Conversation] = []
+
+	private var cancellables: Set<AnyCancellable> = []


assign(to:) 쓰시면 cancellables 필요 없어욥

dacodaco · 2022-08-27T07:56:07Z

CasualConversation/Targets/Presentation/Sources/NoteSetView/NoteDetailView/NoteDetailView.swift

 				.font(.headline)
 				.foregroundColor(.logoDarkGreen)
 		}
+		.disabled(!viewModel.isEdited)
+		.opacity(viewModel.isEdited ? 1 : 0.3)


disabled 활용하면 자동으로 불투명해지는 것으로 알고있는데 아닌가요?!

dacodaco · 2022-08-27T08:01:23Z

...nversation/Targets/Presentation/Sources/NoteSetView/NoteDetailView/NoteDetailViewModel.swift

+		let newItem: Note = .init(
+			id: self.dependency.item.id,
+			original: self.original.trimmingCharacters(in: [" "]),
+			translation: self.translation.trimmingCharacters(in: [" "]


dacodaco · 2022-08-27T08:17:22Z

CasualConversation/Targets/Presentation/Sources/SelectionView/SelectionViewModel.swift

+	func addNote() -> Bool {
+		guard checkConditions() else { return false }
+		let newItem: Note = .init(
+			id: .init(),
+			original: language == .original ? self.inputText : "",
+			translation: language == .translation ? self.inputText : "",
+			category: category == .vocabulary ? .vocabulary : .sentence,
+			references: [self.dependency.item.id],
+			createdDate: Date()
+		)
+		self.dependency.noteUseCase.add(item: newItem) { error in
+			guard error == nil else {
+				print("add Note 실패") // TODO: Error 처리 필요
+				return
+			}
+			self.inputText = ""
+		}
+		return true
+	}


프로퍼티에 isNoteAdded: Bool라고 넣어두고
함수 내부에서 Bool값을 변경하는 방식이 나은 것 같습니다!
함수명과 반환타입이 어색해요!

- isAbleToAdd 연산프로퍼티가 조건을 확인 - AddNote의 반환값 제거

…with_view_input

keeplo added 12 commits August 19, 2022 16:40

feat: RecordView 녹음저장 및 핀 기능구현

7dc6d21

- 각 상황에 맞는 기능 메서드 구현 (ViewModel) - pin 초기화 및 찍기, 제거 동작 구현 - CoreData에 저장 동작 (UseCase) 구현 - 테스트 용 코드 작성

feat: ConversationList 삭제 및 Publish

60f8df2

- CoreData fetch 정렬 내림차순으로 변경 - UseCase DataSource View에 바로 연동 되도록 Publish 동작 구현 - onDelete 와 연동해서 제거 동작 구현

feat: SelectionView 노트저장 기능 구현

8c45791

- 사용자 입력 조건 체크하는 LanguageCheckable 프로토콜 extension 기능 구현 - 사용자 입력에 따라 Note 데이터 저장 기능 구현 - 오타 수정 sentece -> sentence

test: Presentation Layer UnitTest 구현

e12627c

- PresentationDIContatiner 테스트 추가 - RecordViewModel UnitTest 구성 - SelectionViewModel UnitTest 구현

feat: Published 적용 및 연동

5a110a2

- Conversation List Published 연동 - Note List Published 연동 - Debug Repository Published protocol 연동

feat: Note 데이터 제거 구현

ac7d759

- 제거 동작 ViewModel로 이동

chore: �Data Layer Controller 명칭변경

42e94ea

- Local Repository를 다루는 Controller 명칭 변경 Repository -> DataController

feat: SelectionView 노트저장 기능 구현

4e459a5

- 사용자 입력 조건 체크하는 LanguageCheckable 프로토콜 extension 기능 구현 - 사용자 입력에 따라 Note 데이터 저장 기능 구현 - 오타 수정 sentece -> sentence

feat: ConversationInfo Update 기능 구현

a1723ee

- Placeholder 수정 (기본 날짜, 안내문) - CoreData 와 연동해서 Update 기능 구현

feat: Note 데이터 수정 기능

3986767

- NoteDetailView 취소 버튼 제거 - NoteDetailView 완료 버튼 -> 저장 - 저장 버튼 사용성 체크 - NoteSetRow 컨텐츠 onTapGesture 동작 이슈 해결 - NoteUseCase edit(_:_:) 메서드 매개변수 수정

fetat: NoteSet isDone 정렬 및 발음 삭제

d0d1ba7

- 발음 관련 컨텐츠 삭제(MVP) - isDone 연산프로퍼티로 TODO 상태 처리

feat: ConversationRow 메타정보 입력 체크

0100d9b

- Info 입력시 Row 우측에 체크해주는 UI 구현

keeplo added feature New feature or request refactor It'll be refactoring labels Aug 23, 2022

keeplo requested a review from dacodaco August 23, 2022 08:37

keeplo self-assigned this Aug 23, 2022

This was linked to issues Aug 23, 2022

[Feature Request] Conversation UseCase 기능 구현 #34

Closed

[Feature Request] NoteUseCase 기능 구현 #38

Closed

dacodaco reviewed Aug 27, 2022

View reviewed changes

keeplo and others added 2 commits August 27, 2022 17:34

refactor: AddNote 메서드 동작 수정

be85ab7

- isAbleToAdd 연산프로퍼티가 조건을 확인 - AddNote의 반환값 제거

Merge branch 'develop' into feature-implementation_coredata_behavior_…

9561261

…with_view_input

dacodaco merged commit 421f967 into develop Aug 27, 2022

keeplo deleted the feature-implementation_coredata_behavior_with_view_input branch September 18, 2022 12:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feature] Implementation CoreData Behavior with View (User) Input Actions #39

[Feature] Implementation CoreData Behavior with View (User) Input Actions #39

keeplo commented Aug 23, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

dacodaco Aug 27, 2022

[Feature] Implementation CoreData Behavior with View (User) Input Actions #39

[Feature] Implementation CoreData Behavior with View (User) Input Actions #39

Conversation

keeplo commented Aug 23, 2022

구현 배경

기능 설명

공통 사항

Conversation

Note

UseCase 동작 이외에 수정

Data Layer 수정사항

코멘트

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