Disqus - Latest Comments for lukeredpath

Re: 依赖注入，iOS和你

Luke Redpath — Tue, 04 Nov 2014 08:05:13 -0000

Good post, but I think it would be good to demonstrate how you could go one step further with this.

Despite the convenience initialiser, you have now made the dependencies of this class explicit and as somebody says below, you've now made it necessary for anything else that creates this to require knowledge of how to wire it up.

You could of course use a factory but one thing that remains unchanged is that this particular implementation is explicitly coupled to NSURLSession and NSNotificationCenter. Explicit is better than implicit of course but there's still a coupling.

To me, the next step would be to decouple instantiation of this class from the places it's used by creating it in one place and passing it to the classes that need it. Additionally, you could extract the fetch methods into a protocol and only depend on that. This makes it fairly trivial to write a new implementation of the protocol that, for instance, uses AFNetworking or NSURLConnection. You only need to swap out the implementation in one place. You could also create a fake implementation that fetches canned data without having to mess around stubbing NSURLSession or other low level framework classes (only mock types you own).

IMO this would arguably be a more valuable refactoring. Making dependencies explicit is often a good thing and I can see why you might want to configure this class with something other than the default session but I'm not convinced it does anything to aid unit testing (I'm not a fan of mocking framework classes and I think a class like this is better treated as an "adapter" in hexagonal architecture speak and tested using integration tests, possibly with something that records and replays network requests at a low level like VCR for performance reasons.

Re: Building automatic login tokens for emails with Rails and Devise

Luke Redpath — Thu, 14 Aug 2014 07:50:52 -0000

Be careful with #find_by_token - it can be vulnerable to a timing attack. This is why it was deprecated in Devise.

The way I approach this is to create a base64 encoded token that contains a piece of static user-identifiable information - you could use their email or their user ID - and the user's auth token.

The server can decode this, extract the user ID and use it to find the User and then compare the auth token using a secure comparison method (Devise has a secure_compare function that will do this for you).

Re: Hashes and encapsulation: Jon Leighton

Luke Redpath — Thu, 19 Jan 2012 12:53:10 -0000

What's wrong with defining #[] on DOMElement, so attributes can be fetched thus: el["some-attribute"]. Seems reasonable to me. I'm pretty sure this is what Alex meant.

Re: Automating iOS Over The Air Beta Deployment

Luke Redpath — Fri, 21 Jan 2011 13:30:11 -0000

Hi Hector

You might want to check out my betabuilder gem which takes care of a lot of this (it's a Rake task library) and supports uploading to TestFlight too.

http://github.com/lukeredpa...

It's available as a Ruby gem.

Re: http://blog.twoguys.us/post/1487937592

Luke Redpath — Fri, 05 Nov 2010 10:51:40 -0000

Trying it out now, seems to work OK - does it report symbolicated crash backtraces from app store releases?

Re: Is Pusher up?

Luke Redpath — Thu, 24 Jun 2010 07:25:34 -0000

I guess it could be useful to add API support for this, so libraries can check Pusher's status when they have problems, it could be something as simple as http://status.pusherapp.com/api returning 0 or 1 in plain text to save having to parse the HTML. Full JSON in the future if you add more useful information (or both if you don't care why its down).

Re: Bringing device sync to Squeemote | the blog of luke redpath

Luke Redpath — Tue, 17 Feb 2009 22:02:46 -0000

You'll find an updated version of the interface here:
http://skitch.com/lukeredpa...

Re: Productivity Tip: Read Your Blogs in the Afternoon — A Single Programmer's Blog

Luke Redpath — Mon, 16 Feb 2009 09:33:49 -0000

I have to say I'm generally the complete opposite; for the first few hours of a day I'm generally pretty useless, with only the occasional exception.

Re: Remote pair-programming using iChat | the blog of luke redpath

Luke Redpath — Sat, 14 Feb 2009 13:16:21 -0000

Hi Tom, I have seen SubEthaEdit but I'm a bit too attached to TextMate to give it up. Now if TextMate integrated remote collaboration, that would be awesome.

Re: Remote pair-programming using iChat | the blog of luke redpath

Luke Redpath — Wed, 04 Feb 2009 10:36:14 -0000

Naturally, its easy to lose some of the subtleties of face to face communication when using iChat (unless you're using video conferncing of course) but I the better you get on with the person you are working with, the less of a problem this becomes (e.g. you should be reasonably attuned to when a person is joking/being ironic/sarcastic etc.).

I haven't been doing it remotely long enough to say for sure how more or less efficient it is than pairing in the same room but so far I'm pretty pleased with our output and overall velocity.

Re: screen and vim, I'm sure these can be equally good tools for remote pairing if you are comfortable with them. I could probably spend time learning how to use screen and vim fluently but I'd much rather stick with tools I'm already familiar with.

Re: Squeemote, the Squeezebox iPhone remote, ready for sale | the blog of luke redpath

Luke Redpath — Tue, 03 Feb 2009 14:14:52 -0000

I assume you mean putting it into standby using the top button; this is a known bug and will be fixed in the next release.

Re: Squeemote, the Squeezebox iPhone remote, ready for sale | the blog of luke redpath

Luke Redpath — Tue, 03 Feb 2009 13:45:28 -0000

Its not something I've tested personally but I can't see any reason why it wouldn't work. The server detection will not work of course, so you'll need to enter the IP/port numbers manually and ensure these ports are open on your router. I can't vouch for the speed of it though; precaching the albums list if you have a large database might take a bit longer over the net.

Re: Squeemote, the Squeezebox iPhone remote, ready for sale | the blog of luke redpath

Luke Redpath — Fri, 23 Jan 2009 04:30:29 -0000

Random Mix is definitely in the works. Possibly 1.2, otherwise 1.3.