Re: CPropertySheet

From:

"David Ching" <dc@remove-this.dcsoft.com>

Newsgroups:

microsoft.public.vc.mfc

Date:

Mon, 12 Feb 2007 15:12:20 GMT

Message-ID:

<ot%zh.17649$ji1.11315@newssvr12.news.prodigy.net>

"David Wilkinson" <no-reply@effisols.com> wrote in message
news:edplnRpTHHA.480@TK2MSFTNGP02.phx.gbl...

David:

I have to say I do not understand what you are saying here. Did you mean

CPropertySheet sheet;
CMyFirstPage1 page1;
CMyFirstPage page2;

sheet.AddPage(&page1);
sheet.AddPage(&page2);

i.e. use a plain CPropertySheet?

Oops, sorry, yes I meant to use a generic CPropertySheet.

I have never thought of doing it this way. I always derive my own
CPropertySheet class and add the pages inside. The pages are an
implemetation detail of the sheet, no? There is no need for the caller to
know about them.

Well, the property sheet is really a container, the question is, do you need
to derive a new container type? You can get by without doing so, since you
don't need to override any virtual methods to change the behavior of the
container. It's like using a CListBox to hold arbitrary items using
CListBox::SetItemDataPtr() - you don't need to derive a new CListBox to
manage these; the caller can. I suppose it's a matter of how much do you
value encapsulation vs. convenience to the caller. Both approaches are
legitimate, but I don't like to manage another class unless it's really
necessary.

Thanks,
David (MVP)