perf: 复制按钮去掉 10×Sleep 重试,改快失败+直接进兜底窗口
性能优化:之前 CopyMessageButton_Click 在 SetDataObject 失败后会 10×50ms 重试 SetText(最坏 500ms 卡 UI),现在去掉重试,失败立刻弹 CopyFallbackWindow 兜底。 兜底窗口里已经自带 [尝试复制] 按钮可以手动再试,所以这里的客户端重试就是多余 — 既然短时 race 几乎不可能(用户点鼠标到剪贴板写入至少隔了几十毫秒,够任何 race 收敛),直接进兜底比等 500ms 体验更好。 性能对比: - 成功:之前 <1ms,现在 <1ms(没变) - 失败:之前 最坏 500ms,现在 <1ms - 兜底窗口:用户主动 retry,不阻塞主流程 只捕获 Exception 而非只 COMException,因为 SetDataObject 在 OLE 锁时偶尔会抛 UnauthorizedAccessException,一并兜住。
This commit is contained in:
+5
-26
@@ -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();
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user