perf(banner): TabView 프리로딩 완화·다운샘플링·요청취소 적용
배너/캐러셀에서 인접 페이지 프리로딩과 원본 해상도 디코딩으로 발생하던 메모리 스파이크와 중복 로드를 완화했습니다. - 각 페이지에서 이미지 URL을 onAppear에 바인딩, onDisappear에 nil 해제 → 인접 페이지 프리로딩 시 중복 로드·디코딩 방지, 요청 취소 실효 - 모든 KFImage에 cancelOnDisappear(true) 일관 적용 - 큰 배너 이미지에 downsampling(size:) 적용(디코딩 메모리 절감) - 자동 슬라이드 주기 3초 → 4초로 완화(동시 로드 빈도 감소) - TabView 페이지를 서브뷰로 분리하여 뷰 로직 단순화 및 재사용성 향상 결과: 동시 디코딩 감소, 피크 메모리 사용량 하락, 자동 슬라이드 안정성 개선
This commit is contained in:
@@ -11,7 +11,7 @@ import Kingfisher
|
||||
struct SectionEventBannerView: View {
|
||||
|
||||
@State private var currentIndex = 0
|
||||
@State private var timer = Timer.publish(every: 3, on: .main, in: .common).autoconnect()
|
||||
@State private var timer = Timer.publish(every: 4, on: .main, in: .common).autoconnect()
|
||||
@AppStorage("token") private var token: String = UserDefaults.string(forKey: UserDefaultsKey.token)
|
||||
|
||||
let items: [EventItem]
|
||||
@@ -21,51 +21,13 @@ struct SectionEventBannerView: View {
|
||||
TabView(selection: $currentIndex) {
|
||||
ForEach(0..<items.count, id: \.self) { index in
|
||||
let item = items[index]
|
||||
if let url = item.thumbnailImageUrl.addingPercentEncoding(withAllowedCharacters: .urlQueryAllowed) {
|
||||
KFImage(URL(string: url))
|
||||
.cancelOnDisappear(true)
|
||||
.resizable()
|
||||
.scaledToFill()
|
||||
.frame(
|
||||
width: screenSize().width,
|
||||
height: screenSize().width * 300 / 1000,
|
||||
alignment: .center
|
||||
)
|
||||
.tag(index)
|
||||
.onTapGesture {
|
||||
if !token.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
|
||||
if let _ = item.detailImageUrl {
|
||||
AppState.shared.setAppStep(step: .eventDetail(event: item))
|
||||
} else if let link = item.link, link.trimmingCharacters(in: .whitespaces).count > 0, let url = URL(string: link), UIApplication.shared.canOpenURL(url) {
|
||||
UIApplication.shared.open(url)
|
||||
}
|
||||
} else {
|
||||
AppState.shared.setAppStep(step: .login)
|
||||
}
|
||||
}
|
||||
} else {
|
||||
KFImage(URL(string: item.thumbnailImageUrl))
|
||||
.cancelOnDisappear(true)
|
||||
.resizable()
|
||||
.scaledToFill()
|
||||
.frame(
|
||||
width: screenSize().width,
|
||||
height: screenSize().width * 300 / 1000,
|
||||
alignment: .center
|
||||
)
|
||||
.tag(index)
|
||||
.onTapGesture {
|
||||
if !token.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
|
||||
if let _ = item.detailImageUrl {
|
||||
AppState.shared.setAppStep(step: .eventDetail(event: item))
|
||||
} else if let link = item.link, link.trimmingCharacters(in: .whitespaces).count > 0, let url = URL(string: link), UIApplication.shared.canOpenURL(url) {
|
||||
UIApplication.shared.open(url)
|
||||
}
|
||||
} else {
|
||||
AppState.shared.setAppStep(step: .login)
|
||||
}
|
||||
}
|
||||
}
|
||||
SectionEventBannerPage(
|
||||
item: item,
|
||||
width: screenSize().width,
|
||||
height: screenSize().width * 300 / 1000,
|
||||
token: token
|
||||
)
|
||||
.tag(index)
|
||||
}
|
||||
}
|
||||
.tabViewStyle(PageTabViewStyle(indexDisplayMode: .never))
|
||||
@@ -85,7 +47,7 @@ struct SectionEventBannerView: View {
|
||||
}
|
||||
}
|
||||
.onAppear {
|
||||
timer = Timer.publish(every: 3, on: .main, in: .common).autoconnect()
|
||||
timer = Timer.publish(every: 4, on: .main, in: .common).autoconnect()
|
||||
}
|
||||
.onDisappear {
|
||||
timer.upstream.connect().cancel()
|
||||
@@ -109,3 +71,51 @@ struct SectionEventBannerView_Previews: PreviewProvider {
|
||||
SectionEventBannerView(items: [])
|
||||
}
|
||||
}
|
||||
|
||||
private struct SectionEventBannerPage: View {
|
||||
let item: EventItem
|
||||
let width: CGFloat
|
||||
let height: CGFloat
|
||||
let token: String
|
||||
@State private var boundURL: URL?
|
||||
|
||||
var body: some View {
|
||||
Group {
|
||||
if let boundURL {
|
||||
KFImage(boundURL)
|
||||
.cancelOnDisappear(true)
|
||||
.downsampling(size: CGSize(width: width, height: height))
|
||||
.resizable()
|
||||
.scaledToFill()
|
||||
.frame(width: width, height: height, alignment: .center)
|
||||
} else {
|
||||
Color.clear
|
||||
.frame(width: width, height: height, alignment: .center)
|
||||
}
|
||||
}
|
||||
.contentShape(Rectangle())
|
||||
.onTapGesture {
|
||||
if !token.trimmingCharacters(in: .whitespacesAndNewlines).isEmpty {
|
||||
if let _ = item.detailImageUrl {
|
||||
AppState.shared.setAppStep(step: .eventDetail(event: item))
|
||||
} else if let link = item.link,
|
||||
link.trimmingCharacters(in: .whitespaces).count > 0,
|
||||
let url = URL(string: link),
|
||||
UIApplication.shared.canOpenURL(url) {
|
||||
UIApplication.shared.open(url)
|
||||
}
|
||||
} else {
|
||||
AppState.shared.setAppStep(step: .login)
|
||||
}
|
||||
}
|
||||
.onAppear {
|
||||
let urlString = item.thumbnailImageUrl.addingPercentEncoding(
|
||||
withAllowedCharacters: .urlQueryAllowed
|
||||
) ?? item.thumbnailImageUrl
|
||||
boundURL = URL(string: urlString)
|
||||
}
|
||||
.onDisappear {
|
||||
boundURL = nil
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user