WPF自定义UserControl双向绑定仅单向生效问题排查

嗨，我一眼就看出问题出在哪了——你把UI更新逻辑写在CLR属性的setter里了，这可是WPF依赖属性的典型坑！

核心问题 #

WPF在处理绑定更新时，不会调用CLR属性的setter方法，而是直接通过SetValue修改依赖属性的底层存储。所以当MainWindow的IsOn从其他代码修改时，虽然依赖属性的值确实同步了，但你写在setter里的Margin切换代码根本没执行，导致Nipple的位置没更新。

正确的修复步骤 #

1. 把UI更新逻辑移到依赖属性的变更回调中

注册依赖属性时，指定一个PropertyChangedCallback，所有属性变更的逻辑都放在这里：

public static readonly DependencyProperty IsOnProperty = DependencyProperty.Register(
    "IsOn", 
    typeof(bool), 
    typeof(MyToggleSwitch), 
    new FrameworkPropertyMetadata(
        true, 
        FrameworkPropertyMetadataOptions.BindsTwoWayByDefault,
        OnIsOnPropertyChanged // 添加属性变更回调
    )
);

// 实现变更回调方法
private static void OnIsOnPropertyChanged(DependencyObject d, DependencyPropertyChangedEventArgs e)
{
    var toggleSwitch = (MyToggleSwitch)d;
    bool isOn = (bool)e.NewValue;
    toggleSwitch.Nipple.Margin = isOn ? Offset_Right : Offset_Left;
}

2. 简化CLR属性的setter，只保留基础操作

CLR属性只是依赖属性的“便捷包装器”，绝对不能在这里加业务/UI逻辑：

public bool IsOn
{
    get { return (bool)GetValue(IsOnProperty); }
    set { SetValue(IsOnProperty, value); }
}

3. 优化点击事件的处理（可选但推荐）

点击UserControl时，不要直接修改Margin，而是切换IsOn的值，让绑定和变更回调自动处理UI更新：

private void Grid_MouseLeftButtonDown(object sender, MouseButtonEventArgs e)
{
    IsOn = !IsOn;
}

为什么要这么做？ #

WPF的依赖属性系统设计就是绕开CLR属性直接操作底层存储的——不管是绑定、动画还是样式触发，都会直接调用GetValue/SetValue。只有把逻辑放在变更回调里，才能保证无论属性值是怎么被修改的，UI都能同步更新。

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