Программа для тестов - как грамотно подойти к редактированию тестов

@Casper-SC · Регистрация: 27.03.2010

Что я пишу:
Вот пишу приложение для создания тестов, dll с классами самих тестов и приложение, которое будет уже отображать эти тесты, там можно себя проверить например на каком-то тесте, получить баллы за ответы на вопросы теста и прочитать о себе что-то, короче обыкновенные тесты. Программа, которая создаёт тесты должна уметь их редактировать естественно, то есть редактировать сам вопрос, ответы и т.д. Программа, которая будет отображать тесты, она будет грузить тесты из файла и их отображать.

Суть вопроса:
По идее же в программе отображающей тесты, нежелательно давать вообще возможность изменять вопросы, ответы и т.д. То есть меня почему-то постоянно не покидает мысль, что свойства нельзя делать доступными для изменения. Но тогда, в программе редактирующей тест, для изменения теста, мне придётся создавать новый экземпляр класса теста, заменять в тесте все ответы со старого экземпляра в новый и помещать этот экземпляр в коллекцию, удалив старый. А если сделать свойства доступные для редактирования, то все эти беды отпадают, просто меняю свойство и никаких проблем.

То есть мне для избавления от проблем нужно это:

Вариант 1

C#

    /// <summary>Вопрос</summary>
    public class Question
    {
        public Question(string text)
        {
            Text = text;
            Answers = new List<Answer>(4);
        }
 
        public Question(string text, IEnumerable<Answer> answers)
        {
            Text = text;
            Answers = new List<Answer>(answers);
        }
 
        /// <summary>Ответы</summary>
        public List<Answer> Answers { get; protected set; }
        /// <summary>Текст вопроса</summary>
        public string Text { get; protected set; }
    }

C#

    /// <summary>Ответ на вопрос</summary>
    public class Answer
    {
        public Answer(string text, uint marks, bool isCorrect)
        {
            Text = text;
            Marks = marks;
            IsCorrect = isCorrect;
        }
        /// <summary>Определяет правильный ли ответ</summary>
        public bool IsCorrect { get; protected set; }
        /// <summary>Баллы за ответ</summary>
        public uint Marks { get; protected set; }
        /// <summary>Текст ответа</summary>
        public string Text { get; protected set; }
    }

Заменить на это:

Вариант 2

C#

    /// <summary>Вопрос</summary>
    public class Question
    {
        public Question()
            :this(string.Empty)
        {
        }
 
        public Question(string text)
        {
            Text = text;
            Answers = new List<Answer>(4);
        }
 
        public Question(string text, IEnumerable<Answer> answers)
        {
            Text = text;
            Answers = new List<Answer>(answers);
        }
 
        /// <summary>Ответы</summary>
        public List<Answer> Answers { get; protected set; }
        /// <summary>Текст вопроса</summary>
        public string Text { get; set; }
    }

C#

    /// <summary>Ответ на вопрос</summary>
    public class Answer
    {
        public Answer()
        {
        }
 
        public Answer(string text, uint marks, bool isCorrect)
        {
            Text = text;
            Marks = marks;
            IsCorrect = isCorrect;
        }
        /// <summary>Определяет правильный ли ответ</summary>
        public bool IsCorrect { get; set; }
        /// <summary>Баллы за ответ</summary>
        public uint Marks { get; set; }
        /// <summary>Текст ответа</summary>
        public string Text { get; set; }
    }

Как вообще правильнее делать? Понимаю, моожет вообще кому-то покажется вопрос странным, и кто-то делает вообще как удобнее не задумываясь. Просто хочу всё делать правильно и научиться нормально строить архитектуру приложения. Или не стоит делать общедоступными только поля (ну этого вообще никогда и не делаю), которые могу повредить внутреннюю работу класса, то есть даже нельзя их менять извне через методы и свойства. А те свойства, что несут в себе информацию и не более, можно делать доступными для записи?

@I2um1 · 05.01.2012, 19:35

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

@Casper-SC · 05.01.2012, 19:55 **[ТС]**

Сообщение от Blood-Angel

Надеюсь доходчиво объяснил.

Неа, на своём примере, что-то не ясно как применять. Там у меня есть ещё класс самого теста, вот он будет сохраняться в файл:

C#

    /// <summary>Представляет из себя тест предназначенный для тестирования</summary>
    public class Test
    {
        /// <summary>Описание теста, включает в себя дополнительную информацию о тесте и название</summary>
        public Description Description { get; protected set; }
        /// <summary>Вопросы теста</summary>
        public List<Question> Questions { get; protected set; }
    }

Добавлено через 4 минуты
Сейчас попробую оттуда пример скомпилировать и посмотреть в работе, что и как он делает.

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

@I2um1 · 05.01.2012, 20:26

Попробую объяснить на простом примере, чтобы показать суть картины. Есть такая вещь как MVVM (Model-View-ViewModel), если его сравнивать с MVC (Model-View-Controller), то можно груба считать, что Controller - это ViewModel. В нашем же случае это и есть посредник между Model и View. Но особенность MVVM в том, что у него несколько View. (Там есть еще особенности, но в данном случае это не важно)
Вот теперь допустим, что один View для прохождения самого теста, а другой View для редактирования. Причем оба они ссылаются на ViewModel. В таком случае ViewModel разной View предоставляет разный функционал.
Это один из вариантов. Есть еще такой способ как "Инъекция зависимости". Суть ее в том, что мы навязываем интересующему нам классу определенный функционал. К примеру в конструктор этого класса передаем интерфейс контроллера с набором определенного функционала из него, при этом запрещаем создание конструктора по умолчанию, обязывая тем самым передать этот интерфейс.
P.S. Более подробно можно прочесть в книгах Гаммы и Фаулера.

@Casper-SC · 05.01.2012, 21:13 **[ТС]**

Пока-что у меня получилось

вот что.

C#

    public abstract class ObjectEditorBase
    {
        public abstract bool IsEditingInProgress { get; protected set; }
 
        public abstract void StartEdit(object editedObject);
 
        public abstract void ApplyEdit();
 
        public abstract void CancelEdit();
 
        /// <summary>Редактирование началось</summary>
        public abstract event EventHandler EditingStarted;
        /// <summary>Изменения разрешены</summary>
        public abstract event EventHandler<ObjectEditorEventArgs> ChangesArePermitted;
        /// <summary>Изменения отменены</summary>
        public abstract event EventHandler ChangesAreCancelled;
 
        protected abstract void OnEditingStarted();
 
        protected abstract void OnChangesArePermitted();
 
        protected abstract void OnChangesAreCancelled();
    }
 
    public class QuestionEditor : ObjectEditorBase
    {
        Question _editedQuestion;
 
        public override void StartEdit(object editedObject)
        {
            if (!(editedObject is Question))
                throw new ArgumentException();
 
            _editedQuestion = (Question)editedObject;
            OnEditingStarted();
        }
 
        public override void ApplyEdit()
        {
            _editedQuestion = null;
            OnChangesArePermitted();
        }
 
        public override void CancelEdit()
        {
            _editedQuestion = null;
            OnChangesAreCancelled();
        }
 
        public override bool IsEditingInProgress
        {
            get { return _editedQuestion != null; }
            protected set { throw new NotImplementedException(); }
        }
 
        /// <summary>Редактирование началось</summary>
        public override event EventHandler EditingStarted;
        /// <summary>Изменения разрешены</summary>
        public override event EventHandler<ObjectEditorEventArgs> ChangesArePermitted;
        /// <summary>Изменения отменены</summary>
        public override event EventHandler ChangesAreCancelled;
 
        protected override void OnEditingStarted()
        {
            if (EditingStarted != null)
                EditingStarted(this, EventArgs.Empty);
        }
 
        protected override void OnChangesArePermitted()
        {
            if (ChangesArePermitted != null)
                ChangesArePermitted(this, new ObjectEditorEventArgs(_editedQuestion));
        }
 
        protected override void OnChangesAreCancelled()
        {
            if (ChangesAreCancelled != null)
                ChangesAreCancelled(this, EventArgs.Empty);
        }
    }
 
    public class ObjectEditorEventArgs : EventArgs
    {
        public ObjectEditorEventArgs(object editedObject)
        {
            EditedObject = editedObject;
        }
 
        public object EditedObject { get; protected set; }
    }

То есть я буду подписываться на события этого класса в WPF окне, там если изменения подтверждены я буду выполнять те действия, на которые подпишусь (событие ChangesArePermitted), если изменения не подтверждены, то я буду выполнять другие действия с интерфейсом. Я пока ещё не пробовал правда, я только дописал эти классы. Пока не знаю, какие будут проблемы при таком подходе. Буду пробовать в действии. Хотя может это вообще глупая затея с такими классами, посмотрю во что это выльется и стоит ли такое применять. А как сделать по паттерну, что-то я не особо врубился, ну примерно имею представление, но вообще отдалённо.

@I2um1 Злой няш 1949 / 1384 / 508 Регистрация: 05.04.2010 Сообщений: 2,627
	05.01.2012, 19:35	2
	Сделать свойства как для чтения, так и для записи открытыми. После чего реализовать паттерн Посредника. Надеюсь доходчиво объяснил. 1

@I2um1 Злой няш 1949 / 1384 / 508 Регистрация: 05.04.2010 Сообщений: 2,627
	05.01.2012, 20:26	4
	Попробую объяснить на простом примере, чтобы показать суть картины. Есть такая вещь как MVVM (Model-View-ViewModel), если его сравнивать с MVC (Model-View-Controller), то можно груба считать, что Controller - это ViewModel. В нашем же случае это и есть посредник между Model и View. Но особенность MVVM в том, что у него несколько View. (Там есть еще особенности, но в данном случае это не важно) Вот теперь допустим, что один View для прохождения самого теста, а другой View для редактирования. Причем оба они ссылаются на ViewModel. В таком случае ViewModel разной View предоставляет разный функционал. Это один из вариантов. Есть еще такой способ как "Инъекция зависимости". Суть ее в том, что мы навязываем интересующему нам классу определенный функционал. К примеру в конструктор этого класса передаем интерфейс контроллера с набором определенного функционала из него, при этом запрещаем создание конструктора по умолчанию, обязывая тем самым передать этот интерфейс. P.S. Более подробно можно прочесть в книгах Гаммы и Фаулера. 1

Опции темы