Update scrabble.py as per guidelines

Author: nashira26

Scrabble/scrabble.py

@@ -1,77 +1,126 @@
 from PyDictionary import PyDictionary
 
+
 def main():
-    print("\n" + '*'*10 + "Welcome to the Scrabble game" + '*'*10 + "\n" + "Let's start playing!!\n")
+    """
+    Main entry point of the Scrabble game.
+    """
+    print('*' * 10 + "Welcome to the Scrabble game" + '*' * 10)
+    print("Let's start playing!!\n")
     score_board = {}
     for i in range(player_count()):
         player = input(f"Player {i+1}: ")
         score_board[player] = 0
-    print('*'*40 + "\n")
-    print(winner(get_input(score_board)).center(40, " ") +"\n")
-    print('*'*40 + "\n"+"Thank you for your time. Have a Nice day!")
+    print('*' * 40)
+    print(winner(get_input(score_board)).center(40, " "))
+    print('*' * 40)
+    print("Thank you for your time. Have a Nice day!")
 
+    
 def valid(word):
-    if PyDictionary().meaning(word, True) == None :
-        return False
-    else:
-        return True
+"""
+    Checks if a word is valid by checking its meaning using PyDictionary.
+    Args:
+        word (str): The word to be validated.
+    Returns:
+        bool: True if the word is valid, False otherwise.
+    """
+    dictionary = PyDictionary()
+    return dictionary.meaning(word) is not None 
 
+    
 def compute_score(word):
-    score_list = {'a':1, 'b':3, 'c':3, 'd':2, 'e':1, 'f':4, 'g':2, 'h':4, 'i':1, 'j':8, 'k':5, 'l':1, 'm':3,
-                  'n':1, 'o':1, 'p':3, 'q':10, 'r':1, 's':1, 't':1, 'u':1, 'v':4, 'w':4, 'x':8, 'y':4, 'z':10}
+    """
+    Computes the score for a given word based on a score list.
+    Args:
+        word (str): The word for which the score needs to be computed.
+    Returns:
+        int: The computed score for the word.
+    Raises:
+        ValueError: If the word is invalid or contains non-alphabetic characters.
+    """
+    score_list = {
+                'a': 1, 'b': 3, 'c': 3, 'd': 2, 'e': 1, 'f': 4, 'g': 2, 
+                'h': 4, 'i': 1, 'j': 8, 'k': 5, 'l': 1, 'm': 3, 'n': 1, 
+                'o': 1, 'p': 3, 'q': 10, 'r': 1, 's': 1, 't': 1, 'u': 1, 
+                'v': 4, 'w': 4, 'x': 8, 'y': 4, 'z': 10}
     score = 0
     if word.isalpha():
         if valid(word):
-            for i in word:
-                score += score_list[i.lower()]
+            for char in word:
+                score += score_list[char.lower()]
             return score
         else:
-            raise NameError
+            raise ValueError("Invalid word")
     else:
-        raise ValueError
+        raise ValueError("Word should only contain alphabetic characters")
 
+    
 def player_count():
+    """
+    Prompts the user to input the number of players for the game.
+    Returns:
+        int: The number of players.
+    Raises:
+        ValueError: If the user inputs a non-positive number.
+    """
     while True:
         try:
             count = int(input("How many players? "))
-        except ValueError:
-            print("Please type a number in integer format.")
-            continue
-        else:
+            if count < 1:
+                raise ValueError("Please enter a positive number")
             return count
-            break
+        except ValueError as e:
+            print(str(e))
 
+                  
 def get_input(score_board):
+    """
+    Retrieves the word input from each player and updates their scores.
+    Args:
+        score_board (dict): The dictionary storing the scores of each player.
+    Returns:
+        dict: The updated score board.
+    """
     while True:
         for player in score_board:
             while True:
                 try:
                     word = input(f"{player} | Type a word: ")
                     score_board[player] += compute_score(word)
-                except BaseException:
-                    print("Please type a valid word.")
-                    continue
+                except ValueError as e:
+                    print(str(e))
                 else:
                     break
 
-        if input("If exit, type Y: ") == "Y":
-            print('*'*40 + "\n")
+        if input("If exit, type Y: ").lower() == "y":
+            print('*' * 40)
             break
-        else:
-            continue
+        continue
     return score_board
 
+        
 def winner(score_board):
-    sorted_scores = sorted(score_board.items(), key=lambda x: x[1], reverse= True)
-    max = sorted_scores[0][1]
-    winners = [sorted_scores[0][0]]
-    for i in sorted_scores[1:]:
-        if i[1] == max:
-            winners.append(i[0])
-    if len(winners) > 1:
-        return f"It's a tie. The winners are {', '.join(winners)}!!"
-    else:
+    """
+    Determines the winner(s) based on the highest scores.
+    Args:
+        score_board (dict): The dictionary storing the scores of each player.
+    Returns:
+        str: The winner(s) message.
+    Raises:
+        IndexError: If there are no players in the score board.
+    """
+    sorted_scores = sorted(score_board.items(), key=lambda x: x[1], reverse=True)
+    if len(sorted_scores) > 0:
+        max_score = sorted_scores[0][1]
+        winners = [player for player, score in sorted_scores if score == max_score]
+        if len(winners) > 1:
+            return f"It's a tie. The winners are {', '.join(winners)}!!"
         return f"The winner is {winners[0]}!"
+    else:
+        return "No players found. Game over!"
+
 
 if __name__ == "__main__":
-    main()
+    main()
+