Советник - разруливатель локов. Хочу много критики :)

 
Переменная traler - трейлинг в пп, point - точка входа, сдвигается автоматически. Точка входа подсвечивается желтой полосой, поэтому делайте темный профиль.
Файлы:
 

Не делаем локи, и разруливать не надо! Если случается, то не заморачиваемся! Просто тралим каждую позицию независимо! А как достигается общий профит, закрываемся! Зачем нужен специальный советник? И критиковать-то нечего!

 

По объему кода, алгоритм примитивный, а разбирать взломанный код как то желания нет...

 
Так, с вами понятно, кто-нибудь толковый тут есть?
 

Для чего переменная bid?

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

 

Довольно узкий диапазон работоспособности советника, который, к тому-же, никак не контролируется. Комментарии, практически, отсутствуют. Переменные и выражения типа double сравниваются на тождество.

Под локом здесь понимается наличие в терминале ровно двух открытых в противоположном направлении и одинакового объема ордеров. При этом не проверяется совпадение символов инструмента.

Попытка поторговать, пока разруливается лок, приводит к неработоспобности советника. Это можно исправить, если исходить из того, что советник должен разруливать лок, на символе того чарта, где висит. Это не сильно усложнит логику и выборку ордеров, но, на порядок сделает его дружелюбнее.