Re: Lazy optional relationship creation on EO accessor
Re: Lazy optional relationship creation on EO accessor
- Subject: Re: Lazy optional relationship creation on EO accessor
- From: David LeBer <email@hidden>
- Date: Tue, 23 Feb 2010 12:06:13 -0500
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@hidden)
Help/Unsubscribe/Update your Subscription:
This email sent to email@hidden