Foxyk Posted August 31, 2014 Posted August 31, 2014 function enter (thePlayer) accnames = getAccountName ( getPlayerAccount ( thePlayer ) ) if accnames == "..." then moveObject ( Vorota1, 2000, 79.7, 2501.8999, 15.5 ) moveObject ( Vorota2, 2000, 79.75, 2528.5, 15.5 ) else outputChatBox ("#CD0000Доступ запрещён!", thePlayer, 0,255,0,true) end end addEventHandler ("onMarkerHit", getRootElement( ),enter) function leave (thePlayer) accname = getAccountName ( getPlayerAccount ( thePlayer ) ) if accname == "..." then moveObject ( Vorota1, 2000, 79.7, 2509.8999, 15.5 ) moveObject ( Vorota2, 2000, 79.75, 2520.5, 15.5 ) end end addEventHandler ("onMarkerLeave", getRootElement( ),leave) Как можно немного упростить код? А то квадратно как то...
Kernell Posted August 31, 2014 Posted August 31, 2014 Ну хотя бы для начала почитай это: https://forum.multitheftauto.com/viewtopic.php?f=141&t=40703 Специально для таких как ты писали. А потом уже поясни, что в твоём понимании "упростить".
N1kS Posted September 4, 2014 Posted September 4, 2014 Куда тут больше упрощать? Есть смысл наоборот добавить кое что.
TheNormalnij Posted September 5, 2014 Posted September 5, 2014 Сначала надо как-нибудь код стилизовать. Сделать легкое добавление новых ворот. Обозначить области видимости переменных.
Fabervox Posted September 6, 2014 Posted September 6, 2014 У вас обе функции в целом почти повторяются, сделайте одну, но с параметром, можно булевого типа, можно строку "enter", её как второе условие в elseif, первое то же что и в if только ~=, а в else лучше вписать сообщение в консоль об ошибке, т.к. всегда может пойти что-то не так.
TheNormalnij Posted September 6, 2014 Posted September 6, 2014 Не надо так, такие методы написания условий только портят код ИМХО.
Fabervox Posted September 6, 2014 Posted September 6, 2014 Мне кажется, это как раз делает код более абстрактным, и как раз решает задачу ТС.
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now