TreeView拖拽必须先设AllowDrop=true，再分步Remove后Add节点，用HitTest判断位置，禁用拖拽时的自动Expand，并全程同步拖拽状态。

TreeView节点拖拽必须启用AllowDrop

不设 AllowDrop = true，拖拽连MouseDown都触发不了——这是最常卡住的起点。Windows Forms的TreeView默认关闭拖放支持，哪怕你写了ItemDrag、DragEnter、DragDrop三个事件，没这句就全白搭。

实操建议：

• 在设计器里勾选AllowDrop属性，或代码中写treeView1.AllowDrop = true;
• 别只在窗体Load里设，确保控件已初始化（比如放在InitializeComponent()之后）
• 如果树是动态创建的，务必在new TreeView()后立刻赋值AllowDrop，晚一步就可能漏掉

DragDrop事件里不能直接调用Nodes.Add

直接写targetNode.Nodes.Add(draggedNode)会抛ArgumentException: 节点已属于另一个TreeView——因为被拖的节点还挂在其原父节点下，WinForms不允许“跨树移动”这种操作，必须先移除再添加。

正确做法是分两步：先从原位置摘下来，再插入目标位置。常见错误是只做后半步，或者用Clone()导致数据不同步。

实操建议：

• 在ItemDrag事件里用DoDragDrop(e.Item, DragDropEffects.Move)传入原始TreeNode对象
• 在DragDrop中先调draggedNode.Remove()，再调targetNode.Nodes.Add(draggedNode)
• 如果要支持拖到根层级（即无targetNode），检查targetNode是否为null，改用treeView1.Nodes.Add(draggedNode)

拖拽位置判断靠HitTest，不是靠MousePosition

很多人用PointToClient(MousePosition)算坐标再遍历Nodes去比对，结果发现拖到节点间隙时逻辑错乱——TreeView有内建的命中检测逻辑，绕过它等于重写一遍渲染层。

C#

"避开 C# 常见错误——空陷阱、异步陷阱、LINQ 陷阱及资源泄漏。"

下载

HitTest返回的TreeViewHitTestInfo里Location字段才是关键：它区分是拖到了节点文字区（TreeNode）、图标区、空白处还是节点上方/下方间隙。误判Location会导致拖到空白处也强行插入子节点。

实操建议：

• 在DragOver事件里调treeView1.HitTest(e.X, e.Y)
• 只在Location == TreeViewHitTestLocations.Above或Below时允许e.Effect = DragDropEffects.None（禁止放置）
• 拖到OnItem才视为有效目标，此时HitTestInfo.Node才可靠
• 避免用e.X/e.Y直接换算索引——不同DPI、字体缩放下坐标会偏移

多级嵌套拖拽要禁用父节点自动展开

默认情况下，鼠标悬停在折叠节点上0.5秒就会自动展开，拖拽过程中频繁触发展开，不仅干扰视觉，还会让HitTest返回的Node变成刚展开的子节点，导致目标位置错位。

这不是bug，是WinForms的交互设计，但拖拽场景下完全不需要。

实操建议：

• 在BeforeExpand事件里加判断：if (isDragging) e.Cancel = true;
• 用一个私有bool字段isDragging，在ItemDrag开始时设为true，DragDrop或DragLeave结束时设为false
• 别依赖TreeView.IsExpanded状态来判断——它反映的是当前UI状态，不是用户意图

拖拽逻辑真正难的不是API调用，而是状态同步：哪个节点正在被拖、它原本在哪、目标位置是否合法、中间是否发生了Expand/Collapse、取消拖拽时如何回滚。这些细节藏在事件触发顺序和Node引用生命周期里，漏掉任意一环都会出现“看起来动了，实际没变”或者“拖一次，节点消失”的情况。