Scarriffle/onyx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Das Wetter sah nur einmal je Programmstart nach

Browse Source 
Der Abruf hing an `task` der View — und die erscheint bei einem Panel, das
immer existiert, ein einziges Mal. Wer Onyx tagelang laufen lässt, sieht
tagelang dieselbe Temperatur. Gemessen: der Zwischenspeicher trug den Stand
vom 14. August, 15:17, und stand am 17. um 23:13 unverändert da — nachts
35 Grad, weil sie vom Nachmittag drei Tage vorher stammten.

Jetzt sieht das Modell im Takt der Haltbarkeit nach, also alle fünfzehn
Minuten. Teurer wird es dadurch nicht: `load` fragt nur nach, wenn der
gespeicherte Stand abgelaufen ist. Und der Ort wird dabei neu bestimmt — beim
Wechsel der Stadt stand sonst weiter das Wetter der alten da. Genau das war
hier zu sehen: Lemgo, längst verlassen.

Dazu die zweite Hälfte des Fehlers: ein „fertiger" Stand wurde immer als
frisch gezeichnet, egal wie alt er war. Warnzeichen und Zeitstempel gab es nur
bei einem gescheiterten Abruf. Ein Messwert gilt jetzt ab einer Stunde als
alt — großzügiger als die Haltbarkeit, damit ein einzelner verpasster Abruf
noch keine Warnung auslöst.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Scarriffle
2026-08-17 23:23:50 +02:00
parent ee848b86cc
commit 4dee12e4b9
4 changed files with 88 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
Packages/OnyxKit/Sources/WeatherProvider/WeatherWidget.swift
View File

@@ -34,6 +34,8 @@ public final class WeatherModel {
private let defaults: UserDefaults
private var cache = WeatherCache()
private var task: Task<Void, Never>?
/// Der Takt, in dem nachgesehen wird.
private var timer: Timer?
public init(source: WeatherKitSource = WeatherKitSource(),
defaults: UserDefaults = .standard) {
@@ -80,6 +82,8 @@ public final class WeatherModel {
/// Panels. WeatherKit rechnet nach Abrufen ab, und das Wetter ändert sich
/// nicht im Sekundentakt.
public func load(force: Bool = false) {
startTimerIfNeeded()
if !force, let fresh = cache.valid(for: place) {
state = .ready(fresh)
return
@@ -113,9 +117,32 @@ public final class WeatherModel {
}
}
/// Sieht regelmäßig nach.
///
/// **Ohne das lud das Widget genau einmal je Programmstart.** Der Aufruf
/// hing an `task` der View, und die erscheint bei einem Panel, das immer
/// existiert, ein einziges Mal. Wer Onyx tagelang laufen lässt, sah
/// tagelang dieselbe Temperatur — nachts 35 Grad, weil sie vom Nachmittag
/// des Vortags stammte.
///
/// Der Takt entspricht der Haltbarkeit des Zwischenspeichers. Teurer wird
/// es dadurch nicht: `load` fragt nur nach, wenn der gespeicherte Stand
/// abgelaufen ist. Und der Ort wird dabei neu bestimmt — wer die Stadt
/// wechselt, sieht sonst weiter das Wetter der alten.
private func startTimerIfNeeded() {
guard timer == nil else { return }
let timer = Timer(timeInterval: WeatherCache.maxAge, repeats: true) { [weak self] _ in
Task { @MainActor in self?.load() }
}
RunLoop.main.add(timer, forMode: .common)
self.timer = timer
}
public func stop() {
task?.cancel()
task = nil
timer?.invalidate()
timer = nil
}
/// Sucht die Koordinaten zu einem Ortsnamen und übernimmt ihn.
@@ -159,7 +186,11 @@ private struct WeatherWidgetView: View {
Group {
switch model.state {
case .ready(let snapshot):
content(snapshot, isStale: false)
// Auch ein „fertiger" Stand kann alt sein — dann ist er als
// solcher zu kennzeichnen. Genau das fehlte: drei Tage alte
// 35 Grad standen um elf Uhr nachts da, als wären sie eben
// gemessen worden.
content(snapshot, isStale: WeatherSnapshot.isStale(asOf: snapshot.asOf))
case .unavailable(let lastKnown, _):
if let lastKnown {
content(lastKnown, isStale: true)