From a5bfe5f3774fd6874614ce30e67353a743ebca0d Mon Sep 17 00:00:00 2001
From: Tunglies <77394545+Tunglies@users.noreply.github.com>
Date: Thu, 15 Jan 2026 23:59:27 +0800
Subject: [PATCH] perf: enhance profile reordering logic and adjust logging
 level

---
 src-tauri/src/cmd/profile.rs     |  2 +-
 src-tauri/src/config/profiles.rs | 49 +++++++++++++++++++++-----------
 2 files changed, 33 insertions(+), 18 deletions(-)

diff --git a/src-tauri/src/cmd/profile.rs b/src-tauri/src/cmd/profile.rs
index 60bdb09c1..d4950b11a 100644
--- a/src-tauri/src/cmd/profile.rs
+++ b/src-tauri/src/cmd/profile.rs
@@ -117,7 +117,7 @@ pub async fn import_profile(url: std::string::String, option: Option<PrfOption>)
 pub async fn reorder_profile(active_id: String, over_id: String) -> CmdResult {
     match profiles_reorder_safe(&active_id, &over_id).await {
         Ok(_) => {
-            logging!(info, Type::Cmd, "重新排序配置文件");
+            logging!(debug, Type::Cmd, "重新排序配置文件");
             Config::profiles().await.apply();
             Ok(())
         }
diff --git a/src-tauri/src/config/profiles.rs b/src-tauri/src/config/profiles.rs
index 199702326..c035c3608 100644
--- a/src-tauri/src/config/profiles.rs
+++ b/src-tauri/src/config/profiles.rs
@@ -159,28 +159,43 @@ impl IProfiles {
     }
 
     /// reorder items
-    pub async fn reorder(&mut self, active_id: &String, over_id: &String) -> Result<()> {
-        let mut items = self.items.take().unwrap_or_default();
-        let mut old_index = None;
-        let mut new_index = None;
+    pub async fn reorder(&mut self, active_id: &str, over_id: &str) -> Result<()> {
+        if active_id == over_id {
+            return Ok(());
+        }
 
-        for (i, _) in items.iter().enumerate() {
-            if items[i].uid.as_ref() == Some(active_id) {
-                old_index = Some(i);
+        let Some(items) = self.items.as_mut() else {
+            return Ok(());
+        };
+
+        let mut old_idx = None;
+        let mut new_idx = None;
+
+        for (i, item) in items.iter().enumerate() {
+            if let Some(uid) = item.uid.as_ref() {
+                if uid == active_id {
+                    old_idx = Some(i);
+                }
+                if uid == over_id {
+                    new_idx = Some(i);
+                }
             }
-            if items[i].uid.as_ref() == Some(over_id) {
-                new_index = Some(i);
+            if old_idx.is_some() && new_idx.is_some() {
+                break;
             }
         }
 
-        let (old_idx, new_idx) = match (old_index, new_index) {
-            (Some(old), Some(new)) => (old, new),
-            _ => return Ok(()),
-        };
-        let item = items.remove(old_idx);
-        items.insert(new_idx, item);
-        self.items = Some(items);
-        self.save_file().await
+        if let (Some(old), Some(new)) = (old_idx, new_idx) {
+            if old < new {
+                items[old..=new].rotate_left(1);
+            } else {
+                items[new..=old].rotate_right(1);
+            }
+
+            return self.save_file().await;
+        }
+
+        Ok(())
     }
 
     /// update the item value