Skip to content

Create ride_share.rb#40

Open
Naltrexone wants to merge 1 commit into
Ada-C10:masterfrom
Naltrexone:master
Open

Create ride_share.rb#40
Naltrexone wants to merge 1 commit into
Ada-C10:masterfrom
Naltrexone:master

Conversation

@Naltrexone

Copy link
Copy Markdown

ride share

Congratulations! You're submitting your assignment.

Comprehension Questions

Question Answer
What did your data structure look like at first? Did this structure evolve over time? Why? It looked more or less the same, since I was able to retrieve the data from it.
What was your strategy for going through the data structure and gathering information? Iteration through each of the data structures.
What was an example of something that was necessary to store in a variable? What was the scope of each of that variables? Why? While making calculations, it helped to store data in variables for easy readability. All the variables were local variables.
What kinds of iteration did you use? Did you use .map? No, I didn't. I used the '.each' iterators . Would have preferred using more enumerables in my code!
Were some calculations easier than others? Why? Yes, they were easier as data access was not multi-layered.

@dHelmgren

Copy link
Copy Markdown

I like how you structured the data. Take a glance back at this code and think about what you might be able to accomplish in a more succinct way if you were using other kinds of enumerables!

Comment thread ride_share.rb


# calculates number of rides per driver
def number_rides(whatever)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Be wary of giving variables strange names! Will you know what whatever does in three months?

@Naltrexone

Naltrexone commented Aug 17, 2018 via email

Copy link
Copy Markdown
Author

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