C# TreeView获取子项全父节点路径：代码优化建议咨询

我首次尝试使用TreeView控件，目标是实现选中子项的所有父节点路径展示，当前已实现功能，但觉得现有代码还有优化空间，希望得到改进建议。现有实现代码如下：

//Get selected TreeView item and parents
private void TreeviewMain_SelectedItem(object sender, RoutedEventArgs e)
{
    TreeViewItem? selectedParent = e.OriginalSource as TreeViewItem;
    TreeViewItem selectedItem = (TreeViewItem)MainTestList.SelectedItem;
    string parentPath = "";

    if (selectedParent != null)
    {
        parentPath = GetTreeviewParent(selectedParent);

    }

    if (selectedItem != null)
    {
        string finalItemPath = parentPath + selectedItem.Header.ToString();
        CurrentItem.Text = finalItemPath;
    }
}

// Form a string of parent names
public string GetTreeviewParent(TreeViewItem item)
{
    DependencyObject parent = VisualTreeHelper.GetParent(item);
    string parentPath = ">";
    while (!(parent is TreeView))
    {
        parent = VisualTreeHelper.GetParent(parent);

        if (parent != null)
        {
            ItemsControl parentItem = parent as ItemsControl;
            TreeViewItem parentName = parentItem as TreeViewItem;
            if (parentName != null)
            {
                parentPath = ">" + parentName.Header.ToString() + parentPath;
            }
        }
    }

    return parentPath;
}

这是我试过的最简洁版本，但仍觉得代码不够规整，以下是具体优化建议：

1. 消除逻辑冗余，统一选中项获取逻辑 #

原事件方法中同时通过e.OriginalSource和MainTestList.SelectedItem获取选中项，这两个变量指向的是同一个对象，完全可以合并，避免逻辑混淆。同时用C#模式匹配做类型判断，比强制转换更安全。

2. 优化父节点遍历与路径拼接逻辑 #

原GetTreeviewParent方法的字符串拼接方式效率较低，且遍历起始步骤冗余。改用列表收集父节点文本，最后用string.Join拼接，既高效又易读；同时调整遍历顺序，从选中项的父节点开始直接遍历，减少无效操作。

3. 规范命名与可维护性 #

• 方法名GetTreeviewParent改为GetParentPath，更贴合“获取父节点路径”的功能；
• 将路径分隔符>提取为常量，方便后续统一修改；
• 变量名更具语义化，比如parentName改为parentItem，避免歧义。

优化后的代码示例 #

private const string PathSeparator = ">";

private void TreeviewMain_SelectedItem(object sender, RoutedEventArgs e)
{
    // 用模式匹配安全获取选中项
    if (MainTestList.SelectedItem is not TreeViewItem selectedItem)
        return;
        
    string parentPath = GetParentPath(selectedItem);
    // 处理根节点无父路径的情况
    CurrentItem.Text = string.IsNullOrEmpty(parentPath) 
        ? selectedItem.Header.ToString() 
        : $"{parentPath}{PathSeparator}{selectedItem.Header}";
}

/// <summary>
/// 获取指定TreeViewItem的所有父节点路径
/// </summary>
/// <param name="item">目标TreeViewItem</param>
/// <returns>拼接后的父节点路径</returns>
public string GetParentPath(TreeViewItem item)
{
    var parentHeaders = new List<string>();
    DependencyObject currentParent = VisualTreeHelper.GetParent(item);

    while (currentParent != null && currentParent is not TreeView)
    {
        // 直接判断是否为TreeViewItem，减少不必要的类型转换
        if (currentParent is TreeViewItem parentItem)
        {
            parentHeaders.Add(parentItem.Header.ToString());
        }
        currentParent = VisualTreeHelper.GetParent(currentParent);
    }

    // 收集的是从子到父的顺序，反转后得到父到子的路径顺序
    parentHeaders.Reverse();
    return string.Join(PathSeparator, parentHeaders);
}

优化说明 #

• 用List<string>收集父节点文本，避免字符串反复拼接带来的性能损耗；
• 增加XML注释提升代码可读性；
• 处理了根节点无父路径的边界情况，避免出现多余的分隔符；
• 简化类型转换逻辑，减少空值判断的嵌套层级。

内容的提问来源于stack exchange，提问作者Vlad SD