fix: chart

Author: whes1015

lib/app/new_home/_models/home_model.dart

@@ -28,8 +28,6 @@ class HomeModel extends ChangeNotifier {
   RealtimeWeather? _weather;
   List<History> _alerts = const [];
   Map<String, dynamic>? _forecast;
-  bool _isLoading = false;
-  Object? _error;
   Timer? _autoRefreshTimer;
 
   /// Creates a [HomeModel] backed by [settingsLocation].
@@ -44,58 +42,35 @@ class HomeModel extends ChangeNotifier {
     if (_temporaryCode == null) _doRefresh();
   }
 
-  Future<void> _doRefresh() async {
-    final code = _temporaryCode ?? _settingsLocation.code;
-    double? lat;
-    double? lon;
-
-    if (code != null) {
-      final loc = Global.location[code];
-      if (loc != null) {
-        lat = loc.lat;
-        lon = loc.lng;
-      }
+  /// Runs [task] and logs failures under [tag]; returns `null` on error.
+  static Future<T?> _safe<T>(String tag, Future<T> Function() task) async {
+    try {
+      return await task();
+    } catch (e) {
+      TalkerManager.instance.error('HomeModel $tag', e);
+      return null;
     }
+  }
 
-    lat ??= _settingsLocation.coordinates?.latitude;
-    lon ??= _settingsLocation.coordinates?.longitude;
-
+  Future<void> _doRefresh() async {
+    final code = _temporaryCode ?? _settingsLocation.code;
+    final loc = code != null ? Global.location[code] : null;
+    final lat = loc?.lat ?? _settingsLocation.coordinates?.latitude;
+    final lon = loc?.lng ?? _settingsLocation.coordinates?.longitude;
     if (lat == null || lon == null) return;
 
-    _isLoading = true;
-    notifyListeners();
-
     // Fetch weather + alerts + forecast in parallel.
     final results = await Future.wait<Object?>([
-      Global.api.getWeatherRealtimeByCoords(lat, lon).then<Object?>((v) => v).catchError((e) {
-        TalkerManager.instance.error('HomeModel weather', e);
-        return null;
-      }),
-      code == null
-          ? Future<Object?>.value(null)
-          : Global.api.getRealtimeRegion(code).then<Object?>((v) => v).catchError((e) {
-              TalkerManager.instance.error('HomeModel alerts', e);
-              return null;
-            }),
-      code == null
-          ? Future<Object?>.value(null)
-          : Global.api.getWeatherForecast(code).then<Object?>((v) => v).catchError((e) {
-              TalkerManager.instance.error('HomeModel forecast', e);
-              return null;
-            }),
+      _safe('weather', () => Global.api.getWeatherRealtimeByCoords(lat, lon)),
+      if (code != null) _safe('alerts', () => Global.api.getRealtimeRegion(code)) else Future.value(null),
+      if (code != null) _safe('forecast', () => Global.api.getWeatherForecast(code)) else Future.value(null),
     ]);
 
-    final weather = results[0];
-    final alerts = results[1];
-    final forecast = results[2];
-
-    _weather = weather is RealtimeWeather ? weather : _weather;
-    _alerts = alerts is List<History>
-        ? alerts.sorted((a, b) => b.time.send.compareTo(a.time.send))
+    if (results[0] is RealtimeWeather) _weather = results[0] as RealtimeWeather;
+    _alerts = results[1] is List<History>
+        ? (results[1]! as List<History>).sorted((a, b) => b.time.send.compareTo(a.time.send))
         : const [];
-    _forecast = forecast is Map<String, dynamic> ? forecast : _forecast;
-    _error = (weather == null && _weather == null) ? Exception('weather fetch failed') : null;
-    _isLoading = false;
+    if (results[2] is Map<String, dynamic>) _forecast = results[2] as Map<String, dynamic>;
     notifyListeners();
   }
 
@@ -112,12 +87,6 @@ class HomeModel extends ChangeNotifier {
   /// The 24-hour weather forecast for the active location, or `null` if missing.
   Map<String, dynamic>? get forecast => _forecast;
 
-  /// Whether a weather fetch is currently in progress.
-  bool get isLoading => _isLoading;
-
-  /// The error from the last failed fetch, or `null` when the last fetch succeeded.
-  Object? get error => _error;
-
   /// The currently active temporary location code, or `null` when unset.
   String? get temporaryCode => _temporaryCode;
 

lib/app/new_home/_models/weather_params.dart

@@ -56,6 +56,16 @@ double windWeight(RealtimeWeatherData? d) {
   return (d.wind.beaufort / 8.0).clamp(0.0, 1.0);
 }
 
+/// Time-of-day salutation for the given [hour] (0–23, local time).
+///
+/// Used in both the top-of-page [Greeting] and the [AssistantHint] hint text.
+String greetingForHour(int hour) => switch (hour) {
+      < 6 => '夜深了',
+      < 12 => '早安',
+      < 18 => '午安',
+      _ => '晚安',
+    };
+
 /// Solar phase in `[0, 1]` across the visible window from 5am to 7pm.
 ///
 /// `0.0` ≈ sunrise at the eastern edge, `0.5` ≈ noon overhead,

lib/app/new_home/_widgets/assistant_hint.dart

@@ -2,6 +2,7 @@
 library;
 
 import 'package:dpip/app/new_home/_models/home_model.dart';
+import 'package:dpip/app/new_home/_models/weather_params.dart';
 import 'package:dpip/utils/extensions/build_context.dart';
 import 'package:flutter/material.dart';
 import 'package:material_symbols_icons/material_symbols_icons.dart';
@@ -59,14 +60,7 @@ class AssistantHint extends StatelessWidget {
 }
 
 String _buildHintText(double temperature, String weather) {
-  final hour = DateTime.now().hour;
-
-  final greeting = switch (hour) {
-    < 6 => '夜深了',
-    < 12 => '早安',
-    < 18 => '午安',
-    _ => '晚安',
-  };
+  final greeting = greetingForHour(DateTime.now().hour);
 
   final tempComment = switch (temperature) {
     < 10 => '天氣寒冷，注意保暖。',