代码优化

lib/pages/movies/douban_webview_page.dart

@@ -15,7 +15,6 @@ class DoubanWebViewPage extends StatefulWidget {
 class _DoubanWebViewPageState extends State<DoubanWebViewPage> {
   late WebViewController _controller;
   bool _isLoading = true;
-  bool _canExtract = false;
   bool _isExtracting = false; // 防止重复提取
 
   @override
@@ -47,7 +46,6 @@ class _DoubanWebViewPageState extends State<DoubanWebViewPage> {
             if (mounted) {
               setState(() {
                 _isLoading = false;
-                _canExtract = url.contains('douban.com/subject');
               });
             }
           },

lib/pages/movies/movie_detail_page.dart

@@ -3,16 +3,11 @@ import 'dart:ui' as ui;
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:provider/provider.dart';
-import 'package:path_provider/path_provider.dart';
-import 'package:path/path.dart' as path;
 import '../../widgets/fade_in_local_image.dart';
-import 'package:share_plus/share_plus.dart';
-import 'package:cross_file/cross_file.dart';
-import 'package:permission_handler/permission_handler.dart';
 import '../../providers/app_provider.dart';
 import '../../models/data_models.dart';
-import '../../utils/toast_util.dart';
 import '../../utils/user_prefs.dart';
+import '../../utils/toast_util.dart';
 import 'movie_reviews_page.dart';
 import 'movie_posters_page.dart';
 import 'movie_share_page.dart';
@@ -113,7 +108,7 @@ class _MovieDetailPageState extends State<MovieDetailPage> {
               ),
             ),
           ),
-          // 导航栏
+          // 详情页面的标准顶部导航栏
           Positioned(
             top: 0, left: 0, right: 0,
             child: Container(
@@ -387,27 +382,25 @@ class _MovieDetailPageState extends State<MovieDetailPage> {
     required Color backgroundColor,
     required Color foregroundColor,
   }) {
-    return Material(
-      color: Colors.transparent,
-      child: Ink(
-        width: 40,
-        height: 40,
-        decoration: BoxDecoration(
-          color: backgroundColor,
-          shape: BoxShape.circle,
-          boxShadow: [
-            BoxShadow(
-              color: backgroundColor.withValues(alpha: 0.3),
-              blurRadius: 8,
-              offset: const Offset(0, 2),
-            ),
-          ],
-        ),
-        child: IconButton(
-          icon: Icon(icon, size: 18, color: foregroundColor),
-          onPressed: onPressed,
-          padding: EdgeInsets.zero,
-          tooltip: tooltip,
+    return Tooltip(
+      message: tooltip,
+      child: GestureDetector(
+        onTap: onPressed,
+        child: Container(
+          width: 40,
+          height: 40,
+          decoration: BoxDecoration(
+            color: backgroundColor,
+            shape: BoxShape.circle,
+            boxShadow: [
+              BoxShadow(
+                color: backgroundColor.withValues(alpha: 0.3),
+                blurRadius: 8,
+                offset: const Offset(0, 2),
+              ),
+            ],
+          ),
+          child: Icon(icon, size: 18, color: foregroundColor),
         ),
       ),
     );
@@ -1184,11 +1177,15 @@ class _MovieDetailPageState extends State<MovieDetailPage> {
           ),
           ElevatedButton(
             onPressed: () async {
-              await context.read<AppProvider>().removeMovie(widget.movie.id);
+              final provider = context.read<AppProvider>();
+              await provider.removeMovie(widget.movie.id);
               if (!mounted) return;
-              Navigator.pop(context);
-              Navigator.pop(context);
-              ToastUtil.show(context, '已删除');
+              final navigator = Navigator.of(context);
+              navigator.pop();
+              navigator.pop();
+              if (mounted) {
+                ToastUtil.show(context, '已删除');
+              }
             },
             style: ElevatedButton.styleFrom(
               backgroundColor: colors.error,
@@ -1207,27 +1204,6 @@ class _MovieDetailPageState extends State<MovieDetailPage> {
     );
   }
 
-  Future<bool> _requestStoragePermission() async {
-    if (Platform.isAndroid) {
-      final sdkInt = await _getAndroidSdkInt();
-      if (sdkInt >= 33) {
-        final status = await Permission.photos.request();
-        return status.isGranted;
-      } else {
-        var status = await Permission.storage.request();
-        if (status.isDenied) {
-          status = await Permission.storage.request();
-        }
-        return status.isGranted;
-      }
-    }
-    return true;
-  }
-
-  Future<int> _getAndroidSdkInt() async {
-    return 30;
-  }
-
   void _showSharePoster(Movie movie) {
     Navigator.push(
       context,

lib/pages/movies/movie_form_page.dart

@@ -3,7 +3,6 @@ import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 import 'package:image_picker/image_picker.dart';
-import 'package:path_provider/path_provider.dart';
 import 'package:path/path.dart' as p;
 import 'package:provider/provider.dart';
 import 'package:http/http.dart' as http;

lib/pages/movies/movie_posters_page.dart

@@ -1,8 +1,6 @@
 import 'dart:io';
-import 'dart:math';
 import 'package:flutter/material.dart';
 import 'package:image_picker/image_picker.dart';
-import 'package:path_provider/path_provider.dart';
 import 'package:path/path.dart' as p;
 import 'package:provider/provider.dart';
 import 'package:flutter_staggered_grid_view/flutter_staggered_grid_view.dart';
@@ -133,7 +131,6 @@ class _MoviePostersPageState extends State<MoviePostersPage> {
   }
 
   Widget _buildPosterItem(MoviePoster poster, int index) {
-    final colors = Theme.of(context).colorScheme;
     // 根据索引生成不同的高度，实现瀑布流效果
     final heights = [180.0, 220.0, 160.0, 200.0, 240.0, 190.0];
     final height = heights[index % heights.length];

lib/pages/movies/movie_review_detail_page.dart

@@ -1,4 +1,3 @@
-import 'dart:io';
 import 'package:flutter/material.dart';
 import 'package:provider/provider.dart';
 import '../../models/data_models.dart';

lib/pages/movies/movie_review_form_page.dart

@@ -1,4 +1,3 @@
-import 'dart:io';
 import 'package:flutter/material.dart';
 import 'package:provider/provider.dart';
 import '../../providers/app_provider.dart';

lib/pages/movies/movie_reviews_page.dart

@@ -291,18 +291,6 @@ class _MovieReviewsPageState extends State<MovieReviewsPage> {
     ).then((_) => _loadReviews());
   }
 
-  void _navigateToEditReview(MovieReview review) {
-    Navigator.push(
-      context,
-      MaterialPageRoute(
-        builder: (context) => MovieReviewFormPage(
-          movieId: widget.movie.id,
-          review: review,
-        ),
-      ),
-    ).then((_) => _loadReviews());
-  }
-
   void _navigateToReviewDetail(MovieReview review) {
     Navigator.push(
       context,

lib/pages/movies/movie_tab_page.dart

@@ -80,8 +80,6 @@ class _MovieTabPageState extends State<MovieTabPage> {
     }
   }
 
-  String get _currentStatus => _statusMap[context.read<AppProvider>().movieStatusIndex] ?? 'watched';
-
   Future<void> _loadFirst() async {
     final provider = context.read<AppProvider>();
     final statusIdx = provider.movieStatusIndex;

lib/pages/movies/poster_gallery_page.dart

@@ -1,4 +1,3 @@
-import 'dart:io';
 import 'package:flutter/material.dart';
 import '../../models/data_models.dart';
 import '../../widgets/fade_in_local_image.dart';