+9001 way cleaner to read :) On Oct 30, 2012, at 10:49 AM, Matthias Wessendorf wrote: > Hi, > > because builder pattern on iOS is a bit hard to read (as said before, > due to the deeply nested calls), I decided against it: > > id<AGPipe> myPipe = [[[[builder name:@"foo"] baseURL:someURL] > type:@"REST"] build]; > > It could be done with some reflection usage, but that's an issue w/ ARC: > > id<AGPipe> myPipe = [[builder withName:@"foo" withBaseURL:url.....] build]; > > > > == Config object == > > I propose using a config object, passed to a code-block. More details, > see this gist: > > https://gist.github.com/3979665 > > > -M > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev -- qmx _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

+1 for me too for the config object, looks better! On 30 Οκτ 2012, at 2:49 μ.μ., Matthias Wessendorf wrote: > Hi, > > because builder pattern on iOS is a bit hard to read (as said before, > due to the deeply nested calls), I decided against it: > > id<AGPipe> myPipe = [[[[builder name:@"foo"] baseURL:someURL] > type:@"REST"] build]; > > It could be done with some reflection usage, but that's an issue w/ ARC: > > id<AGPipe> myPipe = [[builder withName:@"foo" withBaseURL:url.....] build]; > > > > == Config object == > > I propose using a config object, passed to a code-block. More details, > see this gist: > > https://gist.github.com/3979665 > > > -M > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf > _______________________________________________ > aerogear-dev mailing list > aerogear-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/aerogear-dev _______________________________________________ aerogear-dev mailing list aerogear-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/aerogear-dev

https://github.com/aerogear/aerogear-ios/pull/18 -M On Tue, Oct 30, 2012 at 5:36 PM, Matthias Wessendorf <matzew(a)apache.org&gt; wrote: > Ok, started the work.. > > -M > > On Tue, Oct 30, 2012 at 2:47 PM, Christos Vasilakis <cvasilak(a)gmail.com&gt; wrote: >> +1 for me too for the config object, looks better! >> >> On 30 Οκτ 2012, at 2:49 μ.μ., Matthias Wessendorf wrote: >> >>> Hi, >>> >>> because builder pattern on iOS is a bit hard to read (as said before, >>> due to the deeply nested calls), I decided against it: >>> >>> id<AGPipe> myPipe = [[[[builder name:@"foo"] baseURL:someURL] >>> type:@"REST"] build]; >>> >>> It could be done with some reflection usage, but that's an issue w/ ARC: >>> >>> id<AGPipe> myPipe = [[builder withName:@"foo" withBaseURL:url.....] build]; >>> >>> >>> >>> == Config object == >>> >>> I propose using a config object, passed to a code-block. More details, >>> see this gist: >>> >>> https://gist.github.com/3979665 >>> >>> >>> -M >>> >>> -- >>> Matthias Wessendorf >>> >>> blog: http://matthiaswessendorf.wordpress.com/ >>> sessions: http://www.slideshare.net/mwessendorf >>> twitter: http://twitter.com/mwessendorf >>> _______________________________________________ >>> aerogear-dev mailing list >>> aerogear-dev(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/aerogear-dev >> >> >> _______________________________________________ >> aerogear-dev mailing list >> aerogear-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/aerogear-dev > > > > -- > Matthias Wessendorf > > blog: http://matthiaswessendorf.wordpress.com/ > sessions: http://www.slideshare.net/mwessendorf > twitter: http://twitter.com/mwessendorf -- Matthias Wessendorf blog: http://matthiaswessendorf.wordpress.com/ sessions: http://www.slideshare.net/mwessendorf twitter: http://twitter.com/mwessendorf