C# 筛选的ObservableCollection不更新WPF UI上的ListView

C# 筛选的ObservableCollection不更新WPF UI上的ListView,c#,wpf,listview,mvvm,observablecollection,C#,Wpf,Listview,Mvvm,Observablecollection,我正在尝试筛选WPF UI上列表视图中的ObservableCollection数据索引。但是,当集合刷新时,ListView将变为空白,并且不显示任何内容。我还试图自学MVVM模式。问题可能是我如何不刷新控件,但如何在MVVM模式中让它刷新?我习惯于落后于Winforms和代码,当用户单击Search按钮(调用ApplicationModel.Search方法)时,过滤逻辑就会发生 型号代码: public class AppObject { public string Name {

我正在尝试筛选WPF UI上列表视图中的ObservableCollection数据索引。但是,当集合刷新时,ListView将变为空白,并且不显示任何内容。我还试图自学MVVM模式。问题可能是我如何不刷新控件,但如何在MVVM模式中让它刷新?我习惯于落后于Winforms和代码,当用户单击Search按钮(调用ApplicationModel.Search方法)时,过滤逻辑就会发生

型号代码:

public class AppObject
{
    public string Name { get; set; }
    public string Description { get; set; }
    public string Owner { get; set; }
    public string Email { get; set; }

    public AppObject(string name, string desc, string owner, string email)
    {
        this.Name = name;
        this.Description = desc;
        this.Owner = owner;
        this.Email = email;
    }
}

public class ApplicationsModel : ObservableCollection<AppObject>
{
    private static object threadLocker;
    private static ApplicationsModel current;

    static ApplicationsModel()
    {
        threadLocker = new object();
    }

    public static ApplicationsModel Current
    {
        get
        {
            lock (threadLocker)
            {
                if (current == null)
                {
                    current = new ApplicationsModel();
                }
            }
            return current;
        }
    }

    private ApplicationsModel()
    {
        this.Refresh();
    }

    private ApplicationsModel(IEnumerable<AppObject> collection)
        : base(collection)
    {
    }

    private void Refresh()
    {
                    try
                    {
                            // Query database to get the initial data - this code works fine
                    }
        catch (Exception e)
        {
            string error = String.Format("Could not refresh repository list: {0}", e.Message);
            MessageBox.Show(error, "Error Refreshing", MessageBoxButton.OK, MessageBoxImage.Error);
        }
    }

    public void Search(string key)
    {
        IEnumerable<AppObject> newList = current.Items.Where<AppObject>(t => t.Name.Equals(key));
        foreach (AppObject app in newList)
        {
            Console.WriteLine(app.Name);
            Console.WriteLine(app.Description);
        }
        current.Clear();
        current = null;
        current = new ApplicationsModel(newList);
    }

    public void ClearSearch()
    {
        Console.WriteLine("ClearSearch method called");
    }
}
public class ApplicationViewModel
{
    public ObservableCollection<AppObject> AppCollection { get; set; }
    static string searchString;
    static string emailString;
    public AppObject SelectedApp { get; set; }
    public string AppToSearch
    {
        get
        {
            return searchString;
        }
        set
        {
            searchString = value;
        }
    }
    public string AppToRequest
    {
        get
        {
            return SelectedApp.Email;
        }
        set
        {
            SelectedApp.Email = value;
        }
    }
    private SearchButtonCommand searchButtonCmd;
    private ClearButtonCommand clearButtonCmd;
    private EmailButtonCommand emailButtonCmd;

    public ApplicationViewModel()
    {
        this.AppCollection = ApplicationsModel.Current;
    }

    public ICommand SearchButtonPressed
    {
        get
        {
            if (this.searchButtonCmd == null)
            {
                this.searchButtonCmd = new SearchButtonCommand();
            }
            return this.searchButtonCmd;
        }
    }

    public ICommand ClearButtonPressed
    {
        get
        {
            if (this.clearButtonCmd == null)
            {
                this.clearButtonCmd = new ClearButtonCommand();
            }
            return this.clearButtonCmd;
        }
    }

    public ICommand EmailButtonPressed
    {
        get
        {
            if (this.emailButtonCmd == null)
            {
                this.emailButtonCmd = new EmailButtonCommand();
            }
            return this.emailButtonCmd;
        }
    }

    private class SearchButtonCommand : ICommand
    {
        public event EventHandler CanExecuteChanged;

        public void Execute(object parameter)
        {
            string searchkey = ApplicationViewModel.searchString;
            ApplicationsModel.Current.Search(searchkey);
        }

        public bool CanExecute(object parameter)
        {
            return true;
        }
    }

    private class ClearButtonCommand : ICommand
    {
        public event EventHandler CanExecuteChanged;

        public void Execute(object parameter)
        {
            ApplicationsModel.Current.ClearSearch();
        }

        public bool CanExecute(object parameter)
        {
            return true;
        }
    }

    private class EmailButtonCommand : ICommand
    {
        public event EventHandler CanExecuteChanged;

        public void Execute(object parameter)
        {
            string targetEmail = ApplicationViewModel.emailString;
        }

        public bool CanExecute(object parameter)
        {
            return true;
        }
    }
}

我更改了模型中的刷新方法以在那里进行过滤。

您的ViewModel需要实现
INotifyPropertyChanged
,并在
AppCollection
发生更改时发出通知

参考:

Jeffrey Khan是对的-你错过了INotify。几个问题。1) 为什么要使用螺纹锁固胶?所有这些工作都是在UI线程上完成的,不应该被其他线程触及。2) 为什么每次刷新时都要创建新的视图模型?为什么不刷新视图模型上的可观察集合呢?在处理可能连接到视图模型上的事件时,这样做更容易,也更友好。如果您不手动删除事件处理程序-视图模型永远不会被处理,您的应用程序将消耗内存。是的,这是一个clusterf**k,会导致内存泄漏。时间很晚了,我很累,所以我写了这样的东西。我只是根据您的建议更改了刷新方法,使用一个参数进行筛选,这样就可以正常工作,不会每次都创建一个新集合,我们也不必担心处理问题。成功了!对于任何将此作为指南阅读的人,我编辑了我的文章,以反映我为使其工作所做的更改您的ViewModel不应该实现INPC,这是模型的工作。你提供的链接也没有说要在VM中这样做。
<Window.DataContext>
    <vm:ApplicationViewModel />
</Window.DataContext>

<Grid>
    <Grid.RowDefinitions>
        <RowDefinition Height="Auto" />
        <RowDefinition Height="Auto" />
        <RowDefinition Height="*" />
        <RowDefinition Height="Auto" />
    </Grid.RowDefinitions>
    <Grid.ColumnDefinitions>
        <ColumnDefinition Width="*" />
    </Grid.ColumnDefinitions>
    <Image Grid.Row="0" Height="84" HorizontalAlignment="Left" Margin="0,5,5,5" Name="imgNexusLogo" Stretch="Fill" VerticalAlignment="Top" Width="600" Source="C:\source\Nexus\NexusShop\Images\nexus1bannerlong.png" />
    <Grid Grid.Row="1" HorizontalAlignment="Center" Margin="0,5,5,5" VerticalAlignment="Center">
        <Grid.RowDefinitions>
            <RowDefinition Height="Auto" />
        </Grid.RowDefinitions>
        <Grid.ColumnDefinitions>
            <ColumnDefinition Width="Auto" />
            <ColumnDefinition Width="Auto" />
            <ColumnDefinition Width="Auto" />
            <ColumnDefinition Width="Auto" />
        </Grid.ColumnDefinitions>
        <Label Grid.Row="0" Grid.Column="0" Content="Search for Application">
            <Label.Foreground>
                <SolidColorBrush Color="LightCyan" />
            </Label.Foreground>
        </Label>
        <TextBox Grid.Row="0" Grid.Column="1" Margin="3" Width="500" Text="{Binding AppToSearch}" />
        <Button Grid.Row="0" Grid.Column="2" HorizontalAlignment="Right" Width="100" Height="20" Margin="3" Background="LightCyan" Content="Search" vm:ButtonBehaviour.SearchCommand="{Binding SearchButtonPressed}" />
        <Button Grid.Row="0" Grid.Column="3" HorizontalAlignment="Right" Width="100" Height="20" Margin="3" Background="LightCyan" Content="Clear Search" vm:ButtonBehaviour.ClearCommand="{Binding ClearButtonPressed}"/>
    </Grid>
    <ListView Grid.Row="2" BorderBrush="Black" HorizontalAlignment="Stretch" ItemsSource="{Binding Path=AppCollection}" SelectedItem="{Binding SelectedNexusApp}">
        <ListView.View>
            <GridView>
                <GridViewColumn Header="Application Name" Width="100" DisplayMemberBinding="{Binding Name}"/>
                <GridViewColumn Header="Application Description" Width="800" DisplayMemberBinding="{Binding Description}"/>
                <GridViewColumn Header="Application Owner" Width="100" DisplayMemberBinding="{Binding Owner}"/>
            </GridView>
        </ListView.View>
    </ListView>
    <Button Grid.Row="3" HorizontalAlignment="Center" Width="200" Height="30" Margin="3" Background="LightCyan" Content="Request Application" vm:ButtonBehaviour.EmailCommand="{Binding EmailButtonPressed}" />
</Grid>
    #region INotifyPropertyChanged implementation
    public event PropertyChangedEventHandler PropertyChanged;
    private void NotifyPropertyChanged(string propertyName = "")
    {
        if (PropertyChanged != null)
        {
            PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
    }
    #endregion

ApplicationsModel.Current.Search(this.searchString);
NotifyPropertyChanged();