Избыточные пересчеты переменной/метода

@Beginnner · Регистрация: 02.07.2017

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

Всем привет!

Подскажите как избавиться от избыточных пересчетов переменных/методов.

Model

Кликните здесь для просмотра всего текста

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class Person
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public string Surname { get; set; }
        public int Age { get; set; }
        public DateTime CreatedOn { get; set; }
        public DateTime UpdatedOn { get; set; }
 
        public Person()
        {
            CreatedOn = DateTime.Now;
            UpdatedOn = DateTime.Now;
        }
    }

View

Кликните здесь для просмотра всего текста

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
<Window.Resources>
        <BooleanToVisibilityConverter x:Key="VisibleIfTrueConverter" />
    </Window.Resources>
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="Auto" />
            <RowDefinition Height="Auto" />
            <RowDefinition Height="Auto" />
            <RowDefinition Height="Auto" />
            <RowDefinition Height="Auto" />
            <RowDefinition Height="*" />
        </Grid.RowDefinitions>
        <Grid.ColumnDefinitions>
            <ColumnDefinition Width="Auto" />
            <ColumnDefinition Width="*" />
        </Grid.ColumnDefinitions>
        <TextBlock Text="Имя" />
        <TextBox Text="{Binding Name, UpdateSourceTrigger=PropertyChanged}" Grid.Column="1"/>
        <TextBlock Text="Фамилия" Grid.Row="1" />
        <TextBox Text="{Binding Surname, UpdateSourceTrigger=PropertyChanged}" Grid.Row="1" Grid.Column="1"/>
        <TextBlock Text="Возраст" Grid.Row="2" />
        <TextBox Text="{Binding Age, UpdateSourceTrigger=PropertyChanged}" Grid.Row="2" Grid.Column="1"/>
        <StackPanel Orientation="Horizontal" Grid.Row="3" Grid.ColumnSpan="2" Margin="5">
            <Button Content="Добавить" Command="{Binding AddPersonCommand}"
                    CommandParameter="Add"
                    Margin="5,0"/>
            <Button Content="Заменить" Command="{Binding AddPersonCommand}" 
                    CommandParameter="Edit"
                    Margin="5,0" Visibility="{Binding IsChanged, Converter={StaticResource VisibleIfTrueConverter}}" />
        </StackPanel>
        <TextBlock Margin="5" VerticalAlignment="Center" Grid.Row="4" Grid.ColumnSpan="2">
            <TextBlock.Text>
                <MultiBinding StringFormat="Запись добавлена: {0:dd.MM.yyyy HH:hh:ss}; изменена: {0:dd.MM.yyyy HH:hh:ss}">
                    <Binding Path="CreatedOn"/>
                    <Binding Path="UpdatedOn"/>
                </MultiBinding>
            </TextBlock.Text>
        </TextBlock>
        <DataGrid Grid.Row="5" Grid.ColumnSpan="2" CanUserAddRows="False" IsReadOnly="True" AutoGenerateColumns="False"
            ItemsSource="{Binding People, UpdateSourceTrigger=PropertyChanged}" SelectedItem="{Binding SelectedPerson}" >
            <DataGrid.Columns>
                <DataGridTextColumn Header="Фамилия" Binding="{Binding Surname}" />
                <DataGridTextColumn Header="Имя" Binding="{Binding Name}" />
                <DataGridTextColumn Header="Возраст" Binding="{Binding Age}" />
            </DataGrid.Columns>
        </DataGrid>
    </Grid>

VM

Кликните здесь для просмотра всего текста

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
class PersonVM : BaseInpc
    {
        public PersonVM()
        {
            People = new ObservableCollection<Person>
            {
                new Person() { Id = 0, Name="Ivan", Surname="Ivanov", Age=33 },
                new Person() { Id = 1, Name="Petr", Surname="Petrov", Age=22 }
            };
        }
        public ObservableCollection<Person> People { get; private set; }
 
        private Person _selectedPerson;
        public Person SelectedPerson
        {
            get { return _selectedPerson; }
            set
            {
                _selectedPerson = value; RaisePropertyChanged();
                SetCurrentVulues();
            }
        }
 
        private string _name;
        public string Name
        {
            get { return _name; }
            set
            {
                _name = value; 
                RaisePropertyChanged();
                RaisePropertyChanged("IsChanged");
            }
        }
        private string _surname;
        public string Surname
        {
            get { return _surname; }
            set
            {
                _surname = value;
                RaisePropertyChanged();
                RaisePropertyChanged("IsChanged");
            }
        }
        private int _age;
        public int Age
        {
            get { return _age; }
            set
            {
                _age = value; RaisePropertyChanged();
                RaisePropertyChanged("IsChanged");
            }
        }
        private DateTime _createdOn;
        public DateTime CreatedOn
        {
            get { return _createdOn; }
            set { _createdOn = value; RaisePropertyChanged(); }
        }
        private DateTime _updatedOn;
        public DateTime UpdatedOn
        {
            get { return _updatedOn; }
            set { _updatedOn = value; RaisePropertyChanged(); }
        }
        public bool IsChanged
        {
            get
            {
                if (SelectedPerson == null)
                {
                    return false;
                }
                if (SelectedPerson.Surname == Surname &&
                    SelectedPerson.Name == Name &&
                    SelectedPerson.Age == Age)
                {
                    return false;
                }
                return true;
            }
        }
 
        private RelayCommand _addPersonCommand;
        public RelayCommand AddPersonCommand => _addPersonCommand ?? (_addPersonCommand = new RelayCommand(AddPerson));
        private void AddPerson(object parameter)
        {
            if (!string.IsNullOrEmpty(Surname) && !string.IsNullOrEmpty(Name))
            {
                switch ((string)parameter)
                {
                    case "Add":
                        int nextId = People.Select(x => x.Id).Max() + 1;
                        People.Add(new Person() { Id = nextId, Surname = this.Surname, Name = this.Name, Age = this.Age });
                        SelectedPerson = People[nextId];
                        break;
                    case "Edit":
                        foreach (Person p in People)
                        {
                            if (p.Id == SelectedPerson.Id)
                            {
                                p.Surname = Surname;
                                p.Name = Name;
                                p.Age = Age;
                                p.UpdatedOn = DateTime.Now;
                                SelectedPerson = p;
                                break;
                            }
                        }
                        break;
                }
            }
        }
 
        private void SetCurrentVulues()
        {
            Surname = SelectedPerson.Surname;
            Name = SelectedPerson.Name;
            Age = SelectedPerson.Age;
            CreatedOn = SelectedPerson.CreatedOn;
            UpdatedOn = SelectedPerson.UpdatedOn;
            RaisePropertyChanged("IsChanged");
        }
        
    }

В примере есть переменная IsChanged пирвязанная к свойству видимости элемента кнопки "Заменить".
При изменении переменных отдельно, в текстбоксах к которым они привязаны, расчет IsChanged происходит один раз, как и надо.
А при изменении SelectedPerson - пересчитывается 4 раза, 3 при присвоении привязанных переменных и один раз при изменении SelectedPerson.
Вопрос в том, как избавиться от лишних пересчетов. Например отписаться от RaisePropertyChanged этих переменных перед присвоением и подписаться после или есть другие решения?
Либо (если по другому никак) вообще надо перестроить проект?

@Элд Хасп · 19.09.2024, 23:11

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

Model

Это не модель, а какая-то сущность. Скорее даже DTO.

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

Вопрос в том, как избавиться от лишних пересчетов.

И чем они мешают?
В данном случае, усложнение логики не стоит результата.

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

вообще надо перестроить проект?

Я бы посоветовал изменить реализацию свойств и создания зависимостей.
Вы же используете BaseInpc отсюда: Простые реализации для тем на форуме?

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
    public class Person
    {
        private string name = string.Empty;
        private string surname = string.Empty;
 
        public int Id { get; set; }
        public string Name { get => name; set => name = value ?? string.Empty; }
        public string Surname { get => surname; set => surname = value ?? string.Empty; }
        public int Age { get; set; }
        public DateTime CreatedOn { get; set; } = DateTime.Now;
        public DateTime UpdatedOn { get; set; } = DateTime.Now;
 
    }

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
    public class PersonVM : BaseInpc
    {
        public PersonVM()
        {
            People = new ObservableCollection<Person>
            {
                // Нулевой ID в большинстве фреймворках бывает у новых сущностей,
                // которые не сохранены.
                new Person() { Id = /*0*/ 1, Name="Ivan", Surname="Ivanov", Age=33 },
                new Person() { Id = /*1*/ 2, Name="Petr", Surname="Petrov", Age=22 }
            };
        }
        public ObservableCollection<Person> People { get; /*private set; */}
 
        private Person? _selectedPerson;
        public Person? SelectedPerson
        {
            get => _selectedPerson;
            set => Set(ref _selectedPerson, value);
        }
 
        private string _name = string.Empty;
        public string Name
        {
            get => _name;
            set => Set(ref _name, value ?? string.Empty);
        }
 
        private string _surname = string.Empty;
        public string Surname
        {
            get => _surname;
            set => Set(ref _surname, value ?? string.Empty);
        }
        private int _age;
        public int Age
        {
            get => _age;
            set => Set(ref _age, value);
        }
 
        private DateTime _createdOn;
        public DateTime CreatedOn
        {
            get => _createdOn;
            set => Set(ref _createdOn, value);
        }
 
        private DateTime _updatedOn;
        public DateTime UpdatedOn
        {
            get => _updatedOn;
            set => Set(ref _updatedOn, value);
        }
 
        private bool _isChanged;
        public bool IsChanged
        {
            get => _isChanged;
            set => Set(ref _isChanged, value);
        }
 
        private RelayCommand? _addPersonCommand;
        public RelayCommand AddPersonCommand => _addPersonCommand ?? (_addPersonCommand = new RelayCommand(AddPerson));
        private void AddPerson(object parameter)
        {
            if (!string.IsNullOrEmpty(Surname) && !string.IsNullOrEmpty(Name))
            {
                switch ((string)parameter)
                {
                    case "Add":
                        int nextId = People.Select(x => x.Id).Max() + 1;
                        People.Add(new Person() { Id = nextId, Surname = this.Surname, Name = this.Name, Age = this.Age });
                        // SelectedPerson = People[nextId]; Может выдать исключение.
                        SelectedPerson = People[^1];
                        break;
 
                    case "Edit":
                        //foreach (Person p in People)
                        //{
                        //    if (p.Id == SelectedPerson.Id)
                        //    {
                        //        p.Surname = Surname;
                        //        p.Name = Name;
                        //        p.Age = Age;
                        //        p.UpdatedOn = DateTime.Now;
                        //        SelectedPerson = p;
                        //        break;
                        //    }
                        //}
                        // Зачем цикл, если нужная сущность и так выбрана?
 
                        if (SelectedPerson is not null)
                        {
                            SelectedPerson.Surname = Surname;
                            SelectedPerson.Name = Name;
                            SelectedPerson.Age = Age;
                            SelectedPerson.UpdatedOn = DateTime.Now;
                        }
 
                        break;
                }
            }
        }
 
        //private void SetCurrentVulues()
        //{
        //    Surname = SelectedPerson.Surname;
        //    Name = SelectedPerson.Name;
        //    Age = SelectedPerson.Age;
        //    CreatedOn = SelectedPerson.CreatedOn;
        //    UpdatedOn = SelectedPerson.UpdatedOn;
        //    RaisePropertyChanged("IsChanged");
        //}
 
        protected override void OnPropertyChanged(string propertyName, object oldValue, object newValue)
        {
            base.OnPropertyChanged(propertyName, oldValue, newValue);
 
            if (propertyName is nameof(Surname) or nameof(Name) or nameof(Age))
            {
                IsChanged = !
                    (SelectedPerson is null ||
                    SelectedPerson.Surname == Surname &&
                    SelectedPerson.Name == Name &&
                    SelectedPerson.Age == Age);
            }
 
            if (propertyName is nameof(SelectedPerson))
            {
                Surname = SelectedPerson?.Surname!;
                Name = SelectedPerson?.Name!;
                Age = SelectedPerson?.Age ?? 0;
                CreatedOn = SelectedPerson?.CreatedOn ?? DateTime.Now;
                UpdatedOn = SelectedPerson?.UpdatedOn ?? DateTime.Now;
            }
        }
    }

@Beginnner · 23.09.2024, 19:13 **[ТС]**

Сообщение от Элд Хасп

И чем они мешают?
В данном случае, усложнение логики не стоит результата.

Согласен, в данном случае не мешают.
Просто представил большой класс, где таких пересчетов будет не 2-3, а x*[много]^2!.
Вот и возник интерес, как этого можно избежать.

Сообщение от Элд Хасп

Вы же используете BaseInpc отсюда:

Верно.

Сообщение от Элд Хасп

Я бы посоветовал изменить реализацию свойств и создания зависимостей.

Спасибо за интересный подход!
Но если реализовать кнопку отмены внесенных изменений, все равно же придется использовать метод SetCurrentVаlues() в команде этой кнопки.

@Элд Хасп · 23.09.2024, 21:05

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

Но если реализовать кнопку отмены внесенных изменений, все равно же придется использовать метод SetCurrentVаlues() в команде этой кнопки.

Команда - это МЕТОД. У метода, естественно, есть какое-то тело с логикой.
А вот у свойств нежелательно помещать сложную логику в тело. Создание зависимостей лучше вынести. Всё таки основное назначение свойства - хранить значение. И дополнительная логика усложняет понимание кода, увеличивает вероятность ошибок.

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

Просто представил большой класс, где таких пересчетов будет не 2-3, а x*[много]^2!.

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

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Access VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .