События или наследование?

Question

У меня есть класс процесса, где, если вход содержит значение, он также сделает что-то еще.

Например

Person { name; age } 

ProcessPerson(person) //takes in a person 

в ProcessPerson если возраст = старше 18 лет (для взрослых), а затем отправить по электронной почте.

ProcessPerson уже существует, и базовая логика должна выполняться независимо от того, что (сохранить человека в БД).

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

я могу думать о 2 пути решения этого и хотел Деку сессии:

1. Наследования -> создать производный класс под названием ProcessAdult
2. Событие -> после сохранения человека к БД, повысить обработанное событие. Добавьте обработчик, который отправляет электронное письмо.

Я предпочитаю последний из 2. Или, возможно, есть другой чистый и расширяемый метод, о котором я не думал?

Answer 1

Ваш второй вариант, безусловно, намного лучше.

Answer 2

Я голосую за второй вариант.

Answer 3

Для случая, который вы описываете, мне также нравится # 2. Звучит чище и более конкретно обращается к вашей ситуации.

Answer 4

Возможно, вы ошибаетесь, но я бы не подклассифицировал класс Process, а скорее класс Person и потребовал, чтобы дети реализовали метод PostProcess или что-то в этом роде. Это фактически позволяет вам также требовать «PreProcess», если вы думаете, что через пару лет у вас может возникнуть необходимость в этом. Затем ваш метод Process принимает Person-base, вызывает PreProcess(), вставляет в БД и затем вызывает PostProcess(). Дети должны применять этот метод, но, очевидно, ничего не должны делать.

Но, честно говоря, в зависимости от того, где это живет и количества возможных сценариев, я скорее всего сделаю это изнутри в Process() и не буду заниматься событиями или наследованием.

if(person.Age>=18){}else if(person.Pets){}... 

Answer 5

Может быть, ваше описание проще, чем ваша актуальная проблема, но то, что случилось с:

void ProcessPerson(Person const& p) { 
    // do what you always do 

    if(p.age>=18) { 
     // send mail 
    } 
    // add other "special" handling here 
} 

?

Вы бы поступили из Person, если у вас есть дополнительные данные или специальные методы или оба, но вы этого не сделаете. Затем вы можете реализовать специализацию ProcessPerson для обработки производного класса.

Если вся информация, необходимая для обработки объекта Person, доступна локально, сделайте все необходимое в этом методе.

Если вам нужны разные фрагменты информации для обработки Person, которые не имеют ничего общего друг с другом, распределите обработку. У вас может быть, например,различные обработчики для вашей базы данных и ваша почтовая система и STH как:

Person p; 
foreach(handler in handler list) { 
    handler.Process(p); 
} 

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

Answer 6

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

public delegate void PersonProcessedEventHandler(Person p); 
public event PersonProcessedEventHandler PersonProcessed; 

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

public void EmailAdult(Person p) 
{ 
    if(p.age >= 18) 
    { 
     //send email 
    } 
} 

public void SendWaffles(Person p) 
{ 
    if(p.name = "John Skeet") 
    { 
     //send waffles 
    } 
} 

thePersonProcessor.PersonProcessed += EmailAdult; 
thePersonProcesser.PersonProcessed += SendWaffles;