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

Mindy's scrabble #19

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

Mindy's scrabble #19

wants to merge 3 commits into from

Conversation

mcarson1111
Copy link

So apparently, I forgot to commit the whole day at the coworking space, so that's awesome...
I've been working on getting wave 1 to work.
Sorry it's a few minutes late, I was working on the .shortest function to try to determine the winner in a tie...I'm getting confused about scope in JS and specifically with these last two functions, how and where to return things. Also, no tests yet. :/
Will work on it a bit more tonight then tomorrow.

Player.prototype.total_score = function() {
total = 0
for(var word of this.plays) {
total = total + Scrabble.score(word)
Copy link

Choose a reason for hiding this comment

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

Love how you separated out the score calculations from the total score function!

@CShekta
Copy link

CShekta commented Jun 21, 2016

This was an amazing first javascript project! Yay 👍

@mcarson1111
Copy link
Author

Yay! thank you so much ^_^

On Tue, Jun 21, 2016 at 3:51 PM, CShekta [email protected] wrote:

This was an amazing first javascript project! Yay 👍


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#19 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AMr71354cKETEvT5dbljj7XC-ThEkUZvks5qOGrzgaJpZM4Iz-5r
.

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