INotifyPropertyChanged“双”绑定

Question

我正在尝试将一些 XAML 代码绑定到 ViewModel 中的属性。

<Grid Visibility="{Binding HasMovies, Converter={StaticResources VisibilityConverter}}">
...
</Grid>

我的 ViewModel 设置如下：

private bool _hasMovies;
        public bool HasMovies
        {
            get { return _hasMovies; }
            set { _hasMovies = value; RaisePropertyChanged("HasMovies"); }
        }

在 ViewModel 的构造函数中，我设置了 HasMovies 链接：

MovieListViewModel()
{
   HasMovies = CP.Connection.HasMovies;
}

在 CP 中：

public bool HasMovies
        {
            get { return MovieList != null && MovieList.Count > 0; }
        }

private ObservableCollection<Movie> _movies;
        public ObservableCollection<Movie> MovieList
        {
            get { return _movies; }
            set
            {
                _movies = value;
                RaisePropertyChanged("MovieList");
                RaisePropertyChanged("HasMovies");
                _movies.CollectionChanged += MovieListChanged;
            }
        }

        private void MovieListChanged(object sender, NotifyCollectionChangedEventArgs e)
        {
            RaisePropertyChanged("HasMovies");
        }

我做错了什么？我应该如何更改此绑定，以便它反映 CP.Connection.HasMovies 的当前状态？

Answer 1

要么直接公开 ViewModel 中的对象并直接通过该对象绑定（这样该值就不会只复制一次，就像现在发生的那样），或者订阅 PropertyChanged 事件并将 HasMovies 设置为新值源对象中发生变化的时间。

例如

CP.Connection.PropertyChanged += (s,e) =>
    {
        if (e.PropertyName = "HasMovies") this.HasMovies = CP.Connection.HasMovies;
    };

Answer 2

首先，当您更改集合的内容（即添加/删除项目）时，不会调用集合类型（例如 MovieList 属性）的 setter 。

这意味着 MovieList 属性的所有 setter 代码都是毫无意义的。

其次，这是非常愚蠢的代码。更好的解决方案是使用 NotifyPropertyWeaver。那么你的代码在视图模型中将如下所示：

[DependsOn("MovieList")]
public bool HasMovies
{
    get { return MovieList != null && MovieList.Count > 0; }
}

public ObservableCollection<Movie> MovieList
{
    get;
    private set;
}

或者，当你第一次初始化 MovieList 属性时，你必须为 CollectionChanged 事件添加一个侦听器（没有理由有一个支持属性，真的没有理由！），并且然后在事件处理程序中调用 RaisePropertyChanged("HasMovies")。

例子：

public class CP : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    public CP()
    {
        MovieList = new ObservableCollection<Movie>();
        MovieList.CollectionChanged += MovieListChanged;
    }

    public bool HasMovies
    {
        get { return MovieList != null && MovieList.Count > 0; }
    }

    public ObservableCollection<Movie> MovieList
    {
        get;
        private set;
    }

    private void MovieListChanged(object sender, NotifyCollectionChangedEventArgs e)
    {
        RaisePropertyChanged("HasMovies");
    }

    private void RaisePropertyChanged(string propertyName)
    {
        if (PropertyChanged != null)
        {
            PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
    }
}