@uuu1101
안녕하세요 태태!
박스오피스 II step3 PR입니다.

step3 구현을 완료하여 PR보냅니다.

리뷰 잘 부탁드리겠습니다!

## 고민한 점

### NSCache대신 URLCache를 사용한 이유
NSCache는 in-memory에서 데이터를 가져오기 때문에 on-disk를 사용하는 URLCache보다 빠르다는 장점이 있습니다.
하지만 NSCache는 비휘발성이기에 앱을 종료하고 다시 들어온다면 네트워크와의 연결을 재시도해야합니다. 또한 in-memory는 메모리 청크에 할당하기 때문에 메모리에 많은 데이터들이 올라와있는 상황이라면 성능이 저하될 수 있습니다.

반면에 URLCache는 in-memory와 on-disk방식 둘 다 사용할 수 있고 on-disk의 경우 디스크를 사용하기 때문에 메모리 청크를 할당하지않습니다. 또한 디스크의 크기를 정할 수 있다는 점에서 유연합니다. [NSCache, URLCache](https://medium.com/@master13sust/to-nscache-or-not-to-nscache-what-is-the-urlcache-35a0c3b02598)

그리고 네트워크를 통해 받아온 데이터의 크기는 약 6000바이트로 상당히 크기에 NSCache로 구현한 경우 메모리가 금방 초과되어 제거될 가능성이 높다고 생각했습니다. 따라서 on-disk방식의 URLCache를 사용하는 것으로 결정했습니다.

### Query parameter에 따른 캐시
처음 구현할 때 저희는 query의 순서를 고려하지 않고 forEach문을 사용하여 query를 추가하였습니다.

self.queries.forEach { queryItem in
    let queryItem = URLQueryItem(name: queryItem.key, value: queryItem.value)
    queriesItem.append(queryItem)
}

그 결과 쿼리의 순서가 무작위로 더해져 cache를 할때 cache를 저장하는 값이 달라져 같은 데이터임에도 다른 캐시를 저장할때가 있었습니다. 이러한 문제점을 해결하고자 저희는 sorted메서드를 사용하여 순서를 보장하여 query를 추가하여 같은 캐시를 저장할 수 있게 하였습니다.

self.queries.sorted(by:<).forEach { queryItem in
    let queryItem = URLQueryItem(name: queryItem.key, value: queryItem.value)
    queriesItem.append(queryItem)
}

## 조언 받고 싶은 점
저희가 지금은 잔존기한을 설정해주지 않아서 잔존기한에 따른 자동삭제 부분을 구현하지 못했습니다. 만약 잔존기한을 설정한다고 하면 아래의 코드와 같이 1일정도로 잔존기한을 설정할 거 같습니다.

userInfo: ["expirationDate": Calendar.current.date(byAdding: .day,
                                                   value: 1,
                                                   to: Date()) ?? Date()]

userInfo에는 Any타입을 받기 때문에 이를 사용하여 잔존기한을 설정하는 것이 좋다고 생각했습니다. 여기서 궁금한 점은 이 잔존기한을 이용하여 캐시를 삭제해야하는데 어디서 검사하고 어떤 방식으로 삭제해야하는지 모르겠습니다.

1. 캐시된 데이터가 있는지 확인 후 삭제
캐시된 데이터가 있는지 확인 후에 추가적으로 그 함수 안에서 데이터가 만료가 되었다면 그 캐시를 삭제하면 되기 때문에 on-disk메모리 안에서 지워져서 문제가 되지 않지만,
캐시된 데이터가 있는지 확인을 안한다면(사용자가 한 번 클릭했던 날짜를 오랜기간 누르지 않았다면) 만료된 캐시가 삭제되지 않고 오랜기간 남아있을 것 같습니다. 이 경우에는 만료되었지만 on-disk에 오랜기간 남아있는 것이 일반적일까요?

2. 모든 캐시 잔존기한 확인 후 삭제
1번의 문제로 인해서 2번의 방안을 생각해봤습니다. 앱 실행 시 앱에 존재하는 모든 캐시된 데이터들의 잔존기한을 확인한 후 지난 것 데이터들은 지워주는 방법을 생각했습니다. 하지만 이 방법은 앱 실행 시 모든 캐시들의 만료기한을 확인하기 때문에 비용이 클 수도 있겠다고 생각했습니다.

캐시정책으로 `useProtocolCachePolicy`을 사용하는 경우에는 서버의 정책을 따르게 되는데 현재 API응답헤더에는 정책을 정의해주지 않고 있어서 클라이언트에서 정책을 설정해줘야했습니다. 따라서 1번방법과 2번방법에 대해서 고민을 했는데요!
현재 상황에서 URLCache를 사용하면서 어떤 방법으로 만료된 캐시를 삭제하는 것이 좋을까요?(아니면 만료된 캐시를 굳이 삭제해줄 필요가 없는걸까요?)