为WPF自定义按钮通过Property暴露内部文本是否属于不良实践？

先给你吃个定心丸：这不算绝对的不良实践，在简单场景下完全能用，但从WPF的设计理念和长期维护的角度来看，确实有更优的实现方式，咱们拆解着说：

你的做法的合理性 #

如果只是做个小工具、快速原型，或者控件逻辑特别简单，这种直接暴露内部控件属性的方式非常直观，代码写起来快，也能满足需求——毕竟能跑起来的代码就是好代码（在小场景里）。

潜在的问题（为什么更推荐改进） #

但如果你的控件需要长期维护、扩展，或者要给其他开发者用，这种做法就有几个明显的局限：

• 违反封装原则：你把内部的TextHolder这个TextBlock的细节直接暴露给了外部。以后要是想把TextBlock换成Label、Run甚至自定义文本控件，外部代码因为依赖了这个内部控件的逻辑，可能就得跟着改，耦合度太高了。
• 不支持WPF核心特性：你现在写的是普通CLR属性，没有实现通知机制，也不是依赖属性。如果外部想用数据绑定来设置这个Text，或者想给Text加个动画、样式，都会出问题——WPF的控件属性几乎都是依赖属性，就是为了支持这些核心能力。
• 限制了模板定制性：WPF控件的一大优势是可以通过ControlTemplate完全重写布局。如果以后有人想修改你的按钮模板，把ViewBox换成其他容器，或者把TextBlock换成别的元素，你的Text属性因为直接绑定到TextHolder，就会直接失效，因为模板里已经没有这个控件了。

更符合WPF设计的做法 #

推荐用依赖属性+模板绑定的组合，这是WPF自定义控件的标准玩法：

第一步：给自定义按钮定义Text依赖属性 #

public class CustomButton : Button
{
    // 注册依赖属性
    public static readonly DependencyProperty TextProperty =
        DependencyProperty.Register(
            nameof(Text), 
            typeof(string), 
            typeof(CustomButton), 
            new PropertyMetadata(string.Empty));

    // 对应的CLR包装器
    public string Text
    {
        get => (string)GetValue(TextProperty);
        set => SetValue(TextProperty, value);
    }
}

第二步：在控件模板里用TemplateBinding绑定Text #

<!-- 在资源字典或者窗口资源里定义自定义按钮的样式 -->
<Style TargetType="{x:Type local:CustomButton}">
    <Setter Property="Template">
        <Setter.Value>
            <ControlTemplate TargetType="{x:Type local:CustomButton}">
                <Viewbox>
                    <!-- 用TemplateBinding绑定到自定义的Text依赖属性 -->
                    <TextBlock Text="{TemplateBinding Text}"/>
                </Viewbox>
            </ControlTemplate>
        </Setter.Value>
    </Setter>
</Style>

这样做的好处：

• 完全封装了内部实现，外部只需要关心Text属性，不管你内部用什么控件显示文本都不影响。
• 支持数据绑定、样式设置、动画等所有WPF原生特性。
• 其他人可以自由修改ControlTemplate来定制按钮布局，只要模板里的文本元素绑定了Text属性，就能正常工作。

总结 #

如果是小项目快速实现，你的原始做法完全没问题；但如果是需要长期维护、扩展的控件，或者要遵循WPF的设计规范，依赖属性+模板绑定的方式是更优的选择。

内容的提问来源于stack exchange，提问作者Tristan Trainer