Comments on: A binary clock in Ruby using Shoes http://blog.coryfoy.com/2008/09/a-binary-clock-in-ruby-using-shoes/ Agile Coaching, Ruby, .NET, Debugging, why not? Fri, 03 Sep 2010 01:46:10 +0000 http://wordpress.org/?v=2.9.1 hourly 1 By: ashbb http://blog.coryfoy.com/2008/09/a-binary-clock-in-ruby-using-shoes/comment-page-1/#comment-41 ashbb Wed, 24 Sep 2008 05:11:00 +0000 http://blog.cornetdesign.com/?p=44#comment-41 Fantastic! I love simple code. :) Fantastic! I love simple code. :)

]]> By: Cory Foy http://blog.coryfoy.com/2008/09/a-binary-clock-in-ruby-using-shoes/comment-page-1/#comment-42 Cory Foy Tue, 23 Sep 2008 19:58:00 +0000 http://blog.cornetdesign.com/?p=44#comment-42 Great idea! That's what I get for doing it late at night and without a pair - I missed both the simplest thing, and an interesting pattern. Great idea! That’s what I get for doing it late at night and without a pair – I missed both the simplest thing, and an interesting pattern.

]]> By: Anonymous http://blog.coryfoy.com/2008/09/a-binary-clock-in-ruby-using-shoes/comment-page-1/#comment-43 Anonymous Tue, 23 Sep 2008 14:18:00 +0000 http://blog.cornetdesign.com/?p=44#comment-43 Cool app. Shoes is indeed cool, and as it matures it's definitely something that many people may use.<br /><br />By the way, for the get_image_pos_to_turn_on method, why don't you use a case block?<br /><br />Or even better—since num is just an integer string, just use a single array:<br /><br />IMAGE_POSITIONS = [<br /> [],<br /> [0],<br /> [1],<br /> [0, 1],<br /> [2],<br /> [0, 2],<br /> [1, 2],<br /> [0, 1, 2],<br /> [3],<br /> [0, 3]<br />]<br />end<br /><br />def get_image_pos_to_turn_on num<br /> IMAGE_POSITIONS[num.to_i] or []<br />end<br /><br />...Actually, since the array follows a pattern, you probably could just generate it, if it's worth it. Cool app. Shoes is indeed cool, and as it matures it’s definitely something that many people may use.

By the way, for the get_image_pos_to_turn_on method, why don’t you use a case block?

Or even better—since num is just an integer string, just use a single array:

IMAGE_POSITIONS = [
[],
[0],
[1],
[0, 1],
[2],
[0, 2],
[1, 2],
[0, 1, 2],
[3],
[0, 3]
]
end

def get_image_pos_to_turn_on num
IMAGE_POSITIONS[num.to_i] or []
end

…Actually, since the array follows a pattern, you probably could just generate it, if it’s worth it.

]]>