Leaves - Cloudy

[OhCloud]

JS Adagrams

Congratulations! You're submitting your assignment!

Comprehension Questions

Question	Answer
What patterns were you able to use from your Ruby knowledge to apply to JavaScript?	most things translated outside of the foreach situation which i did use and then have to change to a forloop bc you cant break out of it.
Did you need to use different strategies to find information online about JavaScript than you would for Ruby?	yes bc there is ALOT of information though not all of it is good or still relevant. it was about using things and seeing if they worked, for example _.samplesize(lodash) did not work for me. begrudgingly.
What was a challenge you were able to overcome on this assignment?	the idea that i cant javascript. it was a slow ramp up.
What has been interesting and positive about learning a new programming language?	that its not as hard to understand and implement as previously thought. more than anything though, those gosh dang { {{ [] }} } ughh.
What is something to focus on your learnings in JavaScript in the next week?	how to know think in terms in functions. translating ruby isnt the approach.

[beccaelenzil]

JS Adagrams

What We're Looking For

Feature	Feedback
General
Answered comprehension questions	yes
Small commits with meaningful commit messages	yes
Code Requirements
drawLetters method	yes -- see comment about small bug
Uses appropriate data structure to store the letter distribution	yes -- see comment
All tests for drawLetters pass	yes
usesAvailableLetters method
All tests for usesAvailableLetters pass	yes
scoreWord method
Uses appropriate data structure to store the letter scores	yes
All tests for scoreWord pass	yes
Optional
highestScoreFrom method	N/A
Appropriately handles edge cases for tie-breaking logic	N/A
All tests for highestScoreFrom pass	N/A
Overall	Good work overall. In this project you’ve taken some interesting logic and worked it into JavaScript syntax. I've left a few inline comment for you to review, but overall, you’ve done a great job of practicing syntax.

[beccaelenzil]

This should be a const.

Style note: put each letter on it's own line.

[beccaelenzil]

This should be a const.

Style note: put each letter on it's own line.

allLetters and letterScores are best placed inside the module, but above all the functions.

[beccaelenzil]

Walk through what this line of code is doing. Math.random() shouldn't take an argument.

myShuffledLetters should have a more semantically correct name.

There is a small bug where this could would allow you to pick 10 Z's (for instance), even though there aren't 10 Z's. What chance is necessary?

[beccaelenzil]

This else block isn't necessary.