HI Chad, > "Decorator" in there. Don't you think thats more > understandable from a > developer's point of view?
Well, "FacadeDecorator" is less understandable, I think. > Although I do like FacadeLogic > and FacadeLogicImpl too....how about I change to those names? Yeah, FacadeLogic is cool and describes the real intent behind it. I always like "talking names". :-) So, if you can agree with it - please do! Cheers... Matthias > -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED] On Behalf > Of Chad Brandon > Sent: Saturday, February 28, 2004 10:55 PM > To: Matthias Bohlen; 'Wouter Zoons' > Cc: [EMAIL PROTECTED] > Subject: Re: [Andromda-devel] MMFs: The final battle continues! > > > > ----- Original Message ----- > From: "Matthias Bohlen" <[EMAIL PROTECTED]> > To: "'Chad Brandon'" <[EMAIL PROTECTED]>; "'Wouter Zoons'" > <[EMAIL PROTECTED]> > Cc: <[EMAIL PROTECTED]> > Sent: Saturday, February 28, 2004 4:48 PM > Subject: RE: [Andromda-devel] MMFs: The final battle continues! > > > Hi Chad, > > I see the new objects as facades for metaobjects. The > decorator aspect is not as important as that - we decorate > only because we want to avoid implementation inheritance. So, > if you want the word "Facade" in the interface ... hmmm ... > maybe, we could call the other two "FacadeRealizer" and > "FacadeRealizerImpl"??? > > Or: FacadeLogic and FacadeLogicImpl? > > -- Ignore my check in...I'll revert back if we decide > differently :) I do think its ugly...but again, these should > be ignored by the user/developers (since as you say they are > only there to avoid implementation inheritance)....but they > do use the decorator pattern...correct?...so therefore they > are decorators...thats the reason I wanted to keep > "Decorator" in there. Don't you think thats more > understandable from a > developer's point of view? Although I do like FacadeLogic > and FacadeLogicImpl too....how about I change to those names? > > In any case, I would not like to see "decorator" again. And: > "FacadeDecorator" is ugly because we do not decorate facades, do we? > > Cheers... > Matthias > > > -----Original Message----- > > From: [EMAIL PROTECTED] > > [mailto:[EMAIL PROTECTED] On > Behalf Of Chad > > Brandon > > Sent: Saturday, February 28, 2004 10:05 PM > > To: Matthias Bohlen; 'Wouter Zoons' > > Cc: [EMAIL PROTECTED] > > Subject: Re: [Andromda-devel] MMFs: The final battle continues! > > > > > > > > ----- Original Message ----- > > From: "Matthias Bohlen" <[EMAIL PROTECTED]> > > To: "'Chad Brandon'" <[EMAIL PROTECTED]>; "'Wouter Zoons'" > > <[EMAIL PROTECTED]> > > Cc: <[EMAIL PROTECTED]> > > Sent: Saturday, February 28, 2004 3:47 PM > > Subject: RE: [Andromda-devel] MMFs: The final battle continues! > > > > > > Hi all, > > > > good idea! The class in the facade model should be called > "cleanly", > > i.e. "Classifier". The interface should also be called > "Classifier", > > the facade could be called "ClassifierFacade" and the implementation > > "ClassifierFacadeImpl". I completely agree with that. > > > > ----- Matthias, I was actually thinking this: > ClassifierFacade (in our > > models and interfaces) ClassifierFacadeDecorator (for the generated > > decorator of the facade and ClassifierFacadeDecoractorImpl) for the > > the Impl classes (I didn't want to use the metamodel name > 'Classifier' > > since it may be a pain to keep track of package names within > > the code if we have the same exact name of the meta model > > element)..which way do you think is better? > > > > Wouter: yes, we can leave out the "I" if we name the > classes like Chad > > suggested! > > > > Wouter: the three outlets are for interfaces ("facade-interfaces"), > > generated implementations ("facades") and hand-written > implementations > > ("facade-impls"). > > > > It's good to have three pairs of eyes... :-) > > Matthias > > > > > -----Original Message----- > > > From: [EMAIL PROTECTED] > > > [mailto:[EMAIL PROTECTED] On > > Behalf Of Chad > > > Brandon > > > Sent: Saturday, February 28, 2004 9:06 PM > > > To: Wouter Zoons; 'Matthias Bohlen' > > > Cc: [EMAIL PROTECTED] > > > Subject: Re: [Andromda-devel] MMFs: The final battle continues! > > > > > > > > > Matthias, > > > > > > Could we give the interfaces the "clean" name (i.e. the > > actual facade > > > name), since that is the one people are going to be working > > with. And > > > name the generated facade > > > decorators: <some facade name>Decoractor and then the impl of the > > > facade decorator <some facade name>DecoratorImpl? I think > > that would > > > be easier to understand (since the names are more > > descriptive of what > > > they > > > are) and really the generated decorator shouldn't be even > seen..its > > > the interfaces that should be documented/used. What do you say? > > > > > > Chad > > > > > > ----- Original Message ----- > > > From: "Wouter Zoons" <[EMAIL PROTECTED]> > > > To: "'Matthias Bohlen'" <[EMAIL PROTECTED]>; "'Chad Brandon'" > > > <[EMAIL PROTECTED]> > > > Cc: <[EMAIL PROTECTED]> > > > Sent: Saturday, February 28, 2004 1:48 PM > > > Subject: RE: [Andromda-devel] MMFs: The final battle continues! > > > > > > > > > > It seems to work this time (locally I took all cartridges > > > except meta > > > > and bpm4struts from the build process and still need to fix the > > > > bpm4struts code) > > > > > > > > Some comment though: > > > > > > > > 1. do we really want to prefix our interfaces with 'I', Visual > > > > Age/Eclipse style ? I think we should stick to the widely > > accepted > > > > standard of placing a suffix on the implementation classes > > > with 'Impl' > > > > > > > > You expose an object through an interface, why change the > > > name of that > > > > object when it is passed as an interface ? Shouldn't the > > interface > > > > have the cleanest name of them all ? > > > > > > > > > > > 2. I need to put the 'facades', 'facade-impls' and > > > 'facade-interfaces' > > > > outlets in the build.xml where I use the andromda task... I was > > > > wondering what the first one is needed for > > > > > > > > thanks > > > > > > > > Wouter > > > > > > > > > > > > > > > > > > > > > > > > > > > ------------------------------------------------------- > > > SF.Net is sponsored by: Speed Start Your Linux Apps Now. > Build and > > > deploy apps & Web services for Linux with a free DVD > > software kit from > > > IBM. Click Now! > > http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click > > _______________________________________________ > > Andromda-devel mailing list [EMAIL PROTECTED] > > https://lists.sourceforge.net/lists/listinfo/andromda-devel > > > > > > > > > > ------------------------------------------------------- > > SF.Net is sponsored by: Speed Start Your Linux Apps Now. Build and > > deploy apps & Web services for Linux with a free DVD > software kit from > > IBM. Click Now! > http://ads.osdn.com/?ad_id56&alloc_id438&op=ick > _______________________________________________ > Andromda-devel mailing list [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/andromda-devel > > > > > ------------------------------------------------------- > SF.Net is sponsored by: Speed Start Your Linux Apps Now. > Build and deploy apps & Web services for Linux with > a free DVD software kit from IBM. Click Now! http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click _______________________________________________ Andromda-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/andromda-devel ------------------------------------------------------- SF.Net is sponsored by: Speed Start Your Linux Apps Now. Build and deploy apps & Web services for Linux with a free DVD software kit from IBM. Click Now! http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click _______________________________________________ Andromda-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/andromda-devel ------------------------------------------------------- SF.Net is sponsored by: Speed Start Your Linux Apps Now. Build and deploy apps & Web services for Linux with a free DVD software kit from IBM. Click Now! http://ads.osdn.com/?ad_id56&alloc_id438&op=click _______________________________________________ Andromda-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/andromda-devel
