Почему не двигается объект в pygame?

Question

[Alex_Kuzen]

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

import pygame
import sys
import random
import time

pygame.init()
win = pygame.display.set_mode((800, 800))
white = (200, 200, 200)
red = (220, 51, 102)
green = (120, 120, 200)
black = (0, 0, 0)


class Hunter:
    white = pygame.Color(220, 100, 100)

    def __init__(self, x, y, h, w, color):
        self.x = x
        self.y = y
        self.h = h
        self.w = w
        self.color = color

    def hunter_show(self):
        pygame.draw.rect(win, (self.color), (self.x, self.y, self.h, self.w))

    def huner_move(self, x, y):
        self.x = x
        self.y = y


class Ex:
    def __init__(self, x, y, h, w, color):
        self.x = x
        self.y = y
        self.h = h
        self.w = w
        self.color = color

    def ex_show(self):
        pygame.draw.rect(win, (self.color), (self.x, self.y, self.h, self.w))

    def ex_position(self, x, y):
        self.x = x
        self.y = y


while True:
    color = white
    h_color = red
    pos_x_rand = random.randint(1, 790)
    pos_y_rand = random.randint(1, 790)

    hunter_pos_x = 0
    hunter_pos_y = 0
    time.sleep(0.0007)
    hunter_1 = Hunter(hunter_pos_x, hunter_pos_y, 5, 5, h_color)
    exl = Ex(pos_x_rand, pos_y_rand, 3, 3, color)
    if hunter_pos_y < pos_y_rand:
        hunter_pos_y += 1
    if hunter_pos_x < pos_x_rand:
        hunter_pos_x += 1
    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            sys.exit()


    exl.ex_show()
    hunter_1.hunter_show()
    pygame.display.update()

Answer 1

[o5a]

Потому что сравниваете координаты не объекта hunter_1, а не связанные с ним координаты

if hunter_pos_y < pos_y_rand:
        hunter_pos_y += 1

когда должно быть примерно так

if hunter_1.y < pos_y_rand:
        hunter_1.y += 1

Аналогично и координаты целей (pos_y_rand). Сейчас координаты охотника сравниваются просто с этой рандомной точкой каждый раунд, а не с координатами реального объекта - цели.
К тому же этот самый hunter каждый цикл создается заново, хотя по логике предполагаю он должен быть только 1.
Т.е. создание объекта hunter нужно вынести наружу цикла, а координаты проверять по аттрибутам объекта, его же и двигать.
Во-вторых, эти цели (звёзды?) в текущем виде нигде не сохраняются, каждый цикл предыдущие объекты-цели перезаписываются, остается только их изображение на экране. Т.е. координаты ранее созданных целей взять будет негде. Если все же логика работы была - ходить за этими целями, то их нужно хранить (самое простое - добавлять к списку).

Comments

[Alex_Kuzen]

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

А белые точки это скорее жертвы, а не звезды.

Вот, дописал по совету, он ожил, но все же немного не то. Я в Питоне недавно, могу и тупить. Спасибо за ответ заранее

import pygame
import sys
import random
import time

pygame.init()
win = pygame.display.set_mode((800, 800))
white = (200, 200, 200)
red = (220, 51, 102)
green = (120, 120, 200)
black = (0, 0, 0)


class Hunter:
    white = pygame.Color(220, 100, 100)

    def __init__(self, x, y, h, w, color):
        self.x = x
        self.y = y
        self.h = h
        self.w = w
        self.color = color

    def hunter_show(self):
        pygame.draw.rect(win, (self.color), (self.x, self.y, self.h, self.w))

    '''def huner_move(self, x, y):
        self.x = x
        self.y = y'''


class Ex:
    def __init__(self, x, y, h, w, color):
        self.x = x
        self.y = y
        self.h = h
        self.w = w
        self.color = color

    def ex_show(self):
        pygame.draw.rect(win, (self.color), (self.x, self.y, self.h, self.w))

    def ex_position(self, x, y):
        self.x = x
        self.y = y


h_color = red
hunter_1 = Hunter(1, 1, 5, 5, h_color)

while True:
    color = white
    pos_x_rand = random.randint(1, 790)
    pos_y_rand = random.randint(1, 790)
    

    time.sleep(0.007)

    exl = Ex(pos_x_rand, pos_y_rand, 3, 3, color)
    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            sys.exit()

    if hunter_1.x < pos_x_rand:
        hunter_1.x += 1
    if hunter_1.y < pos_y_rand:
            hunter_1.y += 2
    if hunter_1.y > pos_y_rand:
            hunter_1.y -= 2
    if hunter_1.x > pos_x_rand:
            hunter_1.x -= 1
    if hunter_1.x == pos_x_rand:
        color = black
    if hunter_1.y == pos_y_rand:
        color = black

    exl.ex_show()
    hunter_1.hunter_show()
    pygame.display.update()

[Alex_Kuzen]

На более низких скоростях выяснилась неприятная вещь, он делает только один шаг на появление точки, и идет только в сторону новой точки.

[o5a]

Alex_Kuzen, конечно, я про это тоже писал. Что делает эта проверка?
if hunter_1.x < pos_x_rand:
сравнивает текущую позицию x объекта hunter с текущим значением переменной pos_x_rand. А эта переменная каждый цикл генерируется, создает точку-цель и в следующем цикле ее забывает, т.к. опять же генерируется заново
pos_x_rand = random.randint(1, 790)
Логично же, что каждый цикл курс будет меняться на последнюю сгенерированную точку.
Поэтому вместо сравнения со значением переменной pos_x_rand нужно сравнивать со значением координаты х - аттрибута объекта Ex. Какого именно - я не знаю, как Вы это по логике игры планировали?

А чтобы можно было к этим объектам обращаться, нужно их где-то хранить. Например, складывать в список.

# создаем наш список объектов
targets = []
while True:
    ....
    # и после создания добавляем созданный объект к списку целей
    exl = Ex(pos_x_rand, pos_y_rand, 3, 3, color)
    targets.append(exl)

Тогда можно будет обращаться к любому созданному объекту-цели.
Например, targets[0].x будет выдавать текущую координату самой первой созданной цели.

как сделать так, что бы охотник не оставлял след

Обычно в pygame это делается таким образом, что каждый цикл поле сбрасывается (окрашивается цветом фона)
win.fill(black)
и после этого все нужные объекты отрисовываются заново. Те, что изменили координату, соответственно перерисуются в новом месте, так создастся иллюзия, что они передвинулись, и не будет следа.

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

Answer 2

[Руслан .]

Тут вероятно нужно вынести инициализации положения охотника за пределы цикла while.
Я про вот этот код:
hunter_pos_x = 0
hunter_pos_y = 0

А так у вас на каждой итерации цикла все начинается с 0,0