Архифлеймер
Зарегистрирован: 04 October 2005
Сообщения: 3067
Примеры кода: 0
|
Re: [dev] octalforty Waffle -- форум |
21 February 2008 15:34 |
|
|
|
|
Нахлобуч:
C...R...a...S...HЗачем такой огород с if и as непонятно
Потому что семантически неверно у какого-нибудь DiscussionPost'а иметь свойство EmailAddress -- пусть даже оно всегда null возвращает.
Так по этому мне кажется общие свойства должны находиться в базовом классе, а все что уникальное уже в наследниках.
И поэтому метод BuildRssFeedItem должен на вход принимать не абстрактный IBusinessObject и там смотреть что он из себя представляет, а должет точно определять что он хочет:
___ BuildRssFeedItem(Discussion discussion)
___ BuildRssFeedItem(Forum forum)
Которые, вызывают
___ BuildRssFeedItem(IBusinessObject bo)
А потом дописывают что им надо.
А у Вас получается наоборот.
сначала вы скрываете конкретную реализацию, а потом динамически пытаетесь ее получить.
----------------------------------------
Knowledge is P...O...w...E...R!
My site
Данное сообщение получено с сайта GotDotNet.RU
|
|