Добавить в Combobox после связывания MVVM WPF

Question

Привет, после того, как я свяжу комбобокс наблюдаемую коллекцию, я хочу добавить текстовое поле в комбо. и это дает мне эту ошибку:

System.NullReferenceException: 'Ссылка на объект не установлена ​​на экземпляр объекта.'

WpfApp1.MainWindow.getModel (...) возвратила ноль.

Изображение ОШИБКИ

Модель:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Collections.ObjectModel;

namespace WpfApp1
{
    public class Parts : Changed
    {
        public string name;

        public string Name
        {
            get { return name; }
            set
            {
                if (name != value)
                {
                    name = value;
                    RaisePropertyChanged("Name");
                }
            }            
        }
    }
}

viewModel:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.ComponentModel;
using System.Collections.ObjectModel;

namespace WpfApp1
{
    public class AddViewModel : Changed
    {

        private ObservableCollection<Parts> _persons;
        public string names;
        public AddViewModel()
        {
            Persons = new ObservableCollection<Parts>()
             {
                  new Parts{Name="Nirav"}
                 ,new Parts{Name="Kapil"}
                 ,new Parts{Name="Arvind"}
                 ,new Parts{Name="Rajan"}
             };

        }
        public ObservableCollection<Parts> Persons
        {
            get { return _persons; }
            set {
                if (_persons != value)
                {
                    _persons = value;
                    RaisePropertyChanged("Persons");
                }
            }
        }
        private Parts _sperson;

        public Parts SPerson
        {
            get { return _sperson; }
            set {
                if (_sperson != value)
                {
                    _sperson = value;
                    RaisePropertyChanged("SPerson");
                }
            }
        }

    }
}

MainWindow:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace WpfApp1
{
    /// <summary>
    /// Interaction logic for MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public AddViewModel addviewmodel;

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

        public AddViewModel getModel()
        {
            return addviewmodel;
        }

        private void Button_Click(object sender, RoutedEventArgs e)
        {

                getModel().Persons.Add(new Parts { Name = cmbtxt.Text});

        }

    }
}

XAML:

<Grid>
        <ComboBox ItemsSource="{Binding Persons}" SelectedItem="{Binding Path=SPersons,Mode=TwoWay}" HorizontalAlignment="Left" Margin="391,17,0,0" VerticalAlignment="Top" Width="314" Height="27">
            <ComboBox.ItemTemplate>
                <DataTemplate>
                    <StackPanel Orientation="Horizontal">
                        <TextBlock Text="{Binding Path=Name}" />
                    </StackPanel>
                </DataTemplate>
            </ComboBox.ItemTemplate>
        </ComboBox>
        <TextBox Name="cmbtxt" HorizontalAlignment="Left" Height="23" Margin="24,21,0,0" TextWrapping="Wrap" VerticalAlignment="Top" Width="172" />
        <Button Content="Add" HorizontalAlignment="Left" Margin="24,88,0,0" VerticalAlignment="Top" Width="156" Height="49" Click="Button_Click"/>
    </Grid>

Answer 1

Вы можете сделать addviewmodel приватным полем только для чтения и немедленно инициализировать его. Тогда вам просто нужно установить DataContext для поля в конструкторе.

Кроме того, getModel() не очень C # /. NET дружественный. Используйте свойство, если вам нужно выставить поле:

public partial class MainWindow : Window
{
    private readonly AddViewModel addviewmodel = new AddViewModel();

    public MainWindow()
    {
        InitializeComponent();
        DataContext = addviewmodel;
    }

    public AddViewModel AddViewModel => addviewmodel;

    private void Button_Click(object sender, RoutedEventArgs e)
    {
        addviewmodel.Persons.Add(new Parts { Name = cmbtxt.Text });
    }

}

Используя свойство, вы можете фактически удалить поле:

public partial class MainWindow : Window
{
    public MainWindow()
    {
        InitializeComponent();
        DataContext = AddViewModel;
    }

    public AddViewModel AddViewModel { get; } = new AddViewModel();

    private void Button_Click(object sender, RoutedEventArgs e)
    {
        AddViewModel.Persons.Add(new Parts { Name = cmbtxt.Text });
    }
}

Answer 2

В MainWindow вы никогда не устанавливаете значение addviewmodel, следовательно, оно равно нулю. Вы можете исправить это, изменив конструктор:

public MainWindow()
        {
            InitializeComponent();
            addviewmodel = new AddViewModel()
            DataContext = addviewmodel ;
        }

Вот сообщение о NullReferenceException в целом: Что такое NullReferenceException и как его исправить?