Vincent Massol wrote:

> Actually, there is another example in the pdf book :
> 
> if (initialized || disposed) {
>   throw new IllegalStateException ("Illegal call");
> }
> if (null == this.manager) {
>   this.manager = cmanager;
> }
> 
> Is it the recommended way or do you usually assume that the lifecycle is
> enforced ? I have a feeling that you'll answer the following : it depends.
> If you're using excalibur component manager then you don't need these checks
> but if you're not using any component manager, you'd rather check. maybe not
> ... let's see ... :)
> 
> In any case, I'd like to have your opinion.


Bottom line is this:

You should ALWAYS explicitly enforce your contracts.  This leads to secure
components that cannot be interacted with in ways you had no intention of
allowing.


> 
> -Vincent
> 
> ----- Original Message -----
> From: "Vincent Massol" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Tuesday, November 13, 2001 6:14 PM
> Subject: compose() method and developing with avalon pdf
> 
> 
> 
>>Hi,
>>
>>In the developing with avalon pdf, there is an example of how to implement
>>the compose() method :
>>
>>public void compose(ComponentManager manager) throws ComponentException {
>>  if (this.manager == null) {
>>    this.manager = manager;
>>    myGuard = (Guardian) this.manager.lookup(Guardian.ROLE);
>>  }
>>}
>>
>>I'm wondering why there is a test to verify if the manager is not null
>>
> (i.e.
> 
>>to verify if the compose() method has not already been called) ? I thought
>>that the lifecycle for composable would prevent this method being called
>>twice. Any reason ? If not, shouldn't we amend the doc, as most persons
>>
> will
> 
>>use this doc as their starting point (as I am ... :) ).
>>
>>Thanks
>>-Vincent
>>
>>
>>--
>>To unsubscribe, e-mail:
>>
> <mailto:[EMAIL PROTECTED]>
> 
>>For additional commands, e-mail:
>>
> <mailto:[EMAIL PROTECTED]>
> 
>>
> 
> 
> --
> To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
> 
> .
> 
> 



-- 

"Those who would trade liberty for
  temporary security deserve neither"
                 - Benjamin Franklin


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to