1. 프로젝트 기간: 2022.01.03 - 2022.01.28
2. Ground Rules
   1. 시간
      • 시작시간 10시
      • 점심시간 12시~2시
      • 저녁시간 6시~7시 사이부터 2시간
   • 진행 계획
     • 프로젝트가 중심이 아닌 학습과 이유에 초점을 맞추기
     • 의문점을 그냥 넘어가지 않기
   • 스크럼
     • 10시에 스크럼 시작
3. 커밋 규칙
   1. 단위
      • 기능 단위
   • 메세지
     • 카르마 스타일

• 키워드
• STEP 1 : 네트워킹 타입 구현
  • 고민했던 것
  • 의문점
  • Trouble Shooting
  • 배운 개념
  • PR 후 개선사항
• STEP 2 : 상품 목록 화면 구현
  • 고민했던 것
  • 의문점
  • Trouble Shooting
  • 배운 개념
• STEP Bouns : 로컬 캐시 구현
  • 고민했던 것
  • 의문점
  • Trouble Shooting
  • 배운 개념
  • PR 후 개선사항
• STEP 3 : 상품 등록/수정 화면 구현
  • 고민했던 것
  • 의문점
  • Trouble Shooting
  • 배운 개념
  • PR 후 개선사항
• STEP 4 : 상품 상세화면 구현
  • 고민했던 것
  • 의문점
  • Trouble Shooting
  • 배운 개념
  • PR 후 개선사항

• STEP1
  • 의존성 주입(DI) URLSession URLProtocol URLRequest
  • API HTTP TCP/IP MIME-Type multipart/form-data
  • application/json Result Codable CodingKey Async Test
• STEP2
  • UICollectionView UICollectionViewFlowLayout
  • Supplyment UICollectionReusableView performBatchUpdates
  • reloadData Xib File UISegmentedControl
• STEP Bonus
  • NSCache UIActivityIndicatorView
• STEP3
  • UIRefreshControl UIGraphicsImageRenderer UIImagePicker
  • UIScrollViewDelegate UITextField UIAlertController
• STEP4
  • UIPageControl GestureRecognizer UIFontMetrics
  • UIScrollView zoomScale Dynamic Type UICollectionView Paging
  • UIAlertController UITextFlied

• 네트워크 통신을 담당한 타입을 설계합니다.
• Mock 데이터를 활용하여 단위테스트를 수행합니다.

• 한 타입이 하나의 역할만 할 수 있도록 설계에 많은 고민을 해보았다.

실제 네트워크에서 내려오는 변수명이 스네이크 케이스를 사용하는 변수는 Codingkey를 이용하여 parsing하는 key를 바꿔주었으며 스네이크케이스를 사용하지 않는, 즉 타입의 변수명과 일치하면 rawValue를 명시할 필요가 없어 가독성을 위해 한 줄로 case를 합쳐주었다.

enum CodingKeys: String, CodingKey {
   case id, stock, name, thumbnail, currency, price, images, vendors
   case vendorID = "vendor_id"
   case bargainPrice = "bargain_price"
   case discountedPrice = "discounted_price"
   case createdAt = "created_at"
   case issuedAt = "issued_at"
}

• Network하는 과정에서 역할마다 객체를 구분하여 구현하였다.

  • Network : dataTask()를 통해 SessionDataTask를 서버로 전송해 직접 네트워킹하는 객체

    func execute(request: URLRequest, completion: @escaping (Result<Data?, Error>) -> Void) {
            session.dataTask(with: request) { data, response, error in
            ...

  • NetworkManager : Network의 excute를 통해 data를 받아 decoding하는 fetch()를 가진 객체

    func fetch<T: Decodable>(request: URLRequest,
                                decodingType: T.Type,
                                completion: @escaping (Result<T, Error>) -> Void) {
            
         network.execute(request: request) { result in

• 하드코딩을 개선하기 위해 enum 타입을 만들어 Address와 HTTPMethod의 값들을 분류해주었다.

• Request할 때, 그리고 Response하는 타입이 세부적으로 달라 ProductModification, ProductRegistration 등... 각 타입을 모두 구현하였다.

• 상품 삭제, 등록, 조회 등 여러가지 요청을 request 메소드 하나를 오버로딩을 활용하여 작성하였다.

• 테스트 작성을 위해 의존성 주입을 활용하여 Mock, Stub 객체를 만들어 활용하였다.
• URLProtocol을 상속받은 클래스를 만들고 재정의를 해주었다.
  • 이 방법은 URLSession의 dataTask를 직접 Stub으로 만드는 방법도 있었지만, URLSessionDataTask를 채택한 타입에 init()을 정의하니 deprecated 경고가 떠서 이를 해결하기 위해 삭제 후 URLProtocol을 활용하는 방법으로 로직을 변경하였다.

• 비동기 메서드를 사용하는 동기 메서드는 비동기 메서드 테스트로 진행해야할까?
• URLProtocol과 URLSession의 관계가 정확하게 이해되지 않는다...
  • https://developer.apple.com/videos/play/wwdc2018/417/
• Health Checker의 필요성을 모르겠다..
• 테스트 시 Request의 바디도 체크를 해야할까?

class StubURLSessionDataTask: URLSessionDataTask {
    var dummyData: DummyData?

    // init 부분에서 에러가 났다.
    init(dummy: DummyData?, completionHandler: DataTaskCompletionHandler?) {
        self.dummyData = dummy
        self.dummyData?.completionHandler = completionHandler
    }

    override func resume() {
        dummyData?.completion()
    }
}

• multipart/form-data
• API문서 읽는 방법
• 파싱한 JSON 데이터와 매핑할 모델 설계
  • CodingKeys 프로토콜의 활용
• URL Session을 활용한 서버와의 통신
  • URLRequest를 설정하는 방법
  • Testable한 네트워크 코드 작성하기
    • 네트워크 상황과 무관한 네트워킹 데이터 타입의 단위 테스트(Unit Test)

• 테스트 코드에 중복되는 부분을 개선
  • 빠진 주석 및 줄바꿈을 수정
• Image의 프로퍼티 네이밍을 명확하게 수정
• 하드코딩 되어있는 문자열을 따로 enum 타입으로 빼주어 개선
• 에러의 네이밍을 명확하게 개선
• Parser, Parsable의 네이밍을 JSON을 덧붙혀 명확하게 개선
• 접근제어가 붙어있지 않은 프로퍼티에 접근제어를 추가
• Address의 네이밍을 명확하게 개선 (APIAddress)

상품목록을 볼 수 있는 화면을 구현합니다.

• CollectionView 하나로 Cell 두개를 활용하여 화면을 전환하기

  • Custom Cell을 구현하고, 두개의 레이아웃을 만들어 셀만 바꿔주는 방식으로 목록화면 구성
  • FlowLayout을 활용하여 Cell의 레이아웃을 구성
  • 서버에서 상품 목록을 불러오는 부분과 뷰를 그리는 부분 비동기 처리 구현

• CollectionView cell 각각 xib로 구현

  • CollectionView의 GridCell, ListCell을 각각 xib파일을 생성하여 storyboard로 구현하였고 두개의 xib에 대한 코드는 ProductCell 하나의 cell로 구성

• Network를 통해 data를 가져와 CollectionView를 구성

  • API의 Data를 가져오기 위해 productList Search하는 request 생성하여 networkManager의 fetch()로 network를 진행하였고 가져온 data로 collectionViewload하였다.

    let request = networkManager.requestListSearch(page: 1, itemsPerPage: 10) else {
    ...        
    networkManager.fetch(request: request, decodingType: Products.self) { result in
        switch result {
            case .success(let products):
               self.productList = products.pages
               self.collectionViewLoad()
            ...

• CollectionView를 재구성하는 경우 reloadData() 사용

  • SegmentControl을 이용해 flowlayout을 변경하는 경우 CollectionView를 재구성하기 위해 reloadData를 사용하였다.

    // list -> gird, grid -> list로 변경
    @IBAction private func switchSegmentedControl(_ sender: UISegmentedControl) {
            switch sender.selectedSegmentIndex {
            case 0:
                currentCellIdentifier = ProductCell.listIdentifier
                collectionView.setUpListFlowLayout()
                collectionView.scrollToTop()
                collectionView.reloadData()
    

• alert을 이용한 Error Handling

  • OpenMarket app에서 발생한 error는 alert 창을 띄워 error를 나타내었다.
  • localizedError 프로토콜의 errorDescription을 이용하여 description을 정의하였고 error.localizedDescription으로 error Message를 출력하도록 에러처리.

• 상품등록 버튼 Segue

  • HIG를 참고하여 상품등록 버튼을 눌렀을 때 Navigation 형태가 아니라 Modal로 띄우도록 구성
  • Navigation Bar를 활용하여 취소 버튼을 구성

• collectionview의 flowlayout을 변경할 때 AutoLayout 충돌 관련 경고가 뜨는데, 해결 방법을 모르겠다.
• SegmentControl을 활용하여 List나 Grid를 전환할 때 생기는 약간의 딜레이의 원인을 모르겠다.

• 상황 FlowLayout을 활용하여 화면을 전환할 때, 스크롤이 상단에 위치하는게 아니라 제멋대로인 위치에 가있는 현상이 발생했다.

• 이유 레이아웃이 서로 다르기 때문에 스크롤의 좌표도 다른 것으로 추측이 되었다.

• 해결 따라서 이 부분을 화면을 전환할 때 스크롤의 위치를 상단에 위치하게 설정해주니 해결되었다.

  extension UIScrollView {
      func scrollToTop() {
          let topOffset = CGPoint(x: 0, y: -contentInset.top)
          setContentOffset(topOffset, animated: false)
      }
  }

• UICollectionView UICollectionViewFlowLayout
• Networking을 통한 뷰에 대한 비동기 처리
• reloadData
• Xib File
• UISegmentedControl
• UIActivityIndicatorView

• Asset에 등록되어있는 이미지 설정값 수정
• 전반적인 네이밍 수정
• 삼항연산자로 조건문 개선
• 빠져있는 접근제어 추가
• 동적으로 레이아웃을 잡을 수 있도록 UICollectionViewDelegateFlowLayout을 채택
  • 가로모드, 세로모드에서도 레이아웃이 뭉개지지 않도록 개선

서버에서 받은 데이터를 로컬에 캐시합니다.

• Pagination
  • 스크롤이 하단에 가까워지면 다음 페이지를 로드하도록 구성
  • scrollViewDidScroll()를 이용하여 구현
• Cache

  • 앱이 실행하는 동안 캐시를 가지고 있을 수 있도록 싱글톤 패턴으로 ImageManager 타입을 생성

  • Cell에서 URL로 이미지를 가져오던 부분을 캐싱처리를 하도록 구성

    if let cachedImage = ImageManager.shared.loadCachedData(for: url) {
        productImageView.image = cachedImage
    } else {
        ImageManager.shared.downloadImage(with: url) { image in
        ImageManager.shared.setCacheData(of: image, for: url)
        self.productImageView.image = image
        }
    }

• 스크롤 시 yOffset을 비정상적으로 카운트가 되는 부분이 문제여서 아직 풀리지 않았는데, 해당 부분을 그냥 넘어가도 되는건지[?] 약간의 찝찝함이 남는다.

• 항상 뜨는 에러는 아니고, 간헐적으로 뜨는 에러다. 스크롤을 하단까지 했을 때 디코딩에 실패하는 경우가 있습니다. 디버깅을 해보니 결과는 아래와 같다.
• 
• 
• 
• 뷰컨트롤러 쪽에서 네트워크 매니저의 fetch 메소드를 사용하는 쪽에서 에러가 나는 것 같은데, 네트워크에서는 success로 데이터를 가져오긴 했으나 조회를 해보면 데이터가 비어있는 것을 확인할 수 있었다.
• 
• Response를 확인해보면 204코드로 응답하고 있다.
• 
• 스크롤 하는 부분에 중단점을 찍고 확인해보니 currentPage가 104가 되어있는 것도 확인되었다. 저 조건문이라면 104가 될 수가 없는데.. 어느순간 스크롤을 계산하는 부분(yOffset)에서 에러가 발생해서 비정상적으로 currentPage가 올라가는 것 같다.

if heightRemainBottomHeight < frameHeight ,
   let page = page, page.hasNext, page.pageNumber == currentPage {
    currentPage += 1
    self.requestProducts()
}

• 따라서 위와 같이 조건문을 하나 더 추가해서 안전하게 currentPage를 더해줄 수 있도록 임시방편으로 수정해주었다.

• UIScrollViewDelegate를 이용한 pagination 구현
• Cache의 대한 개념
  • Caching의 범위

• 코딩 컨벤션에 맞추어 메소드의 줄바꿈을 수정
• 사용하지 않는 타입을 삭제
• 네이밍 개선
• API 문서를 다시 검토하여 타입을 개선
• 중복되는 코드를 제거하여 리팩토링

상품등록, 상품수정 화면을 구현합니다.

• CollectionView의 Cell과 Header를 활용하여 View를 구성
• 이미지가 5장이 채워졌을 때 경고얼럿을 띄우도록 구현
• 사용자 입장에서 몇개의 이미지가 추가되었는지 시각적으로 확인할 수 있도록 Label 추가 구현
• 이미지 파일 용량이 300KB 이상일 경우 UIGraphicsImageRenderer를 이용하여 20퍼센트씩 resize

• 요구사항과 API 문서를 참고하여 사용자가 제대로 입력하지 않은 부분이 있다면 얼럿을 통해 어떤 부분이 잘못되었는지 알려줄 수 있도록 구현

• 화면 전환 시 상품 등록, 상품 수정 flag를 전달하여 해당 flag에 대한 분기처리를 통해서 ViewController를 재활용

• 점점 커져가는 ViewController를 다이어트 시키기 위해 DataSource를 따로 타입으로 빼두어 ViewController에 주입시키도록 구현

• ScrollView와 NotificationCenter 두가지를 활용
• ScrollView의 contentInset을 keyboardFrame 사이즈의 높이만큼 버퍼를 추가하여 스크롤뷰를 확장하므로써 키보드 가림현상을 해결
  • scrollIndicatorInsets도 같이 변경해주도록 하였음.
• TextView에서 줄바꿈을 하면서 커서가 내려갈 때 따라갈 수 있도록, ScrollEnabled 기능을 false로 비활성화
• 이후 오토레이아웃으로 TextView의 높이가 늘 고정되어있는 것이 아니라 늘어날 수도 있도록 priority를 조절.
  • 텍스트뷰가 안에 Text가 길어질 수록 높이가 늘어나고, 그에 따라 스크롤도 자동으로 내려온다.

• ImageHeaderView와 RegisterViewController, 각각의 ViewController들 사이에 소통을 위해서 notification을 사용

• 상품 등록, 상품 수정 시 각 ViewController에 notification post하여 update 이벤트를 전달
• 이벤트 전달 시 API에 request하여 데이터를 업데이트
• MainViewController의 CollectionView를 아래로 쓸어내렸을 때 새 데이터를 받아오는 기능 추가 구현

• 상품 등록시에는 API 명세를 따르기 위해 비밀번호가 이미 세팅된 상태로 등록이 되지만, 상품 수정시에는 수정하기 전 비밀번호를 먼저 확인하고, 비밀번호가 맞다면 수정할 수 있도록 구현

• Delegate vs Notification

  view와 Controller 사이에 소통하는 방식으로 delegate pattern을 사용하였는데 DataSource 파일을 분리하면서 DataSource에서 delegate 객체를 이용하는 것에 한계를 느껴 Notificationcenter를 사용

• 키보드를 내리는 방법 중 Recognizer방법을 사용하면 CollectionView의 Delegate 메소드 호출이 먹통이 되는데 어떤 문제인지 모르겠다.

  // Recognizer를 활용해서 뷰컨을 터치하면 키보드 사라지는 메소드
  func hideKeyboard() {
      let tap = UITapGestureRecognizer(target: self, action: #selector(UIViewController.dismissKeyboard))
      view.addGestureRecognizer(tap)
  }
  
  @objc func dismissKeyboard() {
      view.endEditing(true)
  }

• cell을 dequeueReusable 해주는 구간에서 productList를 통해 셀을 구성하고 있는데, MainViewController을 업데이트 할 때 간헐적으로 out of range 에러가 나는데, 정확한 에러 원인을 찾지 못했다.

• 점점 비대해지는 ViewController를 어떻게 다이어트 시켜야하는지...

• 사용자 친화적인 UI/UX 구현
• URLSession을 활용한 multipart-form 요청 전송
• UIImagePickerController를 활용하여 사용자 앨범에 접근하는 방법
• UIGraphicsImageRenderer 를 이미지를 랜더링 하여 압축하는 방법
• UICollectionReusableView를 사용하는 방법
• UIScrollView를 활용하여 키보드가 컨텐츠를 가리는 부분을 해결하는 방법
• UITextField를 가지고 있는 UIAlertController 활용
• UIRefreshControl를 활용하는 방법

• AlertConstant를 제거하고 UIViewController+extension 부분 전체적으로 개선
• CollectionView에서 refreshing할 때 index out of range 에러나는 부분 비동기적으로 처리하여 해결
• 네이밍 부분 전체적으로 개선
• 앨범에 접근할 때 보여지는 description을 수정하여 개선
• HIG를 참고하여 alert, ActionSheet의 버튼 순서를 변경하여 개선
• DetailViewController 메소드 순서를 개선
• 이미지 삭제시 'x'버튼을 클릭했을 때 삭제되도록 수정하여 개선
• 셀을 선택했을 때 변화가 발생하도록 selectedBackgroundView 설정

상품의 상세내용을 확인할 수 있는 화면을 구현합니다

• 사용자가 볼 필요가 없는 에러의 경우(네트워크 에러) 얼럿을 띄우는 대신 print를 호출하도록 하여, 에러처리를 해주었다.

• UIFont가 제공하는 preferredFont를 사용하면 따로 굵기를 지정할 수 없고, 지정된 font만 사용해야한다. 반대로 systemFont를 사용한다면 Dynamic Type이 동작하지 않는다.
  • extension을 통해서 굵기를 지정해도 Dynamic Type을 지원하는 메소드를 구현하여 이 문제를 해결.

• 다른 마켓앱을 살펴보면 상세보기로 넘어갈 때 보고있던 이미지를 그대로 넘겨서 상세보기 화면으로 전환되는 것을 확인할 수 있다. 사용자 입장에서도 보고있던 이미지를 그대로 상세보기로 보는 것이 바람직하다고 판단되어 해당 기능을 구현하였다.

  DispatchQueue.main.asyncAfter(deadline: .now() + 0.01) {
      let indexPath = IndexPath(item: self.currentPage, section: 0)
      self.collectionView.scrollToItem(
          at: indexPath,
          at: [.centeredHorizontally, .centeredVertically],
          animated: false
       )
  }

Make destructive choices prominent. Use red for buttons that perform destructive or dangerous actions, and display these buttons at the top of an action sheet.

• 이 부분을 참고하여, 요구사항처럼 삭제 버튼을 아래에 위치해있는게 아니라 상단에 위치하도록 개선하였다.

• 상세페이지화면에서 이미지를 표시할 때 CollectionView의 isPaging과 가로 스크롤을 통해서 paging을 구현하였다.
• pageControl을 이용하여 현재 페이지의 위치를 나타내고 PageControl의 dot을 탭하여도 페이지가 전환되는 기능을 구현하였다.

• DetailViewController → ImageDetailViewController 로 전환할 때 viewDidLoad에서 collectionView.scrollToItem을 호출 시 제대로 적용되지 않아서 DispatchQueue.main.asyncAfter를 활용했는데, 적절한지 잘 모르겠다.
• 네트워킹 하는 부분이 마치 산맥..처럼 공포의 들여쓰기가 생겨났는데, 이러한 부분을 어떻게 더 개선할 수 있을지 더이상 떠오르지가 않는다.
• CollectionView에 FlowLayout을 따로 코드로 대입해주지 않으면, UICollectionViewDelegateFlowLayout를 채택하여 레이아웃을 설정해주어도 레이아웃이 적용되지 않는데, 왜그런걸까?
• ScrollView를 활용해서 Zoom을 구현하였지만, 디바이스 전체 크기에 맞춰서 Scale을 설정하는 방법을 모르겠다.
  • 이미지가 디바이스 크기보다 커지지 않도록 하고싶다...

• 문제 메인페이지에서 상세페이지로 넘어갈때 이미지가 보이지 않는 현상이 나타났다.

• 이유 이미지를 네트워킹하는 속도보다 화면이 전환되는 시점이 빨라서 나타나는 문제였다.

• 해결 DispatchGroup을 이용하여 네트워킹하는 코드 내 completion 탈출 클로저가 모두 끝난 시점에 View를 띄워주도록 하였다.

  dispatchGroup.enter()
  ImageManager.shared.downloadImage(with: newImage.url) { image in
      ImageManager.shared.setCacheData(of: image, for: newImage.url)
      self.images.append(image)
      dispatchGroup.leave()
  }
  
  dispatchGroup.notify(queue: .main) {
      DispatchQueue.main.async {
          // view 
          ...
      }
  }

• 문제 CollectionView의 isPagingEnabled을 true로 주면 아래 예시와 같이 스크롤 했을 때 cell이 조금씩 밀리는 현상이 나타났다.

  

• 이유 CollectionView의 경우 minimumLineSpacing이 기본적으로 값(10.0)이 들어가있다. 해당 값 때문에 스크롤 시 밀림현상이 있었던 것이였다.

• 해결 minimumLineSpacing을 0으로 설정해주니 스크롤 시 cell이 조금씩 밀리던 현상이 해결되었다.

  

• 문제 UITableView같은 경우에는 기본적으로 seleted 했을 때, 회색 배경이 사라지지 않아서 delegate 메소드를 활용하여 deselect를 해주어야 배경색이 다시 원래대로 돌아왔었다.

• 이유 하지만 UICollectionView 같은 경우에는 아무것도 설정되어있지 않기 때문에 이 부분을 직접 설정을 해주어야 한다.

• 해결 셀을 초기화할 때 selectedBackgroundView를 지정해주고, backgroundColor를 입혀준다.

  cell.selectedBackgroundView = UIView(frame: self.bounds)
  cell.selectedBackgroundView?.backgroundColor = .systemGray5

  

• 하지만 위와 같이 배경색이 바뀐 채로 남아있다. 따라서 Delegate 메소드 중 didSelectItemAt를 구현하여 deselectItem를 호출해서 셀 선택을 해제시켜주어야 한다.

  // UICollectionViewDelegate...
  func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
      collectionView.deselectItem(at: indexPath, animated: true)
  }

  

  • 이렇게 하면 정상적으로 셀을 선택했을 때, 선택되었다는 효과가 일어나면서 화면전환이 되는 것을 확인할 수 있다.

• UIPageControl을 활용하는 방법
• GestureRecognizer를 통해 특정 터치이벤트를 처리하는 방법
• UIScrollView의 Zoom 기능을 활용하는 방법
• UIFontMetrics를 활용하여 Custom Font에도 Dynamic Type을 적용해보는 방법
• UICollectionView를 활용해서 Paging을 구현하는 방법
• UIAlertController의 텍스트 필드를 추가하여 활용하는 방법
  • Handler 활용