В вашем коде отсутствует абстракция .
Вместо набора screen.blit и множества операторов if используйте списки.
Вместо жесткогозакодированные значения (например, cn1+25 < x and cn1 + 50 > x), используют структуры данных (в вашем случае, объекты), которые содержат всю необходимую им информацию.
Вот простой пример.
Допустим, у нас есть этот прекрасный миргде облака движутся в солнечный день:
import pygame
screen = pygame.display.set_mode((300, 300))
clock = pygame.time.Clock()
cloud = pygame.Surface((50, 20))
cloud.set_colorkey((11, 12, 13))
cloud.fill((11, 12, 13))
pygame.draw.ellipse(cloud, pygame.Color('white'), cloud.get_rect())
cloudmove = 20
while True:
for e in pygame.event.get():
if e.type == pygame.QUIT:
quit()
screen.fill(pygame.Color('lightblue'))
screen.blit(cloud, (cloudmove+120,30))
cloudmove += 1
pygame.display.flip()
clock.tick(30)
Но, конечно, мы хотим на больше , чем одно облако.Чего мы не хотим, так это дублирования кода и повышения сложности.Мы могли бы сделать что-то вроде этого:
import pygame
screen = pygame.display.set_mode((300, 300))
clock = pygame.time.Clock()
cloud = pygame.Surface((50, 20))
cloud.set_colorkey((11, 12, 13))
cloud.fill((11, 12, 13))
pygame.draw.ellipse(cloud, pygame.Color('white'), cloud.get_rect())
cloud2 = pygame.Surface((50, 20))
cloud2.set_colorkey((11, 12, 13))
cloud2.fill((11, 12, 13))
pygame.draw.ellipse(cloud2, pygame.Color('white'), cloud2.get_rect())
cloud3 = pygame.Surface((50, 20))
cloud3.set_colorkey((11, 12, 13))
cloud3.fill((11, 12, 13))
pygame.draw.ellipse(cloud3, pygame.Color('white'), cloud3.get_rect())
cloudmove = 20
while True:
for e in pygame.event.get():
if e.type == pygame.QUIT:
quit()
screen.fill(pygame.Color('lightblue'))
screen.blit(cloud, (cloudmove-120,30))
screen.blit(cloud2, (cloudmove-220,40))
screen.blit(cloud3, (cloudmove-170,50))
cloudmove += 1
pygame.display.flip()
clock.tick(30)
Вы уже можете видеть, где это закончится, если мы добавим еще дюжину объектов.
Итак, давайте попробуем использовать здесь объекты, представляющие облака:
import pygame
screen = pygame.display.set_mode((300, 300))
clock = pygame.time.Clock()
class Cloud:
def __init__(self, x, y):
self.image = pygame.Surface((50, 20))
self.image.set_colorkey((11, 12, 13))
self.image.fill((11, 12, 13))
pygame.draw.ellipse(self.image, pygame.Color('white'), self.image.get_rect())
self.x = x
self.y = y
def update(self):
self.x += 1
if self.x > 300:
self.x = 0
clouds = [Cloud(0, 30), Cloud(100, 40), Cloud(50, 50)]
while True:
for e in pygame.event.get():
if e.type == pygame.QUIT:
quit()
screen.fill(pygame.Color('lightblue'))
for cloud in clouds:
screen.blit(cloud.image, (cloud.x, cloud.y))
cloud.update()
pygame.display.flip()
clock.tick(30)
(здесь нет нового изображения, потому что оно в основном выглядит одинаково)
Намного лучше!Теперь, чтобы добавить больше облаков, мы просто добавили бы новый экземпляр Cloud в список clouds.Посмотрите, как мы храним не только изображение облака в классе, но и положение.И мы поместили облако поведение (двигаясь по небу и начнем слева, как только мы отошли от экрана ) в класс.
Но мы все еще можем сделать лучше! Pygame предлагает несколько удобных классов и функций, которые мы можем использовать.Взгляните на следующий код:
import pygame
screen = pygame.display.set_mode((300, 300))
clock = pygame.time.Clock()
class Coin(pygame.sprite.Sprite):
def __init__(self, x, y):
super().__init__()
self.image = pygame.Surface((44, 40))
self.image.set_colorkey((11, 12, 13))
self.image.fill((11, 12, 13))
cx, cy = self.image.get_rect().center
pygame.draw.circle(self.image, pygame.Color('grey'), (cx + 2, cy), 20)
pygame.draw.circle(self.image, pygame.Color('orange'), (cx - 2, cy), 20)
self.rect = self.image.get_rect(topleft=(x, y))
def update(self):
mouse_pos = pygame.mouse.get_pos()
if self.rect.collidepoint(mouse_pos) and pygame.mouse.get_pressed()[0]:
self.kill()
class Cloud(pygame.sprite.Sprite):
def __init__(self, x, y):
super().__init__()
self.image = pygame.Surface((50, 20))
self.image.set_colorkey((11, 12, 13))
self.image.fill((11, 12, 13))
pygame.draw.ellipse(self.image, pygame.Color('white'), self.image.get_rect())
self.rect = self.image.get_rect(topleft=(x, y))
def update(self):
self.rect.move_ip(1, 0)
if not pygame.display.get_surface().get_rect().colliderect(self.rect):
self.rect.right = 0
stuff = pygame.sprite.Group(Cloud(0, 30), Cloud(100, 40), Cloud(50, 50),
Coin(100, 100), Coin(150, 180), Coin(80, 200))
while True:
for e in pygame.event.get():
if e.type == pygame.QUIT:
quit()
screen.fill(pygame.Color('lightblue'))
stuff.draw(screen)
stuff.update()
pygame.display.flip()
clock.tick(30)
Мы абстрагировали отрисовку и обновили наши объекты, используя Pygame's Group и Sprite классы.
И посмотрите, как легко добавлять новые вещи в нашу маленькую игру.Я добавил монеты, которые вы можете нажать, чтобы собрать, и все поведение монет находится в классе Coin.Как только вы щелкаете монетку, вызывается метод kill класса Sprite, который просто удаляет объект из всех его групп, поэтому он больше не входит в группу stuff и больше не рисуется наэкран, и фактически удален из игры (и отвечает на ваш вопрос «Выполнение оператора if только один раз»).