On 2010-02-23, at 11:55 AM, Jon Nolan wrote:

> Is this sane and safe?
> 
>   public Properties properties() {
>       Properties properties = super.properties();
>       if (properties == null) {
>           properties = Properties.createProperties(editingContext());
>           setPropertiesRelationship(properties);
>       }
>       return properties;
>   }
> 
> Seems to work fine and suits my needs perfectly but it makes me uneasy.
> 
> Thanks,
> Jon

Seriously smacks of EOF commandment violations to me:

<http://wiki.objectstyle.org/confluence/display/WO/EOF-Using+EOF-The+EOF+Commandments>

See: #6, #8.

I'd use awakeFromInsertion or create a cover accessor (lazyProperties(), 
setLazyProperties()), and in a purely naming convention vein, try to avoid 
having Entities with plural names, it makes it too easy to confuse them with 
relationships.

;david

--
David LeBer
Codeferous Software
'co-def-er-ous' adj. Literally 'code-bearing'
site:   http://codeferous.com
blog:   http://davidleber.net
profile:        http://www.linkedin.com/in/davidleber
twitter:        http://twitter.com/rebeld
--
Toronto Area Cocoa / WebObjects developers group:
http://tacow.org




 _______________________________________________
Do not post admin requests to the list. They will be ignored.
Webobjects-dev mailing list      ([email protected])
Help/Unsubscribe/Update your Subscription:
http://lists.apple.com/mailman/options/webobjects-dev/archive%40mail-archive.com

This email sent to [email protected]

Reply via email to