siwonkim0 / ios-cloud-notes

📝 동기화 메모장

1. 프로젝트 기간: 2022.02.07 - 2022.02.25
2. 팀원: Allie 조이 제인
3. Ground Rules
   • 학습 시간
     • 시작시간 10시
     • 점심시간 1시~3시
     • 저녁시간 7시~9시
   • 스크럼
     • 10시에 스크럼 시작
4. 커밋 규칙
   1. 단위
      • 기능 단위
   • 메세지
     • 카르마 스타일

🗂 목차

• 📺 실행 화면

• ⌨️ 키워드
• STEP 1 : 리스트 및 메모영역 화면 UI구현
  • 고민했던 것
  • Trouble Shooting
  • 배운 개념
• STEP 2 : 코어데이터 DB 구현
  • 고민했던 것
  • Trouble Shooting
  • 배운 개념

실행화면

새로운 메모 추가 및 수정	메모 삭제 및 공유

Keyword

• Core Data NSPersistentContainer
  • NSFetchRequest NSPredicate NSSortDescriptor
  • NSManagedObject NSManagedObjectContext
• UISplitViewController
• DateFormatter
• UITapGestureRecognizer
• Collection subscript
• NavigationItem UIBarButtonItem
• UIActivityViewController UIAlertController
  • popoverPresentationController
• UITextView
  • UITextViewDelegate
• UITableView
  • UISwipeActionsConfiguration
  • selectRow deleteRows
  • UITableViewCell defaultContentConfiguration
    • NSMutableAttributedString
    • setSelected selectedBackgroundView

STEP 1 : 리스트 및 메모영역 화면 UI구현

리스트 화면과 메모영역 화면을 SplitViewController를 활용하여 구현합니다.

1-1 고민했던 것

양방향 델리게이트로 메모 목록과 상세페이지간 데이터 전달

메모 목록을 테이블뷰 형식으로 가지고있는 MemoListViewController 와, 메모의 내용을 표시하는 memoDetailViewController 간에 데이터 전달을 위하여 양방향으로 delegation 관계를 구현하였습니다.

MemoListViewController는 테이블뷰 셀이 선택되면 UITableViewDelegate 메서드 didSelectRowAt에서 MemoDetailViewControllerDelegate 프로토콜을 채택한 memoDetailViewController 에게 Memo 인스턴스를 전달하여 텍스트뷰에 표시할 데이터를 전달합니다.

memoDetailViewController는 사용자가 메모의 내용을 수정하면 MemoListViewControllerDelegate을 채택한 MemoListViewController 에게 변경된 텍스트뷰의 내용을 전달하여 수정사항을 메모 목록에 반영합니다.

NSAttributedString과 defaultContentConfiguration을 이용한 테이블 뷰 셀 구성

• subtitle에서 날짜와 메모 본문에 다른 attribute를 적용하기 위해 NSAttributedString을 사용했습니다
• 날짜는 footnote, 본문은 caption1 + secondaryLabel 색상으로 구성했습니다.

let attributedString = NSMutableAttributedString()

attributedString.append(NSAttributedString(
    string: dateString + " ",
    attributes: [.font: UIFont.preferredFont(forTextStyle: .footnote)]
))

attributedString.append(NSAttributedString(
    string: truncatedBody,
    attributes: [
        .font: UIFont.preferredFont(forTextStyle: .caption1),
        .foregroundColor: UIColor.secondaryLabel
    ]
))

UISwipeActionsConfiguration 사용

TableView의 Cell을 swipe할 때 공유 및 삭제 기능을 위한 액션버튼이 띄워지도록 구현했습니다.

어플의 실행에 따른 selectRow(at:) 호출

• 앱이 처음 구동될 때 첫번째 셀이 선택되도록 했습니다.
• 메모를 추가했을 때 추가한 새로운 메모를 select 합니다.
• 메모를 삭제했을 때는 삭제한메모의 다음 메모를 자동으로 select 합니다.

어떤 메모를 선택해서 작성하고 있는지 알리기 위해 작성중인 셀이 계속 select 되도록 구현했습니다.

키보드의 text 가림현상 개선

NotificationCenter를 활용하여 키보드가 화면에 표시될 때, textView의 text를 가리지 않도록 contentInset을 키보드의 높이와 같게 조정하고, textView.isEditable을 사용하여 메모가 없을 때 textView를 수정할 수 없도록 구현했습니다.

안전하게 배열 조회

배열에서 존재하지않는 인덱스를 조회했을 때 Crash가 나지 않도록 subscript를 활용하여 안전하게 조회할 수 있도록 했습니다.

extension Collection {
    subscript (safe index: Index) -> Element? {
        return indices.contains(index) ? self[index] : nil
    }
}

1-2 Trouble Shooting

1. GestureRecognizer의 cancelsTouchesInView 기본값 true

• GestureRecognizer를 ViewController에 추가하자 UITableView의 셀이 터치되지 않는 현상이 나타났습니다.
• 원인 그 이유는 GestureRecognizer의 프로퍼티 cancelsTouchesInView의 기본값이 true이기 때문입니다. 제스처만 인식한 후 나머지 터치정보들을 뷰로 전달하지 않고 취소하기 때문에 UITableView의 UITableViewDelegate 메서드가 작동하지 않았습니다.
• 해결 따라서 cancelsTouchesInView값을 false로 할당해줌으로써 제스처를 인식한 후에도 Gesture Recognizer와는 무관하게 터치 정보들을 뷰에 전달할 수 있게 되었습니다.

2. 셀 선택이 유지되지 않는 문제

수정 전	수정 후

• 원인 tableView의 allowsSelectionDuringEditing 프로퍼티의 디폴트가 false였기 때문에 셀선택이 되지않았습니다.

• 해결 allowsSelectionDuringEditing 를 true로 바꿔주었습니다. 셀을 지운 후에도 셀선택이 남아있도록 하기 위해 didEndEditingRowAt에서도 indexPath에 해당하는 row를 select하는 로직을 추가했습니다.

  // 셀을 지우는 동안 editing을 할 수 있도록 true로 변경
  tableView.allowsSelectionDuringEditing = true
  tableView.selectRow(at: indexPath, animated: false, scrollPosition: .top)
  
  // 셀 수정이 끝난 후에도 셀을 select하는 로직 추가
  func tableView(_ tableView: UITableView, didEndEditingRowAt indexPath: IndexPath?) {
    guard let indexPath = indexPath else {
        return
    }
    tableView.selectRow(at: indexPath, animated: false, scrollPosition: .top)
  }

3. 마지막 셀을 지웠을 때 index 오류가 나는 문제

Thread 1:
"Attempted to scroll the table view to an out-of-bounds row (0) when there are only 0 rows in section 0.
Table view: <UITableView: 0x13f031400;
frame = (0 0; 420 834);
clipsToBounds = YES;
autoresize = W+H; gestureRecognizers = <NSArray: 0x600000031680>;
layer = <CALayer: 0x600000ec7b80>; contentOffset: {0, -74};
contentSize: {420, 72.5}; adjustedContentInset: {74, 0, 20, 0};
dataSource: <CloudNotes.MemoListViewController: 0x14880fad0>>"

• 원인 셀을 지운 후 다음 셀을 select해주는데, 맨 마지막 셀을 지우면 select할 row가 남아있지 않아서 index 오류가 발생했습니다.
• 해결 조건문으로 지우려는 셀의 row가 남아있는 메모의 개수보다 작을때만 tableView.selectRow(at: indexPath) 를 하도록 분기해주었습니다. 추가로 맨 마지막 셀을 지우는 경우 detailViewController가 비어있는 텍스트뷰를 보여주도록 구현했습니다.

private func deleteMemo(at indexPath: IndexPath) {
    let deletedMemo = MemoDataManager.shared.memos[indexPath.row]
    MemoDataManager.shared.memos.remove(at: indexPath.row)
    tableView.deleteRows(at: [indexPath], with: .none)
    MemoDataManager.shared.deleteMemo(id: deletedMemo.id)
    
    if indexPath.row < MemoDataManager.shared.memos.count {
        let memo = MemoDataManager.shared.memos[indexPath.row]
        delegate?.memoDetailViewController(showTextViewWith: memo)
        tableView.allowsSelectionDuringEditing = true
        tableView.selectRow(at: indexPath, animated: false, scrollPosition: .top)
    } else {
		// 맨 마지막 셀을 지우는 경우
        delegate?.showEmptyTextView()
    }
}

func showEmptyTextView() {
    textView.isEditable = false
    textView.text = ""
}

1-3 배운 개념

• 코드로 뷰 구현하기: SceneDelegate 에서 initial View Controller 설정

  코드로 뷰 구현하기: SceneDelegate 에서 initial View Controller 설정

  • 스토리보드를 지운 후 SceneDelegate의 scene메서드에서 window의 rootViewController를 앱의 첫화면에 보이는 splitViewController로 설정한합니다.
  • 그리고 makeKeyAndVisible()로 화면에 보이도록 설정하여 스토리보드에서 initial view controller로 지정하는 것을 대신해줄 수 있습니다.

  func scene(_ scene: UIScene, willConnectTo session: UISceneSession, options connectionOptions: UIScene.ConnectionOptions) {
      guard let windowScene = (scene as? UIWindowScene) else { return }
          
      window = UIWindow(windowScene: windowScene)
      let splitViewController = SplitViewController(style: .doubleColumn)
      window?.rootViewController = splitViewController
      window?.makeKeyAndVisible()
  }

• BarButtonItem 활용

  BarButtonItem 활용

  • UIViewController에 있는 navigationItem 프로퍼티를 사용하여 navigationbar에 필요한 item을 설정해줄 수 있다.
  • UIBarButtonItem의 생성자에는 barButtonSystemItem이나 image를 받아서 원하는 대로 설정해 사용할 수 있다.

  private func setupNavigationBar() {
      let addButton = UIBarButtonItem(barButtonSystemItem: .add, 
  																		target: self, 
  																		action: #selector(addMemo))
      navigationItem.rightBarButtonItem = addButton
      navigationItem.title = "메모"
  }
  
  private func setupNavigationItem() {
  		navigationItem.rightBarButtonItem = UIBarButtonItem(image: UIImage(systemName: "ellipsis.circle"),
  																												style: .plain,
  																												target: self,
  																												action: #selector(viewMoreButtonTapped))
  }

• UISplitViewController

  UISplitViewController

  • setViewController(_:for:) : UISplitViewController의 메서드로 Double Column 스타일인 경우에 primary와 secondary 뷰컨트롤러를 지정합니다.
  • 이 메서드로 지정하는 경우에 자동으로 뷰컨트롤러에 네비게이션 컨트롤러를 감싸서 UISplitViewController에 할당해주었습니다.
  • preferredDisplayMode = .oneBesideSecondary 로 앱 초기 화면에서 왼쪽에 메모 목록, 오른쪽엔 메모 상세화면이 같이 나오도록 설정하였습니다.

  class SplitViewController: UISplitViewController {
      private let listViewController = MemoListViewController()
      private let detailViewController = MemoDetailViewController()
      
      override func viewDidLoad() {
          super.viewDidLoad()
          setupChildView()
          setupDisplay()
          listViewController.delegate = detailViewController
          detailViewController.delegate = listViewController
          hideKeyboard()
      }
      
      private func setupChildView() {
          setViewController(listViewController, for: .primary)
          setViewController(detailViewController, for: .secondary)
      }
      
      private func setupDisplay() {
          preferredSplitBehavior = .tile
          preferredDisplayMode = .oneBesideSecondary
      }
  }

STEP 2 : 코어데이터 DB 구현

메모를 위한 코어데이터 모델을 생성합니다.

2-1 고민했던 것

코어데이터를 관리하는 매니저 타입 구현

• 데이터를 관리하는 타입인 MemoDataManager 를 구현했습니다.
  • fetch해온 데이터를 저장해 놓는 memos 배열을 가지고 있습니다.
  • CoreData를 생성, 삭제, 조회, 업데이트를 담당합니다.

사용자 친화적인 UI 구현

메모가 추가되거나 수정될 때 최신순으로 정렬하여 TableView에 보여줄 수 있도록 날짜를 기준으로 정렬했습니다.

메모리스트 뷰에서는 Swipe해서 share 버튼을 눌렀을 때, UIActivityView를 화면 중앙에 보여주도록 해주었고, 상세페이지의 barButtonItem에서 share 버튼을 눌렀을 때는, 해당 버튼에서부터 UIActivityView가 보여지도록 했습니다.

Swipe해서 share 버튼을 눌렀을 때	barButtonItem에서 share 버튼을 눌렀을 때

아이패드에서 popoverPresentationController의 사용

• 오류메세지

아이폰에서는 잘 띄워지던 UIAlertController나 UIActivityViewController가 아이패드 환경에서는 작동하지 않았습니다.

Thread 1: "Your application has presented a UIAlertController (<UIAlertController: 0x10d813a00>) of style UIAlertControllerStyleActionSheet from CloudNotes.SplitViewController (<CloudNotes.SplitViewController: 0x11f7068f0>).
The modalPresentationStyle of a UIAlertController with this style is UIModalPresentationPopover.
You must provide location information for this popover through the alert controller's popoverPresentationController.
You must provide either a sourceView and sourceRect or a barButtonItem.
If this information is not known when you present the alert controller, you may provide it in the UIPopoverPresentationControllerDelegate method -prepareForPopoverPresentation."

popoverPresentationController를 사용하여 얼럿이 띄워질 위치를 sender나 뷰의 특정한 위치로 명시를 하여 해결했습니다.

• sender를 준 경우

private func showAlert(_ sender: UIBarButtonItem) {
    let alert = UIAlertController(title: nil, message: nil, preferredStyle: .actionSheet)
    // 코드 생략
    if let popoverController = alert.popoverPresentationController {
        popoverController.barButtonItem = sender
    }
    present(alert, animated: true, completion: nil)
}

• 좌표를 설정해준 경우

private func showActivityView(indexPath: IndexPath) {
    //코드 생략
    let activityViewController = UIActivityViewController(activityItems: [memoToShare], applicationActivities: nil)
    
    if let popOver = activityViewController.popoverPresentationController {
        popOver.sourceView = splitViewController.view
        popOver.sourceRect = CGRect(x: splitViewController.view.bounds.midX,
                                    y: splitViewController.view.bounds.midY,
                                    width: 0,
                                    height: 0)
        popOver.permittedArrowDirections = []
    }
    present(activityViewController, animated: true)
}

관련 공식문서 링크: Displaying Transient Content in a Popover

2-2 Trouble Shooting

UITableView의 Cell을 deleteRows(at:) 메서드로 삭제했을 때 발생한 에러

Thread 1:
"Invalid update: invalid number of rows in section 0.
The number of rows contained in an existing section after the update (26) must be equal to the number of rows contained in that section before the update (26), plus or minus the number of rows inserted or deleted from that section (0 inserted, 1 deleted) and plus or minus the number of rows moved into or out of that section (0 moved in, 0 moved out).

• 상황 tableView 섹션의 row 개수와 실제 보여줄 섹션의 개수가 맞지 않아서 발생하는 오류입니다.
• 이유 tableView의 셀을 삭제하면서 tableView에 보여줄 데이터도 같이 삭제 해주어야 하는데, 그 과정이 누락이되어 발생한 것으로 보입니다.
• 해결 셀을 추가 및 삭제할 때 tableView에 보여줄 섹션의 개수도 동일하도록 MemoDataManager의 메모와 코어데이터에 저장된 메모데이터도 삭제 해주었습니다.

텍스트뷰가 변할때마다 메모가 저장되는 문제

작성 중일때	앱을 다시 실행해서 코어데이터에서 fetch해왔을 때

• 상황 텍스트뷰에 여러 글자를 적고, 앱을 다시 구동하면 오른쪽 사진처럼 한글자씩 메모가 여러개 생기는 오류입니다.
• 이유 텍스트뷰가 수정이 될 때마다 코어데이터에 저장이 되기 때문에 발생한 문제였습니다.
  • DetailViewController에서 currentMemo 라는 연산프로퍼티로 현재 메모에 접근했는데,
  • 그 안에서 만들어주는 Memo 타입을 이니셜라이즈 할때 코어데이터의 context가 쓰이면서 글자를 수정한 수만큼 메모가 저장이 되고,
  • fetch를 해올 때 한 글자씩 저장 된 모든 메모들을 불러오면서 하나의 메모를 수정했지만, 수정한 글자 수대로 새로운 메모들이 생기는 현상이 발생하였습니다.

<수정 전>

extension MemoDetailViewController: UITextViewDelegate {
		private var currentMemo: Memo {
        let memoComponents = textView.text.split(
            separator: "\n",
            maxSplits: 1
        ).map(String.init)
        
        **let memo = Memo(context: MemoDataManager.shared.viewContext)**
        memo.title = memoComponents[safe: 0] ?? ""
        memo.body = memoComponents[safe: 1] ?? ""
        memo.lastModified = Date()
        
        return memo
    }

    func textViewDidChange(_ textView: UITextView) {
        delegate?.memoListViewController(updateTableViewCellWith: currentMemo)
    }
}

프린트문으로 출력해본 결과 context의 수는 글자가 입력할 때 마다 늘어나지만, 메모의 수는 변함이 없는것을 확인할 수 있었습니다.

extension MemoListViewController: MemoListViewControllerDelegate {
		func memoListViewController(updateTableViewCellWith memo: Memo) {
		    guard let indexPath = tableView.indexPathForSelectedRow else {
		        return
		    }
		    let request = NSFetchRequest(entityName: "Memo")
				//context의 수가 글자 입력할때마다 늘어남 
		    print(try? MemoDataManager.shared.viewContext.count(for: request))
				
		    MemoDataManager.shared.memos[indexPath.row] = memo
				//메모의 수는 그대로 
		    print(MemoDataManager.shared.memos.count)
		    tableView.reloadRows(at: [indexPath], with: .none)
		    tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
		}
}

• 해결 코어데이터의 Memo(context:) 객체를 이용하지 않고 그냥 메모 수정 화면에서 메모 목록으로 title, body를 자체를 전달하여 수정되고 있는 셀의 indexPath로 MemoDataManager에서 Memo 를 가져와서 MemoDataManager.shared.updateMemo 로 해당 Memo를 수정한 후 context를 save하는 방식으로 해결했습니다.

extension MemoDetailViewController: UITextViewDelegate {
		func textViewDidChange(_ textView: UITextView) {
        let memoComponents = textView.text.split(separator: "\n",
                                                 maxSplits: 1)
                                                .map(String.init)
        
        let title = memoComponents[safe: 0] ?? ""
        let body = memoComponents[safe: 1] ?? ""
        let lastModified = Date()
        
        delegate?.memoListViewController(updateTableViewCellWith: title, body: body, lastModified: lastModified)
    }
}

extension MemoListViewController: MemoListViewControllerDelegate {
		func memoListViewController(updateTableViewCellWith title: String, body: String, lastModified: Date) {
        guard let indexPath = tableView.indexPathForSelectedRow,
              let id = MemoDataManager.shared.memos[indexPath.row].id else {
            return
        }
        MemoDataManager.shared.updateMemo(id: id, title: title, body: body, lastModified: lastModified)
        tableView.reloadRows(at: [indexPath], with: .none)
        tableView.selectRow(at: indexPath, animated: true, scrollPosition: .none)
    }
}

class MemoDataManager {
		func updateMemo(id: UUID,
                    title: String,
                    body: String,
                    lastModified: Date)
		    {
        let predicate = NSPredicate(format: "id == %@", id.uuidString)
        guard let memo = fetchMemos(predicate: predicate).first else {
            return
        }
        memo.title = title
        memo.body = body
        memo.lastModified = lastModified
        saveViewContext()
    }
}

마지막 Cell을 삭제했을 때, textView에 text가 남아있는 문제

MemoListViewController의 deleteMemo() 메소드 내부에서 조건문을 통해 마지막 Cell인지 확인 후 detailViewController에 있는 showEmptyTextView() 메소드를 호출해 textView의 text를 초기화 해주었습니다.

private func deleteMemo(at indexPath: IndexPath) {
    let deletedMemo = MemoDataManager.shared.memos[indexPath.row]
    MemoDataManager.shared.memos.remove(at: indexPath.row)
    tableView.deleteRows(at: [indexPath], with: .none)
    MemoDataManager.shared.deleteMemo(id: deletedMemo.id)
        
    if indexPath.row < MemoDataManager.shared.memos.count {
        let memo = MemoDataManager.shared.memos[indexPath.row]
        delegate?.memoDetailViewController(showTextViewWith: memo)
        tableView.allowsSelectionDuringEditing = true
        tableView.selectRow(at: indexPath, animated: false, scrollPosition: .top)
    } else {
        delegate?.showEmptyTextView()
    }
}

---

2-3 배운 개념

• UITableViewDelegate 메서드를 활용한 스와이프 기능

  [UITableViewDelegate 메서드를 활용한 스와이프 기능]

  tableView의 Delegate 메서드를 활용하여 셀을 스와이프 했을 때 선택할 수 있는 옵션을 선택할 수 있습니다.

  • 왼쪽 → 오른쪽 스와이프
  • 오른쪽 → 왼쪽 스와이프

  func tableView(_ tableView: UITableView,
    leadingSwipeActionsConfigurationForRowAt indexPath: IndexPath
    ) -> UISwipeActionsConfiguration? {
        // ...
    }
  
  func tableView(_ tableView: UITableView,
        trailingSwipeActionsConfigurationForRowAt indexPath: IndexPath
    ) -> UISwipeActionsConfiguration? {
        // ...
    }

STEP 2-2 REFACTOR

이번 스텝에서는 각각의 타입의 역할을 명확하게 분리하는데 초점을 두었습니다.

타입의 역할이 잘 분리된다면, 다음과 같은 장점이 있습니다.

1. 어떤 부분을 고쳐야 하는지 금방 파악할 수 있어서 유지보수가 용이해진다.
2. 의존성이 없어지기 때문에 재사용성이 높아진다.
3. 특정 타입에 대한 수정이 다른 타입에 영향을 주지 않는다.

1, 2차 수정을 거쳐서 SplitViewController가 모델인 MemoDataManager와 자식 뷰컨들인 ListViewController / DetailViewController 사이에서 중개를 하는 구조로 refactor하였습니다.

이로써 모델은 모델 관련 로직만, 각 뷰컨은 각자의 뷰 관련 로직만 가지고 있도록 구성하였습니다.

---

1차 수정

설계 목적

• 데이터를 한곳에서 관리하기
• ListViewController / DetailViewController 에서 model 관련 로직을 최대한 덜어내기
• MVC 디자인 패턴으로 구조 짜기

처음에는 SplitViewController를 이용해서 한 곳에서 data를 관리해주는 방법으로 고민을 했습니다. 하지만, SplitViewController도 ListViewController / DetailViewController와 마찬가지로 ViewController라는 점에서 그 또한 데이터를 관리해주는 역할을 맡기에 부적절하는 생각이 들었습니다.

고민끝에, DataManager의 인스턴스를 SplitViewController가 가지고 있고, ListVC / DetailVC 에게 전달해주는 방식으로 구현하기로 결정을 했습니다. 결과적으로 ListVC / DetailVC은 SplitViewController으로부터 받은 DataManager를 가지고 있고, DataManager는 listDelegate / detailDelegate를 가지고 소통을 하는 구조입니다.

• 기존에 ListVC / DetailVC에서 Model 관련 로직이 많았던 부분은 모두 DataManager 안으로 옮겨주었습니다.
  • (ex if indexPath.row < dataManager.memos.count 이런식으로 조건을 확인하는 부분)
• ListVC와 DetailVC는 각각 MemoDataManagerListDelegate, MemoDataManagerDetailDelegate를 채택합니다.
• DataManager는 데이터에 변화가 일어날 때 listDelegate / detailDelegate에게 일을 시킵니다.
  • 예를들어 유저가 셀을 삭제 → 삭제 이벤트를 받은 ListVC가 DataManager의 메모삭제 메서드를 호출 → DataManager는 조건을 확인하여 listDelegate / detailDelegate에게 적절한 일을 시킴

2-1 고민했던 것

indexPath를 활용하여 하나의 메서드로 ListVC와 DetailVC에서 메모 삭제

indexPath를 활용하여 하나의 메서드로 ListVC와 DetailVC에서 메모 삭제

메모를 지울 때 indexPath를 활용해서 지워야 해서 Core Data에서 선택된 메모의 indexPath를 가져오는 방법에 대해 고민하였습니다. ListVC의 메모 목록에서 스와이프해서 삭제할때는 선택된 indexPath의 정보를 같이 전달해줄 수 있지만, DetailCV의 메모 상세페이지에서 더보기 버튼으로 삭제시에는 선택된 셀의 indexPath를 알 수 없어서 indexPath가 있으면 그대로 사용하고, 없으면 listVC의 selectedCellIndex에서 선택된 셀의 인덱스를 가져와서 listVC의 셀도 지우고 detailVC의 텍스트도 지워주도록 구성하였습니다.

final class MemoListViewController: UIViewController {
	func tableView(_ tableView: UITableView,
		       trailingSwipeActionsConfigurationForRowAt indexPath: IndexPath
	) -> UISwipeActionsConfiguration? {
	    let deleteAction = UIContextualAction(style: .destructive, title: "Delete") { _, _, _ in
		self.dataManager.deleteSelectedMemo(at: **indexPath**)
	    }
	    ...
	}
}

final class MemoDetailViewController: UIViewController {
	private func showDeleteAlert(_ sender: UIBarButtonItem) {
	    ...
	    let delete = UIAlertAction(title: "삭제", style: .destructive) { _ in
		self.dataManager.deleteSelectedMemo()
	    }
	    ...
	}
}

final class MemoDataManager {
	...
	func deleteSelectedMemo(at indexPath: IndexPath? = nil) {
	    let selectedIndexPath: IndexPath?
	    if indexPath != nil {
		selectedIndexPath = indexPath
	    } else {
		selectedIndexPath = listDelegate?.selectedCellIndex
	    }

	    guard let selectedIndexPath = selectedIndexPath else {
		return
	    }
	    let deletedMemo = memos[selectedIndexPath.row]
	    deleteMemo(id: deletedMemo.id)
	    listDelegate?.deleteCell(at: selectedIndexPath)

	    if selectedIndexPath.row < memos.count {
		let memo = memos[selectedIndexPath.row]
		detailDelegate?.showTextView(with: memo)
		listDelegate?.selectNextCell(at: selectedIndexPath)
	    } else {
		detailDelegate?.showIneditableTextView()
	    }
	}
	...
}

2-2 Trouble Shooting

앱 첫 실행 화면에서 첫번째 셀이 선택되지 않는 문제

앱 첫 실행 화면에서 첫번째 셀이 선택되지 않는 문제

정상적으로 작동하는 모습

문제 상황

앱을 실행했을 때, selectFirstMemo()가 실행되면서 첫번째 셀이 선택되고 해당 셀의 내용을 detailView에 보여줘야합니다. 원래는 MemoListViewController의 viewDidLoad() 에서 selectFirstMemo()를 호출주었습니다. 그런데 List의 셀은 선택이 되지만, 선택된 셀의 상세페이지는 보이지 않는 문제가 발생했습니다. 반대로 MemoDetailViewController에서 viewDidLoad() 에서 selectFirstMemo()를 호출하면 선택된 셀의 상세페이지는 보이지만 List의 셀은 선택이 되지 않는 문제가 발생합니다.

원인

문제의 원인은 selectFirstMemo()를 호출하는 VC의 delegate 등록만 작동해서 발생하는 것이었습니다.

• ListVC에서 호출하면 listDelegate만 등록되고 detailDelegate는 nil → 첫번째 셀이 선택되지만, 상세메모는 보이지 않음
• DetailVC에서 호출하면 detailDelegate만 등록되고 listDelegate는 nil → 첫번째 상세메모는 보이지만 셀 선택이 안됨

extension MemoDataManager {
    func selectFirstMemo() {
        if memos.isEmpty == false {
            listDelegate?.setupRowSelection()
            detailDelegate?.showTextView(with: memos[0])
        }
    }
}

ListViewController의 viewDidLoad에서 호출하는 모습	detailDelegate가 nil인 모습

해당 메서드를 호출하지 않으면 delegate임을 명시하더라도 무시되는 현상을 확인했습니다. 좌측 사진은 ListViewController의 viewDidLoad()에서 selectFirstMemo()를 호출하고 있는 상태고, listDelegate / detailDelegate를 출력했을 때 detailDelegate 만 nil 이 나오는것을 확인할 수 있었습니다.

해결

listVC의 viewDidAppear에서 dataManager.selectFirstMemo() 호출하는 방식으로 해결했습니다.

private let dataManager = MemoDataManager()

private lazy var listViewController = MemoListViewController(dataManager: dataManager)
private lazy var detailViewController = MemoDetailViewController(dataManager: dataManager)

override func viewDidAppear(_ animated: Bool) {
    super.viewDidAppear(animated)
    dataManager.selectFirstMemo()
}

메모를 삭제할 때 viewContext를 save하기전에 memos 배열에서도 삭제하지 않으면 Crash가 나는 문제

메모를 삭제할 때 viewContext를 save하기전에 memos 배열에서도 삭제하지 않으면 Crash가 나는 문제

saveViewContext를 먼저호출	saveViewContext를 나중에 호출

문제 상황

처음에는 view context 에서 메모를 삭제하고나서 viewContext.delete(memoToDelete) 바로 view context를 save 한 후 saveViewContext() MemoDataManager가 가지고 있는 memos 배열에서도 삭제하였습니다. 그랬더니 Crash가 나며 앱이 중지되었습니다.

해결

MemoDataManager가 가지고 있는 memos 배열에서도 삭제해준 후 view context를 save 하는 방법으로 순서를 바꿔서 해결해주었습니다.

2차 수정

설계 목적

• DataManager는 UI와 관련된 역할을 하지 않는다
• SplitVC만 DataManager를 갖는다
• ListVC / DetailVC에서는 데이터와 관련된 작업을 하지 않는다

1차 수정을 하고 나서, ListVC / DetailVC가 DataManager를 꼭 알아야 하는지에 대해서 많은 고민을 했습니다. DataManager를 아는 타입이 많음으로 인해서 DataManager에 발생하는 일들이 어떻게 관리가 되고 있는지 파악하기 힘들어 지고, 처리가 분산되어 있다고 생각했습니다.

그래서 DataManager를 갖게되는 SplitVC가 이러한 일들을 담당하도록 다시 설계했습니다. SplitVC가 DataManager를 갖는 이유는 Debug view hierarchy를 통해 확인해봤을 때 계층구조가 SplitVC가 자식 뷰컨들을 직접적으로 알고 있기 때문에 모델과 연결되어야 하는 컨트롤러가 SplitVC이라고 생각했습니다.

기존에 ListVC / DetailVC에서 Model 관련 로직들을 모두 없앴습니다. ListVC / DetailVC는 View를 Controller하는 역할만 하고 있습니다. 또, MemoDataManager에 남아있던 UI관련 로직들도 모두 삭제했습니다. 그 중간에서 SplitVC가 DataManager와 자식 뷰컨들을 중개해주고 있습니다.

<최종 구조>

SplitViewController는 DataManager와 childViewControllers와 메시지를 주고 받고, 각각의 childViewController들은 splitViewController와만 메시지를 주고 받는 로직입니다.

ListVC, DetailVC의 delegate = SplitVC

listViewController와 detailViewController의 delegate를 SplitVC로 지정하여 DataManager의 데이터에 따라 SplitVC가 ListVC와 DetailVC가 할 일을 대신 해주도록 구현하였습니다.

SOLID - DIP 원칙 (의존관계 역전 원칙)

상위레벨 모듈은 하위레벨 모듈에 의존하면 안된다는 DIP 원칙에 따라, MemoDataManagable 프로토콜을 정의하여 SplitVC이 생성될 때 MemoDataManagable 프로토콜을 준수하는 어떤 타입이라도 주입될 수 있도록 해주었습니다.

protocol MemoDataManagable {
    ...
}

extension MemoDataManager: MemoDataManagable {
    ...
}

final class SplitViewController: UISplitViewController {
    private let dataManager: MemoDataManagable
	  ...
    init(style: UISplitViewController.Style, dataManager: MemoDataManagable) {
        self.dataManager = dataManager
        super.init(style: style)
    }
    ...
}