Торговля с помощью советника-полуавтомата SS24

Кроме того, таким образом избавился в одном месте от разделения на режим тестирования и торговли. И сделал одинаковый лот для селл и бай. Теперь нужно встроить функцию подсчета открытых ордеров по магику, и убрать тогда:
if ( IsTesting( ) == false)
ВклB = 0; и тому подобное.

[quote="Евгений Субботин"]Наконец-то сподобился в начале этой недели сделать небольшой шажок в сторону модернизации робота. Начал с "головы" ...[/quote]Евгений, ты уж меня извини, но... У меня создаётся впечатление, что я напрасно тратил время на разъяснения в этой теме выше...
Заглянул в твою "новую" версию, а там - ничего принципиально нового, фактически не учтена ни одна моя подсказка, нет попыток использования шаблона и функций. Эксперт по-прежнему остаётся хромым калекой с кучей костылей... Не обижайся, но это так.
По-хорошему, не нужно "дорабатывать" и модифицировать его - нужно поэтапно писать новый.
Рабочее состояние этой и предыдущих версий "висит на волоске" и в любой момент может быть нарушено любым сбоем в работе эксперта, например - обычной перезагрузкой или сменой таймфрейма.

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

- Не нужно писать так:
if ([b]IsTesting() == false[/b] && Ticket > 0)
Функция [b]IsTesting()[/b] возвращает TRUE, если эксперт работает в режиме тестирования, иначе возвращает FALSE, поэтому достаточно писать:
if ([b]!IsTesting()[/b] && Ticket > 0)
а для случая тестирования - без отрицания (восклицательного знака):
if ([b]IsTesting()[/b]...)
Этого уже достаточно.

- У тебя нигде в коде, кроме случая тестирования [b]нет селекции ордера перед его закрытием[/b]...
Это очень грубая ошибка. Посмотри, как правильно это делать в коде скрипта и функции Закрыть Всё, что я выкладывал выше...
- последняя строка return(0) - вообще за закрывающей скобкой функции start()...

В условии
if ( ВклB == 1 && [b]OrdersTotal() < OrdTotal[/b] && iClose(NULL, 1440, 1) < iOpen(NULL, 1440, 1))
Ты проверяешь [b]общее количество ордеров, включая отложки[/b], на соответствие входному параметру OrdTotal... В случае, если на одном счёте имеются ордера, открытые другим советником или вручную, логика работы нарушится.
То же самое и дальше - роботом не используется Magik, поэтому свои ордера от "чужих" он не отличит...

Нет проверки на соответствие параметров торговых функций торговым условиям и ограничениям, в том числе нет нормализации цен. Всё это может приводить к появлению ошибок при отправке торговых запросов. Нет контроля выполнения запроса и, соответственно, повторных запросов в случае ошибок.
Выставление СЛ/ТП одновременно с открытием позиции возможно только на счетах классик (Instant Execution), поэтому на счёте Маркет он уже работать не будет...
Короче - удивительно, что хотя бы иногда этот робот умудряется торговать. Не надо его модифицировать - переписывай его заново с учётом всех требований к роботу, торгующему на реале, а не в тестере. ;)

Теперь посмотрим на логику...
//----------------------------------------------------------блок БАЙ
[b]if[/b] ( ВклB == 1 && OrdersTotal() < OrdTotal && iClose(NULL, 1440, 1) < iOpen(NULL, 1440, 1))// вчерашняя дневка медвежья?
{
[b]double KB =[/b] iStochastic(NULL, 1440, PKB, PDB, 3, MODE_SMA, 0, MODE_MAIN, 1);
[b]double DB =[/b] iStochastic(NULL, 1440, PKB, PDB, 3, MODE_SMA, 0, MODE_SIGNAL, 1);
[b]double XB[/b] = MathPow((200-(KB - DB))/100, (50 - DB)/MathAbs(50 - DB));
}
[b][color=#FF0000]if ( KB > DB && MathAbs(KB - dKB) > SB )[/color][/b]
{
double СЛB = ИсторСЛB *XB *Point;
double ТПB = ИсторТПB *XB *Point;
OrderSend(Symbol(), OP_BUY, Лот, Ask, 3, Ask - СЛB, Ask + ТПB, "SS24", 1, 0);// если условия истинны, то открывем ордер
if ( IsTesting( ) == false) ВклB = 0;
}
Если выполняются такие-то условия, то робот присваивает нескольким переменным значения, [b]А ЕСЛИ НЕ ВЫПОЛНЯЮТСЯ[/b], то не присваивает, но сразу за этим [b][color=#FF0000]начинает их сравнивать[/color][/b] - нелогично, не так ли?)
Всё, на этом заканчиваю копаться в этом эксперте... Нужно переписывать однозначно.

[quote="4x_trader"]Евгений, ты уж меня извини, но... Эксперт по-прежнему остаётся хромым калекой с кучей костылей... Не обижайся, но это так. Рабочее состояние этой и предыдущих версий "висит на волоске" и в любой момент может быть нарушено любым сбоем в работе эксперта, например - обычной перезагрузкой или сменой таймфрейма.[/quote]
Да чего там извиняться, меня этим не выбьешь из седла :) . Я и не хвалюсь, что уже создал идеальный продукт, а сказал, что сделал небольшой шаг, но верный. То, на что пока хватило времени и знаний. А перезагрузка и смена таймфрейма никак не влияли на работу робота. Критический период для этого - первые 15 минут дневки, т.е. вероятность очень низкая, хотя согласен, что и ее надо свести к нулю.

[quote="4x_trader"]...Использование в именах переменных кириллицы, хоть и допускается компилятором, но крайне нежелательно, и не является "хорошим тоном" в программировании...[/quote]
Да, это я понял и уже исправляю. Не зря в основном используют латиницу. Кириллица была, в основном, из чувства патриотизма :)