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

Pcc01 #878

Open
wants to merge 2 commits into
base: community
Choose a base branch
from
Open

Pcc01 #878

Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 38 additions & 0 deletions 01/loukaja/wordvalue.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
from data import DICTIONARY, LETTER_SCORES

def load_words():
"""Load dictionary into a list and return list"""
with open(DICTIONARY) as f:
return (f.read().split())

def calc_word_value(word):
"""Calculate the value of the word entered into function
using imported constant mapping LETTER_SCORES"""
value = 0
for char in word:
if char.isalpha():
value += LETTER_SCORES[char.upper()]

return value

def max_word_value(word_list=None):
"""Calculate the word with the max value, can receive a list
of words as arg, if none provided uses default DICTIONARY"""
if word_list is None:
word_list = load_words()

max_value = 0
max_word = ''

for word in word_list:
if len(word) * 10 < max_value:
continue # no point in calculating if the value can't reach above current max
value = calc_word_value(word)
if value > max_value:
max_word = word
max_value = value

return max_word

if __name__ == "__main__":
pass # run unittests to validate