The hang_man from purplesingularity

Issues

Part 2

`is_word_guessed`

Lines 64 to 67 in 3acdf65

    
           if word_copy == let_copy: 
        
           	return True 
        
           else: 
        
           	return False

Це еквівалентно return word_copy == let_copy.

`get_available_letters`

hang_man/hangman.py

Lines 95 to 99 in 3acdf65

    
           guess = set(letters_guessed) 
        
           let = set(string.ascii_lowercase) 
        
           dif = list(let.difference(guess)) 
        
           dif.sort() 
        
           left = ''.join(dif)

Це явно можна написати простіше, наприклад: dif = l for l in string.ascii_lowercase if l not in letters_guesed. Так у вас принаймні тільки один раз будуть копіюватися значення.

Part 3

hang_man/hangman.py

Lines 143 to 144 in 3acdf65

    
           if inp in letters_guessed or inp not in string.ascii_lowercase: 
        
               letters_tried.append(inp)

Ви перевіряєте letter_guessed, а додаєте у letters_tried. Через це ваша програма працює некоректно.

hang_man/hangman.py

Line 132 in 3acdf65

vow = {'a', 'i', 'e', 'y', 'o', 'u'}

Це порушує умову 3.D.6: y не рахується за голосну при розрахунку втрачених спроб.

hang_man/hangman.py

Line 143 in 3acdf65

if inp in letters_guessed or inp not in string.ascii_lowercase:

Спробуйте виводити текст у залежності від помилки, можливо ці дві умови варто розділити.

Part 4

`match_with_gaps`

hang_man/hangman.py

Lines 197 to 200 in 3acdf65

    
           if my_word[i] != other_word[i] and my_word[i] != "_": 
        
               return False 
        
           elif my_word[i] == '_' and other_word[i] != '_' and other_word[i] in set_let: 
        
           	return False

Ці умови можна спростити об'єднавши. Крім того, other_word[i] != '_' завжди True.

`show_possible_matches`

Іронічно, але ця функція не відповідає умові з 4.3В: вона не має нічого повертати, вона має відразу виводити. Її сигнатура має бути str -> None, а не str -> str.

hang_man/hangman.py

Lines 215 to 218 in 3acdf65

    
           result = "Posible words: " 
        
           for i in wordlist: 
        
               if match_with_gaps(my_word,i): 
        
                   result += i + " "

Так краще не робити: рядки є іммутабельними, тому на кожному присвоєнні створюється новий рядок, що тягне за собою виділення пам'яті і т.д. Простіше додавати усі слова до списку, а потім викликати ' '.join(result).

purplesingularity / hang_man Goto Github PK

hang_man's People

Contributors

hang_man's Issues

Issues

Part 2

`is_word_guessed`

`get_available_letters`

Part 3

Part 4

`match_with_gaps`

`show_possible_matches`

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	guess = set(letters_guessed)
	let = set(string.ascii_lowercase)
	dif = list(let.difference(guess))
	dif.sort()
	left = ''.join(dif)

	if inp in letters_guessed or inp not in string.ascii_lowercase:
	letters_tried.append(inp)

	if my_word[i] != other_word[i] and my_word[i] != "_":
	return False
	elif my_word[i] == '_' and other_word[i] != '_' and other_word[i] in set_let:
	return False

	result = "Posible words: "
	for i in wordlist:
	if match_with_gaps(my_word,i):
	result += i + " "