Fix/#152 인스타그램 이미지 조회 문제 수정 #155
Conversation
| ForEach(unclassifiedContents) { content in | ||
| let isFirst = content == unclassifiedContents.first | ||
| let isLast = content == unclassifiedContents.last | ||
|
|
||
| PokitLinkCard( | ||
| link: content, | ||
| action: { send(.컨텐츠_항목_눌렀을때(content)) }, | ||
| kebabAction: { send(.미분류_케밥_버튼_눌렀을때(content)) } | ||
| ForEachStore( | ||
| store.scope(state: \.contents, action: \.contents) | ||
| ) { store in | ||
| let isFirst = store.state.id == self.store.contents.first?.id | ||
| let isLast = store.state.id == self.store.contents.last?.id | ||
|
|
||
| ContentCardView( | ||
| store: store, | ||
| isFirst: isFirst, | ||
| isLast: isLast |
There was a problem hiding this comment.
ForEachStore대신 ForEach로 할 수 있으니까 수정해주세욥.
수정한 방법이 soft-deprecated되었음..
There was a problem hiding this comment.
ForEachStore대신 ForEach로 할 수 있으니까 수정해주세욥. 수정한 방법이 soft-deprecated되었음..
엇 deprecated된게 맞을텐데.. 자꾸 컴파일 오류가 나서 이렇게 하긴 했는데.. 다시 한번 해보겠습니다..!!
| final class SwiftSoupProvider { | ||
| func parseOGTitle(_ url: URL) async throws -> String? { | ||
| try await parseOGMeta(url: url, type: "og:title") | ||
| } | ||
|
|
||
| func parseOGImageURL(_ url: URL) async throws -> String? { | ||
| try await parseOGMeta(url: url, type: "og:image") | ||
| } | ||
|
|
||
| func parseOGMeta(url: URL, type: String) async throws -> String? { | ||
| let html = try String(contentsOf: url) | ||
| let document = try SwiftSoup.parse(html) | ||
|
|
||
| if let metaData = try document.select("meta[property=\(type)]").first()?.attr("content") { | ||
| return metaData | ||
| } else { | ||
| var request = URLRequest(url: url) | ||
| request.setValue( | ||
| "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36", | ||
| forHTTPHeaderField: "User-Agent" | ||
| ) | ||
|
|
||
| let (data, _) = try await URLSession.shared.data(for: request) | ||
| guard let html = String(data: data, encoding: .utf8) else { | ||
| return nil | ||
| } | ||
| let document = try SwiftSoup.parse(html) | ||
| let metaData = try document.select("meta[property=\(type)]").first()?.attr("content") | ||
|
|
||
| return metaData | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
provider 분리하니까 훨낫네요 뷰에 저 코드들 있던것으로 기억하는데 잘 뺀듯
There was a problem hiding this comment.
provider 분리하니까 훨낫네요 뷰에 저 코드들 있던것으로 기억하는데 잘 뺀듯
감사합니다
| /// - Reducer body | ||
| public var body: some ReducerOf<Self> { | ||
| Reduce(self.core) | ||
| ._printChanges() |
There was a problem hiding this comment.
요런거 까먹지 말구 지워서 올리기
저도 몇개 남겨놧던것같음 ㅋㅋ; 같이지워주세엽
There was a problem hiding this comment.
요런거 까먹지 말구 지워서 올리기 저도 몇개 남겨놧던것같음 ㅋㅋ; 같이지워주세엽
넵!
| var contents: IdentifiedArrayOf<BaseContentItem>? { | ||
| guard let contentList = domain.contentList.data else { | ||
| return nil | ||
| } | ||
| var identifiedArray = IdentifiedArrayOf<BaseContentItem>() | ||
| contentList.forEach { content in | ||
| identifiedArray.append(content) | ||
| } | ||
| return identifiedArray | ||
| } | ||
| var contents: IdentifiedArrayOf<ContentCardFeature.State> = [] |
모듈이 많아진 느낌이라 정말 고민 많이 했습니다... 그래서 더 늦어진 것도 있는거 같아요... |
아....횟수제한이면 어쩔수없네요 😂 지금 할 수 있는 최선을 한거였네..! 채고입니다 |
#️⃣연관된 이슈
📝작업 내용
SwiftSoupClient로직 변경FeatureContentCard모듈,ConentCardFeature,ContentCardView추가PokitLinkCard컴포넌트 추가 로직 작성스크린샷 (선택)
💬리뷰 요구사항(선택)
SwiftSoupClient로직을 변경하였습니다.og이미지와 제목을 한번에 파싱하는 기존 로직과 다르게 이미지, 제목 파싱을 따로 작성하였습니다.og이미지, 제목 파싱을 분리하면서,SwiftSoupClient의Dependecies주입이 복잡해질 가능성이 있어, 내부 로직은SwiftSoupProvider로 분리하였습니다.Completion방식 자체가 가독성이 떨어지기도 하고, 이미지, 제목으로 분리 했다보니 코드 가독성이 더 떨어질 가능성이 있어, 기존Compleion방식에서async/await방식으로 바꿨습니다.FeatureContentCard모듈,ConentCardFeature,ContentCardView추가하였습니다.PokitLinkCard컴포넌트는 디자인 시스템에 있다보니 비지니스 로직을 넣을 수 가 없습니다.또한, 리스트를 탐색해서 이미지 url 파싱을 다시 하면, 링크 개수가 많이지면 많아 질 수 록 비용이 많이 들 가능성이 있습니다.
그래서, 리스트 항목 단위로 이미지 오류을 판단하고 재조회하는 로직을 넣기 위해,
ConentCardFeature,ContentCardView를 추가하였습니다.RemindFeature에는 추천링크 3개, 즐겨찾기 링크 3개, 읽지않음 링크 3개가 고정이기 때문에,RemindFeature만 예외적으로 이미지 url 파싱 로직을Reducer안에 바로 넣었습니다.PokitLinkCard컴포넌트 추가 로직 작성하였습니다.fetchMetaData라는 함수를 추가적으로 받습니다.LazyImage의phase의error가 발생했을때의 뷰의onAppear시점으로 이미지 url 파싱을 진행합니다.문제점
인스타그램을 기준으로, 여러번 테스트를 해보니, 짧은 시간에 너무 많은 요청을 인스타그램으로 보내면, ip 단위로 요청을 막아버리는 것 같습니다.
인스타그램, 페이스북에서는 이러한 직접적인 파싱을 권장하지 않고, 자체 API를 제공하고 있습니다.
close #152