From 17480f683ebd7db60b77fb9e2ea4fe2e9f78232d Mon Sep 17 00:00:00 2001
From: kennethcheng <a1478925630@163.com>
Date: Mon, 7 Sep 2026 00:23:44 +0800
Subject: [PATCH] =?UTF-8?q?perf:=20=E5=A4=8D=E5=88=B6=E6=8C=89=E9=92=AE?=
 =?UTF-8?q?=E5=8E=BB=E6=8E=89=2010=C3=97Sleep=20=E9=87=8D=E8=AF=95,?=
 =?UTF-8?q?=E6=94=B9=E5=BF=AB=E5=A4=B1=E8=B4=A5+=E7=9B=B4=E6=8E=A5?=
 =?UTF-8?q?=E8=BF=9B=E5=85=9C=E5=BA=95=E7=AA=97=E5=8F=A3?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

性能优化:之前 CopyMessageButton_Click 在 SetDataObject 失败后会 10×50ms 重试 SetText(最坏 500ms 卡 UI),现在去掉重试,失败立刻弹 CopyFallbackWindow 兜底。

兜底窗口里已经自带 [尝试复制] 按钮可以手动再试,所以这里的客户端重试就是多余 — 既然短时 race 几乎不可能(用户点鼠标到剪贴板写入至少隔了几十毫秒,够任何 race 收敛),直接进兜底比等 500ms 体验更好。

性能对比:
- 成功:之前 <1ms,现在 <1ms(没变)
- 失败:之前 最坏 500ms,现在 <1ms
- 兜底窗口:用户主动 retry,不阻塞主流程

只捕获 Exception 而非只 COMException,因为 SetDataObject 在 OLE 锁时偶尔会抛 UnauthorizedAccessException,一并兜住。
---
 MainWindow.xaml.cs | 31 +++++--------------------------
 1 file changed, 5 insertions(+), 26 deletions(-)

diff --git a/MainWindow.xaml.cs b/MainWindow.xaml.cs
index 63bf2a7..6eb9ee1 100644
--- a/MainWindow.xaml.cs
+++ b/MainWindow.xaml.cs
@@ -293,10 +293,9 @@ public partial class MainWindow : Window
         if (HistoryList.SelectedItem is not GotifyMessage msg) return;
         var text = BuildFullMessageText(msg);
 
-        // 三段降级:
-        // 1) 现代路径 Clipboard.SetDataObject(copy: true),用 IDataObject COM 接口(可能跟 OLE 路径分开排队)
-        // 2) 失败回退到 Clipboard.SetText + 10 次 50ms 重试(常见 race,RDP/杀软短暂持有)
-        // 3) 还失败(典型:网易UU远程 / 剪贴板工具长期持有)弹手动复制窗口,文本预填 + 全选,让用户自己 Ctrl+C
+        // 快失败策略:1 次 SetDataObject,失败立刻进兜底窗口。
+        // 兜底窗口里点 "尝试复制" 可以再试(对方进程可能刚好释放),
+        // 所以这里不再做 N×Sleep 重试,避免点复制后卡 500ms。
         Exception? lastEx = null;
         bool ok = false;
 
@@ -311,26 +310,6 @@ public partial class MainWindow : Window
             _logger?.Warn($"Clipboard.SetDataObject failed: {ex.GetType().Name} 0x{ex.HResult:X8}");
         }
 
-        if (!ok)
-        {
-            const int MaxAttempts = 10;
-            const int DelayMs = 50;
-            for (int attempt = 1; attempt <= MaxAttempts; attempt++)
-            {
-                try
-                {
-                    Clipboard.SetText(text);
-                    ok = true;
-                    break;
-                }
-                catch (System.Runtime.InteropServices.COMException ex)
-                {
-                    lastEx = ex;
-                    System.Threading.Thread.Sleep(DelayMs);
-                }
-            }
-        }
-
         if (ok)
         {
             _logger?.Info($"Copied message to clipboard. id={msg.Id}");
@@ -352,8 +331,8 @@ public partial class MainWindow : Window
         }
         else
         {
-            // 长期占用的兜底:弹个预填 + 全选的窗口,用户自己 Ctrl+C
-            _logger?.Error("Clipboard locked persistently. Showing fallback copy dialog.", lastEx);
+            // 长期占用(UU远程 / RDP / 剪贴板工具):弹预填 + 全选的窗口,用户自己 Ctrl+C
+            _logger?.Error("Clipboard locked. Showing fallback copy dialog.", lastEx);
             var dialog = new CopyFallbackWindow(text) { Owner = this };
             dialog.ShowDialog();
         }