Skip to content

Bo's JS Scrabble - #15

Open
botrethewey wants to merge 7 commits into
Ada-C7:masterfrom
botrethewey:master
Open

Bo's JS Scrabble#15
botrethewey wants to merge 7 commits into
Ada-C7:masterfrom
botrethewey:master

Conversation

@botrethewey

Copy link
Copy Markdown

JS Scrabble

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? The basic conditional logics were pretty much the same.
What was a challenge you were able to overcome on this assignment? This assignment helped me get a better understanding of object constructor properties and prototypes
What is your favorite thing about learning a new programming language? Gaining new knowledge and being able to understand the various similarities and differences between languages
What is your least favorite thing about learning a new programming language? Most of the roadblocks came from working with an unfamiliar syntax, but it was a good opportunity to get familiar with JS
Do you have any recommendations on how we could improve this project for the next cohort? I think this assignment would have been a good opportunity for us to use some basic tests in JS

@kariabancroft

Copy link
Copy Markdown

JS Scrabble

What We're Looking For

Feature Feedback
Core Requirements
Git hygiene Yes
Comprehension questions Yes
General
score calculates score, has appropriate params and return value I'm curious why you would want to allow the score function to be used without a word passed in?
highestScoreFrom calculates highest scoring word, has appropriate params and return value Yes - nice job with the scoping of this within your forEach loop. If you are going to need the indexes in the loop, you should use the for loop not the forEach loop.
Player object
Has name and plays properties Good
Has play, totalScore, hasWon functions Good
Has highestScoringWord and highestWordScore functions Good
Overall Nice job hitting the major learning goals of this assignment

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