質問
WPF用にINotifyPropertyChanged
を実装した自作のコレクションを作成したのですが、XAMLでCount
プロパティをバインディングしても、変更が反映されませんでした。
ObservableCollection
のソースコードを参考にして変更が反映されるよう修正することはできたのですが、なぜこれで上手く動くのかがわかりませんでした。
上手く動く場合と動かない場合で違いが出る原因を知りたいです。
どちらでもINotifyPropertyChanged.PropertyChanged
が発火できているところまでは確認しています。
再現手順
[Add]ボタンをクリックすると、CollectionAのカウントには変化がありませんが、CollectionBのカウントには変化があります。
ソースコード
Count
の変更が反映されないコレクション(CollectionA)
internal class MyCollectionA<T> : IList<T>, IReadOnlyList<T>, INotifyCollectionChanged, INotifyPropertyChanged
{
private readonly ObservableCollection<T> collection = [];
public MyCollectionA()
{
}
public event NotifyCollectionChangedEventHandler? CollectionChanged
{
add => collection.CollectionChanged += value;
remove => collection.CollectionChanged -= value;
}
event PropertyChangedEventHandler? INotifyPropertyChanged.PropertyChanged
{
add => ((INotifyPropertyChanged)collection).PropertyChanged += value;
remove => ((INotifyPropertyChanged)collection).PropertyChanged -= value;
}
public T this[int index] { get => collection[index]; set => collection[index] = value; }
public int Count => collection.Count;
public bool IsReadOnly => false;
public bool Contains(T item) => collection.Contains(item);
public int IndexOf(T item) => collection.IndexOf(item);
public void CopyTo(T[] array, int arrayIndex) => collection.CopyTo(array, arrayIndex);
public void Add(T item) => collection.Add(item);
public void Insert(int index, T item) => collection.Insert(index, item);
public bool Remove(T item) => collection.Remove(item);
public void RemoveAt(int index) => collection.RemoveAt(index);
public void Clear() => collection.Clear();
public IEnumerator<T> GetEnumerator() => collection.GetEnumerator();
IEnumerator IEnumerable.GetEnumerator() => collection.GetEnumerator();
}
Count
の変更が反映されるコレクション(CollectionB)
internal class MyCollectionB<T> : IList<T>, IReadOnlyList<T>, INotifyCollectionChanged, INotifyPropertyChanged
{
private readonly ObservableCollection<T> collection = [];
public MyCollectionB()
{
// protectedのPropertyChangedを経由するようにした点が変更点です。
((INotifyPropertyChanged)collection).PropertyChanged += (sender, args) => PropertyChanged?.Invoke(this, args);
}
public event NotifyCollectionChangedEventHandler? CollectionChanged
{
add => collection.CollectionChanged += value;
remove => collection.CollectionChanged -= value;
}
protected event PropertyChangedEventHandler? PropertyChanged;
event PropertyChangedEventHandler? INotifyPropertyChanged.PropertyChanged
{
add => PropertyChanged += value;
remove => PropertyChanged -= value;
}
public T this[int index] { get => collection[index]; set => collection[index] = value; }
public int Count => collection.Count;
public bool IsReadOnly => false;
public bool Contains(T item) => collection.Contains(item);
public int IndexOf(T item) => collection.IndexOf(item);
public void CopyTo(T[] array, int arrayIndex) => collection.CopyTo(array, arrayIndex);
public void Add(T item) => collection.Add(item);
public void Insert(int index, T item) => collection.Insert(index, item);
public bool Remove(T item) => collection.Remove(item);
public void RemoveAt(int index) => collection.RemoveAt(index);
public void Clear() => collection.Clear();
public IEnumerator<T> GetEnumerator() => collection.GetEnumerator();
IEnumerator IEnumerable.GetEnumerator() => collection.GetEnumerator();
}
ViewModel
internal class MyViewModel : INotifyPropertyChanged
{
private MyCollectionA<string> _collectionA = [];
private MyCollectionB<string> _collectionB = [];
public MyViewModel()
{
AddCommand = new MyCommand { Action = p => { CollectionA?.Add("A"); CollectionB?.Add("B"); } };
}
public event PropertyChangedEventHandler? PropertyChanged;
public MyCollectionA<string> CollectionA { get => _collectionA; set { _collectionA = value; OnPropertyChanged(); } }
public MyCollectionB<string> CollectionB { get => _collectionB; set { _collectionB = value; OnPropertyChanged(); } }
public ICommand AddCommand { get; }
private void OnPropertyChanged([CallerMemberName] string? propertyName = null) => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
private sealed class MyCommand : ICommand
{
public MyCommand() { }
public required Action<object?> Action { get; set; }
public event EventHandler? CanExecuteChanged;
public bool CanExecute(object? parameter) => true;
public void Execute(object? parameter) => Action?.Invoke(parameter);
}
}
XAML
<Window
x:Class="WpfSandbox.MainWindow"
xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
xmlns:local="clr-namespace:WpfSandbox"
Title="MainWindow" Height="200" Width="200">
<Window.DataContext>
<local:MyViewModel/>
</Window.DataContext>
<StackPanel>
<TextBlock Text="{Binding CollectionA.Count, StringFormat={}CollectionA.Count:{0}}"/>
<TextBlock Text="{Binding CollectionB.Count, StringFormat={}CollectionB.Count:{0}}"/>
<Button Command="{Binding AddCommand}" Content="Add"/>
</StackPanel>
</Window>
環境
- Windows 11
- Visual Studio 2022
- .NET 8
※ ソースを昔っぽく修正すれば、Visual Studio 2013、.NET Framework 4.5 でも再現できました。