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

Немножко покритикую твой "блок выхода"...
[size=50] //-----------------------------------------------------------общий блок выхода
if (IsTesting( ) == false && Ticket > 0)
{
if (DayOfWeek( )== 1||2||3||4 && MathMod(TimeCurrent(), 86400) >= MathMod(StrToTime("23:55"), 86400))
{
OrderClose(Ticket, Лот_b, Bid, 3);
OrderClose(Ticket, Лот_s, Ask, 3);
}

if (DayOfWeek( )== 5 && MathMod(TimeCurrent(), 86400) >= MathMod(StrToTime("20:00"), 86400))
{
OrderClose(Ticket, Лот_b, Bid, 3);
OrderClose(Ticket, Лот_s, Ask, 3);
}
}

if (IsTesting( ) == true)
{
OrderSelect(0, SELECT_BY_POS, MODE_TRADES);

if (DayOfWeek( )== 1||2||3||4 && TimeCurrent() - OrderOpenTime( ) >=86400)
{
if(OrderType() == OP_BUY )
OrderClose(OrderTicket(), Лот_b, Bid, 3);
if(OrderType() == OP_SELL )
OrderClose(OrderTicket(), Лот_s, Ask, 3);
}

if (DayOfWeek( )== 5 && TimeCurrent() - OrderOpenTime( ) >=72000)
{
if(OrderType() == OP_BUY )
OrderClose(OrderTicket(), Лот_b, Bid, 3);
if(OrderType() == OP_SELL )
OrderClose(OrderTicket(), Лот_s, Ask, 3);
}
}[/size]
1. Снова вижу деление на для теста/для реала - зачем???
2. Используется переменная Ticket, задаваемая вручную во входных параметрах советника... Выходит, что для закрытия нужно заранее ввести тикет и перезапустить советника, а если ордера два???))) Тогда дождаться закрытия первого и указать тикет второго? Нелогично.
Нужно просто использовать [b]функцию ЗакрытьВсеОрдеры()[/b], вызывая её в нужное время и контролируя Магик при этом, чтобы не закрыть ордеры других советников или открытые вручную. Изобретать её нет необходимости - такие давно написаны.

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

[quote="4x_trader"]- код не структурирован, поэтому логика плохо читается...[/quote]
Вячеслав, ты имеешь ввиду под структурированностью, что везде нужны подробные комментарии и разделение на более мелкие блоки?

[quote="4x_trader"]...Вот фрагмент последней версии твоего эксперта:
[b]...
if ([color=#FF0000]DayOfWeek( )== 1||2||3||4[/color] && MathMod(TimeCurrent(), 86400) >= MathMod(StrToTime("23:55"), 86400))
...[/b]
Чтобы было понятнее и нагляднее, в чём ошибка, я набросал тебе скриптик с аналогичной логикой... Тоже кинь его на чарт и убедись, что [color=#FF0000][b]такая[/b][/color] конструкция не работает...[/quote]
Я проверил скрипт. Ну да, там, где нет 5 и 6, он выдает в моем варианте единицы, как-бы находит несуществующие цифры (если я правильно понял тебя). Но советник все же исправно закрывал сделки с понедельника по четверг в 23.55, а пятницу в 20.00. Видимо повезло, что ошибка оказалась некритической :)

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

[quote="Евгений Субботин"]... повезло, что ошибка оказалась некритической :)[/quote]Дело не в том - "критическая/не критическая", а в том, что с логическими выражениями надо быть поосторожнее. Везде, где только можно, ставить скобки. Вместо одного сложного выражения - лучше несколько простых. Тогда ошибок не будет. В сомнительных случаях проверять расчёт выражений на простых скриптах.

Выше я тебе советовал использовать [b]функцию для закрытия всех позиций[/b]. Выкладываю скрипт, в котором я использовал такую. Универсальная функция закрытия позиций и отложек (если понадобится), сгодится и для твоего советника. В скрипте описаны подробности применения.[attachment=0]sCloseAllBuySell.mq4[/attachment]

[quote="4x_trader"]...зачем у тебя в коде такое обилие [b]IsTesting( )[/b]???...[/quote]
Это довольно запутанная история, поэтому прошу терпения :) . Как бы понятнее объяснить?... Если коротко, то в тестере в режиме "по ценам открытия" и на реале тики имеют разное происхождение. В тестере - тик это начало нового бара, на реале - это непредсказуемый приход новой котировки. Мой советник хоть классически и не определяет новый бар, но имитирует это, работая по началу времени бара. Поэтому я его тестирую "по ценам открытия". Сделки совершает редко на дневном графике, поэтому нужна история лет хотя бы за 5-6. На часовке такую историю уже непросто найти, на дневке есть, но это будет грубо, т.к. не учитывается внутридневное движение и сделки всегда интрадей. Поэтому компромисс - Н4, достаточно глубокая история и приемлемая точность для такого алгоритма, в тестере 6 тиков в сутки.

[b]Что будет, если не разделять тестер и реал?[/b]

1. Условие открытия по времени:
if (
(IsTesting() == true && StrToTime("00:00")== TimeCurrent())
||(IsTesting()==false && MathMod(StrToTime("00:00"),86400)< MathMod(TimeCurrent(),86400) && iVolume(NULL,1440,0) > 1 && iVolume(NULL,1440,0) < 10)
)
Если сделать условие на реале StrToTime("00:00")== TimeCurrent()), то будут неточные расчеты с прежними косяками. Если сделать <=, то несмотря на блокировку после совершения сделки, в случае обрыва связи и снова подключения, на реале откроется лишняя сделка на следующем тике в течение времени ожидания. А в тестере открывается правильно один раз – в 00:00.

Если сделать условие в тестере
MathMod(StrToTime("00:00"),86400)< MathMod(TimeCurrent(),86400) && iVolume(NULL,1440,0) > 1 && iVolume(NULL,1440,0) < 10,
то сделки вообще открываться не будут.

2. Блокировка советника в реале после совершения сделки:
extern bool Вкл_b = 1;
…
if ( Вкл_b == 1 && …)
…
OrderSend(…);
if ( IsTesting( ) == false)
Вкл_b = 0;

Без этого на реале в течение времени ожидания будут открываться лишние сделки. В то же время, в тестере блокировать советник не нужно, т. к. к следующему тику текущее время будет уже больше, чем заданное время открытия (00:00) и лишняя сделка не откроется. А когда текущее станет снова 00:00, то опять одна откроется. Перезапуск счета или терминала снимает блокировку.

3. Закрытие ордеров по рынку.
Как сделать без ручного указания тикета (по магику) пока не знаю, поэтому пока так:
extern int Ticket = 0;
…
if (IsTesting( ) == false && …)
{
…
OrderClose(Ticket, …);
… }

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

if (IsTesting( ) == true)
{
OrderSelect(0, SELECT_BY_POS, MODE_TRADES);
…
if(OrderType() == OP_BUY )
OrderClose(OrderTicket(), Лот_b, Bid, 3);
…
}
…

Конечно, я понимаю, что все это может делаться проще и грамотнее, но … Пока практика сильно опережает теорию :) . Поэтому приходится выдумывать. А главное, на реале теперь результат абсолютно совпадает с тестом. Но конечно, если я научусь, то переделаю лучше :) .

[quote="Евгений Субботин"]... то в тестере в режиме "по ценам открытия" и на реале тики имеют разное происхождение...
[b]Что будет, если не разделять тестер и реал?[/b]

1. Условие открытия по времени:...

2. Блокировка советника в реале после совершения сделки:

3. Закрытие ордеров по рынку.
Как сделать без ручного указания тикета (по магику) пока не знаю...
... я понимаю, что все это может делаться проще...если я научусь, то переделаю лучше :) .[/quote]Ну, давай по порядку...
1. Проблемы тестер/реал ты себе создал сам по большому счёту, написав такие хитромудрые условия для открытия по времени. Этот ответ, похоже, ты писал не посмотрев как следует [b]советник-шаблон, который я тебе выложил[/b]. В нём я вставил открытие по времени и... (см.п.2) Уверен, с ним не будет таких проблем и нестыковок реала с тестером.
2. Блокировка открытия "лишних" позиций должна производиться [b]подсчётом уже открытых[/b].))) Пример этого - в том же советнике-шаблоне.
3. Закрытие позиций - смотри в последнем выложенном скрипте.
Этой информации уже более, чем достаточно, чтобы "причесать" твой советник. Точнее - используя шаблон и функции, которые я тебе вчера-сегодня здесь выложил, можно ПЕРЕПИСАТЬ советник, чтобы он действительно выполнял твою задумку без "танцев с бубнами")
Перечитай и прощупай на демке (поглядывая в логи) работу эксперта и скрипта. Пиши, спрашивай, если что не понятно.
Логикой займёмся чуть позже, когда скомпонуешь первую версию.

Строчка из советника последней версии:

double X_b = MathPow((200-(K_b - D_b))/100, [b][u](50 - D_b)/MathAbs(50 - D_b));[/u][/b]

Здесь всегда будет 1 и X_b = число в степени 1, зачем проводить такие серьезные вычисления, что бы получить значение в первой степени ?!?