Заполнение данными полей формы и ComboBox

@Hugonavy · Регистрация: 04.02.2016

Студворк — интернет-сервис помощи студентам

Здравствуйте!
Пытался искать, но все вопросы, почему-то, связаны с ComboBox vs DataGrid.

Суть проблемы: есть форма добавления/изменения информации о заказчике. Таблица заказчика связана с таблицей транспорта (разные заказчики могут пользоваться одним транспортом). Соответственно, в окне добавления хочу добавить ComboBox с имеющимся в БД транспортом.

В поем понимании логика должна быть следующая: при открытии окна добавления у меня либо есть входящая информация (изменение записи), либо ее нет. Далее я делаю запрос к таблице транспорта и в первом случае сравниваю ключи с полученной информацией и вывожу в список записанный в БД транспорт, во втором - ничего не отображаю (выбор за пользователем).

Делал по примеру с сайта метанит, поэтому окно добавления пока без ViewModel

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
public partial class AddDeliveryWindow : Window
    {
        ApplicationContext db;
        public Delivery Delivery { get; private set; }
        public IEnumerable<Transport> Transports{ get; set; }
        
        public AddDeliveryWindow(Delivery dv)
        {
            InitializeComponent();
 
            db = new ApplicationContext();
            Delivery = db.Deliveries
                .Include(d => d.Transport)
                .Where(d => d.DeliveryId == dv.DeliveryId)
                .FirstOrDefault();
            Transports = db.Transports.ToList();
 
            comboTransports.ItemsSource = Transports;
 
            this.DataContext = Delivery;
 
            //DataContext = new AddDeliveryViewModel(dv);
        }
 
        private void Accept_Click(object sender, RoutedEventArgs e)
        {
            this.DialogResult = true;
        }
    }

Дальше у меня ступор, т.к. у заказчика я могу вытащить свойства через точку (напр., Delivery.DeliveryId), а у транспорта нет.
Подскажите пожалуйста, в чем причина такого поведения?

P.S.: текстовые поля все заполняются как надо, проблема только с ComboBox

@novikov.ea · 06.02.2020, 18:48

Transports - это список, коллекция. Чтобы выбрать нужный для заказчика транспорт, его нужно найти.

C#
1
public Transport SelectedTransport { get; set; }

C#
1
SelectedTransport = Transports.FirstOrDefault(x => x.Id == Delivery.TransportId);

Добавлено через 45 секунд
И уже у SelectedTransport можно обращаться к свойствам

C#
1
string name = SelectedTransport.Name;

@Hugonavy · 07.02.2020, 09:09 **[ТС]**

novikov.ea, спасибо. Только я все равно немного не понимаю, для ComboBox.ItemsSource мне надо указать коллекцию, а для SelectedItem - один элемент этой коллекции. При этом Transports и SelectedTransports друг о друге не знают.

C#
1
2
comboTransports.ItemsSource = Transports;
comboTransports.SelectedItem = Transports.FirstOrDefault(x => x.Id == Delivery.TransportId);

В итоге в список выводится перечень из строк, содержащих TestProject.Transport

@novikov.ea · 07.02.2020, 09:37

Сообщение от Hugonavy

В итоге в список выводится перечень из строк, содержащих TestProject.Transport

ComboBox не знает, как отображать объекты типа TestProject.Transport. Поэтому просто выводит на экран название типа. Следует определить ItemTemplate для ComboBox. Посмотрите пример здесь, как это делается

@Элд Хасп · 07.02.2020, 14:15

Сообщение от Hugonavy

Только я все равно немного не понимаю, для ComboBox.ItemsSource мне надо указать коллекцию, а для SelectedItem - один элемент этой коллекции. При этом Transports и SelectedTransports друг о друге не знают.

Не совсем так.
Для ComboBox.ItemsSource-Transports вы указываете глобальный список Транспорта. Этот список одного уровня с Заказчиком.
ComboBox.SelectedItem-SelectedTransports - это свойство Заказчика.

В итоге вы получаете необходимость привязки одного UI элемента к данным находящимся на разных уровнях иерархии Контекста Данных. И так как Transports и SelectedTransports находятся на разных уровнях иерархии, то знания их друг о друге ограничены. Знают они только то, что SelectedTransports обязательно элемент списка Transports. Больше ничего они друг о друге не знают.

Такая проблема типична для всех списочных UI элементов. И есть множество типовых способов её решения.

Добавлено через 3 минуты

Сообщение от Hugonavy

В итоге в список выводится перечень из строк, содержащих TestProject.Transport

Для простых случаев можно использовать свойство ComboBox.DisplayMemberPath - указать в нём отображаемое свойство Контекста Данных.
Свойство DisplayMemberPath есть у всех списочных элементов, оно наследуется от ItemsControl.DisplayMemberPath

Добавлено через 2 минуты
Так же для привязки выделенного элемента можно использовать не полностью элемент ComboBox, а одно из свойств этого элемента.
Для этого используются свойства Selector.SelectedValuePath и Selector.SelectedValue

@Hugonavy · 07.02.2020, 16:42 **[ТС]**

Элд Хасп, спасибо!

Сделал пока так (костыль)

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
public partial class AddDeliveryWindow : Window
    {
        ApplicationContext db;
        public Delivery Delivery { get; private set; }
        public List<Transport> Transports { get; set; }
        public AddDeliveryWindow(Delivery dv)
        {
            InitializeComponent();
 
            db = new ApplicationContext();
            Transports = db.Transports.ToList();
 
            //ComboBox
            cbTransport.ItemsSource = Transports;
 
            if(dv != null)
            {
                Delivery = db.Deliveries
                    .Where(d => d.DeliveryId == dv.DeliveryId)
                    .FirstOrDefault();
 
                cbTransport.SelectedItem = Transports.FirstOrDefault(s => s.TransportId == dv.TransportId);
            }
            else
            {
                Delivery = new Delivery();
            }
            this.DataContext = Delivery;
 
            //DataContext = new AddDeliveryViewModel(dv);
        }
 
        private void cbTransport_Selected(object sender, RoutedEventArgs e)
        {
            ComboBox comboBox = (ComboBox)sender;
            int selectedItem = comboBox.SelectedIndex;
 
            Delivery.TransportId = selectedItem + 1;
            //MessageBox.Show("TransportId: " + Delivery.TransportId.ToString());
        }
        private void Accept_Click(object sender, RoutedEventArgs e)
        {
            this.DialogResult = true;
        }
    }

Если я открываю запись на изменение, все работает. Проблема с добавлением записи (когда открывается пустая форма).

И еще вопрос: когда я выводил все записи таблицы, то в ViewModel использовал конструкцию

C#
1
Deliveries = db.Deliveries.Local.ToBindingList();

Но это работает только для коллекции. Как из ViewModel передать в DataContext единичную запись?

@Элд Хасп · 07.02.2020, 16:51

Hugonavy, мне трудно читать ваш код.
Да, наверное, не только мне.

С одной стороны вы используете Контекст Данных и привязки к нему, с другой - напрямую лезете в UI элементы.

Это очень неправильный подход, и для такой реализации вы будете создавать костыль за костылём.
Вынесите из CB окна всё что относится к данным.

@Hugonavy · 10.02.2020, 11:02 **[ТС]**

Элд Хасп, доброго утра)) Я принял Ваше замечание к сведению и, надеюсь, у меня получилось более "читабельно".

Класс AppModelClass

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
public class AppModelClass
    {
        static AppModelClass _instance;
        static ApplicationContext db;
        static double _koeff = 0.44;
        
        private AppModelClass()
        {
            db = new ApplicationContext();
            
        }
        public static AppModelClass getInstance()
        {
            if (_instance == null)
                _instance = new AppModelClass();
            return _instance;
        }
        //Get Functions
        public List<Year> GetYears()
        {
            db.Years.Load();
            var y = db.Years.Local.ToList();
            return y;
        }
        public List<Transport> GetTransports()
        {
            db.Transports.Load();
            var tr = db.Transports.Local.ToList();
            return tr;
        }
        public List<Delivery> GetDeliveries()
        {
            db.Deliveries.Load();
            var d = db.Deliveries.Local.ToList();
            return d;
        }
 
        //Add Functions
        public void AddYear(Year y)
        {
            db.Years.Add(y);
            db.SaveChanges();
        }
        public bool AddTransport(Transport t)
        {
            if (!CheckTransport(t))
                return false;
            db.Transports.Add(t);
            db.SaveChanges();
            return true;
        }
        public bool AddDelivery(Delivery d)
        {
            if (!CheckDelivery(d))
                return false;
            db.Deliveries.Add(d);
            db.SaveChanges();
            return true;
        }
 
        //Edit Functions
        public bool EditTransport(Transport t)
        {
            if (!CheckTransport(t))
                return false;
            var findtransport = db.Transports.FirstOrDefault(x => x.TransportId == t.TransportId);
 
            findtransport.TransportName = t.TransportName;
            findtransport.Value = t.Value;
            findtransport.ClearanceTime = t.ClearanceTime;
            db.Entry(findtransport).State = EntityState.Modified;
            db.SaveChanges();
 
            return true;
 
        }
        public bool EditDelivery(Delivery d)
        {
            if (!CheckDelivery(d))
                return false;
            var finddelivery = db.Deliveries.FirstOrDefault(x => x.DeliveryId == d.DeliveryId);
 
            finddelivery.DeliveryName = d.DeliveryName;
            finddelivery.DeliveryType = d.DeliveryType;
            finddelivery.LoadVolume = d.LoadVolume;
            finddelivery.LoadTime = d.LoadTime;
            finddelivery.WLoads = d.WLoads;
            finddelivery.SLoads = d.SLoads;
            finddelivery.DeliveryVolume = d.LoadVolume * (d.WLoads + d.SLoads) * _koeff;
            db.Entry(finddelivery).State = EntityState.Modified;
            db.SaveChanges();
 
            return true;
 
        }
 
        //Delete Functions
        public bool DeleteYear(Year y)
        {
            var findYear= db.Years.FirstOrDefault(x => x.YearId == y.YearId);
            db.Years.Remove(findYear);
            db.SaveChanges();
            return true;
        }
        public bool DeleteTransport(Transport t)
        {
            if (!CheckTransport(t))
                return false;
            var findTransport = db.Transports.FirstOrDefault(x => x.TransportId == t.TransportId);
            db.Transports.Remove(findTransport);
            db.SaveChanges();
            return true;
        }
        public bool DeleteDelivery(Delivery d)
        {
            if (!CheckDelivery(d))
                return false;
            var findDelivery = db.Deliveries.FirstOrDefault(x => x.DeliveryId == d.DeliveryId);
            db.Deliveries.Remove(findDelivery);
            db.SaveChanges();
            return true;
        }
 
        //Check Functions
        bool CheckTransport(Transport t)
        {
            var error = "";
            if (t == null)
                error = "Null transport";
            else
            {
                if (t.TransportName == null || t.TransportName.Length == 0 || string.IsNullOrWhiteSpace(t.TransportName))
                    error = "Некорректное наименование";
                else if (db.Transports.Any(x => x.TransportName == t.TransportName))
                    error = "Указанное наименование уже присутствует в базе данных";
            }
            if (error.Length == 0)
                return true;
            System.Windows.MessageBox.Show(error, "Error", System.Windows.MessageBoxButton.OK, System.Windows.MessageBoxImage.Error);
            return false;
        }
        bool CheckDelivery(Delivery d)
        {
            var error = "";
            if (d == null)
                error = "Null delivery";
            else
            {
                if (d.DeliveryName == null || d.DeliveryName.Length == 0 || string.IsNullOrWhiteSpace(d.DeliveryName))
                    error = "Empty delivery name";
                if (!db.Years.Any(x => x.YearId == d.YearId))
                    error = "Не указан контрактный год";
                if (!db.Transports.Any(x => x.TransportId == d.TransportId))
                    error = "Не указан транспорт";
            }
            if (error.Length == 0)
                return true;
            System.Windows.MessageBox.Show(error, "Error", System.Windows.MessageBoxButton.OK, System.Windows.MessageBoxImage.Error);
            return false;
        }
    }
 
    public static class Extensions
    {
        public static Delivery GetCopy(this Delivery from)
        {
            return new Delivery
            {
                DeliveryId = from.DeliveryId,
                DeliveryName = from.DeliveryName,
                DeliveryType = from.DeliveryType,
                LoadVolume = from.LoadVolume,
                LoadTime = from.LoadTime,
                WLoads = from.WLoads,
                SLoads = from.SLoads,
                DeliveryVolume = from.DeliveryVolume,
                YearId = from.YearId,
                TransportId = from.TransportId
            };
        }
 
        public static Transport GetCopy(this Transport from)
        {
            return new Transport
            {
                TransportId = from.TransportId,
                TransportName = from.TransportName,
                Value = from.Value,
                ClearanceTime = from.ClearanceTime
            };
        }
    }

Класс EditDeliveryViewModel

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
class EditDeliveryViewModel : INotifyPropertyChanged
    {
        AppModelClass _appModel;
        public Delivery Delivery { get; set; }
        public List<Year> Year { get; set; }
        public List<Transport> Transport { get; set; }
 
        public Year Years
        {
            get { return Year.FirstOrDefault(x => x.YearId == Delivery.YearId); }
        }
        public Transport Transports
        {
            get { return Transport.FirstOrDefault(x => x.TransportId == Delivery.TransportId); }
        }
 
        public EditDeliveryViewModel(Delivery delivery)
        {
            _appModel = AppModelClass.getInstance();
            Delivery = delivery.GetCopy();
            Year = _appModel.GetYears();
            Transport = _appModel.GetTransports();
            OnPropertyChanged("Delivery");
            OnPropertyChanged("Year");
            OnPropertyChanged("Transport");
        }
 
        public ICommand SaveDeliveryCommand => new Command(obj => Edit(obj));
 
        void Edit(object obj)
        {
            if (_appModel.EditDelivery(Delivery))
                Close(obj, true);
        }
 
        public ICommand CancelCommand => new Command(obj => Close(obj, false));
 
        void Close(object obj, bool dialogResult)
        {
            var w = ((System.Windows.Window)obj);
            w.DialogResult = dialogResult;
            w.Close();
        }
 
        public event PropertyChangedEventHandler PropertyChanged;
 
        public void OnPropertyChanged(string propertyName)
        {
            PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
        }
    }

И, наконец, разметка

XML
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
<Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="auto"/>
            <RowDefinition Height="auto"/>
            <RowDefinition Height="auto" />
        </Grid.RowDefinitions>
        <Grid Grid.Row="0">
            <Grid.RowDefinitions>
                <RowDefinition/>
                <RowDefinition/>
            </Grid.RowDefinitions>
            <Grid.ColumnDefinitions>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="3*"/>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="*"/>
                <ColumnDefinition Width="2*"/>
            </Grid.ColumnDefinitions>
            <Label Content="Year" HorizontalAlignment="Center" Grid.Row="0"/>
            <ComboBox Grid.Row="1" ItemsSource="{Binding Year}" DisplayMemberPath="CurrentYear" 
                      SelectedValue="{Binding Delivery.YearId , Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
                      SelectedValuePath="YearId" />
 
            <Label Content="Delivery Name" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="1"/>
            <TextBox Text="{Binding Path=Delivery.DeliveryName}" Grid.Row="1" Grid.Column="1" Margin="1" />
 
            <Label Content="Delivery Type" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="2"/>
            <TextBox Text="{Binding Path=Delivery.DeliveryType}" Grid.Row="1" Grid.Column="2" Margin="1" />
 
            <Label Content="Load volume" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="3"/>
            <TextBox Text="{Binding Path=Delivery.LoadVolume}" Grid.Row="1" Grid.Column="3" Margin="1" />
 
            <Label Content="Load time" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="4"/>
            <TextBox Text="{Binding Path=Delivery.LoadTime}" Grid.Column="4" Margin="1" Grid.Row="1"/>
 
            <Label Content="Winter Loads" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="5"/>
            <TextBox Text="{Binding Path=Delivery.WLoads}" Grid.Column="5" Margin="1" Grid.Row="1"/>
 
            <Label Content="Summer Loads" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="6"/>
            <TextBox Text="{Binding Path=Delivery.SLoads}" Grid.Column="6" Margin="1" Grid.Row="1"/>
 
            <Label Content="Transport name" HorizontalAlignment="Center" Grid.Row="0" Grid.Column="7"/>
            <ComboBox ItemsSource="{Binding Transport}" DisplayMemberPath="TransportName" 
                      SelectedValue="{Binding Delivery.TransportId , Mode=TwoWay, UpdateSourceTrigger=PropertyChanged}" 
                      SelectedValuePath="TransportId" Grid.Row="1" Grid.Column="7"/>
        </Grid>
        
 
        <StackPanel Grid.Row="2" Orientation="Horizontal" HorizontalAlignment="Center">
            <Button Command="{Binding SaveDeliveryCommand}" CommandParameter="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type Window}}}" Content="OK" Height="30" Width="100" />
            <Button Command="{Binding CancelCommand}" CommandParameter="{Binding RelativeSource={RelativeSource FindAncestor, AncestorType={x:Type Window}}}" Content="Отмена" Width="100" Height="30" />
        </StackPanel>
    </Grid>

А теперь, практически финальный вопрос)) Я не могу понять следующее: если я вношу изменения в текстовые поля - изменения вносятся. Но если я выберу что-то другое в выпадающих списках (comboBox), то запись в БД не изменяется. Как можно с этим разобраться?

@Элд Хасп · 10.02.2020, 12:42

Сообщение от Hugonavy

Как можно с этим разобраться?

Так гораздо понятнее.

Судя по коду, вроде всё правильно.
И должно работать.
Наверное что-то упускаю..., но не могу понять что.

Надо прогнать в дебагере ваш проект, но для этого нужен какой-то тестовый вариант.
Такой чтобы и вам от него польза была и чтобы его можно было скинуть архивом и я мог его запустить у себя.

Ну, или можно на вашем компе запустить ваш проект через TeamViewer.
Если такой вариант устраивает - напишите в личку.

@Hugonavy · 10.02.2020, 13:00 **[ТС]**

Элд Хасп, все, вопрос решен)) Я по невнимательности забыл в AppModelClass в функции EditDelivery добавлять TransportId и YearId. Теперь все работает

@Элд Хасп · 10.02.2020, 13:21

Сообщение от Hugonavy

все, вопрос решен

Ну, и здорово, что сами разобрались!
Удачи!

Добавлено через 3 минуты
P.S. Hugonavy, советую немного притормозить и получше изучить азы.
У вас всё сделано криво: XAML, C#, разделение слоёв и т.д.

Могу помочь, но быстро не получится.
Судя по фрагментам вашего кода, скорее всего, придётся переделывать полностью.

Новые блоги и статьи Все статьи Все блоги /
Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .	Расчёт токов в цепи постоянного тока igorrr37 05.01.2026 / * Дана цепь постоянного тока с сопротивлениями и напряжениями. Надо найти токи в ветвях. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и решает её. Последовательность действий:. . .
Новый CodeBlocs. Версия 25.03 palva 04.01.2026 Оказывается, недавно вышла новая версия CodeBlocks за номером 25. 03. Когда-то давно я возился с только что вышедшей тогда версией 20. 03. С тех пор я давно снёс всё с компьютера и забыл. Теперь. . .	Модель микоризы: классовый агентный подход anaschu 02.01.2026 Раньше это было два гриба и бактерия. Теперь три гриба, растение. И на уровне агентов добавится между грибами или бактериями взаимодействий. До того я пробовал подход через многомерные массивы,. . .	Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .

@Элд Хасп Модератор 16118 / 11239 / 2887 Регистрация: 21.04.2018 Сообщений: 33,043 Записей в блоге: 2
	07.02.2020, 16:51
	Hugonavy, мне трудно читать ваш код. Да, наверное, не только мне. С одной стороны вы используете Контекст Данных и привязки к нему, с другой - напрямую лезете в UI элементы. Это очень неправильный подход, и для такой реализации вы будете создавать костыль за костылём. Вынесите из CB окна всё что относится к данным. 0

@Hugonavy 180 / 95 / 25 Регистрация: 04.02.2016 Сообщений: 308 Записей в блоге: 3
	10.02.2020, 13:00 [ТС]
	Элд Хасп, все, вопрос решен)) Я по невнимательности забыл в AppModelClass в функции EditDelivery добавлять TransportId и YearId. Теперь все работает 1