Develop

[2014-12-09 13:11:00 EEvL] Pull Request current develop branch into master.
Personally I like to have clean readable code that the next engineer can pick up and run with. If you have variables sporadically defined throughout your methods it makes it hard for the next engineer to know what variables and/or methods are already defined to be able to reuse. But, you are absolutely correct as I have defined the static dispatch_once_t inside the same method your suggesting.

> On Dec 10, 2014, at 5:33 AM, arctouch-caiofonseca <notifications@github.com> wrote:
> 
> In ArcTouchCodeChallenge/ArcTouchCodeChallenge/Sources/Managers/WebServiceManager.m <https://github.com/hockeygod/ArcTouchCodeChallenge/pull/1#discussion-diff-21602852>:
> 
> > +
> > +#import "WebServiceManager.h"
> > +#import "../Entities/Departure.h"
> > +#import "../Entities/Route.h"
> > +#import "../Entities/Stop.h"
> > +
> > +@interface WebServiceManager ()
> > +#pragma mark - WebServiceManager Private Instance Methods -
> > +- (id)queryForRequestURL:(NSURL *)requestURL requestHTTPBody:(NSData *)requestHTTPBody error:(NSError **)error;
> > +
> > +@end
> > +
> > +@implementation WebServiceManager
> > +
> > +#pragma mark - WebServiceManager Class Variable -
> > +static  WebServiceManager   *_webServiceManagerSingleton = nil;
> this static variable declaration can be moved into the method itself.
> 
> —
> Reply to this email directly or view it on GitHub <https://github.com/hockeygod/ArcTouchCodeChallenge/pull/1/files#r21602852>.
>