修复未取消事件订阅导致的内存泄漏问题

Author: CodingOctocat

Firefly/Behaviors/AutoRowNumberBehavior.cs

@@ -28,19 +28,26 @@ public static void SetEnableAutoRowNumbers(DependencyObject obj, bool value)
 
     private static void ItemsControl_Loaded(object sender, RoutedEventArgs e)
     {
-        if (sender is not ItemsControl itemsControl)
+        if (sender is not ItemsControl itemsControl || itemsControl.ItemsSource is not INotifyCollectionChanged notifyCollection)
         {
             return;
         }
 
-        if (itemsControl.ItemsSource is not INotifyCollectionChanged collection)
+        void CollectionChangedHandler(object? s, NotifyCollectionChangedEventArgs args)
         {
-            return;
+            // DispatcherPriority.Render 确保 UI 更新后再执行 RefreshRowNumbers()
+            itemsControl.Dispatcher.Invoke(() => RefreshRowNumbers(itemsControl), DispatcherPriority.Render);
+        }
+
+        notifyCollection.CollectionChanged += CollectionChangedHandler;
+
+        void UnloadedHandler(object s, RoutedEventArgs args)
+        {
+            notifyCollection.CollectionChanged -= CollectionChangedHandler;
+            itemsControl.Unloaded -= UnloadedHandler;
         }
 
-        // DispatcherPriority.Render 确保 UI 更新后再执行 RefreshRowNumbers()
-        collection.CollectionChanged += (s, args)
-            => itemsControl.Dispatcher.Invoke(() => RefreshRowNumbers(itemsControl), DispatcherPriority.Render);
+        itemsControl.Unloaded += UnloadedHandler;
 
         RefreshRowNumbers(itemsControl);
     }

Firefly/Behaviors/AutoScrollBehavior.cs

@@ -44,7 +44,7 @@ public static void SetIsEnabled(DependencyObject element, bool value)
 
     private static void ItemsControl_Loaded(object sender, RoutedEventArgs e)
     {
-        if (sender is not ItemsControl itemsControl)
+        if (sender is not ItemsControl itemsControl || itemsControl.ItemsSource is not INotifyCollectionChanged notifyCollection)
         {
             return;
         }
@@ -59,26 +59,31 @@ private static void ItemsControl_Loaded(object sender, RoutedEventArgs e)
         bool allowPause = GetAllowPause(itemsControl);
         bool autoScroll = true;
 
-        scrollViewer.ScrollChanged += (s, ev) => {
-            if (scrollViewer.VerticalOffset == scrollViewer.ScrollableHeight)
-            {
-                autoScroll = true;
-            }
-            else
+        void ScrollChangedHandler(object s, ScrollChangedEventArgs args)
+        {
+            autoScroll = scrollViewer.VerticalOffset == scrollViewer.ScrollableHeight || !allowPause;
+        }
+
+        scrollViewer.ScrollChanged += ScrollChangedHandler;
+
+        void CollectionChangedHandler(object? s, NotifyCollectionChangedEventArgs args)
+        {
+            if (autoScroll && args.Action == NotifyCollectionChangedAction.Add)
             {
-                autoScroll = !allowPause;
+                scrollViewer.ScrollToEnd();
             }
-        };
+        }
 
-        if (itemsControl.ItemsSource is INotifyCollectionChanged notifyCollection)
+        notifyCollection.CollectionChanged += CollectionChangedHandler;
+
+        void UnloadedHandler(object s, RoutedEventArgs args)
         {
-            notifyCollection.CollectionChanged += (s, ev) => {
-                if (autoScroll && ev.Action == NotifyCollectionChangedAction.Add)
-                {
-                    scrollViewer.ScrollToEnd();
-                }
-            };
+            scrollViewer.ScrollChanged -= ScrollChangedHandler;
+            notifyCollection.CollectionChanged -= CollectionChangedHandler;
+            itemsControl.Unloaded -= UnloadedHandler;
         }
+
+        itemsControl.Unloaded += UnloadedHandler;
     }
 
     private static void OnIsEnabledChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)