Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Panda.Tiger.Eagle(complete) #11

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

var114
Copy link

@var114 var114 commented Dec 27, 2013

No description provided.

@answer_select = []
@answers = ["She can code in Ruby", "She has a dog named Lulu", "She never quite learned how to ride a bike",
"She's seen all Die Hard Movies", "She only uses chopsticks to eat her food"]
@answers.each do |x|
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it looks like you are making a copy of answers, in order to sample it later.

You likely don't need to --- since "sample" is a safe method, it won't change the array you're calling it on.

@answers = ["She can code in Ruby", "She has a dog named Lulu", "She never quite learned how to ride a bike",
                    "She's seen all Die Hard Movies", "She only uses chopsticks to eat her food"]
@reply = @answers.sample

@jwo
Copy link
Member

jwo commented Dec 30, 2013

Quite excellent! Is this online anywhere? I'd like to check it out.

@var114
Copy link
Author

var114 commented Jan 2, 2014

Hey Jesse,

Happy New Year.
Oh No. I’m having difficulty getting it to work…Heroku is uploading it but giving me an error message when I go to the page. I’ll fix it that soon. Also did you get my Episode9 assignment?

Best,
Patil
On Dec 30, 2013, at 8:20 AM, Jesse Wolgamott [email protected] wrote:

Quite excellent! Is this online anywhere? I'd like to check it out.


Reply to this email directly or view it on GitHub.

@jwo
Copy link
Member

jwo commented Jan 2, 2014

@var114 -- I'll take a look now

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