Почему не выводит список WPF?

Question

[jostario]

Столкнулся с проблемой DataGrid не выводит содержимое поля .

XAML

<DataGrid x:Name="Jobs" Grid.Column="0" AutoGenerateColumns="False" ItemsSource="{Binding Path = Jobs}">
            <DataGrid.Columns >
                <DataGridTextColumn Binding="{Binding Path = Name}" Header="Job" Width="200*"  IsReadOnly="False"></DataGridTextColumn>
            </DataGrid.Columns>
        </DataGrid>

ViewModel

class MainViewModel : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;
        public void OnPropertyChanged([CallerMemberName] string prop = "")
        {
            if (PropertyChanged != null)
                PropertyChanged(this, new PropertyChangedEventArgs(prop));
        }
        public List<Job> Jobs;

        public MainViewModel() 
        {

            Jobs = new List<Job>() { new Job() { Name = "sdfs" } ,
            new Job() { Name = "sdytrfs" }};
            
        }
    }

И сам класс Job

class Job : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;
        public void OnPropertyChanged([CallerMemberName] string prop = "")
        {
            if (PropertyChanged != null)
                PropertyChanged(this, new PropertyChangedEventArgs(prop));
        }
        public string Name
        {
            get { return name; }
            set
            {
                name = value;
                OnPropertyChanged("Name");
            }
        }
       private string name;
        
        List<JobTitle> jobs;
    }

Comments

[Борис Животное]

Зачем ты реализуешь везде INotifyPropertyChanged по новой? Вообще используй MvvmLight библиотеку для начала, а там посмотришь что лучше для тебя использовать. В крайнем случае сделай класс ViewModelBase и там реализуй этот функционал.

[Борис Животное]

А где MainViewModel создаётся? Где он добавляется в DataContext окна? (ItemsSource="{Binding Path = Jobs}" )

[jostario]

Борис Животное,

public MainWindow()
        {
            InitializeComponent();
            DataContext = new MainViewModel();
        }

как я понимаю это и есть добавление в DataContext

[Борис Животное]

jostario, держи ViewModelBase. Если никогда не будешь использовать OnDispose, то можешь удалить интерфейс IDisposable и связанные методы OnDispose и Dispose.

Использовать так:

namespace WpfAppDataGridHelp.ViewModels
{
    internal class JobViewModel : ViewModelBase
    {
        private string _name = "";

        public string Name
        {
            get => _name;
            set => Set(ref _name, value);
        }
    }
}

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Diagnostics;
using System.Runtime.CompilerServices;

namespace WpfAppDataGridHelp.ViewModels;

public abstract class ViewModelBase : INotifyPropertyChanged, IDisposable
{
    protected ViewModelBase()
    {
    }

    public virtual string? DisplayName { get; protected set; }

    protected virtual bool ThrowOnInvalidPropertyNames { get; private set; }

    [field: NonSerialized]
    public event PropertyChangedEventHandler? PropertyChanged;

    [Conditional("DEBUG")]
    [DebuggerStepThrough]
    public void VerifyPropertyName(string propertyName)
    {
        if (TypeDescriptor.GetProperties(this)[propertyName] == null)
        {
            string msg = "Invalid property name: " + propertyName;

            if (ThrowOnInvalidPropertyNames) throw new Exception(msg);

            Debug.Fail(msg);
        }
    }

    protected void RaisePropertyChanged(string propertyName)
    {
        VerifyPropertyName(propertyName);
        PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    public virtual void RaisePropertyChanged<T>(
        [CallerMemberName] string? propertyName = null, 
        T? oldValue = null,
        T? newValue = null) 
        where T : class
    {
        if (string.IsNullOrEmpty(propertyName))
            throw new ArgumentException("This method cannot be called with an empty string", nameof(propertyName));

        RaisePropertyChanged(propertyName);
    }

    protected bool Set<T>(
        ref T? field, 
        T? newValue,
        [CallerMemberName] string? propertyName = null)
        where T : class
    {
        if (EqualityComparer<T>.Default.Equals(field, newValue))
            return false;

        var oldValue = field;
        field = newValue;
        RaisePropertyChanged(propertyName, oldValue, field);
        return true;
    }

    protected virtual void OnDispose()
    {
    }

    public void Dispose()
    {
        OnDispose();
    }
}

[Борис Животное]

jostario, я проект рабочий уже набросал с этим классом ViewModelBase и ViewModelLocator, но смотрю тебе ответили уже.

[Роман]

Борис Животное, а зачем вообще в VM IDisposable? В VM использовать unsafe и неуправляемые ресурсы, так себе идея.

[Борис Животное]

Роман, использование IDisposable и using не ограничивается только перечисленными кейсами использования. Я взял часть кода из старого проекта, который брал за основу код из автоматически создаваемого шаблона для проекта с MvvmLight, там создавался класс ViewModelLocator, в котором создавалось два свойства: public static MainViewModel MainStatic { get; private set; } и public static MainViewModel Main { get; private set; } = MainStatic; и был статический метод ViewModelLocator.Cleanup(), в котором, если я не путаю ничего, вызывался этот MainStatic.Dispose, или я где-то это нагуглил, а там отродясь вызывался метод MainStatic.Cleanup интерфейса ICleanup, что сути не меняет, тот же Dispose, но без возможности использовать using.

Update: посмотрел историю гита, скорее всего, всё-таки, я где-то это нагуглил в таком виде когда-то давно.
https://github.com/lbugnion/mvvmlight/blob/master/...

Answer 1

[Роман]

Привязка делается к свойствам, а не полям.

private List<Job> _jobs;
public List<Job> Jobs
{
    get {return _jobs;}
    set {
                _jobs = value;
                OnPropertyChanged("Jobs");        
    }
}