Foxyk Posted August 31, 2014 Share 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) Как можно немного упростить код? А то квадратно как то... Link to comment
Kernell Posted August 31, 2014 Share Posted August 31, 2014 Ну хотя бы для начала почитай это: https://forum.multitheftauto.com/viewtopic.php?f=141&t=40703 Специально для таких как ты писали. А потом уже поясни, что в твоём понимании "упростить". Link to comment
N1kS Posted September 4, 2014 Share Posted September 4, 2014 Куда тут больше упрощать? Есть смысл наоборот добавить кое что. Link to comment
TheNormalnij Posted September 5, 2014 Share Posted September 5, 2014 Сначала надо как-нибудь код стилизовать. Сделать легкое добавление новых ворот. Обозначить области видимости переменных. Link to comment
Fabervox Posted September 6, 2014 Share Posted September 6, 2014 У вас обе функции в целом почти повторяются, сделайте одну, но с параметром, можно булевого типа, можно строку "enter", её как второе условие в elseif, первое то же что и в if только ~=, а в else лучше вписать сообщение в консоль об ошибке, т.к. всегда может пойти что-то не так. Link to comment
TheNormalnij Posted September 6, 2014 Share Posted September 6, 2014 Не надо так, такие методы написания условий только портят код ИМХО. Link to comment
Fabervox Posted September 6, 2014 Share Posted September 6, 2014 Мне кажется, это как раз делает код более абстрактным, и как раз решает задачу ТС. Link to comment
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