Требуется критика кода

@lexcher · Регистрация: 12.04.2018

Студворк — интернет-сервис помощи студентам

Доброго времени суток, уважаемые форумачане. Хотел бы вас попросить покритиковать код. Версия python 3.6. Код во вложении.
В заранее благодарен.

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
import random
 
 
def search_position_letter(l, w):
    letter_position = w.index(l)
    return letter_position
 
def string_word_to_list_word(w):
    '''str(obj) => list(obj)'''
    list_word = list(w)
    return list_word
 
def list_word_to_string_word(w):
    '''list(obj) => str(obj)'''
    string_word = ''.join(w)
    return string_word
 
def same_letter(user_answer, word):
    '''Находит и возврощает список с индексами повторяющихся символов'''
    letter_list = []
    word = list(word)
    while user_answer in word:
        letter_list.append(word.index(user_answer))
        number = len(letter_list)
        letter_list[-1] += number - 1
        del word[word.index(user_answer)]
    return letter_list
 
def user_attempts_choice(user_attempts):
    if user_attempts == 1:
        attempts = 7
        return attempts
    elif user_attempts == 2:
        attempts = 5
        return attempts
    elif user_attempts == 3:
        attempts = 3
        return attempts
    else:
        return 'Неверный ввод.' 
 
 
words_list = ['книга', 'месяц', 'ручка', 'шарик', 'олень', 'носок']
 
while len(words_list) != 0:
    word = random.choice(words_list)
    crypto_word = '\u25A0' * len(word)
    user_attempts = int(input('Выберите уровень сложности: 1-легкий; 2-средний; 3-сложный: '))
    attempts = user_attempts_choice(user_attempts)
    print(f'В слове {len(word)} букв.')
    print(f'У вас есть {attempts} попыток, чтобы угадать слово.')
    while '\u25A0' in crypto_word:
        if attempts == 0:
            print('К сожалению у вас закончились попытки!')
            break
        user_answer = input('Введите букву или слово целиком: ')
        if user_answer == word:
            break
        if user_answer in word:
            crypto_word = string_word_to_list_word(crypto_word)
            if word.count(user_answer) > 1:
                letter_list = same_letter(user_answer, word)
                for i in letter_list:
                    crypto_word[i] = user_answer
            letter_position = search_position_letter(user_answer, word)
            crypto_word[letter_position] = user_answer
            crypto_word = list_word_to_string_word(crypto_word)
            print(crypto_word)
        else:
            print('Нет такой буквы!')
            attempts -= 1
            print(f'Осталось попыток: {attempts}')
    print(f'Поздравляем, это и правда слово "{word}"')
    del words_list[words_list.index(word)]
print('Поздравляем, вы угадали все доступные слова.')

@dondublon · 12.04.2018, 18:13

Сообщение от lexcher

del word[word.index(user_answer)]

Не знаю, какой у вас там размер списка, но лучше так не искать, использовать словарь или множество.

@lexcher · 12.04.2018, 18:23 **[ТС]**

Сообщение от dondublon

Не знаю, какой у вас там размер списка, но лучше так не искать, использовать словарь или множество.

В данной строке, я нахожу индекс слова в списке исходя из правильного ответа пользователя, затем удаляю его из списка, чтобы продолжить игру, и так до тех пор пока список не станет пустым. По заданию было указанно, что требуется создать именно список со словами. Пока что еще не дотумкал каким образом еще можно исключить из списка уже 'отыгранное' слово.

@woldemas · 12.04.2018, 18:29

lexcher, первые две функции лишние, на мой взгляд
вторые две можно упростить как нибудь так:

Python
1
2
3
4
5
6
def same_letter(user_answer, word):
    '''Находит и возврощает список с индексами повторяющихся символов'''
    return [idx for idx, ch in enumerate(word) if user_answer == ch]
 
def user_attempts_choice(user_attempts):
    return {1:7, 2:5, 3:3}.get(user_attempts, 'Неверный ввод')

Вот этот блок можно убрать и рассматривать много угаданных букв как частный случай одной:

Python
1
2
letter_position = search_position_letter(user_answer, word)
crypto_word[letter_position] = user_answer

Зачем каждый раз перегонять crypto_word из списка в строку, храните всегда в списке и только на экран выводите через str.join.

Да и вообще как-то не очень, условных операторов много - зачем, если есть словари и полиморфизм, щас так не пишут, вроде.
Хотя я питон недавно изучать начал, но это то что бросается в глаза.

@lexcher · 12.04.2018, 18:37 **[ТС]**

Сообщение от woldemas

Вот этот блок можно убрать и рассматривать много угаданных букв как частный случай одной:

letter_position = search_position_letter(user_answer, word)
crypto_word[letter_position] = user_answer

Не совсем понял.

@woldemas · 12.04.2018, 18:52

Сообщение от lexcher

Не совсем понял.

Ну вот я чуть подчистил, что мне в глаза бросается, но я бы вообще по-другому написал

Кликните здесь для просмотра всего текста

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
import random
   
def same_letter(user_answer, word):
    '''Находит и возврощает список с индексами повторяющихся символов'''
    return [idx for idx, ch in enumerate(word) if user_answer == ch]
 
def user_attempts_choice(user_attempts):
    return {1:7, 2:5, 3:3}.get(user_attempts, 'Неверный ввод')
 
words_list = ['книга', 'месяц', 'ручка', 'шарик', 'олень', 'носок']
 
while len(words_list) != 0:    
    word = random.choice(words_list)    
    crypto_word = ['\u25A0'] * len(word)     
    user_attempts = int(input('Выберите уровень сложности: 1-легкий; 2-средний; 3-сложный: '))
    attempts = user_attempts_choice(user_attempts)
    print(f'В слове {len(word)} букв.')
    print(f'У вас есть {attempts} попыток, чтобы угадать слово.')
    while '\u25A0' in crypto_word:
        if attempts == 0:
            print('К сожалению у вас закончились попытки!')
            break
        user_answer = input('Введите букву или слово целиком: ')
        if user_answer == word:
            break                         
        letter_list = same_letter(user_answer, word)
        if len(letter_list):            
            for i in letter_list:
                crypto_word[i] = user_answer            
            print(''.join(crypto_word))
        else:
            print('Нет такой буквы!')
            attempts -= 1
            print(f'Осталось попыток: {attempts}')
    print(f'Поздравляем, это и правда слово "{word}"')
    words_list.remove(word)
print('Поздравляем, вы угадали все доступные слова.')

@lexcher · 12.04.2018, 18:54 **[ТС]**

Сообщение от woldemas

Ну вот я чуть подчистил, что мне в глаза бросается, но я бы вообще по-другому написал

Благодарю, сейчас ознакомлюсь. Сразу оговорюсь с ООП еще не знаком, на данный момент писал код таким каким он мне казался очевидным))

@woldemas · 12.04.2018, 18:55

Сообщение от lexcher

Сразу оговорюсь с ООП еще не знаком

Там нет ООП, я ваш код мало поменял

@lexcher · 12.04.2018, 19:00 **[ТС]**

Сообщение от woldemas

Там нет ООП, я ваш код мало поменял

Да, да я понял. Последнюю тему которую изучал были Функции, и как я понимаю я создал лишние функции. Пытался уделять им больше внимания, в то время как они только нагромождали код.

@woldemas · 12.04.2018, 19:19

Сообщение от lexcher

как я понимаю я создал лишние функции

Что касается функций,например, в данном случае можно выделить в функцию один раунд игры, например таким образом, тогда код читабельнее будет (еще я там выход добавил):

Кликните здесь для просмотра всего текста

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
def begin_round(word, attempts):
    crypto_word = ['\u25A0'] * len(word)     
    while '\u25A0' in crypto_word: 
        if attempts == 0:
            print('К сожалению у вас закончились попытки!')
            return False            
        user_answer = input('Введите букву или слово целиком: ')
        if user_answer == word: 
            return True            
        letter_list = same_letter(user_answer, word)
        if len(letter_list):            
            for i in letter_list: crypto_word[i] = user_answer            
            print(''.join(crypto_word))
        else:
            print('Нет такой буквы!')
            attempts -= 1
            print(f'Осталось попыток: {attempts}')
    return True
 
words_list = ['книга', 'месяц', 'ручка', 'шарик', 'олень', 'носок']
while len(words_list) != 0:    
    word = random.choice(words_list)         
    attempts = int(input('Выберите уровень сложности: 1-легкий; 2-средний; 3-сложный: '))
    if begin_round(word, user_attempts_choice(attempts)):
        print(f'Поздравляем, это и правда слово "{word}"')
    else: 
        print(f'Вы не угадали слово "{word}"')
    words_list.remove(word)
    if input('Продолжить: д/н?').lower() != 'д': 
        break

@lexcher · 12.04.2018, 19:24 **[ТС]**

Сообщение от woldemas

def begin_round(word, attempts):
* * crypto_word = ['\u25A0'] * len(word) * *
* * while '\u25A0' in crypto_word:
* * * * if attempts == 0:
* * * * * * print('К сожалению у вас закончились попытки!')
* * * * * * return False * * * * * *
* * * * user_answer = input('Введите букву или слово целиком: ')
* * * * if user_answer == word:
* * * * * * return True * * * * * *
* * * * letter_list = same_letter(user_answer, word)
* * * * if len(letter_list): * * * * * *
* * * * * * for i in letter_list: crypto_word[i] = user_answer * * * * * *
* * * * * * print(''.join(crypto_word))
* * * * else:
* * * * * * print('Нет такой буквы!')
* * * * * * attempts -= 1
* * * * * * print(f'Осталось попыток: {attempts}')
* * return True

Это нормальная практика, чтобы функция что то печатала через print, а не только возвращала посредством return?

@woldemas · 12.04.2018, 19:29

Сообщение от lexcher

Это нормальная практика, чтобы функция что то печатала через print

не знаю, но для данной задачи - пойдет
Можно туда в функцию еще и вот это вывод засунуть, логичнее будет:

Python
1
2
3
        print(f'Поздравляем, это и правда слово "{word}"')
    else: 
        print(f'Вы не угадали слово "{word}"'

@dondublon · 13.04.2018, 11:52

Сообщение от lexcher

В данной строке, я нахожу индекс слова в списке исходя из правильного ответа пользователя, затем удаляю его из списка, чтобы продолжить игру,

Я говорю не о ваших целях, а о структуре данных, которую вы для этого используете. Вы используете список, надо словарь или множество.

@lexcher · 13.04.2018, 14:13 **[ТС]**

Сообщение от dondublon

Я говорю не о ваших целях, а о структуре данных, которую вы для этого используете. Вы используете список, надо словарь или множество.

Сообщение от lexcher

По заданию было указанно, что требуется создать именно список со словами.

Но все равно благодарю, за совет.

Новые блоги и статьи Все статьи Все блоги /
Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .
Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .	Модель микоризы: классовый агентный подход 3 anaschu 06.01.2026 aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114	Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ФедосеевПавел 06.01.2026 Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .	Модель микоризы: классовый агентный подход 2 anaschu 06.01.2026 репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .