I keep on getting error message "your function fails on scrabble_score(“xenophobia”). It returns “23” when it should return “24”.."

Question

0.00/5 (No votes)

See more:

This is my code:

score = {"a": 1, "c": 3, "b": 3, "e": 1, "d": 2, "g": 2,
         "f": 4, "i": 1, "h": 4, "k": 5, "j": 8, "m": 3,
         "l": 1, "o": 1, "n": 1, "q": 10, "p": 3, "s": 1,
         "r": 1, "u": 1, "t": 1, "w": 4, "v": 4, "y": 4,
         "x": 8, "z": 10}

def scrabble_score(word):
    word = (word.lower())
    final_score = 0
    for key in score:
        if key in word:
            final_score = final_score + score[key]
    return(int(final_score))

print(scrabble_score("xenophobia"))

What I have tried:

I have tried removing the if statement and just using the for loop, I've also tried removing final_score.

Posted 29-Oct-22 11:20am

Naveen Krishna1

Updated 29-Oct-22 12:16pm

Add a Solution

Comments

Richard Deeming 31-Oct-22 6:36am

NB: This appears to be an extremely over-simplified Scrabble score calculator. Aside from the fact that it ignores the bonus squares (double/triple word/letter), there is no way to know how many letters the user played, so no way of knowing whether to award them the bonus for playing all seven letters.

I can only hope this is part of a programming course, and the additional complexity will be introduced later. :)

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Accepted Answer · 2022-10-29T12:16:00

Instead of removing things, you should have been walking through the code line by line, executing each statement in your head and probably keeping track of variable content on paper. That way you get a better understanding of the code instead of guessing at what's wrong.

The problem is your algorithm. You're taking each letter in the dictionary and seeing if it in the word when you should have been taking each letter in the word and looking it up in the dictionary. That's the entire point of the dictionary!

Mike Hankey · Accepted Answer · 2022-10-29T11:33:00

Solution 1

I'm not real familiar with Python but it looks like your going through all the scores and seeing if the letter is in the word. Since there are 2 'o's it will only get recorded one time.

Better to go through the word and look up the letter in score and add!

Posted 29-Oct-22 11:33am

Mike Hankey

Comments

Naveen Krishna1 29-Oct-22 22:15pm

Thanks for the help!