Форум программистов, компьютерный форум, киберфорум
Python: PyGame
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.69/13: Рейтинг темы: голосов - 13, средняя оценка - 4.69
0 / 0 / 0
Регистрация: 01.11.2020
Сообщений: 8

Оцените пожалуйста мой небольшой проект по Python. Что можно исправить и доработать?

11.03.2021, 23:53. Показов 2703. Ответов 1

Студворк — интернет-сервис помощи студентам
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
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
import pygame as pg
import random
import pickle
from tkinter import *
import sys
from PIL import Image, ImageTk
 
"""I used Tkinter just for practise, I know 
that with only pygame game will be better
but I wanted to combine these two modules.
PIL was used to add image on tkinter window"""
 
HEIGHT, WIDTH = 640, 900
clock = pg.time.Clock()
best_result = 0
time = 10  # time = 60/time sec
lvl = 1
 
 
def start_window():
    global lvl
    """Create main window, where you can choose difficult
    :return None"""
    root = Tk()
    root.geometry(f'900x640+{310}+{80}')
    root["bg"] = '#35b056'
    root.title("The Snake")
    root.iconbitmap("C:\Programming\Python\Games\Snake\project\icon.ico")
 
    # https://наклейкибум.рф/wp-content/uploads/13150_0-250x250.png
    img = ImageTk.PhotoImage(Image.open("C:\Programming\Python\Games\Snake\project\Snake3.png"))
    img_lab = Label(image=img)  # creating the snake on the main window, but I couldn't make it
    img_lab.place(x=150, y=150)                                              # without white bg
 
    r_var = IntVar()
    r_var.set(lvl)
 
    start_but = Button(text="Start", width=32, height=3, background='#c5ff2e', font=("Arial 32", 12))
    r1 = Radiobutton(text="Easy", variable=r_var, value=1, background='#35b056',
                     activebackground='#35b056', font=("Arial 32", 16))
    r2 = Radiobutton(text="Normal", variable=r_var, value=2, background='#35b056',
                     activebackground='#35b056', font=("Arial 32", 16))
    r3 = Radiobutton(text="Hard", variable=r_var, value=3, background='#35b056',
                     activebackground='#35b056', font=("Arial 32", 16))
 
    start_but.place(y=150, x=460)
    r1.place(y=450, x=220)
    r2.place(y=450, x=450)
    r3.place(y=450, x=680)
 
    def change(event):  # changing difficulties
        global game_status, time, lvl
        if r_var.get() == 1:
            time = 10  # repeating of cycle = (60/time)
            lvl = 1
        elif r_var.get() == 2:
            time = 17
            lvl = 2
        elif r_var.get() == 3:
            time = 24
            lvl = 3
        root.destroy()
        game_status = True
 
    def but_animation_enter(event):  # if mouse on the button change bg
        start_but['bg'] = '#e6e200'
 
    def but_animation_leave(event):
        start_but['bg'] = '#c5ff2e'
 
    def on_exit():
        sys.exit()
 
    root.protocol('WM_DELETE_WINDOW', on_exit)  # if player clicked quit, close window
    start_but.bind('<Button-1>', change)
    start_but.bind('<Enter>', but_animation_enter)
    start_but.bind('<Leave>', but_animation_leave)
    root.mainloop()
 
 
try:
    with open('pb.pickle', 'rb') as r:  # checking best score
        best_result = pickle.load(r)
except EOFError:                        # if no pb.pickle
    with open('pb.pickle', 'wb') as w:  # or pb.pickle clear, best_result = 1
        pickle.dump('1', w)
except FileNotFoundError:
    with open('pb.pickle', 'wb') as w:
        pickle.dump('1', w)
 
start_window()
 
x, y = 440, 320
SIZE = 20  # size of all squares on the field
directions = {
    'U': (0, -SIZE),
    'D': (0, SIZE),
    'R': (SIZE, 0),
    'L': (-SIZE, 0)}
dir2 = {'U': False, 'D': False, 'R': False, 'L': False}
dx, dy = 0, 0
 
pg.font.init()  # creating fonts for all text
score_font = pg.font.Font(None, 36)
lose_font = pg.font.Font(None, 80)
lose_text = lose_font.render("You lost :(", True, (1, 1, 1))
 
restart_img = pg.image.load('restart.png')  # text of window
pg.display.set_caption('The Snake')
 
 
SNAKE = [(x, y)]
length = 1  # number of sections in Snake
ax, ay = random.randrange(0, 900, SIZE), random.randrange(0, 640, SIZE)  # cords of apple
 
bg = pg.display.set_mode((WIDTH, HEIGHT))  # creating window
 
isQuit = False
game_status = True
 
while not isQuit:
    bg.fill((53, 176, 86))
 
    while game_status:
        bg.fill((53, 176, 86))
        score = score_font.render(str(length), True, (1, 1, 1))
        bg.blit(score, (870, 612))
 
        for i in SNAKE:  # Snake moving
            pg.draw.rect(bg, (181, 252, 2), (i[0], i[1], SIZE, SIZE))
 
        pg.draw.rect(bg, (192, 85, 85), (ax, ay, SIZE, SIZE))  # the appearance of apple
        x += dx
        y += dy
        SNAKE.append((x, y))
        SNAKE = SNAKE[-length:]
 
        for i in range(0, len(SNAKE) - 3):  # if snake eat itself
            if SNAKE[i][0] == SNAKE[-1][0] and SNAKE[i][1] == SNAKE[-1][1]:
                game_status = False
 
        if x >= 900 or y >= 640 or x <= -10 or y <= -10:  # if touch boundaries
            game_status = False
 
        if x == ax and y == ay:  # if eat apple, add new one, increase length and add score
            pg.draw.rect(bg, (192, 85, 85), (ax, ay, 20, SIZE))
            length += 1
            ax, ay = random.randrange(0, 900, SIZE), random.randrange(0, 640, SIZE)
            for i in SNAKE:  # reduce probability of creating apple in snake
                if i[0] == ax and i[1] == ay:
                    ax, ay = random.randrange(0, 900, SIZE), random.randrange(0, 640, SIZE)
 
        pg.display.flip()
        clock.tick(time)
        time += 0.02  # every cycle speed increase
 
        for i in pg.event.get():  # checking keyboard pressing
            if i.type == pg.QUIT:
                sys.exit()
            elif i.type == pg.KEYDOWN:
                if i.key == pg.K_a and dir2['R'] is False:
                    dx, dy = directions['L'][0], 0
                    dir2 = {'U': False, 'D': False, 'R': False, 'L': True}
                elif i.key == pg.K_d and dir2['L'] is False:
                    dx, dy = directions['R'][0], 0
                    dir2 = {'U': False, 'D': False, 'R': True, 'L': False}
                elif i.key == pg.K_w and dir2['D'] is False:
                    dy, dx = directions['U'][1], 0
                    dir2 = {'U': True, 'D': False, 'R': False, 'L': False}
                elif i.key == pg.K_s and dir2['U'] is False:
                    dy, dx = directions['D'][1], 0
                    dir2 = {'U': False, 'D': True, 'R': False, 'L': False}
 
    bg.blit(lose_text, (355, 120))  # if lose appears text "You lost"
 
    if length > int(best_result):  # if result > past best score, new best = result
        best_result = length
        with open('pb.pickle', 'wb') as w:
            pickle.dump(str(best_result), w)
 
    best_score = score_font.render(f"Best score: {best_result}", True, (1, 1, 1))
    bg.blit(best_score, (410, 190))  # show best score
    bg.blit(restart_img, (375, 340))  # restart icon
 
    clock.tick(10)
    pg.display.flip()
 
    mouse_pos = pg.mouse.get_pos()
    for i in pg.event.get():  # checking if we quit in main window
        if i.type == pg.QUIT:
            sys.exit()
        if 380 <= mouse_pos[0] <= 570 and 348 <= mouse_pos[1] <= 524:  # if player pressed on icon,
            if i.type == pg.MOUSEBUTTONUP:  # restart game
                start_window()
                x, y = 440, 320  # reinitialization of variables
                SNAKE = [(x, y)]
                dx, dy = 0, 0
                length = 1
                game_status = True
Я начинающий программист, решил немного попрактиковаться и сделать змейку. Использовал PyGame. Хочу узнать мнение опытных программистов. На моём этапе очень важна критика, чтоб мне помогли и подсказали мои ошибки. Если вам не сложно оцените мою работу и укажите мои ошибки пожалуйста. Как мне сделать код более читабельным и что вообще изменить в нём.
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
11.03.2021, 23:53
Ответы с готовыми решениями:

Оцените код. Что можно исправить?
Всем прив. Что можно поменять?Упростить? Вам понятно,как работает код? Создайте вектор, хранящий десять строковых значений...

Оцените код крестиков-ноликов. Что можно исправить/добавить?
#include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;windows.h&gt; using namespace std; const int height = 5, leight = 11; char...

Зависает, подскажите что исправить пожалуйста, и доработать чуть чуть, очень надо
Разработать программу , которая в диалоговом режиме позволяет а ) создать файл , каждая запись которого содержит информацию о книге в...

1
Эксперт Python
5438 / 3859 / 1215
Регистрация: 28.10.2013
Сообщений: 9,552
Записей в блоге: 1
12.03.2021, 00:06
Лучший ответ Сообщение было отмечено DamirFilretov как решение

Решение

Цитата Сообщение от DamirFilretov Посмотреть сообщение
подсказали мои ошибки
1) Не используй global. Это ведет к путанице, случайным непредсказуемым изменениям и плохой инкапсуляции данных.
Данные нужно передавать либо из функции в функцию, либо через экземпляры классов.

2) Не используй переменные вида x1, x2, x3 и т.д. Если у тебя много однотипных переменных - заведи список или словарь.
Для того, чтобы создать 10 кнопок не надо писать 10 раз одно и тоже. Нужно написать цикл из 10 итераций и создать в нем все нужные кнопки, а результат создания записывать в список или словарь.

3) Не надо хардкодить статические данные прямо в код: пути к файлам, настройки виджетов (позиции, значения атрибутов ,имена и и т.д.). Заведи для них специальные конфиг и доставай данные оттуда. Конфиг в Python принято создавать как отдельный .py файл внутри которого есть либо список переменных (не очень хорошо), либо специальный конфиг класс, который ты импортируешь в главном модуле и используешь по назначению поля его экземпляра там где нужно что-то инициировать.
2
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
12.03.2021, 00:06
Помогаю со студенческими работами здесь

Оцените пожалуйста мой ПК
------------------------------------------------------------------------------------------------ Компьютер: Тип компьютера...

Оцените пожалуйста мой ПК
Всем доброго времени суток! Собрался продавать свой компьютер сборки 2016/декабрь 2017 года (что-то купил раньше, что-то позже). ...

Оцените пожалуйста мой пк
1. Простой корпус 3ccot 2.ga 970 ds3p 3.fx 6300 4.gt 740 ddr5 2gb 5.deepcool gammaxx s40 6.chieftec eco series 500w 80 bronze ...

Оцените пожалуйста мой системник
Ребята,добрый вечер. Подскажите пожалуйста за сколько можно продать этот системник: 1. Процессор CPU Intel Core i5-3470 3.2...

Оцените пожалуйста мой компьютер!
Core i7 860 (2.80/4 8GT/sec/8Mb) OEM Asus P7p55-M Soc-1156 iP55 DDRIII mATX AC'97 8ch LAN-Gbt+1394+RaiD (White Box) 850W ATX Zalman...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Новые блоги и статьи
Установка Qt Creator для C и C++: ставим среду, CMake и MinGW без фреймворка Qt
8Observer8 05.04.2026
Среду разработки Qt Creator можно установить без фреймворка Qt. Есть отдельный репозиторий для этой среды: https:/ / github. com/ qt-creator/ qt-creator, где можно скачать установщик, на вкладке Releases:. . .
AkelPad-скрипты, структуры, и немного лирики..
testuser2 05.04.2026
Такая программа, как AkelPad существует уже давно, и также давно существуют скрипты под нее. Тем не менее, прога живет, периодически что-то не спеша дополняется, улучшается. Что меня в первую очередь. . .
Отображение реквизитов в документе по условию и контроль их заполнения
Maks 04.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеСпецтехники", разработанного в конфигурации КА2. Данный документ берёт данные из другого нетипового документа. . .
Фото всей Земли с борта корабля Orion миссии Artemis II
kumehtar 04.04.2026
Это первое подобное фото сделанное человеком за 50 лет. Снимок называют новым вариантом легендарной фотографии «The Blue Marble» 1972 года, сделанной с борта корабля «Аполлон-17». Новое фото. . .
Вывод диалогового окна перед закрытием, если документ не проведён
Maks 04.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать программный контроль на предмет проведения документа. . .
Программный контроль заполнения реквизитов табличной части документа
Maks 02.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: 1. Реализовать контроль заполнения реквизита. . .
wmic не является внутренней или внешней командой
Maks 02.04.2026
Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/
Программная установка даты и запрет ее изменения
Maks 02.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru