АвторСообщение



Сообщение: 43
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 16.12.16 05:39. Заголовок: Паттерны ПрайсЭкшен через структуру


Игорь добрый день!
Перенесу сюда свои мытарства с перепрограммированием индикатора по паттернам под свои нужды. В той ветке я показал, что кое-какие сдвиги к получению желаемого мной результата есть. В текущем варианте как вы видели по картинкам он уже отрисовывает паттерны и их цели, способен перекрасить паттерны, если его таргет пробит. Вопрос по удалению перекрытых паттернов пока откладываю на потом, поскольку возник более актуальный. Повторю здесь кусок кода, где происходят главные события с паттернами:
Скрытый текст

Отображает он все как мне хотелось бы. При наведении мышки на паттерн отображает информацию из функции GetUniqueID:

 цитата:
string GetUniqueID(const Pattern & pattern, string middlePart)
{

return(PREFIX + EnumToString(pattern.patternName)+EnumToString(pattern.getTarget)+"_"
+EnumToString(pattern.patternType)+ middlePart + string(Time[pattern.rightBarIndex]));
}


Но тут где-то собака зарылась, не пойму относительно отображения EnumToString(pattern.getTarget), которая определяется отдельной функцией (кстати подобными строками перекрашиваются паттерны):

 цитата:
int IsActualPattern(Pattern &pattern)
{
for(int i = ArraySize(g_patterns)-1; i >= 0; i--)
{
if(g_patterns.patternType == BEAR_TYPE)
{
for(int k = g_patterns.rightBarIndex-1; k >= 0; k--)
{
if(iLow(NULL, 0, k) < g_patterns.patternTarget)
g_patterns.getTarget = WORKED;
else
g_patterns.getTarget = ACTUAL;
}
}
if(g_patterns.patternType == BULL_TYPE)
{
for(int k = g_patterns.rightBarIndex-1; k >= 0; k--)
{
if(iHigh(NULL, 0, k) > g_patterns.patternTarget)
g_patterns.getTarget = WORKED;
else
g_patterns.getTarget = ACTUAL;
}
}
}
return (g_patterns[ArraySize(g_patterns)-1].getTarget);
}


Так вот... Eсли медвежий паттерн отработан (WORKED) - то окрашивается он как отработанный, и в подписи к нему EnumToString(pattern.getTarget) пишет WORKED, а вот с бычьими почему-то не пишет... окрашивает как отработанный, но в подписи и в принте пишет что он актуальный (см. рис). Как побороть это недоразумение? Сдается мне что тут вина в implicit enum conversion в строке
g_patterns[patternArray].getTarget = getTarget, его энум
Скрытый текст

Кроме этого таких предупреждений еще два штуки есть по коду в строках:
g_patterns[patternArray].patternType = patternDir, его Энум выглядит так
Скрытый текст

и
g_patterns[patternArray].patternName = patternName, его энум
Скрытый текст

Ну и все они в структуре объявлены и инициализированы вот так
patternType = NONE_TYPE;
patternName = NONE_INDEX;
getTarget = NONE;
И на всякий случай функция GetData
Скрытый текст

Ткните, плиз носом, как победить этот недуг


Просмотрев немного по истории, точной закономерности не выявил, но вот покажу участок, где стрелками показаны два по рассраске отработанных бычьих паттерна, между ними 8 свечей (две стрелки указывают на один и тот паттерн). Но подписи у них разные... тот что ниже слева - пишет WORKED, а тот что через 8 свечей справа - ACTUAL.


Чо к чему... не пойму А главное как так-то? Может в коде какого дополнительного условия не хватает в функции IsActualPattern? Или это не понятные примудрусти какие-то в функции строки подписи?
Скрытый текст



Спасибо: 0 
ПрофильЦитата Ответить
Ответов - 87 , стр: 1 2 3 4 5 6 All [только новые]





Сообщение: 54
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 01.01.17 02:31. Заголовок: evbut пишет: Посмот..


evbut пишет:

 цитата:
Посмотрю уже в следующем году


С Новым Годом, Игорь! Спасибо Вам за неизменную поддержку и Ваш профессионализм в прошлом году! В новом году желаю успеха в реализации Ваших задумок, благополучия и мирного неба Вам и вашей семье!

Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2418
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 05.01.17 17:53. Заголовок: evbut пишет: Тынц С..


evbut пишет:

 цитата:
Тынц Ссыль действительна 6 дней


Посмотрел. Не увидел в коде участка, отвечающего за перерисовку паттерна. Так, функция ProcessBar обслуживает только новые бары. Она никогда не повторяет расчет одного и того же бара. Поэтому паттерн может быть только найден. Его дополнительная проверка не производится.

Возможно проверка задумывалась в функции IsActualPattern(), но эта функция нигде в коде не вызывается. Даже, если она будет вызвана, то это ни к чему не приведет, т. к. функция меняет лишь статус паттерна, но не производит его перерисовку.

Спасибо: 1 
ПрофильЦитата Ответить



Сообщение: 55
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 07.01.17 18:36. Заголовок: Scriptong пишет: Во..


Scriptong пишет:

 цитата:
Возможно проверка задумывалась в функции IsActualPattern(), но эта функция нигде в коде не вызывается. Даже, если она будет вызвана, то это ни к чему не приведет, т. к. функция меняет лишь статус паттерна, но не производит его перерисовку.



Собственно да, один из вариантов реализации отображения отработки паттернов было как раз через эту функцию, что-то в этом духе

 цитата:
void ReDrawPattern(Pattern &pattern)
{
IsActualPattern(pattern);
if(ShowWorkedPattern && pattern.getTarget == WORKED &&
(pattern.patternType == BEAR_TYPE || pattern.patternType == BULL_TYPE))
{
pattern.patternColor = WorkColor;
}
else
{
if(pattern.patternType == BEAR_TYPE)
pattern.patternColor = RegcolorBears;
else
pattern.patternColor = RegcolorBulls;
}
ShowPatterns(pattern);


Далее в OnCalculate так прописал

 цитата:
int limit=GetRecalcIndex(total); // Определим первый расчетный бар

for (int i = limit; i > 0; i--)
{
FillPatternsDB(i); //функция наполнения базы паттернов типа void

for(int k = ArraySize(g_patterns)-1; k>=0; k--)
ReDrawPattern(g_patterns[k]); //собственно отрисовка
}

WindowRedraw();


От вашего варианта как во Всплеск и Полка пока отошел... и по сему понимаю, что кривая конструкция, но с праздниками этими все в голове перемешалось... В тестере вроде рисуется с использованием этой функции... Новый паттерн отрисовывает в сине-красных цветах, и даже отработанные показывает, только не на том месте где он действительно есть, а со смещением к нулевому бару, а точней к тому бару на котором была зафиксирована его отработка паттерна (ВИДЕОзаписал ).

Решение кажется нашлось... в функции GetUniqueID было внесено отображение факта отработки паттерна. ТАк вот получалось, что индикатор не находил прямоугольник патерна с подписью ACTUAL, потому что он уже был WORKED, поэтому рисовал его поновой в другом месте..




Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2423
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 13.01.17 21:07. Заголовок: evbut пишет: Собств..


evbut пишет:

 цитата:
Собственно да, один из вариантов реализации отображения отработки паттернов было как раз через эту функцию, что-то в этом духе

 цитата:
void ReDrawPattern(Pattern &pattern)
{
IsActualPattern(pattern);
if(ShowWorkedPattern && pattern.getTarget == WORKED &&
(pattern.patternType == BEAR_TYPE || pattern.patternType == BULL_TYPE))
{
pattern.patternColor = WorkColor;
}
else
{
if(pattern.patternType == BEAR_TYPE)
pattern.patternColor = RegcolorBears;
else
pattern.patternColor = RegcolorBulls;
}
ShowPatterns(pattern);


Далее в OnCalculate так прописал


Ну это уж совсем никудышное решение. Во-первых, производится повторение одних и тех же действий (вызов функции FillDBPatterns() теперь будет дважды на одном и том же баре и, приводит к дублированию паттернов в массиве - см. журнал). Во-вторых, дважды производится определение отработки паттерна (логика размазана по двум функциям RecolorPattern() и ReDrawPattern()), что затрудняет понимание кода.

Также в процессе просмотра кода увидел основную проблему: при нахождении паттерна запонимаются номера баров, где он найден (члены leftBarIndex и rightBarIndex структуры Pattern). При таком подходе требуется поддерживать номера баров в дальнейшем, т. к. номер бара - не уникальный идентификатор. Номера баров изменяются в момент прихода нового бара. В итоге тот бар, который сохранен для паттерна оказывается невалидным. Поэтому нужна поддержка индексов. Или же, лучший подход, запоминать не индекс бара, в время открытия баров. Это уже уникальные данные, которые никогда не изменяться.

Спасибо: 0 
ПрофильЦитата Ответить



Сообщение: 50
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 22.12.16 08:16. Заголовок: Игорь, подскажите, д..


Игорь, подскажите, достаточно ли будет на этом этапе в функции наполнения БД паттернов

 цитата:
int patternArray=ArraySize(g_patterns);
for (int i = patternArray - 1; i >= 0; i--)
if (g_patterns.patternType == newPattern &&
g_patterns.patternName == newPatternName &&
g_patterns.leftBarIndex == startBar &&
g_patterns.rightBarIndex == endBar
)
return FUNCTION_RESULT_ALRIGHT;


прописать для проверки только эти характеристики паттерна или же нужно проверить все согласно прописанного в структуре (за исключением patternTarget, getTarget и patternColor?

 цитата:
ENUM_PATTERN_TYPE patternType; // Тип паттерна (бычий, медвежий)
ENUM_PATTEN_NAME patternName; // Индекс имени паттерна
ENUM_GET_TARGET getTarget; // Отработан ли патерна
int leftBarIndex; // Индекс левого бара паттерна
int rightBarIndex; // Индекс правого бара паттерна
double patternLowPrice; // Нижняя цена паттерна
double patternHighPrice; // Верхняя цена паттерна
double patternHeight; // Высота паттерна
double patternTarget; // Уровень цели бычьего паттерна
color patternColor; // ЦВет паттерна


И еще несколько вопросов по коду функции из инидкатора Всплеск и полка

 цитата:
ShowIndicatorData(int limit, int total)
{
if (limit > 1)
{
ArrayResize(g_patterns, 0, ARRAY_RESERVE_SIZE);
g_lastBuyPatternRegFractalTime = 0;
g_lastSellPatternRegFractalTime = 0;
}

if (limit == 1)
for (int i = ArraySize(g_patterns) - 1; i >= 0; i--)
{
g_patterns.leftBarIndex++;
g_patterns.rightBarIndex++;
}

for (int i = limit; i >= 0; i--)
if (!ProcessBar(i))
return false;


Под limit подразумевается количество баров или собственно ограничение в барах? Да и в целом я правильно понимаю результат этой функции:
при limit > 1 задаем новый размер массива паттернов, при limit == 1 - изменяем номера баров границы паттернов, и в последнем случае отрисовываем что желаем?

Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2402
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 26.12.16 13:25. Заголовок: evbut пишет: Игорь,..


evbut пишет:

 цитата:
Игорь, подскажите, достаточно ли будет на этом этапе в функции наполнения БД паттернов
прописать для проверки только эти характеристики паттерна или же нужно проверить все согласно прописанного в структуре (за исключением patternTarget, getTarget и patternColor?


Пока суть вопроса неясна. Конкретизируйте, пожалуйста.

Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2403
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 26.12.16 13:33. Заголовок: evbut пишет: Под li..


evbut пишет:

 цитата:
Под limit подразумевается количество баров или собственно ограничение в барах?


В данном случае то, что представляет собой переменная limit, описано в функции OnCalculate:

 цитата:
int total;
int limit = GetRecalcIndex(total, rates_total, prev_calculated);

if (!ShowIndicatorData(limit, total))
g_activate = false;


В свою очередь, GetRecalcIndex прокомментирована: Determination of bar index which needed to recalculate. Это означает, что переменная limit получает индекс бара, с которого на текущем тике необходимо вычислять значения индикатора.
Так, при первом запуске индикатора limit получит индекс самого левого бара в истории минус 2 (ratesTotal - 3), если значение параметра "Количество баров отображения" равно 0. В пртивном случае переменная limit будет равна значению этого параметра.
При повторном вызове функции OnCalculate индикатора значение limit будет равно 0, если на текущем баре уже были вычисления, и 1, если образовался новый бар. Больше 1 при повторном вызове OnCalculate переменная limit будет только в случае подкачки истории котировок.
Такой подход позволяет оптимизировать работу индикатора: с каждым новым тиком производится перерасчет только вновь поступивших данных. В противном случае индикатор на каждом тике производил бы перерасчет всей имеющейся истории.


Спасибо: 1 
ПрофильЦитата Ответить



Сообщение: 53
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 28.12.16 05:42. Заголовок: Scriptong пишет: Та..


Scriptong пишет:

 цитата:
Такой подход позволяет оптимизировать работу индикатора: с каждым новым тиком производится перерасчет только вновь поступивших данных. В противном случае индикатор на каждом тике производил бы перерасчет всей имеющейся истории.


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

Спасибо: 0 
ПрофильЦитата Ответить



Сообщение: 56
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 11.01.17 03:47. Заголовок: Игорь, не подскажите..


Игорь, не подскажите, почему при смене тайм-фрейма остаются элементы с которого ушел?


Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2424
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 13.01.17 21:17. Заголовок: evbut пишет: Игорь,..


evbut пишет:

 цитата:
Игорь, не подскажите, почему при смене тайм-фрейма остаются элементы с которого ушел?


У меня такое не наблюдается. Проверил на последней версии (со вставкой приведенного Вами кода).

Спасибо: 1 
ПрофильЦитата Ответить



Сообщение: 57
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 13.01.17 22:34. Заголовок: Scriptong пишет: У ..


Scriptong пишет:

 цитата:
У меня такое не наблюдается.


Странно очень... Такое отмечается, когда переключаешься с меньшего фрейма на больший. Однако в функцию ShowIndicatorData прописал

 цитата:
if(limit > 1)
ArrayResize(g_patterns,0);



и кажется перестало такое безобразие мельтешить... Не критично конечно, но мешает.

Такой еще момент. Решил добавить опционально удаление перекрытых паттернов. Код такой

 цитата:
//+-----------------------------------------------------------------------------+
//| Process specify pattern |
//+-----------------------------------------------------------------------------+
bool IsNeedToDeletePattern(const Pattern &pattern, int index)
{
if(OverLaidPattern)
return false;

for (int i = index; i < pattern.leftBarIndex; i++)
{
if(NamePattern(i) < 0)
{ return false;
i--; }
}
return true;
}
//+-----------------------------------------------------------------------------+
//| Delete specify element from array |
//+-----------------------------------------------------------------------------+
template<typename T>
void DeleteElementFromArray(T &array[], int elementToDelete)
{
int arraySize = ArraySize(array);
if (arraySize == 0)
return;

array[elementToDelete] = array[arraySize - 1];
ArrayResize(array, arraySize - 1, ARRAY_RESERVE_SIZE);
}
//+-----------------------------------------------------------------------------+
//| Process on overlaid pattern |
//+-----------------------------------------------------------------------------+
void ProcessPatterns(int index)
{
for (int i = ArraySize(g_patterns) - 1; i >= 0; i--)
if (IsNeedToDeletePattern(g_patterns, index-1))
DeleteElementFromArray(g_patterns, i);
}


Судя по картинкам - удаляет те двухсвечные, паттерны, которые находятся как бы внутри трехсвечного паттерна. Однако "чистка" среди трехсвечных не происходит (подчеркнул на картинке явное наложение. По идее синий паттерн должен быть удален). Каким образом заставить и такие участки удалять или где-то в коде косякнул?



Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2425
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 17.01.17 17:24. Заголовок: evbut пишет: Странн..


evbut пишет:

 цитата:
Странно очень... Такое отмечается, когда переключаешься с меньшего фрейма на больший. Однако в функцию ShowIndicatorData прописал


Проверил, переключившись а М1 на Н1. Все ОК.

evbut пишет:

 цитата:
и кажется перестало такое безобразие мельтешить... Не критично конечно, но мешает.


Это, кстати, правильный ход. Только лучше такой код разместить в GetRecalcIndex() после строки ObjectsDeleteAll(). Ведь объекты удаляются, а массив g_patterns в итоге будет содержать паттерны-дубликаты.

evbut пишет:

 цитата:
Такой еще момент. Решил добавить опционально удаление перекрытых паттернов. Код такой


Нужно знать, откуда вызывается этот код. В самом же коде есть логическое упущение - в функции IsNeedToDeletePattern() проверяется только совпадение правых краев паттернов, а далее - нет.
Кстати, в чем смысл такой конструкции?

 цитата:
if(NamePattern(i) < 0)
{ return false;
i--; }


Во-первых, после return ничего не может выполняться. Во-вторых, если допустить, что требуется декремент значения i при выполнении условия NamePattern, то все равно смысла нет - происходит выход из функции, значение переменной i теряется. Или же декремент должен был быть за фигурной скобкой? Но тогда получим бесконечный цикл...



Спасибо: 1 
ПрофильЦитата Ответить



Сообщение: 58
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 18.01.17 02:18. Заголовок: Scriptong пишет: Кс..


Scriptong пишет:

 цитата:
Кстати, в чем смысл такой конструкции?


В нее заложен смысл, что если нет именованного паттерна, то и проверять не имеет смысла.


Спасибо: 0 
ПрофильЦитата Ответить





Сообщение: 2427
Зарегистрирован: 03.03.13
Откуда: Украина, Каменское (Днепродзержинск)
Репутация: 3
ссылка на сообщение  Отправлено: 24.01.17 11:03. Заголовок: evbut пишет: В нее ..


evbut пишет:

 цитата:
В нее заложен смысл, что если нет именованного паттерна, то и проверять не имеет смысла.


Так а зачем после return происходит декремент? Ведь после return ничего не может выполняться в том же блоке. Возможно, скобкой ошиблись. Но тогда, когда исправите положения скобок, получите зависание индикатора.

Спасибо: 0 
ПрофильЦитата Ответить



Сообщение: 59
Зарегистрирован: 11.04.16
Откуда: Иркутск
Репутация: 0
ссылка на сообщение  Отправлено: 19.01.17 18:50. Заголовок: И снова здрасьте... ..


И снова здрасьте... С удалением перекрытых паттернов еще однако разгребать и разгребать... Но вот решил сделать выбор отображения паттернов по отдельности, а не кучей... Уже и не помню ту версию что высылал вам... В общем функция ShowIndicatorData выглядит теперь так

 цитата:
void ShowIndicatorData(int limit, int total)
{
if(limit > 1)
ArrayResize(g_patterns,0);

for (int i = limit; i > 0; i--)
{
FillPatternsDB(i);
ProcessPatterns(i);
for(int j = ArraySize(g_patterns)-1; j >= 0; j--)
{
if((IsShowOVBPattern && g_patterns[j].patternName == OVB_) ||
(IsShowPPRPattern && g_patterns[j].patternName == PPR_) ||
(IsShowRailsPattern && g_patterns[j].patternName == RAILS_))
ReDrawPattern(g_patterns[j]);
}
}
}


Включаю по отдельности отображение то рельс, то ППР... И вот при отображении только паттерна Рельсы на некоторых участках встречается вот такое как для бычьих так и для медвежьих направлений (см рис), что совсем не понятно откуда всплывает - Прямоугольник показывает ППР да и выглядит это комбинация свечей как ППР, а в подписи Рельсы (( В настройках параметры Рельс стоят 20 и 10



Спасибо: 0 
ПрофильЦитата Ответить
Ответов - 87 , стр: 1 2 3 4 5 6 All [только новые]
Ответ:
1 2 3 4 5 6 7 8 9
большой шрифт малый шрифт надстрочный подстрочный заголовок большой заголовок видео с youtube.com картинка из интернета картинка с компьютера ссылка файл с компьютера русская клавиатура транслитератор  цитата  кавычки моноширинный шрифт моноширинный шрифт горизонтальная линия отступ точка LI бегущая строка оффтопик свернутый текст

показывать это сообщение только модераторам
не делать ссылки активными
Имя, пароль:      зарегистрироваться    
Тему читают:
- участник сейчас на форуме
- участник вне форума
Все даты в формате GMT  2 час. Хитов сегодня: 6
Права: смайлы да, картинки да, шрифты да, голосования нет
аватары да, автозамена ссылок вкл, премодерация откл, правка нет