Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed HangingLamp #411

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Fixed HangingLamp #411

wants to merge 6 commits into from

Conversation

Graff46
Copy link
Contributor

@Graff46 Graff46 commented Feb 12, 2022

Поправил HangingLamp, теперь методы turn_off, turn_on можно применять к лампам в любое время. Раньше, если методы применить слишком "рано", например из метода биндера самих ламп on_spawn был вылет, потому что объект источника света и кости не успевал создаться
Поправил код чтобы в замыкании метода использовать одну переменную сущности, раньше её получали несколько раз.
Локально проверено - работает.

xrSimpodin and others added 6 commits June 29, 2021 13:57
Поправил HangingLamp, теперь методы `turn_off`, `turn_on` можно применять к лампам в любое время. Раньше, если методы применить слишком "рано", например из метода биндера самих ламп `on_spwn` был вылет, потому что объект источника света и кости не успевал создаться

fix
@ForserX
Copy link
Contributor

ForserX commented Feb 16, 2022

image

Выглядит как-то странно. Типо, если лампа включена, то включаем.

@Graff46
Copy link
Contributor Author

Graff46 commented Feb 16, 2022

image

Выглядит как-то странно. Типо, если лампа включена, то включаем.

Вот оригинальный код:
изображение
Если лампа "жива" то включить её, иначе выключить. Я всего лишь флаг добавил.

@ForserX
Copy link
Contributor

ForserX commented Feb 17, 2022

В этом и соль. Если ты выключить лампу, то флаг станет false. И больше не включится. Т.к. проверяется при включении на true

@Graff46
Copy link
Contributor Author

Graff46 commented Feb 17, 2022

В этом и соль. Если ты выключить лампу, то флаг станет false. И больше не включится. Т.к. проверяется при включении на true

Чтобы включить лампу нужно вызвать метод turnOn он переключит флаг и активирует источник света и его кость. При возврате из оффлайна экземпляр класса лампы создается по новой и флаг по умолчанию будет true

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants