Problem with example from pickax book

ruby-talk
1

Working through the Pickax Ruby programing book and cant get the
following example 'compile':-

irb(main):040:0* class SongList
irb(main):041:1> MAX_TIME = 5*60 # 5 minutes
irb(main):042:1>
irb(main):043:1* def Songlist.is_too_long(song)
irb(main):044:2> return song.duration > MAX_TIME
irb(main):045:2> end
irb(main):046:1> end
(irb):41: warning: already initialized constant MAX_TIME
NameError: uninitialized constant SongList::Songlist
        from (irb):43

Any idea why?

Ben

···

from :0
--
Ben Edwards - Bristol, UK
http://www.flickr.com/photos/41303704@N00/ - have a look at my pics
If you have a problem emailing me use
http://www.gurtlush.org.uk/profiles.php?uid=4
(email address this email is sent from may be defunct)

2

Dear Ben,

irb(main):040:0* class SongList
irb(main):041:1> MAX_TIME = 5*60 # 5 minutes
irb(main):042:1>
irb(main):043:1* def Songlist.is_too_long(song)

                         ^^^^^^^^^

Just leave out the underlined "Songlist."
Best regards,

Axel

···

--
Psssst! Schon vom neuen GMX MultiMessenger gehört?
Der kanns mit allen: http://www.gmx.net/de/go/multimessenger

3

Songlist needs to be SongList. Or just change it to "self". :slight_smile:

···

On Jul 13, 4:51 pm, "Ben Edwards" <funkyt...@gmail.com> wrote:

Working through the Pickax Ruby programing book and cant get the
following example 'compile':-

irb(main):040:0* class SongList
irb(main):041:1> MAX_TIME = 5*60 # 5 minutes
irb(main):042:1>
irb(main):043:1* def Songlist.is_too_long(song)
irb(main):044:2> return song.duration > MAX_TIME
irb(main):045:2> end
irb(main):046:1> end
(irb):41: warning: already initialized constant MAX_TIME
NameError: uninitialized constant SongList::Songlist

4

Dear Ben,

> irb(main):040:0* class SongList
> irb(main):041:1> MAX_TIME = 5*60 # 5 minutes
> irb(main):042:1>
> irb(main):043:1* def Songlist.is_too_long(song)
                         ^^^^^^^^^

Just leave out the underlined "Songlist."

Its actually supposed to be a class method, but ive spotted the error.
It should be SongList, not Songlist (the L needs to be upper case).

Ta,
Ben

···

On 13/07/07, Axel Etzold <AEtzold@gmx.de> wrote:

Best regards,

Axel
--
Psssst! Schon vom neuen GMX MultiMessenger gehört?
Der kanns mit allen: http://www.gmx.net/de/go/multimessenger

--
Ben Edwards - Bristol, UK
Ben Edwards | Flickr - have a look at my pics
If you have a problem emailing me use
http://www.gurtlush.org.uk/profiles.php?uid=4
(email address this email is sent from may be defunct)