From: Jacob Fugal <lukfugl@...>
Date: 2005-04-14T06:01:21+09:00
Subject: Re: Fixnum playing cards

On 4/13/05, Douglas Livingstone <rampant@gmail.com> wrote:
> [1] Code listing:
> 
> module PlayingCards
> 
>         def black?
>                 self > 26
>         end
> 
>         # ...snip...
>         def face
>                 # ...snip...
>                 return case self
>                         when 53; :Red
>                         when 54; :Black
>                 end unless self > 54
>         end
> 
>         # ...snip...
> end

It looks like 53 (Red Joker) would be classified as black (53.black?
== true) since 53 > 26. Maybe change to:

  def black?
     self > 26 and self != 53
  end

Also, if we're defining black? why not define red? as well:

  def red?
    not black?
  end

Jacob Fugal