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

import win32gui
import pyautogui
import winsound

def get_pixel_colour(i_x, i_y):
    i_desktop_window_id = win32gui.GetDesktopWindow()
    i_desktop_window_dc = win32gui.GetWindowDC(i_desktop_window_id)
    long_colour = win32gui.GetPixel(i_desktop_window_dc, i_x, i_y)
    i_colour = int(long_colour)
    return (i_colour & 0xff), ((i_colour >> 8) & 0xff), ((i_colour >> 16) & 0xff)

def is_top_free(x, y):
    if all(i in range(35,46) for i in get_pixel_colour(x+258, y+108) and get_pixel_colour(x+283, y+108) and get_pixel_colour(x+283, y+139) and get_pixel_colour(x+258, y+139)) and all(i in range(80, 90) for i in get_pixel_colour(x+264, y+131)):
        return True
    else:
        return False

def is_bottom_free(x, y):
    if all(i in range(35, 46) for i in get_pixel_colour(x+256, y+788) and get_pixel_colour(x+286, y+788) and get_pixel_colour(x+256, y+820) and get_pixel_colour(x+286, y+820)) and all(i in range(80, 90) for i in get_pixel_colour(x+264, y+808)):
        return True
    else:
        return False

def get_top_seat(x, y):
    current = pyautogui.position()
    pyautogui.click(x+267, y+126)
    pyautogui.moveTo(current[0], current[1])
    winsound.Beep(500, 500)

def get_bottom_seat(x, y):
    current = pyautogui.position()
    pyautogui.click(x+267, y+802)
    pyautogui.moveTo(current[0], current[1])
    winsound.Beep(500, 500)

def callback(hwnd, extra):
    rect = win32gui.GetWindowRect(hwnd)
    x = rect[0]
    y = rect[1]
    w = rect[2] - x
    h = rect[3] - y
    if win32gui.GetWindowText(hwnd) in accounts:
        if is_top_free(x, y):
            get_top_seat(x, y)
        elif is_bottom_free(x, y):
            get_bottom_seat(x,y)

accounts = ['3948700830', '3949029392', '3949074503', '3949131493', '3949174547', '3949206036']

try:
    while True:
        win32gui.EnumWindows(callback, None)
except KeyboardInterrupt:
    print('End')

Этот сценарий ищет кнопку и по возможности нажимает на нее.

Компьютер работает все медленнее и медленнее при выполнении этого сценария. Я предполагаю, что некоторые переменные перезаписываются без удаления из памяти. Я попытался добавить в цикл немного сна, но это не решило проблему - он замедляется медленнее, но все равно регулярно. У вас есть идеи, что может вызвать это?

Это не связано напрямую с вашей проблемой, но я не думаю, что ваши условия, начинающиеся с if all, будут работать должным образом. Вы не можете связать вместе несколько вызовов get_pixel_colour и заставить for i in... перебирать их все. Учтите: если вы сделаете (11,12,13) and (4,5,6) в интерактивной подсказке, результатом будет (4,5,6), а не (11,12,13,4,5,6).

Kevin 10.12.2018 21:21

Хм, исходя из документации GetWindowDC, вы должны вызвать ReleaseDC, когда закончите его использовать. Я не вижу ни одного, что может быть проблемой. Я бы также подумал об оптимизации получения цвета пикселей. В идеале для каждого окна вы должны вызвать GetWindowDC один раз, извлечь все необходимые значения пикселей и затем отпустить его. (Кроме того, я бы действительно подумал о том, чтобы переписать условие if в is_bottom_free и is_top_free ... это, по меньшей мере, неприятно разбирать)

Dan Mašek 10.12.2018 21:22

Они исправно работают. Он находит кнопку, когда она появляется, и нажимает на нее. Проблема только в эффективности.

Jakub Bielan 10.12.2018 21:23

Спасибо, Дэн, я проверю.

Jakub Bielan 10.12.2018 21:25

Позвольте мне уточнить. Когда я говорю, что они не будут работать должным образом, я не имею в виду, что программа выйдет из строя или что-то в этом роде. Я имею в виду, что код будет выполнять меньше проверок, чем должен, и иногда сообщать, что окно совпадает, когда этого не должно быть. По сути, ваше первое условное выражение проверяет только результат get_pixel_colour(x+258, y+139) и полностью игнорирует все предшествующие ему вызовы.

Kevin 10.12.2018 21:25
Почему в Python есть оператор "pass"?
Почему в Python есть оператор "pass"?
Оператор pass в Python - это простая концепция, которую могут быстро освоить даже новички без опыта программирования.
Некоторые методы, о которых вы не знали, что они существуют в Python
Некоторые методы, о которых вы не знали, что они существуют в Python
Python - самый известный и самый простой в изучении язык в наши дни. Имея широкий спектр применения в области машинного обучения, Data Science,...
Основы Python Часть I
Основы Python Часть I
Вы когда-нибудь задумывались, почему в программах на Python вы видите приведенный ниже код?
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
LeetCode - 1579. Удаление максимального числа ребер для сохранения полной проходимости графа
Алиса и Боб имеют неориентированный граф из n узлов и трех типов ребер:
Оптимизация кода с помощью тернарного оператора Python
Оптимизация кода с помощью тернарного оператора Python
И последнее, что мы хотели бы показать вам, прежде чем двигаться дальше, это
Советы по эффективной веб-разработке с помощью Python
Советы по эффективной веб-разработке с помощью Python
Как веб-разработчик, Python может стать мощным инструментом для создания эффективных и масштабируемых веб-приложений.
2
5
149
1

Ответы 1

Это помогло решить проблему:

def get_pixel_colour(i_x, i_y):
    i_desktop_window_id = win32gui.GetDesktopWindow()
    i_desktop_window_dc = win32gui.GetWindowDC(i_desktop_window_id)
    long_colour = win32gui.GetPixel(i_desktop_window_dc, i_x, i_y)
    i_colour = int(long_colour)
    win32gui.ReleaseDC(i_desktop_window_id, i_desktop_window_dc)
    return (i_colour & 0xff), ((i_colour >> 8) & 0xff), ((i_colour >> 16) & 0xff)

Спасибо, Дан Машек! Все дело в понимании контекста устройства.

Другие вопросы по теме