Skip to content

Conversation

@LaneHesser
Copy link

ATTENTION: before clicking "Create Pull Request" please submit some meta data, thanks!

Difficulty level (1-10): [2]
Estimated time spent (hours): [2]
Completed (yes/no): [Yes]
I stretched my coding skills (if yes what did you learn?): []
Other feedback (what can we improve?): []

@bbelderbos
Copy link
Collaborator

Looks good to me, I like how you considered the shorter max and sum

@bbelderbos bbelderbos merged commit 464d542 into pybites:community Dec 12, 2020
for letter in word:
letter = letter.upper()
if letter in ascii_uppercase:
value += LETTER_SCORES[letter]
Copy link
Collaborator

Choose a reason for hiding this comment

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

This works here, if a letter would not be in the LETTER_SCORES dict, you could use LETTER_SCORES.get(letter, 0)

# ])


if __name__ == "__main__":
Copy link
Collaborator

Choose a reason for hiding this comment

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

No code, could remove

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