Ошибки, баги, вопросы - страница 886

[Yedelkin]

G001: У меня только 3 деления:

Вот здесь потенциально м.б. деление на ноль:

/AccountInfoDouble(ACCOUNT_BALANCE)

Нужна элементарная проверка.

G001: Что такое выход за пределами массива? Извиняюсь за мое незнание. 

Это когда, например:

double array[3];
for (int i=0; i<=3; i++) array[i];

На последней итерации будет выход за пределы массива. 

[[Удален]]

Yedelkin:

Вот здесь потенциально м.б. деление на ноль:

Нужна элементарная проверка.

Это когда, например:

На последней итерации будет выход за пределы массива. 

Спасибо.

Круто, надо проверять если функция работает правильно?

AccountInfoDouble(ACCOUNT_BALANCE)

 

[Yedelkin]

G001: Круто, надо проверять если функция работает правильно?

Ну а что произойдёт, когда баланс уйдёт в ноль, а функция сработает правильно? :)

[[Удален]]

 В таком счетчике нет проблем?

int TotalBullStopOrders()
{
  int BullCounter=0;
  int Total = OrdersTotal();
  for(i = 0; i < Total; i++)
  {
    if(OrderSelect(OrderGetTicket(i)))
    {
      if(OrderGetString(ORDER_SYMBOL)==Symbol() && OrderGetInteger(ORDER_MAGIC)==Magic)
      {
        if(OrderGetInteger(ORDER_TYPE)==ORDER_TYPE_BUY_STOP)
        {
          BullCounter++;
        }
      }
    }
  }
  return(BullCounter);
}

[[Удален]]

Yedelkin:
Ну а что произойдёт, когда баланс уйдёт в ноль, а функция сработает правильно? :)

Вы правы, исправлю там.

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

[Yedelkin]

G001: В таком счетчике нет проблем?

У оператора for не указан тип переменной i. Переменная Magic не определена. В строчке

if(OrderSelect(OrderGetTicket(i)))

функция OrderSelect() - лишняя. Больше ничего не заметил.

[Test Account]

AndreyS:

вопрос с ошибкой (Ошибка CopyBuffer - номер ошибки: 4806) - снимаю, излечиваю его задержкой  Sleep(1000); - списываю на то что индикатор не успевает создатся.

Вопрос с ошибкой (4194304 bytes not available), остаётся. 

Создаться-то он скорее всего создался. Только перед копированием данных надо бы проверить наличие получаемых данных (а индикатор успел расчитаться?). Функция BarsCalculated должна помочь.

Далее вы запрашиваете все таймфреймы по символу. Если эти данные готовы - они поднимаются с диска в память, если нет - начинают строиться (тоже память расходуется) + 100 индикаторов с разным периодом на каждый ТФ и все это в памяти. Данные из памяти выгружаются не сразу (а вдруг вы еще раз запросите их)...

Уменьшите кол-во баров на чарте.

[[Удален]]

Yedelkin:

У оператора for не указан тип переменной i. Переменная Magic не определена. В строчке

функция OrderSelect() - лишняя. Больше ничего не заметил.

 

Спасибо.

Есть вне оператора.

input int    Magic          = 55;

 

MqlTradeRequest request={0};
MqlTradeResult result={0};
MqlTradeCheckResult check={0};
//-----
int LotsDigits;
int Indicator;
double IndicatorVal[5];
double Ask;
double Bid;
int i;
int pos;
double TradeLot;
ulong StopLevel;
double MeansFree;
double MaxLotAllowed;
//+------------------------------------------------------------------+
//| Expert initialization function                                   |
//+------------------------------------------------------------------+

[Andriy Sydoruk]

alexvd:

Создаться-то он скорее всего создался. Только перед копированием данных надо бы проверить наличие получаемых данных (а индикатор успел расчитаться?). Функция BarsСalculated должна помочь.

Далее вы запрашиваете все таймфреймы по символу. Если эти данные готовы - они поднимаются с диска в память, если нет - начинают строиться (тоже память расходуется) + 100 индикаторов с разным периодом на каждый ТФ и все это в памяти. Данные из памяти выгружаются не сразу (а вдруг вы еще раз запросите их)...

Уменьшите кол-во баров на чарте.

 

        while(BarsCalculated(m_handle)==-1 && !IsStopped()) Sleep(1); // Спасибо alexvd - откоректировал строку с учётом замечаний

Количество баров ничего не меняет,

проблема спрятана в периодах, для варианта

for(int iperiod=5; iperiod<=19; iperiod++) 

не работает, 19-5=14 - разных периодов,

если их уменьшить до 4- то все работает, например:

for(int iperiod=16; iperiod<=19; iperiod++)

 Хотя фу-я IndicatorRelease(m_handle) - дествительно некорректно используется.

Думаю даже если не освобождать дескрипторы функцией  IndicatorRelease, всеравно должно работать,

получается проблема в количестве разных периодов.

Или... 

[Andriy Sydoruk]

По поводу моего вопроса проблема дествительно в нехватке памяти, необходимо иметь функцию которая удаляет индюк скорее чем 30 мин. как это делает IndicatorRelease, необходима ф-я немедленного удаления, кто знаэт есть такая?