нужна помощь

Посмотрите с помощью Print() какие значения переменных (машек и МАКАД-а) участвуют в формировании значения переменной Signal. И вообще, объявили бы Вы эту пременную булевым типом и разделили бы по сигналам, навроде как Signal_UP/Signal_DOWN. И вот ещё что, после того, как Вы проинициализировали свою переменную Signal значением, Вы в коде её используете. После этого она уже не нужна. Но на следующем тике она хранит значение, которое осталось от предыдущего тика. Поэтому на каждом новом тике лучше бы обнулить её в первой же строке функции start() - это будет гарант того, что значение, оставшееся от предыдущего тика, сброшено.

добавив функцию ни чего не поменялось. решил протестировать с разрешением только в лонг, выдает ошибку при попытке открытия "4111", я так понимаю что сигналы есть! а вот открывать отказывается.

ERR_SHORTS_NOT_ALLOWED 4111 Короткие позиции не разрешены. Необходимо проверить свойства эксперта.

https://docs.mql4.com/ru/constants/errors

проверить свойства! на что конкретно в свойствах мне нужно обратить внимание

Конкретно прямо на это и обратить.

Чтобы там не стояло лонг онли.

благодарю за ответ, но в начале темы было прописано что не открывается в лонг, то есть эксперт открывает короткие только короткие и все  , а нужно что бы открывались еще и длинные. в свойствах стоит лонг&шорт

Если бы в свойствах стоял  лонг&шорт, то ошибка с таким номером не выскочила бы. 

Видимо рано автор топика ник сменил