Вы видите какие-либо (анти) узоры в моем дизайне? Как распознать шаблон?

Question

У меня есть веб-приложение, закодированное в ABAP/BSP. Мне интересно доставить прекрасный код, поэтому, возможно, вы можете помочь мне распознать хороший или плохой шаблон. На самом деле я делаю следующее. Объявите объект базового класса и динамически создайте его на основе некоторого параметра с подклассом. У базового класса есть некоторые методы для извлечения данных() под-классы используют эти данные и, возможно, манипулируют им.

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

alt text http://img35.imageshack.us/img35/4351/designkdp.jpg

ли Somthing как этот ОК? Есть ли какой-либо известный шаблон скрытый или я должен рефакторировать и использовать шаблон? Мне нравятся некоторые предложения. Я уже прочитал статью в Википедии, но я не вижу точно такой образ. (Также заказал несколько книг, но вы, ребята уверены, что быстрее :))

Answer 1

Кажется factory method

Answer 2

Это шаблон фабрики из книги «Design Patterns: Элементы Многоразовые объектно-ориентированного программного обеспечения» от Gamma, Helm , Джонсон и Влиссидес.

Answer 3

BaseClass и подклассы, мне кажется, как имеющее что-то как (псевдокод)

interface IDataProvider{ 
    GetData(); 
} 

class SubClass1 implements IDataProvider{ 
    GetData(){ 
    ... 
    } 
} 

Стратегия конкретизации вы используете, кажется, работает как Factory. Что вы можете сделать - это сделать рендеринг в каком-нибудь HTMLRenderingEngine или что-то еще (я не знаком с ABAP/BSP), просто чтобы разделить презентацию от остальной части логики.

Answer 4

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

Что делать, если у вас был «DataClass», в котором вы могли бы установить processData объект (или нет). Объект processData фактически будет интерфейсом с единственным методом «processData».

При вызове GetData, это может выглядеть примерно так:

if(processDataObject == null) 
    return getInformation(); 
else 
    return processDataObject.processData(getInformation()); 

Heck, если у вас есть «Сквозной» объект ProcessData определил, что ничего не делает, и использовать его в качестве значения по умолчанию «processDataObject», то весь метод GetData становится:

return processDataObject.processData(getInformation()); 

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

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