Создание программы - ООП модель, MVP

@Storm23 · Регистрация: 11.01.2015

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

Пример разработки приложения с нуля под WinForms

Рассматриваются такие аспекты разработки как:

Создание ООП модели предметной области.
Разработка Unit-тестов.
Архитектура приложения под WinForms.
Разработка пользовательского интерфейса с разделением модели и представления (MVP).
Разработка UserControl для построения пользовательского интерфейса.
Рефакторинг, непрерывная интеграция.
Организация проекта в VisualStudio.

В этом примере НЕ рассматриваются шаблоны типа фабрик, не рассматривается внедрение зависимостей (DI).
Используется простой вариант шаблона Model-View-Presenter, адаптированный под специфику WinForms.

Этот топик перекликается с этим FAQ, и является практическим примером к нему.
Комментарии и вопросы - приветствуются.

Постановка задачи

Требуется создать приложение - опросник.

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

Приложение должно состоять из двух частей - конструктора опросника и приложения в котором респондент может ответить на вопросы готового опросника.

Результаты опроса должны выгружаться в виде, пригодном для дальнейшей обработки.

@MIRAMIX · 02.01.2019, 18:30

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

Не понимаю почему, но все работает теперь вроде корректно))

оказалось нет. Уже выбранная альтернатива не сбрасывалась при нажатии назад.

В том же методе, в том же классе interview.anketa.Remove(prevAnswer); заменил на interview.anketa.Remove(interview.Curren tAnswer);
Экспорт теперь вроде работает корректно.

Еще подкорректировал механизм возврата.
У кнопки btBack остались теже методы :

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

C#
1
2
interviewManipulator.GoToPrevQuestion();
Build();

А в конце метода Build формы прохождения опросника добавил :

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

C#
1
2
3
4
5
6
7
8
9
btNext.Parent = interview.IsFinished ? null : pnAnswers;
btBack.Parent = interview.IsFinished ? null : pnAnswers;
btFinish.Parent = interview.IsFinished ? pnAnswers : null;
 
if (interview.PassedAnswers.Count != 0) //отсюда
{btBack.Visible = true;}
else btBack.Visible = false; //до сюда
 
helper.ResumeDrawing();

Добавлено через 8 минут
if-else заменил на

C#
1
btBack.Visible = (interview.PassedAnswers.Count != 0) ? true : false;

@Storm23 · 02.01.2019, 18:49 **[ТС]**

MIRAMIX,
Все правильно вы сделали (в последнем варианте)
НО, вот это выражение interview.PassedAnswers.Count != 0 - это логика. А логики не должно быть в интерфейсе. Логика должна быть в модели.
Поэтому в InterviewManipulator нужно сделать метод CanGoToPrevQuestion:

C#
1
2
3
4
public bool CanGoToPrevQuestion()
{
   return interview.PassedAnswers.Count > 0;
}

А в интерфейсе, в методе Build только проверять этот метод:

C#
1
btBack.Visible = interviewManipulator.CanGoToPrevQuestion();

@MIRAMIX · 02.01.2019, 21:08

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

А логики не должно быть в интерфейсе.

Боялся остальное разрушить)

Код огонь, никаких if else, а получаем правду/ложь.
Несколько раз пересмотрел, чтоб понять))

Где всему этому научиться? Явно не по книжкам

Добавлено через 2 часа 0 минут
Storm23, ну никак не могу понять - как запретить проге принимать пустой ответ в строке ввода.
Если пользователь ничего не ввел - вставлять в эксель дефолтную фразу null.

Все файлы перелопатил. Логически, подходит вот это из InterviewManipulator :

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

C#
1
2
3
//добавляем текущий ответ в список отвеченных вопросов
if(interview.CurrentAnswer != null)
interview.PassedAnswers.Add(interview.CurrentAnswer);

Вроде есть условие - принимаем не пустые ответы, но на деле прохождение опросника идет как по маслу(

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

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

@MIRAMIX · 08.01.2019, 22:21

Storm23, прошу помощи с проверкой на пустой ответ с ручным вводом.

Сделал вот так у кнопки перехода к следующему вопросу :

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

C#
1
2
3
4
5
6
if (interview.CurrentAnswer.Text != null)
{
       interviewManipulator.GoToNextQuestion();
       Build();
}
else MessageBox.Show("Пустой ответ!");

Так месседжбокс выходит и при выборе ответа из списка.

В идеале - при пустом ответе отправлять ответ типа прочерка, но пытался хотя бы информировать, чтоб был дан ответ, но не работает(

@Storm23 · 09.01.2019, 10:00 **[ТС]**

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

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

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

В идеале - при пустом ответе отправлять ответ типа прочерка

Ну так эту проблемы экспорта а не модели. Если внешняя БД не поддерживает пустые значения, нужно проверять пустые ответы при экспорте и делать замену на какое-то значение (не в модели а в экспортируемом значении!).

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

хотя бы информировать, чтоб был дан ответ

Ответ хранится либо в поле Answer.AlternativeCode либо в поле Answer.Text.
Для проверки того, что ответ пустой сделайте в классе Answer свойство IsEmpty:

C#
1
2
3
4
public bool IsEmpty
{
   get{ return AlternativeCode == null && string.IsNullOrEmpty(Text); }
}

Затем, в интерфейсе проверяйте это свойство.

@MIRAMIX · 09.01.2019, 22:31

Storm23, огромное спасибо)

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

Затем, в интерфейсе проверяйте это свойство.

ничего, что я в классе InterviewManipulator вставил эту проверку + подмену ответа?
2я половина метода GoToNextQuestion :

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

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
                    //добавляем текущий ответ в список отвеченных вопросов
                    if(interview.CurrentAnswer != null)
                    {
                        if (interview.CurrentAnswer.IsEmpty)
                        {
                            interview.CurrentAnswer.Text = "-";
                        }
                        interview.PassedAnswers.Add(interview.CurrentAnswer);
                    }
                        
 
                    //создаем новый Answer для нового вопроса
                    interview.CurrentAnswer = new Answer { QuestId = quest.Id };
                    interview.anketa.Add(interview.CurrentAnswer);
                    //
                    return;
                }
            }
 
            //нет следующего вопроса, анкета завершена
            if (interview.CurrentAnswer != null)
            {
                if (interview.CurrentAnswer.IsEmpty)
                {
                    interview.CurrentAnswer.Text = "-";
                }
                interview.PassedAnswers.Add(interview.CurrentAnswer);
                interview.CurrentAnswer = null;
            }
            interview.IsFinished = true;
        }

На локальном тесте вроде все ок. Если эту проверку вставлять в обработчик кнопки перехода к следующему вопросу при прохождении опроса - нужно нажать 2 раза, чтоб ответ ("-") вставился и закрепился.

@Storm23 · 09.01.2019, 22:33 **[ТС]**

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

ничего, что я в классе InterviewManipulator вставил эту проверку + подмену ответа?

Если все делать через одно место, то ничего страшного.

@MIRAMIX · 09.01.2019, 22:50

Не по теме:

Storm23, в силу своей наивности, не всегда правильно понимаю речевые обороты. Надеюсь вы не о том, откуда руки у некоторых растут :)

@Soft17 · 11.08.2020, 15:25

Storm23,
1. Можно ли рассматривать сообщения 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP ?
2. В данных сообщениях актуальные решения или нужно собирать по всей теме изменения к 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP?
3. Или есть обновлённое решение в одном месте?

Задаю вопрос, потому, что бегло прочитал сообщения других пользователей.... многие критикуют данное решение.

@Storm23 · 12.08.2020, 10:40 **[ТС]**

Soft17,

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

1. Можно ли рассматривать сообщения 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP ?

Строго говоря нет. Это один из вариантов паттернов MV*. Но отнести именно к MVP - нельзя, потому что здесь нет отдельных классов - презентеров. Хотя все это очень расплывчато и зависит от определений.

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

2. В данных сообщениях актуальные решения или нужно собирать по всей теме изменения к 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP?

Все версии коммулятивны и включают все предыдущие правки, можно брать любую версию - она рабочая.

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

3. Или есть обновлённое решение в одном месте?

Берите мои версии в постах. Другие версии - не знаю.

@LunaticCalm · 07.07.2021, 13:10

Storm23, добрый день. Подскажите, пожалуйста, правильное с Вашей точки зрения решение проблемы замены юзерконтрола в зависимости от выбранного значения комбобокса в создающемся по умолчанию юзерконтролу с возможностью возврата к изначальному. Если не сложно, то пример кода был бы полезным. Заранее благодарю.

@Storm23 · 07.07.2021, 15:06 **[ТС]**

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

Подскажите, пожалуйста, правильное с Вашей точки зрения решение проблемы замены юзерконтрола в зависимости от выбранного значения комбобокса в создающемся по умолчанию юзерконтролу с возможностью возврата к изначальному

Что значит замены юзерконтрола?
Редактируется объект одного и того же класса? Или в зависимости от комбобокса нужно создавать/редактировать объекты разных классов? Если второе - то нужно просто создать юзерконтрол для нужного класса и передвать в его Build(...) нужный объект.
Если первое - то не очень понятно, почему у объекта разные редакторы. Ну в принципе решается также - создаются разные юзерконтролы, в их Build передается один и тот же объект.
В чем проблема то?

@LunaticCalm · 07.07.2021, 15:15

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

В чем проблема то?

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

@lonelyhunter · 15.12.2021, 16:21

Может вопрос уже где-то задавался, но внятного ответа я не нашел.

Как быть, если построение модели занимает время? (например, загружаются данные из Интернета)

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
        private Weather w;
 
        private void BuildDataSource()
        {
            w = Weather.GetWeather();
        }
 
        private async void BuildInterface()
        {
            CityControl moscowCityControl = new CityControl() { Parent = this };    
            moscowCityControl.Build(w);
            Controls.Add(moscowCityControl);          
        }

как правильно реализовать получение данных чтобы форма при этом не висла, например, при медленном Интернете?
в моем случае, пока BuildDataSource() не отработает, форма будет мертва.

@live_4_ever · 16.12.2021, 17:07

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

в моем случае, пока BuildDataSource() не отработает, форма будет мертва.

Думаю тогда можете задействовать асинхронность на этом методе:

C#
1
2
3
4
private async void BuildDataSource()
        {
            w = await Weather.GetWeather();
        }

@lonelyhunter · 18.12.2021, 20:03

И ещё. Подскажите пожалуйста, почему тут - https://www.cyberforum.ru/atta... 1445438693 предпочитаются не конструкторы а статические классы для создания объектов модели?

@Storm23 · 19.12.2021, 10:19 **[ТС]**

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

И ещё. Подскажите пожалуйста, почему тут - https://www.cyberforum.ru/atta... 1445438693 предпочитаются не конструкторы а статические классы для создания объектов модели?

Статический метод Create вместо конструктора используется тогда, когда тип создаваемого объекта заранее неизвестен и зависит от входных параметров.

Например, для gif картинок вы захотите добавить свойство "продолжительность анимации". Для этого вы сделаете наследник class AnimatedFileItem : FileItem. И в этом классе пропишите свойство duration.

Но как теперь создавать AnimatedFileItem вместо FileItem? Если использовать просто конструкторы, то получается, что решение о том какой класс создавать будет принимать внешний (по отношению к FileItem) код. Это нехорошо, потому что логика создания отдается на откуп внешнего кода, нарушается инкапсуляция.

А вот если использовать статический метод Create внутри FileItem, то класс FileItem сам решает какой конкретно тип объекта создать, создает и возвращает его. При этом принятие решение об использовании конкретного наследника FileItem принимает сам FileItem. Сохраняется инкапсуляция; внешний код менять не потребуется.

@lonelyhunter · 19.12.2021, 11:09

Storm23, но в этом примере все заранее известно. Поэтому все еще не понимаю. Или я просто взял пример, когда разработка еще кончилась?

@Storm23 · 19.12.2021, 12:36 **[ТС]**

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

но в этом примере все заранее известно.

Хорошая архитектура должна быть расширяемой. В этом примере тип всегда один. Но если вы захотите расширить, то при использовании конструкторов не сможете этого сделать не меняя внешний код. А с Create - сможете.
Это просто задел на будущее.

@lonelyhunter · 19.12.2021, 12:56

Понял, благодарю.

Новые блоги и статьи Все статьи Все блоги /
Модель здравосохранения 14. Собираем всю модель вместе. anaschu 22.05.2026 Модель собрана. В будущих постах на видео я покажу, как она работает. В этом посте запускаем её, проверяем результаты и разбираем что можно с ней делать дальше. Перед запуском проверяем. . .	Модель здравоохранения 13. Добавление самой системы здравоохранения. anaschu 22.05.2026 В предыдущем посте мы настроили болезни. Теперь добавим события, которые управляют здоровьем всего коллектива, а также настроим рабочий график и расчёт финансов. В Main создаём четыре события. . . .	Модель здравоохранения 12. добавление болезней через ресурпул, как аварии anaschu 22.05.2026 Болезни — это ключевая часть нашей модели. Нам нужно, чтобы работник периодически уходил на больничный, его задание при этом зависало, а после выздоровления работа возобновлялась. Реализуем это двумя. . .	Модель здравоохранения 11. Создаём классы Задание и Работник anaschu 22.05.2026 В AnyLogic каждая заявка и каждый ресурс — это объект определённого класса. Нам нужно создать два класса: Задание (заявка) и Работник (ресурс). Класс Задание В дереве проекта нажимаем правой. . .
Модель здравоохранения 10. Новая модель, смотрим, как добавлять логические блоки, и что писать внутри anaschu 22.05.2026 Открываем AnyLogic, создаём новый проект. В дереве проекта появляется класс Main — это главный агент, в котором будет жить вся наша логика. Палитра блоков Слева находится палитра. Нас интересует. . .	модель ЗдравоСохранения 9. Новая модель, разбираемся, как ее создавать anaschu 22.05.2026 В этой серии постов мы построим модель небольшого рабочего коллектива. Сотрудники получают задания, выполняют их, иногда болеют — и мы хотим посчитать, сколько это стоит компании. Метод. . .	[golang] Linked list alhaos 22.05.2026 Связный список / Linked list Связный список структура данных позволяющая хранить список значений, в отличии от массива в памяти хранится не сплошным куском, а отдельными частями которые ссылаются. . .	[golang] Двоичная куча, min-heap alhaos 20.05.2026 Двоичная куча Двоичная куча — структура данных, которая всегда держит самый важный элемент наготове. Представьте очередь к хилеру в игре, и очередь из игроков в приоритете те у кого меньше. . .

@Storm23 10428 / 5158 / 1825 Регистрация: 11.01.2015 Сообщений: 6,226 Записей в блоге: 34

	Создание программы - ООП модель, MVP 30.10.2018, 11:10. Показов 72943. Ответов 100 Метки mvc, mvp, oop, unit-тест, ООП (Все метки) Пример разработки приложения с нуля под WinForms Рассматриваются такие аспекты разработки как: Создание ООП модели предметной области. Разработка Unit-тестов. Архитектура приложения под WinForms. Разработка пользовательского интерфейса с разделением модели и представления (MVP). Разработка UserControl для построения пользовательского интерфейса. Рефакторинг, непрерывная интеграция. Организация проекта в VisualStudio. В этом примере НЕ рассматриваются шаблоны типа фабрик, не рассматривается внедрение зависимостей (DI). Используется простой вариант шаблона Model-View-Presenter, адаптированный под специфику WinForms. Этот топик перекликается с этим FAQ, и является практическим примером к нему. Комментарии и вопросы - приветствуются. Постановка задачи Требуется создать приложение - опросник. Опросник будет состоять из нескольких вопросов с различными вариантами ответов. Вопросы могут быть двух видов - с выбором одного ответа из набора возможных, и открытые вопросы, в которых респондент может указать произвольный ответ в виде текста. В опроснике нужно предусмотреть возможность задавать некоторые вопросы и альтернативы только при выполнении некоторых условий, например если в предыдущем вопросе респондент выбрал определенный ответ. Приложение должно состоять из двух частей - конструктора опросника и приложения в котором респондент может ответить на вопросы готового опросника. Результаты опроса должны выгружаться в виде, пригодном для дальнейшей обработки. 19

@MIRAMIX
	09.01.2019, 22:50
	Не по теме: Storm23, в силу своей наивности, не всегда правильно понимаю речевые обороты. Надеюсь вы не о том, откуда руки у некоторых растут :) 0

@Soft17 5 / 5 / 3 Регистрация: 15.01.2017 Сообщений: 690
	11.08.2020, 15:25
	Storm23, 1. Можно ли рассматривать сообщения 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP ? 2. В данных сообщениях актуальные решения или нужно собирать по всей теме изменения к 1-13, 38 (Шаг 11: Добавление экспорта в БД) как пример ООП MVP? 3. Или есть обновлённое решение в одном месте? Задаю вопрос, потому, что бегло прочитал сообщения других пользователей.... многие критикуют данное решение. 0

@LunaticCalm 0 / 0 / 0 Регистрация: 25.02.2021 Сообщений: 2
	07.07.2021, 13:10
	Storm23, добрый день. Подскажите, пожалуйста, правильное с Вашей точки зрения решение проблемы замены юзерконтрола в зависимости от выбранного значения комбобокса в создающемся по умолчанию юзерконтролу с возможностью возврата к изначальному. Если не сложно, то пример кода был бы полезным. Заранее благодарю. 0

@lonelyhunter 67 / 65 / 61 Регистрация: 11.11.2015 Сообщений: 395
	18.12.2021, 20:03
	И ещё. Подскажите пожалуйста, почему тут - https://www.cyberforum.ru/atta... 1445438693 предпочитаются не конструкторы а статические классы для создания объектов модели? 0

@lonelyhunter 67 / 65 / 61 Регистрация: 11.11.2015 Сообщений: 395
	19.12.2021, 11:09
	Storm23, но в этом примере все заранее известно. Поэтому все еще не понимаю. Или я просто взял пример, когда разработка еще кончилась? 0

@lonelyhunter 67 / 65 / 61 Регистрация: 11.11.2015 Сообщений: 395
	19.12.2021, 12:56
	Понял, благодарю. 0