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

:lol: :lol: :lol:
зачем тебе полуавтомат, когда до автомата один шажочек ;)

я б тебе его причесал... хотя тут не причёсывание, а полная переписка с оставлением идеи получилась бы :)
но не буду, иначе не научишься ;)
так что на основные ошибки я тебе указал, давай причёсывай и тестируй :)

[quote="WWolf"]суть то я понял :) но написал ты не правильно :) ...
надо было
if (DayOfWeek( )!= 6 || DayOfWeek( )!=7)[/quote]
Дошло :) Хотел покороче :) Правда на работу это не повлияло. Тогда это тоже неправильно?
if (DayOfWeek( )== 1||2||3||4 &&... Я вроде где-то такое видел.
А так можно?
if (DayOfWeek( )== (1||2||3||4) &&...

[quote="Евгений Субботин"]Тогда это тоже неправильно?
if (DayOfWeek( )== 1||2||3||4 &&...[/quote]
угу
[quote="Евгений Субботин"]Я вроде где-то такое видел. [/quote]
там скорее какие то флаги логические сравнивались и в том примере оно скорее всего имело смысл :)
а с числовыми значениями это теряет всякий смысл :)

[quote="Евгений Субботин"]А так можно?
if (DayOfWeek( )== (1||2||3||4) &&... [/quote]
никакой разницы с первым примером :)

Все, разобрался почему не открывала третья версия :) Оказывается если условие написать по правилам привычной математики, то с ним не работает:
10 > iVolume(NULL, 1440, 0) > 1 // было так.

А если по-отдельности, то работает почему-то:
iVolume(NULL,1440,0) > 1 && iVolume(NULL,1440,0) < 10 // стало так в 4 версии

Определение первого бара, как объяснял Дима, я сегодня пока не разбирал, но пусть опять коряво, но результата добился. Главное, что уже завтра со 2-го тика должен открывать [b]правильно [/b] :)

хых :) а я и не заметил :lol:
маленький монитор = зло :)

[quote="Евгений Субботин"]... готов к более детальному обсуждению.[/quote]Ну, вот и молодец!)
Замечания по последней версии:
- код не структурирован, поэтому логика плохо читается...
- отработку кода я бы начал на твоём месте с написания шаблона и проверки на скриптах [b]отдельных функци[/b]й, т.к. их применение облегчает проверку логики работы советника в дальнейшем.
Для начала не помешали бы такие:
NewBar(), OpenBuy(), OpenSell(), CloseBuy(), CloseSell(), CloseAll() и т.п.

Вот шаблон эксперта, в котором реализовано разделение на функции, работающие на выбранном новом баре (независимо от ТФ, на который брошен эксперт):[attachment=0]NewBar&EveryTick.mq4[/attachment]Кинь его на чарт и поиграйся с настройками.
Потом сможешь начать заполнять его своими функциями...)

Теперь несколько слов о логике...
Вот фрагмент последней версии твоего эксперта:
[b]...
if ([color=#FF0000]DayOfWeek( )== 1||2||3||4[/color] && MathMod(TimeCurrent(), 86400) >= MathMod(StrToTime("23:55"), 86400))
...[/b]
Чтобы было понятнее и нагляднее, в чём ошибка, я набросал тебе скриптик с аналогичной логикой:[attachment=0]test_logiki.mq4[/attachment]
Пояснения в коде. Тоже кинь его на чарт и убедись, что [color=#FF0000][b]такая[/b][/color] конструкция не работает...

Вопрос на засыпку: скажи, зачем у тебя в коде такое обилие [b]IsTesting( )[/b]???
У тебя логика работы советника в тестере и на реале должны отличаться?) Очень любопытно, но самому докапываться некогда...

Ещё один совет: [b]постарайся избегать применения в именах переменных знака подчёркивания[/b]. В последних билдах метаэдитора имеется некая проверка "на вшивость" - при наличии в коде имён, похожих на генерируемые декомпилятором, компиляция не производится и выдаётся сообщение:[attachment=0]декомпилы запрещены.PNG[/attachment]Примеры имён из декомпила:
int li_12;
int lia_16[];
int l_global_var_32;
double gda_708[14];
double g_ord_open_price_712;
bool gi_720 = TRUE;
int g_datetime_728 = 0;
... и т.п.
В один прекрасный момент может стать обидно, что твой родной исходник вдруг перестанет компилироваться...