Ошибка в коде линейки кластерных индикаторов от Семен Семёновича

 

Здравствуйте!

Скачал здесь кластерные индикаторы, выложенные здесь https://www.mql5.com/ru/articles/1464

Но обнаружил небольшую ошибку в коде.

На форуме МТ5 http://ruforum.mt5.com/threads/59537-indikator-ccfp-risuet-pochemu-to-cherez-bar?p=9857794#post9857794 все довольно подробно расписал.

Знаю, нужно было сразу постить здесь тему. Просто не думал, что ошибку в коде найду.

Прошу внести исправление в исходный код индикаторов которые лежат в открытом доступе.

С уважением,

Александр.

 
Aleks_:

Здравствуйте!

Скачал здесь кластерные индикаторы, выложенные здесь https://www.mql5.com/ru/articles/1464

Но обнаружил небольшую ошибку в коде.

На форуме МТ5 http://ruforum.mt5.com/threads/59537-indikator-ccfp-risuet-pochemu-to-cherez-bar?p=9857794#post9857794 все довольно подробно расписал.

Знаю, нужно было сразу постить здесь тему. Просто не думал, что ошибку в коде найду.

Прошу внести исправление в исходный код индикаторов которые лежат в открытом доступе.

С уважением,

Александр.

Почитайте комментарии на этой странице, там были исправления
 
r772ra:
Почитайте комментарии на этой странице, там были исправления


Здравствуйте!

Ссылка которую вы разместили ведет к скачиванию индикатора CCFp - кстати с существующей ошибкой.

 
Aleks_:


Здравствуйте!

Ссылка которую вы разместили ведет к скачиванию индикатора CCFp - кстати с существующей ошибкой.


Ссылку поправил, в старых билдах с этими исправлениями индикаторы работали нормально.

 
r772ra:


Ссылку поправил, в старых билдах с этими исправлениями индикаторы работали нормально.

Да никогда они не работали нормально. Я их ещё в 2012 году правил. Лениво искать ссылки на forexsystems.ru
 
r772ra:


Ссылку поправил, в старых билдах с этими исправлениями индикаторы работали нормально.



Открыл ссылку. Даа похоже в 2012 году Crazy_Fox: предлагал править, правда совсем по другому.

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

-------------------------------------------

Теперь в начале файла ищем объявление параметров:

extern int All_Bars = 0;
extern int Last_Bars = 0;

Они нам теперь не нужны, вместо них вставим строчку:

extern int Bars.Count = 0;

Всё, теперь никаких проблем не будет. Это применимо для всех трёх индикаторов.

--------------------------------------------

На последнем билдере это не пройдет. Переменные вида Bars.Count считаются ошибкой, нужно будет вместо "." поставить например "_", тогда откомпилируется. Кроме того, я то предлагаю все сохранить, даже переменную Last_Bars. И пусть все работает как задумал сам автор. Внесите уж исправление в скачиваемую версию индикаторов которые лежат в открытом доступе. Тем более, что на данный момент скачать исправленную версию индикаторов по предложенному методу Crazy_Fox: нельзя, ссылка не работает.

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

 

Лепишь не понятно что... Там вся проблема в не правильном применении IndicatorCounted() и направлении заполнения массивов. Потому и грузили комп не по-детски.

А учитывая обновление до 6хх билдов, появятся ещё проблемы которые при компиляции определены не будут.

Причина обращения: