Skip to content

Conversation

@adamxlowe
Copy link

No description provided.

@jwo
Copy link
Member

jwo commented Dec 2, 2013

Looks great.

I do think this exercise shows one note:

train = Train.new("Los Angeles", 1, 15, 1)

Looking at this method, its extremely hard to tell what is what.

We can change this to this for readability:

train = Train.new("Los Angeles", engines=1, cars=15, caboose=1)

thoughts?

@adamxlowe
Copy link
Author

It does make reading it a lot more clear. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants