Ошибка интерпритатора python'a или моего мозга?

Question

[Михаил Мунтян]

Вообщем, решил написать авто-бота на пайтоне, пишу пишу, и делаю активаю/деактивацию, и тут пайтон выдает что переменная не видится в функции, но я ее делаю глобальной, созрел вопрос? Код:

import pyautogui
import keyboard
from threading import Thread
from time import sleep

def Run():
    i = 0
    while switch:
        pass

def Click():
    while switch:
        pyautogui.click()
        sleep(1)

def Switch():
    while True:
        if keyboard.is_pressed('End'):
            if switch:
                switch = False
                pyautogui.keyUp('shift')
            else:
                switch = True
                pyautogui.keyDown('shift')

global switch
switch = False

thread1 = Thread(target=Switch)
thread2 = Thread(target=Run)
thread3 = Thread(target=Click)

thread1.start()
thread2.start()
thread3.start()

thread1.join()
thread2.join()
thread3.join()

Comments

[Vindicar]

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

Answer 1

[Руслан Гильфанов]

Кажется Вы неправильно поняли как работает оператор global.

Для начала, прочитайте про LEGB в Python.

Если к именам из глобальной области ниже происходит обращение на чтение -- использовать оператор global не имеет смысла.

Если именам из глобальной области во вложенном пространстве имён присваиваются другие объекты -- нужно объявить во вложенном пространстве оператор global и перечислить имена из глобальной области через запятую.

Аналогично с объёмлющими (англ. enclosed) областями -- для них предусмотрен оператор nonlocal.

Касательно вашего кода, попробуйте изменить так:

import pyautogui
import keyboard
from threading import Thread
from time import sleep


switch = False


def Run():
    global switch
    i = 0
    while switch:
        pass

def Click():
    global switch
    while switch:
        pyautogui.click()
        sleep(1)

def Switch():
    global switch
    while True:
        if keyboard.is_pressed('End'):
            if switch:
                switch = False
                pyautogui.keyUp('shift')
            else:
                switch = True
                pyautogui.keyDown('shift')


thread1 = Thread(target=Switch)
thread2 = Thread(target=Run)
thread3 = Thread(target=Click)

thread1.start()
thread2.start()
thread3.start()

thread1.join()
thread2.join()
thread3.join()

Comments

[Михаил Мунтян]

До этого и сейчас удаление global не помогло

[Руслан Гильфанов]

Михаил Мунтян, а где Вы нашли у меня удаление global?

У меня скорее размножение и перемещение.

[Михаил Мунтян]

Извиняюсь, не увидел. Но теперь не работает функция Click()

[Руслан Гильфанов]

Михаил Мунтян, отметьте мой ответ решением и пишите новый вопрос =))

[Михаил Мунтян]

Руслан Гильфанов, Может не будем засорять habr и решим все тут? Но если вы настаиваете я не против..

[Руслан Гильфанов]

Михаил Мунтян, настаиваю.

[Михаил Мунтян]

Руслан Гильфанов, Ловите

Answer 2

[UberPool]

Что за чушь ты пишешь?Какая ошибка интерпретатора.Переменную нужно создать глобально и зачем ты используешь global
Приведу пример:

test = 0

def add_user():
    print(test)

Если что-то не понятно спрашивай

Почитай про global и старайся не использовать его

Comments

[Михаил Мунтян]

Чушь..

[UberPool]

Михаил Мунтян, добавь global switch
в функцию Switch.Почитай документацию прежде чем писать элементарные вопросы.