Clean-up ACV sample to use x:Bind, remove redundant code in md file with embedded sample.

Author: michael-hawker

components/Collections/samples/AdvancedCollectionView.md

@@ -1,8 +1,8 @@
 ---
 title: AdvancedCollectionView
 author: nmetulev
-description: The AdvancedCollectionView is a collection view implementation that support filtering, sorting and incremental loading. It's meant to be used in a viewmodel.
-keywords: AdvancedCollectionView, data, sorting, filtering, Collections
+description: The AdvancedCollectionView is a collection view implementation that support filtering, sorting and incremental loading. It's meant to be used in a view or viewmodel.
+keywords: AdvancedCollectionView, CollectionViewSource, data, sorting, filtering, Collections
 dev_langs:
   - csharp
 category: Helpers
@@ -12,70 +12,23 @@ issue-id: 0
 icon: Assets/AdvancedCollectionView.png
 ---
 
-> [!Sample AdvancedCollectionViewSample]
-
 ## Usage
 
-In your viewmodel instead of having a public [IEnumerable](https://learn.microsoft.com/dotnet/core/api/system.collections.generic.ienumerable-1) of some sort to be bound to an eg. [Listview](https://learn.microsoft.com/uwp/api/Windows.UI.Xaml.Controls.ListView), create a public AdvancedCollectionView and pass your list in the constructor to it. If you've done that you can use the many useful features it provides:
+In your view or viewmodel instead of having a public [IEnumerable](https://learn.microsoft.com/dotnet/core/api/system.collections.generic.ienumerable-1) of some sort to be bound to an eg. [Listview](https://learn.microsoft.com/uwp/api/Windows.UI.Xaml.Controls.ListView), create a public AdvancedCollectionView and pass your list in the constructor to it. If you've done that you can use the many useful features it provides:
 
 * sorting your list using the `SortDirection` helper: specify any number of property names to sort on with the direction desired
 * filtering your list using a [Predicate](https://learn.microsoft.com/dotnet/core/api/system.predicate-1): this will automatically filter your list only to the items that pass the check by the predicate provided
 * deferring notifications using the `NotificationDeferrer` helper: with a convenient _using_ pattern you can increase performance while doing large-scale modifications in your list by waiting with updates until you've completed your work
 * incremental loading: if your source collection supports the feature then AdvancedCollectionView will do as well (it simply forwards the calls)
 * live shaping: when constructing the `AdvancedCollectionView` you may specify that the collection use live shaping. This means that the collection will re-filter or re-sort if there are changes to the sort properties or filter properties that are specified using `ObserveFilterProperty`
 
-## Example
-
-```csharp
-using CommunityToolkit.WinUI.Collections;
-
-// Grab a sample type
-public class Person
-{
-    public string Name { get; set; }
-}
+The `AdvancedCollectionView` is a good replacement for WPF's `CollectionViewSource`.
 
-// Set up the original list with a few sample items
-var oc = new ObservableCollection<Person>
-{
-    new Person { Name = "Staff" },
-    new Person { Name = "42" },
-    new Person { Name = "Swan" },
-    new Person { Name = "Orchid" },
-    new Person { Name = "15" },
-    new Person { Name = "Flame" },
-    new Person { Name = "16" },
-    new Person { Name = "Arrow" },
-    new Person { Name = "Tempest" },
-    new Person { Name = "23" },
-    new Person { Name = "Pearl" },
-    new Person { Name = "Hydra" },
-    new Person { Name = "Lamp Post" },
-    new Person { Name = "4" },
-    new Person { Name = "Looking Glass" },
-    new Person { Name = "8" },
-};
-
-// Set up the AdvancedCollectionView with live shaping enabled to filter and sort the original list
-var acv = new AdvancedCollectionView(oc, true);
-
-// Let's filter out the integers
-int nul;
-acv.Filter = x => !int.TryParse(((Person)x).Name, out nul);
-
-// And sort ascending by the property "Name"
-acv.SortDescriptions.Add(new SortDescription("Name", SortDirection.Ascending));
-
-// Let's add a Person to the observable collection
-var person = new Person { Name = "Aardvark" };
-oc.Add(person);
+## Example
 
-// Our added person is now at the top of the list, but if we rename this person, we can trigger a re-sort
-person.Name = "Zaphod"; // Now a re-sort is triggered and person will be last in the list
+The following is a complete example of how to perform ...
 
-// AdvancedCollectionView can be bound to anything that uses collections. 
-YourListView.ItemsSource = acv;
-```
+> [!Sample AdvancedCollectionViewSample]
 
 ## Remarks
 

components/Collections/samples/AdvancedCollectionViewSample.xaml

@@ -1,9 +1,9 @@
-<!--  Licensed to the .NET Foundation under one or more agreements. The .NET Foundation licenses this file to you under the MIT license. See the LICENSE file in the project root for more information.  -->
+<!--  Licensed to the .NET Foundation under one or more agreements. The .NET Foundation licenses this file to you under the MIT license. See the LICENSE file in the project root for more information.  -->
 <Page x:Class="CollectionsExperiment.Samples.AdvancedCollectionViewSample"
       xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
       xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
       xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
-      xmlns:local="using:AdvancedCollectionViewExperiment.Samples"
+      xmlns:local="using:CollectionsExperiment.Samples"
       xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
       mc:Ignorable="d">
 
@@ -17,8 +17,9 @@
                 <Setter Property="CornerRadius" Value="{StaticResource ControlCornerRadius}" />
             </Style.Setters>
         </Style>
-        <DataTemplate x:Key="PersonDataTemplate">
-            <TextBlock Text="{Binding Name}" />
+        <DataTemplate x:Key="PersonDataTemplate"
+                      x:DataType="local:Person">
+            <TextBlock Text="{x:Bind Name}" />
         </DataTemplate>
     </Page.Resources>
     <Grid ColumnSpacing="12"
@@ -51,7 +52,8 @@
         <Grid Grid.Row="2"
               Style="{StaticResource CardStyle}">
             <ListView x:Name="LeftList"
-                      ItemTemplate="{StaticResource PersonDataTemplate}" />
+                      ItemTemplate="{StaticResource PersonDataTemplate}"
+                      ItemsSource="{x:Bind Original}" />
         </Grid>
         <TextBlock Grid.Row="1"
                    Grid.Column="1"
@@ -60,7 +62,8 @@
               Grid.Column="1"
               Style="{StaticResource CardStyle}">
             <ListView x:Name="RightList"
-                      ItemTemplate="{StaticResource PersonDataTemplate}" />
+                      ItemTemplate="{StaticResource PersonDataTemplate}"
+                      ItemsSource="{x:Bind CollectionView}" />
         </Grid>
     </Grid>
 </Page>

components/Collections/samples/AdvancedCollectionViewSample.xaml.cs

@@ -3,24 +3,29 @@
 // See the LICENSE file in the project root for more information.
 
 using CommunityToolkit.WinUI.Collections;
+using System.Diagnostics.CodeAnalysis;
 
 namespace CollectionsExperiment.Samples;
 
-[ToolkitSample(id: nameof(AdvancedCollectionViewSample), "AdvancedCollectionView", description: $"A sample for showing how to create and use a {nameof(AdvancedCollectionView)}.")]
+[ToolkitSample(id: nameof(AdvancedCollectionViewSample), "AdvancedCollectionView", description: $"A sample for showing how to create and use a {nameof(AdvancedCollectionView)} for sorting and filtering.")]
 public sealed partial class AdvancedCollectionViewSample : Page
 {
-    public ObservableCollection<Person>? oc;
+    public ObservableCollection<Person> Original { get; private set; }
+
+    public AdvancedCollectionView CollectionView { get; private set; }
 
     public AdvancedCollectionViewSample()
     {
         this.InitializeComponent();
         Setup();
     }
 
+    [MemberNotNull(nameof(Original))]
+    [MemberNotNull(nameof(CollectionView))]
     private void Setup()
     {
         // left list
-        oc = new ObservableCollection<Person>
+        Original = new ObservableCollection<Person>
             {
                 new Person { Name = "Staff" },
                 new Person { Name = "42" },
@@ -40,22 +45,20 @@ private void Setup()
                 new Person { Name = "8" },
             };
 
-        LeftList.ItemsSource = oc;
-
         // right list
-        var acv = new AdvancedCollectionView(oc);
+        var acv = new AdvancedCollectionView(Original);
         int nul;
         acv.Filter = x => !int.TryParse(((Person)x).Name, out nul);
         acv.SortDescriptions.Add(new SortDescription("Name", SortDirection.Ascending));
 
-        RightList.ItemsSource = acv;
+        CollectionView = acv;
     }
 
     private void Add_Click(object sender, RoutedEventArgs e)
     {
         if (!string.IsNullOrWhiteSpace(NewItemBox.Text))
         {
-            oc!.Insert(0, new Person { Name = NewItemBox.Text });
+            Original.Insert(0, new Person { Name = NewItemBox.Text });
             NewItemBox.Text = "";
         }
     }

components/Collections/samples/Collections.Samples.csproj

@@ -23,4 +23,10 @@
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </Content>
   </ItemGroup>
+  <ItemGroup>
+    <PackageReference Include="PolySharp" Version="1.14.1">
+      <PrivateAssets>all</PrivateAssets>
+      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
+    </PackageReference>
+  </ItemGroup>
 </Project>